SG-38126 Update CI - Test results and code coverage#1082
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #1082 +/- ##
==========================================
- Coverage 79.79% 79.61% -0.18%
==========================================
Files 198 198
Lines 20776 20623 -153
==========================================
- Hits 16579 16420 -159
- Misses 4197 4203 +6
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
This PR aligns the repository with a new CI testing and code coverage strategy, updating the test execution process to use pytest with unique test results and coverage files, enabling parallel test execution with proper result combination.
Changes:
- Updated integration test runner to use pytest with unique test result and coverage filenames
- Removed redundant environment variable assignments for Shotgun credentials
- Applied Black formatting updates (removed unnecessary tuple unpacking parentheses and blank lines)
Reviewed changes
Copilot reviewed 12 out of 12 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| tests/integration_tests/run_integration_tests.py | Implemented pytest-based test execution with UUID-based unique filenames for test results and coverage |
| setup/tank_api_proxy/tank/init.py | Removed extra blank line (Black formatting) |
| setup/tank_api_proxy/sgtk/init.py | Removed extra blank line (Black formatting) |
| scripts/tank_cmd.py | Removed unnecessary tuple unpacking parentheses (Black formatting) |
| hooks/default_storage_root.py | Added blank line after docstring (Black formatting) |
| hooks/context_additional_entities.py | Added blank line after docstring (Black formatting) |
| developer/utils/authentication.py | Removed extra blank line (Black formatting) |
| developer/populate_bundle_cache.py | Removed unnecessary tuple unpacking parentheses and condensed format chain (Black formatting) |
| developer/build_plugin.py | Removed unnecessary tuple unpacking parentheses and condensed format chain (Black formatting) |
| developer/bake_config.py | Changed to f-string, but with incomplete template variable substitution |
| azure-pipelines.yml | Updated tk-ci-tools reference to feature branch |
| .pre-commit-config.yaml | Updated pre-commit-hooks and black versions |
Comments suppressed due to low confidence (1)
developer/bake_config.py:1
- The f-string at line 214 has incorrect syntax. The 'f' prefix should be outside the string content. The variable
automated_setup_documentationis treated as a literal string "f{automated_setup_documentation}" instead of being interpolated. Additionally, the previous.format()chain was removed but the f-string conversion is incomplete. Either use proper f-string syntax with the variable inside the f-string template (line 194), or revert to the original.format()approach.
# Copyright (c) 2018 Shotgun Software Inc.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Align new CI test results and code coverage strategy