Skip to content

Conversation

@maxritter
Copy link
Owner

This commit adds comprehensive E2E testing infrastructure to verify the install.sh script works correctly after any changes:

  • Add --non-interactive flag to install.sh for CI/CD compatibility

    • Skip all interactive prompts (Python support, settings overwrite)
    • Support INSTALL_PYTHON and OVERWRITE_SETTINGS env vars
    • Add --skip-env flag to skip environment setup
    • Add --help flag with usage information
  • Create E2E test suite (tests/e2e/)

    • test-install.sh: Comprehensive test script with multiple test cases
    • test-install-docker.sh: Docker-based testing for clean environments
    • Tests cover: basic install, Python flag, help, idempotency, invalid args
  • Integrate E2E tests into CI/CD pipelines

    • Update quality.yml: Run E2E tests after QLTY checks for PRs
    • Update release.yml: Run E2E tests before release (after QLTY)
    • E2E tests are now required for PR merge and releases
  • Clean up redundant files

    • Remove scripts/release.sh (now handled by semantic-release)
    • Remove redundant e2e-tests.yml workflow (integrated into quality.yml)

All tests passing successfully (32/32 tests)

This commit adds comprehensive E2E testing infrastructure to verify the install.sh script works correctly after any changes:

- Add --non-interactive flag to install.sh for CI/CD compatibility
  - Skip all interactive prompts (Python support, settings overwrite)
  - Support INSTALL_PYTHON and OVERWRITE_SETTINGS env vars
  - Add --skip-env flag to skip environment setup
  - Add --help flag with usage information

- Create E2E test suite (tests/e2e/)
  - test-install.sh: Comprehensive test script with multiple test cases
  - test-install-docker.sh: Docker-based testing for clean environments
  - Tests cover: basic install, Python flag, help, idempotency, invalid args

- Integrate E2E tests into CI/CD pipelines
  - Update quality.yml: Run E2E tests after QLTY checks for PRs
  - Update release.yml: Run E2E tests before release (after QLTY)
  - E2E tests are now required for PR merge and releases

- Clean up redundant files
  - Remove scripts/release.sh (now handled by semantic-release)
  - Remove redundant e2e-tests.yml workflow (integrated into quality.yml)

All tests passing successfully (32/32 tests)
@maxritter maxritter changed the title Add end-to-end testing for install script Fix: Add end-to-end testing for install script Nov 16, 2025
@maxritter maxritter changed the title Fix: Add end-to-end testing for install script fix: Add end-to-end testing for install script Nov 16, 2025
@maxritter maxritter merged commit 75c6230 into main Nov 16, 2025
4 of 6 checks passed
@maxritter maxritter deleted the claude/add-install-e2e-tests-01TNZ9mTD2u7gsBB5iDSBcfe branch November 16, 2025 20:18
@github-actions
Copy link
Contributor

🎉 This PR is included in version 2.1.10 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants