Skip to content
Snippets Groups Projects

dinput/tests: Cleanup mouse and keyboard tests and merge them into device8.c.

Merged Rémi Bernon requested to merge rbernon/wine:mr/dinput-tests into master
1 unresolved thread

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
    • Contributor

      Marvin replied on the mailing list:

      Hi,
      
      While running your changed tests, I think I found new failures.
      Being a bot and all I'm not very good at pattern recognition, so I might be
      wrong, but could you please double-check?
      
      Full results can be found at:
      https://testbot.winehq.org/JobDetails.pl?Key=116665
      
      Your paranoid android.
      
      
      === w8 (32 bit report) ===
      
      dinput:
      driver_bus.c:290: Test failed: id 1: force_feedback.c:4020 got spurious packet
      driver_bus.c:1098: Test failed: id 1: force_feedback.c:4020 expect[0]: got 0xb000f, expected 0
      driver_bus.c:1099: Test failed: id 1: force_feedback.c:4020 expect[0]: got id 6
      driver_bus.c:1100: Test failed: id 1: force_feedback.c:4020 expect[0]: got len 2
      
      === w8 (32 bit report) ===
      
      dinput:
      joystick8.c:1836: Test failed: 0x700: state[0]: WaitForSingleObject succeeded
  • added 9 commits

    • 904c6970...b0731f6e - 3 commits from branch wine:master
    • 6a95003b - dinput/tests: Add some broken joystick axis results.
    • d4ea5f98 - dinput: Fix DIPROP_FFGAIN mouse and keyboard tests.
    • fd46b4bb - dinput/tests: Run mouse info tests for every dinput version.
    • 74b9bd67 - dinput/tests: Merge mouse.c tests into device8.c.
    • cda4596c - dinput/tests: Run keyboard info tests for every dinput version.
    • e1083de9 - dinput/tests: Merge keyboard.c tests into device8.c.

    Compare with previous version

  • Alexandre Julliard approved this merge request

    approved this merge request

Please register or sign in to reply
Loading