Fix non-variant default name for tests-weekly (#1435)

Azure · Feb 19, 2021 · 0367c14 · 0367c14
1 parent fe0e62f
commit 0367c14
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/...ure.Sdk.Tools.PipelineGenerator/Conventions/WeeklyIntegrationTestingPipelineConvention.cs b/...ure.Sdk.Tools.PipelineGenerator/Conventions/WeeklyIntegrationTestingPipelineConvention.cs
@@ -11,7 +11,11 @@ public WeeklyIntegrationTestingPipelineConvention(ILogger logger, PipelineGenera
 
         protected override string GetDefinitionName(SdkComponent component)
         {
-            return component.Variant == null ? $"{Context.Prefix} - {component.Name} - tests" : $"{Context.Prefix} - {component.Name} - tests-weekly.{component.Variant}";
+            var definitionName = $"{Context.Prefix} - {component.Name} - tests-weekly";
+            if (component.Variant != null) {
+                definitionName += $".{component.Variant}";
+            }
+            return definitionName;
         }
 
         protected override Schedule CreateScheduleFromDefinition(BuildDefinition definition)