Skip to content

Fixed saucelabs reporting of failed tests

decentral1se requested to merge correct-saucelabs-reporting into master

Created by: birdsarah

Previously, the correct test status was not being passed to saucelabs. Now we use a fixture to get the py.test reported test status instead of trying to set manually during the test. Also improved the test name so that we can see the test name clearly on the saucelabs test reports page.

This turned out to not be that bad, thanks to the lovely documentation here: http://pytest.org/latest/example/simple.html#making-test-result-information-available-in-fixtures

This is definitely better than what we were doing before as its picking up the actual test results rather than our attempt to manually set them.

Also, checkout the nicer Session name: https://saucelabs.com/tests

If you like the pull request, we should fix the test that we've been arbitrarily failing in test_edit_discussions.py in order to work on this.

Merge request reports