Skip to content

Separate integration tests into standalone workflow with build tag#166

Merged
fredbi merged 4 commits intomasterfrom
copilot/add-separate-test-workflow
Mar 6, 2026
Merged

Separate integration tests into standalone workflow with build tag#166
fredbi merged 4 commits intomasterfrom
copilot/add-separate-test-workflow

Conversation

Copy link
Contributor

Copilot AI commented Mar 6, 2026

  • Add //go:build testintegration build tag to all test files in internal/testintegration/ (3 files: doc_test.go, helpers_spec_test.go, spec_test.go)
  • Create new standalone workflow .github/workflows/integration-test.yml with:
    • Triggers: push to master + pull_request (same as go-test.yml)
    • Simplified matrix: linux + windows, stable Go only (2 jobs instead of 6)
    • No -race flag
    • Build tags: testintegration
    • Coverage upload via artifacts (no test report)
    • No fuzz test
  • Add doc.go to internal/testintegration/ to satisfy linter (no Go source without build tag)
  • Validated: without tag, no integration tests run; with tag, all 6 tests found
  • Validated: root module tests unaffected
  • Code review passed (no comments)
  • Security scan passed (0 alerts)

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Copilot AI and others added 2 commits March 6, 2026 21:55
- Add //go:build testintegration to all test files in internal/testintegration/
- Create .github/workflows/integration-test.yml as standalone workflow
  - Matrix: linux + windows, stable Go only (2 jobs)
  - No -race, no fuzz, no test report
  - Coverage upload via artifacts

Co-authored-by: fredbi <14262513+fredbi@users.noreply.github.com>
Co-authored-by: fredbi <14262513+fredbi@users.noreply.github.com>
Copilot AI changed the title [WIP] Add separate workflow for testintegration Separate integration tests into standalone workflow with build tag Mar 6, 2026
@fredbi fredbi marked this pull request as ready for review March 6, 2026 21:58
@fredbi
Copy link
Member

fredbi commented Mar 6, 2026

@copilot now with the tags the linter complains there is no go code in this module.

Please add an empty doc.go with just a package level comment.

The build tag on all test files made the linter complain about
no Go code in the module. Adding a doc.go with just a package
declaration fixes this.

Co-authored-by: fredbi <14262513+fredbi@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Mar 6, 2026

@copilot now with the tags the linter complains there is no go code in this module.

Please add an empty doc.go with just a package level comment.

Added internal/testintegration/doc.go with a package-level comment in 82a4ec3. This provides a non-test Go source file so the linter no longer complains about an empty module.

@codecov
Copy link

codecov bot commented Mar 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.49%. Comparing base (18e543f) to head (82a4ec3).
⚠️ Report is 2 commits behind head on master.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #166      +/-   ##
==========================================
- Coverage   91.54%   91.49%   -0.05%     
==========================================
  Files          17       17              
  Lines        2034     2034              
==========================================
- Hits         1862     1861       -1     
- Misses        114      115       +1     
  Partials       58       58              

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

@fredbi fredbi merged commit c32fa38 into master Mar 6, 2026
23 checks passed
@fredbi fredbi deleted the copilot/add-separate-test-workflow branch March 6, 2026 22:13
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