Skip to content

ci(documentation): Update metadata-ingestion:testScripts to add coverage and standardized test reporting#16239

Open
devashish2203 wants to merge 3 commits intomasterfrom
dc/test-summary-metadata-ingestion
Open

ci(documentation): Update metadata-ingestion:testScripts to add coverage and standardized test reporting#16239
devashish2203 wants to merge 3 commits intomasterfrom
dc/test-summary-metadata-ingestion

Conversation

@devashish2203
Copy link
Contributor

@devashish2203 devashish2203 commented Feb 17, 2026

Summary

  • Update :metadata-ingestion:testScripts task to produce coverage and junit output.
  • Migrate documentation workflows to the report-test-results composable action for consistent test result reporting and job summaries
  • Consolidate all pytest JUnit XML output into a test-results/ directory across all test tasks (testQuick, testSingle, testIntegrationBatch*, testScripts)
  • Move shared pytest args (--durations, -vv, --continue-on-collection-errors, --junit-xml) from build.gradle into setup.cfg and a new scripts/pytest.ini
  • Fix coverage filename separator in python-coverage.gradle (coverage-<project><name>coverage-<project>-<name>)
  • Refactor clean task into a proper cleanGeneratedFiles Delete task, including cleanup for test-results/, scripts/.coverage, and scripts/test-results

Test plan

  • Verify metadata-ingestion CI matrix jobs produce JUnit XML under metadata-ingestion/test-results/ and publish a test summary
  • Verify documentation CI job produces JUnit XML under metadata-ingestion/scripts/test-results/ and publishes a test summary
  • Confirm coverage reports still upload correctly to Codecov with the fixed filename separator

🤖 Generated with Claude Code

@github-actions github-actions bot added ingestion PR or Issue related to the ingestion of metadata devops PR or Issue related to DataHub backend & deployment labels Feb 17, 2026
@devashish2203 devashish2203 self-assigned this Feb 17, 2026
@datahub-cyborg datahub-cyborg bot added the needs-review Label for PRs that need review from a maintainer. label Feb 17, 2026
@devashish2203 devashish2203 force-pushed the dc/test-summary-metadata-ingestion branch from 49266be to 3526792 Compare February 17, 2026 18:48
@devashish2203 devashish2203 changed the title ci(metadata-ingestion): add test reporting and coverage for metadata-ingestion ci(metadata-ingestion): Update metadata-ingestion:testScripts to add coverage and standardized test reporting Feb 17, 2026
@codecov
Copy link

codecov bot commented Feb 17, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@devashish2203 devashish2203 changed the title ci(metadata-ingestion): Update metadata-ingestion:testScripts to add coverage and standardized test reporting ci(documentation): Update metadata-ingestion:testScripts to add coverage and standardized test reporting Feb 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

devops PR or Issue related to DataHub backend & deployment ingestion PR or Issue related to the ingestion of metadata needs-review Label for PRs that need review from a maintainer.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant