Skip to content

Refactor terragrunt module and add dedicated test pipeline#1569

Merged
james00012 merged 2 commits intomainfrom
refactor/terragrunt-module-improvements
Jul 9, 2025
Merged

Refactor terragrunt module and add dedicated test pipeline#1569
james00012 merged 2 commits intomainfrom
refactor/terragrunt-module-improvements

Conversation

@james00012
Copy link
Copy Markdown
Contributor

Summary

  • Simplified Options vs ExtraArgs API pattern for better clarity
  • Added dedicated terragrunt test pipeline in CircleCI
  • Consolidated duplicate code and improved documentation

Key Changes

  • API Simplification: Use single ExtraArgs []string instead of per-command fields
  • Clear Separation: Options for test framework config, ExtraArgs for terragrunt command-line args
  • Dedicated Pipeline: Added terragrunt_test job in CircleCI for isolated terragrunt testing
  • Code Cleanup: Consolidated duplicate functions, added validation, extracted constants

Test Plan

  • All existing tests pass
  • New simplified API tested in all test files
  • CircleCI pipeline includes dedicated terragrunt test job

- Simplify Options vs ExtraArgs pattern for clearer API
- Remove terragrunt command flags from Options struct
- Use single ExtraArgs []string field instead of per-command fields
- Add comprehensive documentation and examples
- Consolidate duplicate command execution functions
- Add input validation for required fields
- Extract constants and remove magic strings
- Add dedicated terragrunt_test job in CircleCI pipeline
- Exclude terragrunt tests from main test jobs to avoid duplication
@james00012 james00012 self-assigned this Jul 7, 2025
@james00012 james00012 merged commit d88e37f into main Jul 9, 2025
2 of 3 checks passed
@james00012 james00012 deleted the refactor/terragrunt-module-improvements branch July 9, 2025 21:09
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.

2 participants