abderrahim opened a new pull request, #1880:
URL: https://github.com/apache/buildstream/pull/1880

   It was previously using pytest_runtest_setup to skip tests, but this only 
ran for tests that were in the tests/ directory, and not for the sourcetests.
   
   Instead use pytest_collection_modifyitems which runs earlier and has access 
to all collected tests. Since this runs earlier than the tests, we add a skip 
marker to those tests we want to skip.
   
   Comparing the tests that ran before and after this change, we see
   * Before: `1 failed, 1258 passed, 200 skipped, 85 warnings in 116.92s 
(0:01:56)`
   * After: `1258 passed, 201 skipped, 85 warnings in 119.58s (0:01:59)`
   
   So `source_determinism.py::test_deterministic_source_umask[tar]` is indeed 
the only test affected by this.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to