chore: Replace myapp with mock Component in Tests (#1213)

* replacing myapp with mock

* fix tests

* vendoring configuration for mock

* vendoring configuration for mock

* ignore vendored files

* fixed atmos_vendor_pull

* fix acceptance tests

* fix acceptance tests

* fix acceptance tests

* fix acceptance tests

* fixing tests

* fixing tests

* fixing tests

* fixing tests

* fix acceptance tests

* fix acceptance tests

* revert examples and correct test-cases to support

* fixing tests

* fixed test

* reset examples/quick-start-advanced/Dockerfile

---------

Co-authored-by: Erik Osterman (CEO @ Cloud Posse) <[email protected]>

Author: milldr

.gitignore

@@ -39,9 +39,10 @@
 **/cache.*.txt
 
 # Ignore components vendored during tests
-tests/fixtures/scenarios/vendor/components/**
 examples/demo-vendoring/components/**
+tests/fixtures/scenarios/vendor-globs/components/**
+tests/fixtures/scenarios/vendor/components/**
 
 # Coverage files
 coverage.out
-coverage.html
+coverage.html

internal/exec/vendor_utils_test.go

@@ -129,25 +129,16 @@ func TestExecuteVendorPull(t *testing.T) {
 		"./components/terraform/github/stargazers/main/versions.tf",
 		"./components/terraform/github/stargazers/main/README.md",
 		"./components/terraform/test-components/main/main.tf",
-		"./components/terraform/test-components/main/outputs.tf",
 		"./components/terraform/test-components/main/README.md",
-		"./components/terraform/test-components/main/variables.tf",
-		"./components/terraform/test-components/main/versions.tf",
 		"./components/terraform/weather/main/main.tf",
 		"./components/terraform/weather/main/outputs.tf",
 		"./components/terraform/weather/main/providers.tf",
 		"./components/terraform/weather/main/variables.tf",
 		"./components/terraform/weather/main/versions.tf",
 		"./components/terraform/weather/main/README.md",
-		"./components/terraform/myapp2/outputs.tf",
 		"./components/terraform/myapp2/main.tf",
-		"./components/terraform/myapp2/variables.tf",
-		"./components/terraform/myapp2/versions.tf",
 		"./components/terraform/myapp2/README.md",
-		"./components/terraform/myapp1/outputs.tf",
 		"./components/terraform/myapp1/main.tf",
-		"./components/terraform/myapp1/variables.tf",
-		"./components/terraform/myapp1/versions.tf",
 		"./components/terraform/myapp1/README.md",
 	}
 	success, file := verifyFileExists(t, files)

pkg/stack/stack_processor_test.go

@@ -243,8 +243,8 @@ func TestStackProcessorRelativePaths(t *testing.T) {
 	components := mapConfig1["components"].(map[string]any)
 	terraformComponents := components["terraform"].(map[string]any)
 
-	myappComponent := terraformComponents["myapp"].(map[string]any)
-	assert.NotNil(t, myappComponent)
+	mockComponent := terraformComponents["mock"].(map[string]any)
+	assert.NotNil(t, mockComponent)
 }
 
 func TestProcessYAMLConfigFile(t *testing.T) {

pkg/vender/vendor_config_test.go

@@ -28,7 +28,7 @@ func TestVendorConfigScenarios(t *testing.T) {
 	atmosConfig.Logs.Level = "Trace"
 
 	// Setup test component directory
-	componentPath := filepath.Join(testDir, "components", "terraform", "myapp")
+	componentPath := filepath.Join(testDir, "components", "terraform", "mock")
 	err := os.MkdirAll(componentPath, 0o755)
 	assert.Nil(t, err)
 
@@ -41,7 +41,7 @@ metadata:
   name: test-vendor-config
 spec:
   sources:
-    - component: myapp
+    - component: mock
       source: github.com/cloudposse/terraform-null-label.git//exports?ref={{.Version}}
       version: 0.25.0
       included_paths:
@@ -60,12 +60,12 @@ spec:
 		// Verify the component exists in vendor config
 		var found bool
 		for _, source := range vendorConfig.Spec.Sources {
-			if source.Component == "myapp" {
+			if source.Component == "mock" {
 				found = true
 				break
 			}
 		}
-		assert.True(t, found, "Component 'myapp' should be defined in vendor.yaml")
+		assert.True(t, found, "Component 'mock' should be defined in vendor.yaml")
 
 		// Clean up
 		err = os.Remove(vendorYamlPath)
@@ -78,7 +78,7 @@ spec:
 		componentYaml := `apiVersion: atmos/v1
 kind: ComponentVendorConfig
 metadata:
-  name: myapp-vendor-config
+  name: mock-vendor-config
 spec:
   source:
     uri: github.com/cloudposse/terraform-null-label.git//exports?ref={{.Version}}
@@ -89,7 +89,7 @@ spec:
 		assert.Nil(t, err)
 
 		// Test component vendoring
-		componentConfig, compPath, err := e.ReadAndProcessComponentVendorConfigFile(&atmosConfig, "myapp", "terraform")
+		componentConfig, compPath, err := e.ReadAndProcessComponentVendorConfigFile(&atmosConfig, "mock", "terraform")
 		assert.Nil(t, err)
 		assert.NotNil(t, componentConfig)
 		assert.Equal(t, componentPath, compPath)
@@ -108,7 +108,7 @@ spec:
 		assert.False(t, exists)
 
 		// Test component vendoring
-		_, _, err = e.ReadAndProcessComponentVendorConfigFile(&atmosConfig, "myapp", "terraform")
+		_, _, err = e.ReadAndProcessComponentVendorConfigFile(&atmosConfig, "mock", "terraform")
 		assert.Error(t, err)
 		assert.Contains(t, err.Error(), "does not exist")
 	})
@@ -122,7 +122,7 @@ metadata:
   name: test-vendor-config
 spec:
   sources:
-    - component: myapp
+    - component: mock
       source: github.com/cloudposse/terraform-null-label.git//exports?ref={{.Version}}
       version: 0.25.0
 `

tests/cli_terraform_test.go

@@ -39,7 +39,7 @@ func TestCLITerraformClean(t *testing.T) {
 	}()
 
 	// Define the work directory and change to it
-	workDir := "../examples/quick-start-simple"
+	workDir := "fixtures/scenarios/basic"
 	if err := os.Chdir(workDir); err != nil {
 		t.Fatalf("Failed to change directory to %q: %v", workDir, err)
 	}
@@ -55,24 +55,23 @@ func TestCLITerraformClean(t *testing.T) {
 	// Clean everything
 	runTerraformCleanCommand(t, binaryPath)
 	// Clean specific component
-	runTerraformCleanCommand(t, binaryPath, "station")
+	runTerraformCleanCommand(t, binaryPath, "mycomponent")
 	// Clean component with stack
-	runTerraformCleanCommand(t, binaryPath, "station", "-s", "dev")
+	runTerraformCleanCommand(t, binaryPath, "mycomponent", "-s", "nonprod")
 
 	// Run terraform apply for prod environment
 	runTerraformApply(t, binaryPath, "prod")
-	verifyStateFilesExist(t, []string{"./components/terraform/weather/terraform.tfstate.d/prod-station"})
+	verifyStateFilesExist(t, []string{"./components/terraform/mock/terraform.tfstate.d/prod-mycomponent"})
 	runCLITerraformCleanComponent(t, binaryPath, "prod")
-	verifyStateFilesDeleted(t, []string{"./components/terraform/weather/terraform.tfstate.d/prod-station"})
+	verifyStateFilesDeleted(t, []string{"./components/terraform/mock/terraform.tfstate.d/prod-mycomponent"})
 
-	// Run terraform apply for dev environment
-	runTerraformApply(t, binaryPath, "dev")
+	// Run terraform apply for nonprod environment
+	runTerraformApply(t, binaryPath, "nonprod")
 
 	// Verify if state files exist before cleaning
 	stateFiles := []string{
-		"./components/terraform/weather/.terraform",
-		"./components/terraform/weather/terraform.tfstate.d",
-		"./components/terraform/weather/.terraform.lock.hcl",
+		"./components/terraform/mock/.terraform",
+		"./components/terraform/mock/terraform.tfstate.d",
 	}
 	verifyStateFilesExist(t, stateFiles)
 
@@ -85,7 +84,7 @@ func TestCLITerraformClean(t *testing.T) {
 
 // runTerraformApply runs the terraform apply command for a given environment.
 func runTerraformApply(t *testing.T, binaryPath, environment string) {
-	cmd := exec.Command(binaryPath, "terraform", "apply", "station", "-s", environment)
+	cmd := exec.Command(binaryPath, "terraform", "apply", "mycomponent", "-s", environment)
 	envVars := os.Environ()
 	envVars = append(envVars, "ATMOS_COMPONENTS_TERRAFORM_APPLY_AUTO_APPROVE=true")
 	cmd.Env = envVars
@@ -96,7 +95,7 @@ func runTerraformApply(t *testing.T, binaryPath, environment string) {
 	err := cmd.Run()
 	t.Log(stdout.String())
 	if err != nil {
-		t.Fatalf("Failed to run terraform apply station -s %s: %v", environment, stderr.String())
+		t.Fatalf("Failed to run terraform apply mycomponent -s %s: %v", environment, stderr.String())
 	}
 }
 
@@ -143,7 +142,7 @@ func verifyStateFilesDeleted(t *testing.T, stateFiles []string) {
 }
 
 func runCLITerraformCleanComponent(t *testing.T, binaryPath, environment string) {
-	cmd := exec.Command(binaryPath, "terraform", "clean", "station", "-s", environment, "--force")
+	cmd := exec.Command(binaryPath, "terraform", "clean", "mycomponent", "-s", environment, "--force")
 	var stdout, stderr bytes.Buffer
 	cmd.Stdout = &stdout
 	cmd.Stderr = &stderr

tests/fixtures/components/terraform/mock/README.md

@@ -0,0 +1,51 @@
+# Mock Component
+
+This is a mock Terraform component used exclusively for testing purposes. It provides a simple configuration with input variables and corresponding outputs, making it ideal for testing Atmos component functionality and behavior.
+
+## Usage
+
+This component is intended for testing only and should not be used in production environments. It is typically used in conjunction with test fixtures and validation scenarios.
+
+Example stack usage
+
+```yaml
+components:
+  terraform:
+    mock:
+      vars:
+        foo: "default value"
+        bar: "default value"
+        baz: "default value"
+```
+
+## Requirements
+
+No requirements.
+
+## Providers
+
+No providers.
+
+## Modules
+
+No modules.
+
+## Resources
+
+No resources.
+
+## Inputs
+
+| Name | Description | Type | Default | Required |
+|------|-------------|------|---------|:--------:|
+| <a name="input_bar"></a> [bar](#input\_bar) | n/a | `string` | `"bar"` | no |
+| <a name="input_baz"></a> [baz](#input\_baz) | n/a | `string` | `"baz"` | no |
+| <a name="input_foo"></a> [foo](#input\_foo) | n/a | `string` | `"foo"` | no |
+
+## Outputs
+
+| Name | Description |
+|------|-------------|
+| <a name="output_bar"></a> [bar](#output\_bar) | n/a |
+| <a name="output_baz"></a> [baz](#output\_baz) | n/a |
+| <a name="output_foo"></a> [foo](#output\_foo) | n/a |