fix: Updated Pester template test file check & added change to key va…

…ult to enable republishing (#1236)

## Description

- Updated Pester template test file check 
- Added change to key vault to enable republishing
- Fixed incorrect conflict resolution of pe & max tests

## Pipeline Reference

<!-- Insert your Pipeline Status Badge below -->

| Pipeline |
| -------- |
|
[![avm.res.key-vault.vault](https://github.com/AlexanderSehr/bicep-registry-modules/actions/workflows/avm.res.key-vault.vault.yml/badge.svg?branch=users%2Falsehr%2Fe2eTestCheck&event=workflow_dispatch)](https://github.com/AlexanderSehr/bicep-registry-modules/actions/workflows/avm.res.key-vault.vault.yml)
|

---------

Co-authored-by: Erika Gressi <56914614+eriqua@users.noreply.github.com>

avm/res/key-vault/vault/main.bicep

@@ -209,7 +209,7 @@ module keyVault_secrets 'secret/main.bicep' = [for (secret, index) in secretList
  name: secret.name
  value: secret.value
  keyVaultName: keyVault.name
- attributesEnabled: secret.?attributesEnabled ?? true
+ attributesEnabled: secret.?attributesEnabled
  attributesExp: secret.?attributesExp
  attributesNbf: secret.?attributesNbf
  contentType: secret.?contentType
@@ -223,7 +223,7 @@ module keyVault_keys 'key/main.bicep' = [for (key, index) in (keys ?? []): {
  params: {
  name: key.name
  keyVaultName: keyVault.name
- attributesEnabled: key.?attributesEnabled ?? true
+ attributesEnabled: key.?attributesEnabled
  attributesExp: key.?attributesExp
  attributesNbf: key.?attributesNbf
  curveName: key.?curveName ?? 'P-256'

avm/res/key-vault/vault/main.json

@@ -6,7 +6,7 @@
  "_generator": {
  "name": "bicep",
  "version": "0.25.53.49325",
- "templateHash": "18302928297760675801"
+ "templateHash": "5604749365634791124"
  },
  "name": "Key Vaults",
  "description": "This module deploys a Key Vault.",
@@ -1126,7 +1126,7 @@
  "value": "[parameters('name')]"
  },
  "attributesEnabled": {
- "value": "[coalesce(tryGet(variables('secretList')[copyIndex()], 'attributesEnabled'), true())]"
+ "value": "[tryGet(variables('secretList')[copyIndex()], 'attributesEnabled')]"
  },
  "attributesExp": {
  "value": "[tryGet(variables('secretList')[copyIndex()], 'attributesExp')]"
@@ -1403,7 +1403,7 @@
  "value": "[parameters('name')]"
  },
  "attributesEnabled": {
- "value": "[coalesce(tryGet(coalesce(parameters('keys'), createArray())[copyIndex()], 'attributesEnabled'), true())]"
+ "value": "[tryGet(coalesce(parameters('keys'), createArray())[copyIndex()], 'attributesEnabled')]"
  },
  "attributesExp": {
  "value": "[tryGet(coalesce(parameters('keys'), createArray())[copyIndex()], 'attributesExp')]"

avm/res/key-vault/vault/tests/e2e/max/main.test.bicep

@@ -292,3 +292,5 @@ module testDeployment '../../../main.bicep' = [for iteration in [ 'init', 'idem'
  diagnosticDependencies
  ]
 }]
+
+output resourceId string = testDeployment[0].outputs.resourceId

avm/utilities/pipelines/staticValidation/compliance/module.tests.ps1

@@ -214,8 +214,9 @@ Describe 'File/folder tests' -Tag 'Modules' {
 
  $e2eTestFolderPathList = Get-ChildItem -Directory (Join-Path -Path $moduleFolderPath 'tests' 'e2e')
  foreach ($e2eTestFolderPath in $e2eTestFolderPathList) {
- $pathExisting = Test-Path (Join-Path -Path $e2eTestFolderPath 'main.test.bicep')
- $pathExisting | Should -Be $true
+ $filePath = Join-Path -Path $e2eTestFolderPath 'main.test.bicep'
+ $pathExisting = Test-Path $filePath
+ $pathExisting | Should -Be $true -Because "path [$filePath] is expected to exist."
  }
  }
  }
@@ -348,7 +349,7 @@ Describe 'Module tests' -Tag 'Module' {
  }
  }
 
- It '[<moduleFolderName>] Compiled ARM template should be latest.' -TestCases $armTemplateTestCases {
+ It '[<moduleFolderName>] The [main.json] ARM template should be based on the current [main.bicep] Bicep template.' -TestCases $armTemplateTestCases {
 
  param(
  [string] $moduleFolderName,