Mir

acceptance-tests FAIL: TestClientInput.hidden_clients_do_not_receive_pointer_events

Bug #1227683 reported by Daniel van Vugt on 2013-09-19
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mir
Fix Released
High
Robert Carr
mir (Ubuntu)
High
Unassigned

Bug Description

valgrind acceptance-tests FAIL: TestClientInput.hidden_clients_do_not_receive_pointer_events

[ RUN ] TestClientInput.hidden_clients_do_not_receive_pointer_events
/home/dan/bzr/mir/trunk/tests/acceptance-tests/test_client_input.cpp:916: Failure
Actual function call count doesn't match EXPECT_CALL(*handler, handle_input(MotionEventWithPosition(2, 2)))...
         Expected: to be called once
           Actual: never called - unsatisfied and active
Google Test trace:
/home/dan/bzr/mir/trunk/tests/mir_test_framework/testing_process_manager.cpp:110: Client
==23634==
==23634== HEAP SUMMARY:
==23634== in use at exit: 7,179 bytes in 125 blocks
==23634== total heap usage: 22,473 allocs, 22,348 frees, 1,574,144 bytes allocated
==23634==
==23634== LEAK SUMMARY:
==23634== definitely lost: 0 bytes in 0 blocks
==23634== indirectly lost: 0 bytes in 0 blocks
==23634== possibly lost: 2,787 bytes in 45 blocks
==23634== still reachable: 4,392 bytes in 80 blocks
==23634== suppressed: 0 bytes in 0 blocks
==23634== Rerun with --leak-check=full to see details of leaked memory
==23634==
==23634== For counts of detected and suppressed errors, rerun with: -v
==23634== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 2 from 2)
/home/dan/bzr/mir/trunk/tests/mir_test_framework/testing_process_manager.cpp:127: Failure
Value of: result.succeeded()
  Actual: false
Expected: true
client terminate error=process::Result(child_terminated_normally, failure(1))
==23635==
==23635== HEAP SUMMARY:
==23635== in use at exit: 7,171 bytes in 124 blocks
==23635== total heap usage: 22,469 allocs, 22,345 frees, 1,580,777 bytes allocated
==23635==
==23635== LEAK SUMMARY:
==23635== definitely lost: 0 bytes in 0 blocks
==23635== indirectly lost: 0 bytes in 0 blocks
==23635== possibly lost: 2,787 bytes in 45 blocks
==23635== still reachable: 4,384 bytes in 79 blocks
==23635== suppressed: 0 bytes in 0 blocks
==23635== Rerun with --leak-check=full to see details of leaked memory
==23635==
==23635== For counts of detected and suppressed errors, rerun with: -v
==23635== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 2 from 2)
==23632==
==23632== HEAP SUMMARY:
==23632== in use at exit: 310 bytes in 7 blocks
==23632== total heap usage: 12,764 allocs, 12,757 frees, 824,254 bytes allocated
==23632==
==23632== LEAK SUMMARY:
==23632== definitely lost: 0 bytes in 0 blocks
==23632== indirectly lost: 0 bytes in 0 blocks
==23632== possibly lost: 118 bytes in 4 blocks
==23632== still reachable: 192 bytes in 3 blocks
==23632== suppressed: 0 bytes in 0 blocks
==23632== Rerun with --leak-check=full to see details of leaked memory
==23632==
==23632== For counts of detected and suppressed errors, rerun with: -v
==23632== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 2 from 2)
[ FAILED ] TestClientInput.hidden_clients_do_not_receive_pointer_events (6312 ms)

Related branches

Daniel van Vugt (vanvugt) wrote :

Looks like bug 1212518 wasn't really fixed.

Changed in mir:
importance: Undecided → High
Daniel van Vugt (vanvugt) wrote :
Download full text (3.3 KiB)

More detailed logging...

[ RUN ] TestClientInput.hidden_clients_do_not_receive_pointer_events
[II, android-input] Device added: id=0, name='', sources=0x00000101
[II, android-input] Device added: id=1, name='', sources=0x00002002
[II, input] Published motion event (seq_id, event_time) to fd 28: (1, 1379689387423953762)
[II, input] Published motion event (seq_id, event_time) to fd 28: (2, 1379689387423953762)
/home/dan/bzr/mir/trunk/tests/acceptance-tests/test_client_input.cpp:916: Failure
Actual function call count doesn't match EXPECT_CALL(*handler, handle_input(MotionEventWithPosition(2, 2)))...
         Expected: to be called once
           Actual: never called - unsatisfied and active
Google Test trace:
/home/dan/bzr/mir/trunk/tests/mir_test_framework/testing_process_manager.cpp:110: Client
[II, input] Received event finished (seq_id) from fd 28: 1
[II, input] Received event finished (seq_id) from fd 28: 2
[II, android-input] Dropping event because there is no touched window.
==17535==
==17535== HEAP SUMMARY:
==17535== in use at exit: 7,179 bytes in 125 blocks
==17535== total heap usage: 22,590 allocs, 22,465 frees, 1,578,420 bytes allocated
==17535==
==17535== LEAK SUMMARY:
==17535== definitely lost: 0 bytes in 0 blocks
==17535== indirectly lost: 0 bytes in 0 blocks
==17535== possibly lost: 2,787 bytes in 45 blocks
==17535== still reachable: 4,392 bytes in 80 blocks
==17535== suppressed: 0 bytes in 0 blocks
==17535== Rerun with --leak-check=full to see details of leaked memory
==17535==
==17535== For counts of detected and suppressed errors, rerun with: -v
==17535== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 2 from 2)
/home/dan/bzr/mir/trunk/tests/mir_test_framework/testing_process_manager.cpp:127: Failure
Value of: result.succeeded()
  Actual: false
Expected: true
client terminate error=process::Result(child_terminated_normally, failure(1))
==17536==
==17536== HEAP SUMMARY:
==17536== in use at exit: 7,171 bytes in 124 blocks
==17536== total heap usage: 22,586 allocs, 22,462 frees, 1,585,053 bytes allocated
==17536==
==17536== LEAK SUMMARY:
==17536== definitely lost: 0 bytes in 0 blocks
==17536== indirectly lost: 0 bytes in 0 blocks
==17536== possibly lost: 2,787 bytes in 45 blocks
==17536== still reachable: 4,384 bytes in 79 blocks
==17536== suppressed: 0 bytes in 0 blocks
==17536== Rerun with --leak-check=full to see details of leaked memory
==17536==
==17536== For counts of detected and suppressed errors, rerun with: -v
==17536== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 2 from 2)
==17533==
==17533== HEAP SUMMARY:
==17533== in use at exit: 310 bytes in 7 blocks
==17533== total heap usage: 12,958 allocs, 12,951 frees, 833,618 bytes allocated
==17533==
==17533== LEAK SUMMARY:
==17533== definitely lost: 0 bytes in 0 blocks
==17533== indirectly lost: 0 bytes in 0 blocks
==17533== possibly lost: 118 bytes in 4 blocks
==17533== still reachable: 192 bytes in 3 blocks
==17533== suppressed: 0 bytes in 0 blocks
==17533== Rerun with --leak-check=full to see details of leaked memory
==17533==
==17533== For counts of detected an...

Read more...

Changed in mir:
assignee: nobody → Robert Carr (robertcarr)
kevin gunn (kgunn72) wrote :

ended up being jenkins shennanigans.
solution - Francis Ginther is awesome!...he provisioned 2 dedicated build/ci hosts which has solved this problem

Changed in mir:
status: New → Fix Committed
status: Fix Committed → Fix Released
Daniel van Vugt (vanvugt) wrote :

Nothing is fixed. It's happening on my laptop.

Changed in mir:
status: Fix Released → Triaged
assignee: Robert Carr (robertcarr) → nobody
Daniel van Vugt (vanvugt) wrote :

Verified the bug still happens with development-branch r1094.

tags: added: testsfail
tags: added: qa-touch
Robert Carr (robertcarr) on 2013-10-28
Changed in mir:
status: Triaged → In Progress
Changed in mir:
assignee: nobody → Robert Carr (robertcarr)
milestone: none → 0.1.1
summary: - valgrind acceptance-tests FAIL:
+ acceptance-tests FAIL:
TestClientInput.hidden_clients_do_not_receive_pointer_events
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:~mir-team/mir/development-branch at revision None, scheduled for release in mir, milestone Unknown

Changed in mir:
status: In Progress → Fix Committed
Changed in mir:
status: Fix Committed → Fix Released
Changed in mir (Ubuntu):
status: New → Triaged
importance: Undecided → High
Changed in mir (Ubuntu):
status: Triaged → Fix Committed
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package mir - 0.1.1+14.04.20131120-0ubuntu1

---------------
mir (0.1.1+14.04.20131120-0ubuntu1) trusty; urgency=low

  [ Daniel van Vugt ]
  * New upstream release 0.1.1
    - Add unit tests for V/H scroll events.
    - surfaces: avoid publishing some internal headers, tidy up default
      configuration, integrate surfaces report.
    - client: Add mir_connection_drm_set_gbm_device()
    - graphics: avoid publishing some internal headers.
    - Fixed: unity-system-compositor FTBFS on trusty against new Mir
      (libmirserver9) (LP: #1244192)
    - compositor: avoid publishing some internal headers.
    - shell: Add set_lifecycle_state() to the Session interface.
    - frontend: avoid publishing some internal headers
    - logging: avoid publishing some internal headers.
    - Allow specifying the nested server name by passing --name= or setting
      MIR_SERVER_NAME=.
    - graphics,gbm: Inform the EGL platform about the used gbm device when
      using the native GBM platform
    - examples: Restore GL state after initializing buffers, fixing crashes
      observed in render_surfaces (LP: #1234563)
    - Continue refactoring the mir android display classes.
    - shell: Hoist focus control functions needed by unity-mir into
      FocusController interface
    - client: Remove the timeout for detecting server crashes
    - Avoid a race condition that could lead to spurious failures of server
      shutdown tests (LP: #1245336)
    - test_client_input.cpp: Bump reception time-out in client test fixture.
      (LP: #1227683)
    - Ensure StubBufferAllocator returns buffers with the properties requested,
      and not the same old hardcoded constants.
    - Update docs and scripting for trusty.
    - compositor: Make DefaultDisplayBufferCompositorFactory private to the
      compositor component.
    - Ignore warnings clang treats as errors, about unused functions being
      generated from macros in <lttng/tracepoint.h> (LP: #1246590)
    - Add resize() support to BufferBundle. This is the first step and lowest
      level of surface resize support.
    - Clean up constants relating to SwitchingBundle.
    - Fix the armhf chroot setup script to point to the right library, so
      cross compiling can work again (LP: #1246975)
    - shell: avoid publishing some internal headers.
    - input: avoid publishing some internal headers.
  * Bump timeouts used in socket testing. It seems 100ms isn't always
    enough, which leads to spurious test failures (LP: #1252144) (LP:
    #1252144)
  * Fix uninitialized variable causing random drm_auth_magic test
    failures. (LP: #1252144). (LP: #1252144)

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 1165
 -- Ubuntu daily release <email address hidden> Wed, 20 Nov 2013 07:36:15 +0000

Changed in mir (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers