Code review comment for lp:~ted/indicators-client/tests

Revision history for this message
Ɓukasz Zemczak (sil2100) wrote :

hm, I see some issues here.

First, let's not remove the usr/lib/python*/dist-packages/chewie_client/* and indicators-client-autopilot.install. These are autopilot integration tests, they're different and I guess they shouldn't fail.

Second, hm, would it be possible to just comment out the test that is failing? i.e. commenting out declare_test(dynamicmenutest) in the CMakeLists.txt ? The other tests are fine, no need to kill them all I think ;)

review: Needs Fixing

« Back to merge proposal