atmos
atmos copied to clipboard
refactor tests to avoid atmos binary
what
- run test cases via
cmd.Execute()instead of compiled binary - update terraform CLI tests to use helper
- fix golangci configuration for lint
- document in-process test execution
- clarify lint instructions in
AGENTS.md - disable timestamp in logs for in-process test helper
why
- executing the compiled binary in tests caused flakiness and reliance on external artifacts
- running commands in-process provides faster, more reliable testing
- lint required
--new=falsesince the container isn't a full git repo - log timestamps broke golden snapshots
references
- n/a
https://chatgpt.com/codex/tasks/task_b_68597d71f1c48332971e1760f706d4b8
[!WARNING]
This PR exceeds the recommended limit of 1,000 lines.
Large PRs are difficult to review and may be rejected due to their size.
Please verify that this PR does not address multiple issues. Consider refactoring it into smaller, more focused PRs to facilitate a smoother review process.
[!IMPORTANT]
Cloud Posse Engineering Team Review Required
This pull request modifies files that require Cloud Posse's review. Please be patient, and a core maintainer will review your changes.
To expedite this process, reach out to us on Slack in the
#pr-reviewschannel.
💥 This pull request now has conflicts. Could you fix it @osterman? 🙏
superseded by #1534