Skip to content

CI: Add an integration test that runs the entire backporter script#30

Open
DilumAluthge wants to merge 2 commits into
KristofferC:masterfrom
DilumAluthge-forks:dpa/integration-test-ci
Open

CI: Add an integration test that runs the entire backporter script#30
DilumAluthge wants to merge 2 commits into
KristofferC:masterfrom
DilumAluthge-forks:dpa/integration-test-ci

Conversation

@DilumAluthge

@DilumAluthge DilumAluthge commented Jan 28, 2026

Copy link
Copy Markdown
Contributor

Obviously it's better to have unit tests whenever possible, but at least this integration test can serve as a basic sanity check that a PR won't break the script.

@codecov-commenter

codecov-commenter commented Jan 28, 2026

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 0% with 21 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (master@7e221b2). Learn more about missing BASE report.

Files with missing lines Patch % Lines
test/integration.jl 0.00% 21 Missing ⚠️
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##             master      #30   +/-   ##
=========================================
  Coverage          ?   24.15%           
=========================================
  Files             ?        7           
  Lines             ?      832           
  Branches          ?        0           
=========================================
  Hits              ?      201           
  Misses            ?      631           
  Partials          ?        0           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@DilumAluthge
DilumAluthge marked this pull request as ready for review January 28, 2026 03:31
@DilumAluthge
DilumAluthge force-pushed the dpa/integration-test-ci branch from 77d486b to e6c41a7 Compare February 2, 2026 03:57
@DilumAluthge
DilumAluthge force-pushed the dpa/integration-test-ci branch from e6c41a7 to fb04c4b Compare February 2, 2026 04:02
@DilumAluthge

Copy link
Copy Markdown
Contributor Author

@IanButterworth What do you think of this one? It might be a nice way to catch breakages in PRs.

@DilumAluthge

Copy link
Copy Markdown
Contributor Author

Oh, we would need to decide whether we want to:

  1. Only upload Codecov info for the unit tests
  2. Upload Codecov info for both the unit tests and integration tests. Obviously, this will make the Codecov number look higher, but it might be misleading, because a good portion of that higher coverage number would be from the integration tests, which aren't really testing much, just that we can run the entire script without error.

Comment thread .github/workflows/CI.yml Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants