superset icon indicating copy to clipboard operation
superset copied to clipboard

Update python_tests.sh

Open PasunuriSrinidhi opened this issue 2 years ago • 1 comments

Added --cov-report=html flag to generate an HTML report of code coverage. and --numprocesses=auto flag to run tests in parallel, using all available CPU cores. and --random-order flag to randomize the order of tests.

SUMMARY

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

  • [ ] Has associated issue:
  • [ ] Required feature flags:
  • [ ] Changes UI
  • [ ] Includes DB Migration (follow approval process in SIP-59)
    • [ ] Migration is atomic, supports rollback & is backwards-compatible
    • [ ] Confirm DB migration upgrade and downgrade tested
    • [ ] Runtime estimates and downtime expectations provided
  • [ ] Introduces new feature or API
  • [ ] Removes existing feature or API

PasunuriSrinidhi avatar Apr 12 '23 17:04 PasunuriSrinidhi

Closing/reopening this PR see how this does with CI. What format was the report in before if not HTML? I'm just wondering if there's some other automation that runs off the result.

As for the randomizing of test order, that sounds like it might wreak havoc. If we had perfect tests, that seems great, but I'm not sure this'll work out well. Adding reviewers who might know better than I do about the pros/cons.

rusackas avatar Feb 09 '24 20:02 rusackas