fix arm template validation for service/apis/operations (#688)

Co-authored-by: Dmitrii Korolev <dmkorolev@microsoft.com>

Author: DeagleGross

src/ArmTemplates/Extractor/EntityExtractors/ApiOperationExtractor.cs

@@ -44,7 +44,6 @@ public async Task<List<ApiOperationTemplateResource>> GenerateApiOperationsResou
                 apiOperation.OriginalName = apiOperation.Name;
 
                 apiOperation.Name = $"[concat(parameters('{ParameterNames.ApimServiceName}'), '/{apiName}/{apiOperation.OriginalName}')]";
-                apiOperation.DependsOn = new[]{ $"[resourceId('Microsoft.ApiManagement/service/apis/operations', parameters('{ParameterNames.ApimServiceName}'), '{apiName}')]" };
                 apiOperation.ApiVersion = GlobalConstants.ApiVersion;
                 apiOperation.Scale = null;
 
@@ -70,6 +69,8 @@ public async Task<List<ApiOperationTemplateResource>> GenerateApiOperationsResou
                         }
                     }
                 }
+
+                apiOperation.DependsOn = operationDependsOn.ToArray();
             }
 
             return apiOperations;

tests/ArmTemplates.Tests/Extractor/Scenarios/ApiExtractorTests.cs

@@ -123,6 +123,8 @@ public async Task GenerateApiTemplates_ProperlyLaysTheInformation()
             apiTemplate.TypedResources.ApiOperations.Count().Should().Be(2);
             apiTemplate.TypedResources.ApiOperations.All(x => x.Type == ResourceTypeConstants.APIOperation).Should().BeTrue();
             apiTemplate.TypedResources.ApiOperations.All(x => x.Properties is not null).Should().BeTrue();
+            apiTemplate.TypedResources.ApiOperations.SelectMany(x => x.DependsOn).Any(x => x.Contains($"'{ResourceTypeConstants.API}'")).Should().BeTrue();
+            apiTemplate.TypedResources.ApiOperations.SelectMany(x => x.DependsOn).Any(x => x.Contains($"'{ResourceTypeConstants.APIOperation}'")).Should().BeFalse();
 
             // api operations policies
             apiTemplate.TypedResources.ApiOperationsPolicies.Count().Should().Be(2);