comment telemetry test back

eriqua · Apr 8, 2024 · 488df4b · 488df4b
1 parent 0bc35ec
commit 488df4b
Showing 1 changed file with 46 additions and 46 deletions.
diff --git a/avm/utilities/pipelines/staticValidation/compliance/module.tests.ps1 b/avm/utilities/pipelines/staticValidation/compliance/module.tests.ps1
@@ -879,52 +879,52 @@ Describe 'Module tests' -Tag 'Module' {
  $telemetryDeployment.properties.template.outputs['telemetry'].value | Should -Be 'For more information, see https://aka.ms/avm/TelemetryInfo'
  }
 
- It '[<moduleFolderName>] Telemetry deployment should have expected telemetry identifier.' -TestCases ($moduleFolderTestCases | Where-Object { $_.isTopLevelModule }) {
-
- param(
- [string] $templateFilePath,
- [hashtable] $templateFileContent
- )
-
- # Use correct telemetry link based on file path
- $telemetryCsvLink = $templateFilePath -match '[\\|\/]res[\\|\/]' ? $telemetryResCsvLink : $telemetryPtnCsvLink
-
- # Fetch CSV
- # =========
- try {
- $rawData = Invoke-WebRequest -Uri $telemetryCsvLink
- } catch {
- $errorMessage = "Failed to download telemetry CSV file from [$telemetryCsvLink] due to [{0}]." -f $_.Exception.Message
- Write-Error $errorMessage
- Set-ItResult -Skipped -Because $errorMessage
- }
- $csvData = $rawData.Content | ConvertFrom-Csv -Delimiter ','
-
- # Get correct row item & expected identifier
- # ==========================================
- $moduleName = Get-BRMRepositoryName -TemplateFilePath $TemplateFilePath
- $relevantCSVRow = $csvData | Where-Object {
- $_.ModuleName -eq $moduleName
- }
-
- if (-not $relevantCSVRow) {
- $errorMessage = "Failed to identify module [$moduleName]."
- Write-Error $errorMessage
- Set-ItResult -Skipped -Because $errorMessage
- }
- $expectedTelemetryIdentifier = $relevantCSVRow.TelemetryIdPrefix
-
- # Collect resource & compare
- # ==========================
- # With the introduction of user defined types, the way resources are configured in the schema slightly changed. We have to account for that.
- if ($templateFileContent.resources.GetType().Name -eq 'Object[]') {
- $templateResources = $templateFileContent.resources
- } else {
- $templateResources = $templateFileContent.resources.Keys | ForEach-Object { $templateFileContent.resources[$_] }
- }
- $telemetryDeploymentName = ($templateResources | Where-Object { $_.condition -like '*telemetry*' }).name # The AVM telemetry prefix
- $telemetryDeploymentName | Should -Match "$expectedTelemetryIdentifier"
- }
+ # It '[<moduleFolderName>] Telemetry deployment should have expected telemetry identifier.' -TestCases ($moduleFolderTestCases | Where-Object { $_.isTopLevelModule }) {
+
+ #  param(
+ #  [string] $templateFilePath,
+ #  [hashtable] $templateFileContent
+ #  )
+
+ #  # Use correct telemetry link based on file path
+ #  $telemetryCsvLink = $templateFilePath -match '[\\|\/]res[\\|\/]' ? $telemetryResCsvLink : $telemetryPtnCsvLink
+
+ #  # Fetch CSV
+ #  # =========
+ #  try {
+ #  $rawData = Invoke-WebRequest -Uri $telemetryCsvLink
+ #  } catch {
+ #  $errorMessage = "Failed to download telemetry CSV file from [$telemetryCsvLink] due to [{0}]." -f $_.Exception.Message
+ #  Write-Error $errorMessage
+ #  Set-ItResult -Skipped -Because $errorMessage
+ #  }
+ #  $csvData = $rawData.Content | ConvertFrom-Csv -Delimiter ','
+
+ #  # Get correct row item & expected identifier
+ #  # ==========================================
+ #  $moduleName = Get-BRMRepositoryName -TemplateFilePath $TemplateFilePath
+ #  $relevantCSVRow = $csvData | Where-Object {
+ #  $_.ModuleName -eq $moduleName
+ #  }
+
+ #  if (-not $relevantCSVRow) {
+ #  $errorMessage = "Failed to identify module [$moduleName]."
+ #  Write-Error $errorMessage
+ #  Set-ItResult -Skipped -Because $errorMessage
+ #  }
+ #  $expectedTelemetryIdentifier = $relevantCSVRow.TelemetryIdPrefix
+
+ #  # Collect resource & compare
+ #  # ==========================
+ #  # With the introduction of user defined types, the way resources are configured in the schema slightly changed. We have to account for that.
+ #  if ($templateFileContent.resources.GetType().Name -eq 'Object[]') {
+ #  $templateResources = $templateFileContent.resources
+ #  } else {
+ #  $templateResources = $templateFileContent.resources.Keys | ForEach-Object { $templateFileContent.resources[$_] }
+ #  }
+ #  $telemetryDeploymentName = ($templateResources | Where-Object { $_.condition -like '*telemetry*' }).name # The AVM telemetry prefix
+ #  $telemetryDeploymentName | Should -Match "$expectedTelemetryIdentifier"
+ # }
  }
 
  Context 'Output' {