Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feat: Create CLI e2e tests with assertions and CI integration #125

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
t0mmylam merged 4 commits into from
Dec 17, 2025
Merged

feat: Create CLI e2e tests with assertions and CI integration #125

t0mmylam merged 4 commits into from
Dec 17, 2025

Conversation

@t0mmylam
Copy link
Collaborator

@t0mmylam t0mmylam commented Dec 10, 2025

Summary

Wrote CLI e2e test suites with proper state assertions and added CI integration.

Changes

Tests

  • 3 node command tests into 1 comprehensive test with 5 test steps
  • Unified package tests with status, logs, and rerun functionality

Proper Assertions

  • Added chainsaw assertions for state verification
  • Node tests: Verify ignore/unignore labels and reset annotation removal
  • Package tests: Verify rerun sets packages to "in_progress" state

Infrastructure Improvements

  • Added CLI e2e tests to GitHub Actions CI pipeline (runs on K8s 1.34.0)
  • CLI tests run automatically on PRs affecting operator code

Test Coverage

  • Node commands: list, status, ignore, unignore, reset
  • Package commands: status, logs, rerun --stage apply
  • All state-changing commands have proper assertions
  • Comprehensive coverage of CLI functionality

@t0mmylam t0mmylam changed the base branch from main to node-ops December 16, 2025 00:17
@t0mmylam
feat(cli): Add lifecycle management commands (pause, resume, disable,…
ac9018f 
… enable) (#127)

* feat(cli): Add lifecycle management commands (pause, resume, disable, enable)

* update k8s tests

* change to patch
Base automatically changed from node-ops to package-ops December 16, 2025 23:15
@t0mmylam t0mmylam merged commit 18716a6 into package-ops Dec 17, 2025
1 check passed
@t0mmylam t0mmylam deleted the cli-tests branch December 17, 2025 00:19
t0mmylam added a commit that referenced this pull request Dec 17, 2025
@t0mmylam
feat: Create CLI e2e tests with assertions and CI integration (#125)
eab3a15 
* feat(cli): Add node management commands and ignore label support

* feat: Consolidate CLI e2e tests with proper assertions and CI integration

* feat(cli): Add lifecycle management commands (pause, resume, disable, enable) (#127)

* feat(cli): Add lifecycle management commands (pause, resume, disable, enable)

* update k8s tests

* change to patch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lockwobr lockwobr Awaiting requested review from lockwobr

@mskalka mskalka Awaiting requested review from mskalka

@ayuskauskas ayuskauskas Awaiting requested review from ayuskauskas

@riceriley59 riceriley59 Awaiting requested review from riceriley59

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@t0mmylam