Skip to content

Conversation

@DevelopmentCats
Copy link
Contributor

Description

Remove test from cloud-dev template since templates generally have no tests.

Type of Change

  • New module
  • New template
  • Bug fix
  • Feature/enhancement
  • Documentation
  • Other

Template Information

Path: registry/nboyers/templates/cloud-dev

Testing & Validation

  • Tests pass (bun test)
  • Code formatted (bun fmt)
  • Changes tested locally

Related Issues

The test file had multiple issues:
- Located in test/ subdirectory instead of alongside main.tf
- Used function calls in override_data (not allowed)
- Used invalid 'command = validate' (must be plan or apply)

Templates in this repo don't have tests, so removing it entirely.
Copilot AI review requested due to automatic review settings January 5, 2026 21:42
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes the test file from the cloud-dev template to align with the convention that templates in the registry generally do not include tests. The test directory has been completely removed from the template.

  • Removes 87 lines of Terraform test configuration
  • Eliminates the test directory from the cloud-dev template
  • Aligns with registry template conventions

@DevelopmentCats DevelopmentCats merged commit cbb39bd into main Jan 5, 2026
10 checks passed
@DevelopmentCats DevelopmentCats deleted the fix/cloud-dev-test-removal branch January 5, 2026 21:59
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