Skip to content

test: added add-ons test #278

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 19 commits into from
Sep 30, 2025
Merged

test: added add-ons test #278

Show file tree
Hide file tree
Changes from 11 commits
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
76b7e7f
added add-ons
HarikaPonna Sep 11, 2025
15167fd
testing
HarikaPonna Sep 11, 2025
b159736
ran pre commit
Sep 18, 2025
9b0a9cc
Merge branch 'main' into add-ons-test
HarikaPonna Sep 18, 2025
bc10a7f
removed TestDependencyPermutations
Sep 18, 2025
f22ad59
renamed prefix to test
Sep 18, 2025
42f7b70
fix: pre-commit
Aayush-Abhyarthi Sep 18, 2025
41a71e6
fix: pre-commit
Aayush-Abhyarthi Sep 18, 2025
38ef304
fix: addons test
Aayush-Abhyarthi Sep 18, 2025
8d6499c
use existing sm instance
Aayush-Abhyarthi Sep 23, 2025
710134e
Merge branch 'main' into add-ons-test
Aayush-Abhyarthi Sep 23, 2025
2dd6522
fix: addon test
Aayush-Abhyarthi Sep 23, 2025
f34d6ae
update terratest
Aayush-Abhyarthi Sep 29, 2025
976aeb0
few changes
Aayush-Abhyarthi Sep 30, 2025
373c427
update dependency variable
Aayush-Abhyarthi Sep 30, 2025
1729f7e
update sm version
Aayush-Abhyarthi Sep 30, 2025
cc9bb7b
fix: remove common dev assets
Aayush-Abhyarthi Sep 30, 2025
f0c6eaa
resolve comment
Aayush-Abhyarthi Sep 30, 2025
74aa03a
resolve conflicts
Aayush-Abhyarthi Sep 30, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion common-dev-assets
Open in desktop
Submodule common-dev-assets updated 13 files
+1 −1 commonRenovateConfig.json
+1 −1 da-assets/da-dependency-updater/update_da_dependencies.py
+222 −223 module-assets/.pre-commit-config.yaml
+144 −12 module-assets/ci/install-deps.sh
+2 −2 module-assets/ci/terraformConfigInspect.py
+3 −5 module-assets/ci/terraformDocGoMod.py
+6 −8 module-assets/ci/terraformDocOverview.py
+3 −3 module-assets/ci/terraformDocs.py
+3 −3 module-assets/ci/terraformDocsUtils.py
+1 −1 module-assets/ci/validateIbmCatalogJson.py
+2 −4 module-assets/ci/validateJsonTemplate.py
+2 −2 scripts/update-source/run-update-src.py
+3 −4 stack-assets/stack-updater/update_stack_definition.py
2 changes: 1 addition & 1 deletion ibm_catalog.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@
"url": "https://raw.githubusercontent.com/terraform-ibm-modules/terraform-ibm-secrets-manager-public-cert-engine/main/reference-architecture/secrets_manager_public_cert_engine.svg",
"type": "image/svg+xml"
},
"description": "This architcture supports creating secrets manager public certificates engine within a secrets manager instance."
"description": "This architecture supports creating secrets manager public certificates engine within a secrets manager instance."
}
]
},
Expand Down
45 changes: 36 additions & 9 deletions tests/pr_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,16 +2,18 @@
package test

import (
"log"
"os"
"testing"

"github.com/gruntwork-io/terratest/modules/logger"
"github.com/gruntwork-io/terratest/modules/terraform"
"github.com/stretchr/testify/assert"
"github.com/stretchr/testify/require"
"github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper/cloudinfo"
"github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper/common"
"github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper/testaddons"
"github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper/testhelper"
"github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper/testschematic"
"log"
"os"
"testing"
)

// Define a struct with fields that match the structure of the YAML data
Expand Down Expand Up @@ -174,11 +176,11 @@ func TestPlanValidation(t *testing.T) {
options.TerraformOptions.NoColor = true
options.TerraformOptions.Logger = logger.Discard
options.TerraformOptions.Vars = map[string]interface{}{
"prefix": options.Prefix,
"existing_secrets_manager_crn": permanentResources["secretsManagerCRN"],
"acme_letsencrypt_private_key": "PRIVATE_KEY_VALUE", // pragma: allowlist secret
"skip_iam_authorization_policy": true,
"provider_visibility": "public",
"prefix": options.Prefix,
"existing_secrets_manager_crn": permanentResources["secretsManagerCRN"],
"acme_letsencrypt_private_key_secrets_manager_secret_crn": permanentResources["acme_letsencrypt_private_key_secret_crn"], // pragma: allowlist secret
"skip_iam_authorization_policy": true,
"provider_visibility": "public",
}

// Init
Expand All @@ -190,3 +192,28 @@ func TestPlanValidation(t *testing.T) {
assert.Nil(t, planErr, "Terraform plan should not error")
assert.NotNil(t, planOutput, "Expected Terraform plan output")
}

func TestSecretManagerDefaultConfiguration(t *testing.T) {
t.Parallel()

options := testaddons.TestAddonsOptionsDefault(&testaddons.TestAddonOptions{
Testing: t,
Prefix: "pbsme",
ResourceGroup: resourceGroup,
QuietMode: true, // Suppress logs except on failure
Copy link
Contributor

@whoffler whoffler Sep 30, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

Have a look at similar PR where we set inputs on SMs dependencies to avoid creating too many routes in our account.

Make sure to set option OverrideInputMappings: core.BoolPtr(true) in the TestAddonOptions, it allows us to set inputs on variables that would otherwise contain a reference (otherwise SM existing_secrets_manager_crn may not be set as expected)

})

options.AddonConfig = cloudinfo.NewAddonConfigTerraform(
options.Prefix,
"deploy-arch-secrets-manager-public-cert-engine",
"fully-configurable",
map[string]interface{}{
"prefix": options.Prefix,
"existing_secrets_manager_crn": permanentResources["secretsManagerCRN"],
"acme_letsencrypt_private_key_secrets_manager_secret_crn": permanentResources["acme_letsencrypt_private_key_secret_crn"], // pragma: allowlist secret
},
)

err := options.RunAddonTest()
require.NoError(t, err)
}