Mir

Merge lp:~alan-griffiths/mir/revert-workaround-1239955 into lp:mir

Proposed by Alan Griffiths
Status: Superseded
Proposed branch: lp:~alan-griffiths/mir/revert-workaround-1239955
Merge into: lp:mir
Prerequisite: lp:~alan-griffiths/mir/workaround-1239955
Diff against target: 13 lines (+1/-2)
1 file modified
tests/integration-tests/graphics/android/test_display_integration.cpp (+1/-2)
To merge this branch: bzr merge lp:~alan-griffiths/mir/revert-workaround-1239955
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Needs Fixing
Mir development team Pending
Review via email: mp+201042@code.launchpad.net

This proposal has been superseded by a proposal from 2014-02-04.

Commit message

tests: RE-enable test disabled to workaround lp:1239955

Description of the change

tests: RE-enable test disabled to workaround lp:1239955

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Kevin DuBois (kdub) wrote :

This is just to bounce off CI at the moment? (or to remind us to fix?)

Revision history for this message
Alan Griffiths (alan-griffiths) wrote :

> This is just to bounce off CI at the moment? (or to remind us to fix?)

Yes (both)

Revision history for this message
Kevin DuBois (kdub) wrote :

I'll probably merge these two tests into one test that checks the default rendering mode for the platform. It seems neither the nexus 10 nor the nexus 4 support the fb fallback anymore (and shouldn't be checked) in the android 4.4.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'tests/integration-tests/graphics/android/test_display_integration.cpp'
2--- tests/integration-tests/graphics/android/test_display_integration.cpp 2014-01-09 16:47:21 +0000
3+++ tests/integration-tests/graphics/android/test_display_integration.cpp 2013-12-18 02:19:19 +0000
4@@ -68,8 +68,7 @@
5 std::shared_ptr<mga::ResourceFactory> AndroidGPUDisplay::display_resource_factory;
6 }
7
8-// disabled to unblock CI - see lp:1239955
9-TEST_F(AndroidGPUDisplay, DISABLED_gpu_display_ok_with_gles)
10+TEST_F(AndroidGPUDisplay, gpu_display_ok_with_gles)
11 {
12 auto should_use_fb_fallback = true;
13 auto buffer_initializer = std::make_shared<mg::NullBufferInitializer>();

Subscribers

People subscribed via source and target branches