[regression] GLRenderer* unit tests have recently become noisy
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Mir |
Fix Released
|
Low
|
Mir development team | ||
mir (Ubuntu) |
Fix Released
|
Low
|
Unassigned |
Bug Description
[regression] unit test: GLRendererSetup
[==========] Running 6 tests from 2 test cases.
[----------] Global test environment set-up.
[----------] 3 tests from GLRendererSetup
[ RUN ] GLRendererSetup
GMOCK WARNING:
Uninteresting mock function call - returning directly.
Function call: glDeleteShader(1)
Stack trace:
[ OK ] GLRendererSetup
[ RUN ] GLRendererSetup
GMOCK WARNING:
Uninteresting mock function call - returning directly.
Function call: glDeleteShader(2)
Stack trace:
GMOCK WARNING:
Uninteresting mock function call - returning directly.
Function call: glDeleteShader(1)
Stack trace:
[ OK ] GLRendererSetup
[ RUN ] GLRendererSetup
GMOCK WARNING:
Uninteresting mock function call - returning directly.
Function call: glDeleteProgram(1)
Stack trace:
GMOCK WARNING:
Uninteresting mock function call - returning directly.
Function call: glDeleteShader(2)
Stack trace:
GMOCK WARNING:
Uninteresting mock function call - returning directly.
Function call: glDeleteShader(1)
Stack trace:
[ OK ] GLRendererSetup
[----------] 3 tests from GLRendererSetup
Related branches
- PS Jenkins bot (community): Needs Fixing (continuous-integration)
- Alan Griffiths: Approve
-
Diff: 39 lines (+8/-0)1 file modifiedtests/unit-tests/compositor/test_gl_renderer.cpp (+8/-0)
- Alberto Aguirre (community): Approve
- Alexandros Frantzis (community): Approve
- Robert Carr (community): Needs Information
- PS Jenkins bot (community): Approve (continuous-integration)
-
Diff: 1061 lines (+439/-216)20 files modifiedexamples/demo-shell/demo_renderer.cpp (+4/-2)
examples/demo-shell/demo_renderer.h (+1/-1)
examples/demo-shell/demo_shell.cpp (+9/-2)
include/platform/mir/graphics/gl_program_factory.h (+50/-0)
include/server/mir/compositor/gl_renderer.h (+6/-3)
include/server/mir/default_server_configuration.h (+3/-0)
include/server/mir/graphics/gl_program.h (+4/-4)
src/server/compositor/CMakeLists.txt (+0/-1)
src/server/compositor/default_configuration.cpp (+2/-2)
src/server/compositor/gl_renderer.cpp (+16/-14)
src/server/compositor/gl_renderer_factory.cpp (+9/-4)
src/server/compositor/gl_renderer_factory.h (+6/-10)
src/server/graphics/CMakeLists.txt (+2/-0)
src/server/graphics/default_configuration.cpp (+11/-0)
src/server/graphics/gl_program.cpp (+8/-8)
src/server/graphics/program_factory.cpp (+32/-0)
src/server/graphics/program_factory.h (+47/-0)
tests/unit-tests/compositor/test_gl_renderer.cpp (+17/-165)
tests/unit-tests/graphics/CMakeLists.txt (+1/-0)
tests/unit-tests/graphics/test_program_factory.cpp (+211/-0)
Changed in mir: | |
assignee: | nobody → Daniel van Vugt (vanvugt) |
summary: |
- [regression] unit test: GLRendererSetupProcess has recently become noisy + [regression] GLRenderer unit tests have recently become noisy |
summary: |
- [regression] GLRenderer unit tests have recently become noisy + [regression] GLRendererSetupProcess unit tests have recently become + noisy |
Changed in mir: | |
status: | Fix Committed → Fix Released |
Changed in mir (Ubuntu): | |
assignee: | nobody → Daniel van Vugt (vanvugt) |
assignee: | Daniel van Vugt (vanvugt) → nobody |
importance: | Undecided → Low |
status: | New → Triaged |
The existing description is more accurate as "GLRendererSetu pProcess" did not exist until the regression occurred.