Skip to content

test: remove brittle CI configuration and script tests - #11185

Merged
vicancy merged 1 commit into
dotnet:mainfrom
vicancy:codex/relax-ci-workflow-assertions
Sep 25, 2026
Merged

vicancy merged 1 commit into
dotnet:mainfrom
vicancy:codex/relax-ci-workflow-assertions

Conversation

@vicancy

@vicancy vicancy commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

Routine workflow edits, including the Action upgrade in #11182, fail compatibility tests that lock configuration and implementation details. Remove assertions about workflow/documentation text, command ordering, request counts, GitHub output delimiters, and tests that extract PowerShell statements by internal variable names or control-flow structure.

Retain report validation, archive/provenance, and CLI behavior tests. Log export and HTTP selection tests now use sample log paths instead of extracting script assignments. Actual package installation and distribution verification remain in CI.

Validation: node --test test/compatibility/report.test.mjs — all 43 remaining tests passed.

@vicancy
vicancy force-pushed the codex/relax-ci-workflow-assertions branch from 9f6ab34 to 8609756 Compare September 24, 2026 23:48
@vicancy vicancy changed the title test: decouple CI checks from action versions and step names test: remove brittle workflow source assertions Sep 24, 2026
@vicancy
vicancy force-pushed the codex/relax-ci-workflow-assertions branch from 8609756 to 8e3828f Compare September 24, 2026 23:55
@vicancy vicancy changed the title test: remove brittle workflow source assertions test: remove brittle CI configuration and script tests Sep 24, 2026
@vicancy
vicancy merged commit 220e977 into dotnet:main Sep 25, 2026
19 checks passed
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.

1 participant