Spurious Failure in ClientLibrary.highly_threaded_client
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Mir |
Fix Released
|
High
|
Alberto Aguirre | ||
| mir (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
Bug Description
I noticed ClientLibrary.
Note: Google Test filter = ClientLibrary.
[==========] Running 1 test from 1 test case.
[----------] Global test environment set-up.
[----------] 1 test from ClientLibrary
[ RUN ] ClientLibrary.
/home/kdub/
Value of: mir_surface_
Expected: is equal to 7
Actual: 2 (of type MirSurfaceState)
[ FAILED ] ClientLibrary.
[----------] 1 test from ClientLibrary (25 ms total)
[----------] Global test environment tear-down
[==========] 1 test from 1 test case ran. (25 ms total)
[ PASSED ] 0 tests.
[ FAILED ] 1 test, listed below:
[ FAILED ] ClientLibrary.
1 FAILED TEST
Related branches
- PS Jenkins bot (community): Needs Fixing (continuous-integration)
- Alberto Aguirre (community): Disapprove
-
Diff: 37 lines (+19/-0)2 files modifiedsrc/server/shell/canonical_window_manager.cpp (+2/-0)
tests/acceptance-tests/test_surface_modifications.cpp (+17/-0)
- PS Jenkins bot (community): Approve (continuous-integration)
- Alan Griffiths: Approve
- Alexandros Frantzis (community): Approve
-
Diff: 69 lines (+3/-12)5 files modifiedexamples/server_example_canonical_window_manager.cpp (+1/-3)
src/server/scene/basic_surface.cpp (+0/-2)
src/server/shell/canonical_window_manager.cpp (+1/-3)
tests/acceptance-tests/test_client_library.cpp (+1/-1)
tests/unit-tests/scene/test_basic_surface.cpp (+0/-3)
| Changed in mir: | |
| importance: | Undecided → High |
| Daniel van Vugt (vanvugt) wrote | #1 |
| Changed in mir: | |
| assignee: | nobody → Alexandros Frantzis (afrantzis) |
| Changed in mir: | |
| status: | New → Triaged |
| assignee: | Alexandros Frantzis (afrantzis) → Alberto Aguirre (albaguirre) |
| PS Jenkins bot (ps-jenkins) wrote | #2 |
| Changed in mir: | |
| status: | Triaged → Fix Committed |
| Launchpad Janitor (janitor) wrote | #3 |
| Changed in mir (Ubuntu): | |
| status: | New → Fix Released |
| Changed in mir: | |
| status: | Fix Committed → Fix Released |
That test is a few years old so I thought it was solid. Did we break MirWaitHandle (or surface state getting)?