Skip to content

test(ci): cover missing cosign installer guard - #26

Merged
Maxerns merged 1 commit into
getsotto:mainfrom
LouisDeconinck:test/missing-cosign
Sep 10, 2026
Merged

Maxerns merged 1 commit into
getsotto:mainfrom
LouisDeconinck:test/missing-cosign

Conversation

@LouisDeconinck

Copy link
Copy Markdown
Contributor

Fixes #24.

Add focused offline CI coverage for the installers' existing missing-cosign precondition on Ubuntu and Windows. The tests isolate executable lookup, intercept download attempts, verify the exact error, and assert that installation and GitHub output files remain untouched.

Validation:

  • Linux missing-cosign scenario passed locally.
  • YAML parsing passed locally.
  • git diff --check passed.
  • PowerShell and actionlint are unavailable in this environment; the Windows scenario will run in CI.

AI assistance: Codex assisted with implementation and test drafting; the patch was reviewed before submission.

Signed-off-by: Louis Deconinck <louis.dck@gmail.com>
Assisted-by: Codex

@Maxerns Maxerns left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @LouisDeconinck verified both legs against the installers ill approve the ci now and merge when its green

@Maxerns
Maxerns merged commit 5b58507 into getsotto:main Sep 10, 2026
30 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.

CI: test the installer error when cosign is unavailable

2 participants