autopilot ignores class-level conditional skip() decorator

[Olivier Tilloy]

The webbrowser_app test suite has a few tests that are conditionally skipped depending on the platform, such as:

    @unittest.skipIf(model() == "Desktop", "on devices only")
    def test_swipe_partway_switches_tabs(self):
        …

This works well for skipping one specific test method, but it doesn’t seem to work for skipping an entire TestCase. We’re adding support for keyboard shortcuts in the browser, which obviously makes sense only on platforms with a physical keyboard, so we would like to skip the entire test case when not on desktop, but it seems the decorator is ignored by autopilot, and it tries to run all the tests:

    @unittest.skipIf(model() != "Desktop", "on desktop only")
    class TestKeyboard(PrepopulatedDatabaseTestCaseBase):
        …

[Brendan Donegan (brendan-donegan)]

This should probably work, but have you tried using testtools.skipIf? This seems to work for me

[Olivier Tilloy (osomon)]

testtools.skipIf() seems to work indeed. Why would unittest.skipIf() not work though?

[Christopher Lee (veebers)]

Confirmed that unittest.skipIf doesn't skip a whole class (testtools does though). Have marked as medium.