Skip to content

Conversation

@westonplatter
Copy link
Member

@westonplatter westonplatter commented Nov 7, 2025

Pre-Mege Todos

Changes

Adds BATS test infrastructure to Taskit and renames the terraform library to tf with support for switching between OpenTofu and Terraform.

Test Infrastructure

  • GitHub Actions Workflow: Automated testing on PRs and main branch pushes (.github/workflows/test.yaml)
  • BATS Test Suite: Comprehensive tests for terraform:plan task (tests/terraform_plan.bats)
  • Documentation: Concise testing guide matching main README tone (tests/README.md)

Gowiem
Gowiem previously approved these changes Nov 7, 2025
Copy link
Member

@Gowiem Gowiem left a comment

Choose a reason for hiding this comment

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

@westonplatter I hate and love it at the same time. Let's :shipit: 😎

@westonplatter
Copy link
Member Author

westonplatter commented Nov 7, 2025

@westonplatter I hate and love it at the same time. Let's :shipit: 😎

I feel about the same. Asking a couple of hard core linux friends if there's a better way to get to the same end goal.

@westonplatter westonplatter requested a review from Gowiem November 7, 2025 19:27
@westonplatter
Copy link
Member Author

Thanks for the reviews @Gowiem and @gberenice.

While bats gives a higher degree of confidence, we heard from others that it was hard to scale it and other shell testing to handle medium to high complexity. Closing this while take their advice and look further into ways to test taskfiles.

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.

4 participants