Fix APIM deployment issue

Author: pascalgn

lib/arm.js

@@ -48,6 +48,10 @@ function generateTemplate(input) {
     return `[resourceId('Microsoft.ApiManagement/service/apis', '${serviceName}', '${api.fullName}')]`;
   }
 
+  function resourceIdOperation(serviceName, api, operation) {
+    return `[resourceId('Microsoft.ApiManagement/service/apis/operations', '${serviceName}', '${api.fullName}', '${operation.name}')]`;
+  }
+
   function resourceIdProduct(serviceName, product) {
     return `[resourceId('Microsoft.ApiManagement/service/products', '${serviceName}', '${product.fullName}')]`;
   }
@@ -69,6 +73,8 @@ function generateTemplate(input) {
   for (const environment of input.environments) {
     const { serviceName } = environment.configuration;
 
+    let previousApiId = null;
+
     for (const api of environment.apis) {
       const apiResourceId = resourceIdApi(serviceName, api);
 
@@ -83,6 +89,7 @@ function generateTemplate(input) {
           protocols: ["https"],
           isCurrent: true
         },
+        dependsOn: previousApiId == null ? [] : [previousApiId],
         resources: toArray(api.policies, {
           type: "policies",
           name: "policy",
@@ -108,11 +115,16 @@ function generateTemplate(input) {
         }
       }
 
+      let previousOperationId = null;
+
       for (const operation of api.operations) {
         addResource({
           type: "Microsoft.ApiManagement/service/apis/operations",
           name: `${serviceName}/${operation.fullName}`,
-          dependsOn: [apiResourceId],
+          dependsOn:
+            previousOperationId == null
+              ? [apiResourceId]
+              : [apiResourceId, previousOperationId],
           properties: {
             displayName: operation.fullDisplayName,
             method: operation.method,
@@ -133,7 +145,10 @@ function generateTemplate(input) {
             }
           })
         });
+        previousOperationId = resourceIdOperation(serviceName, api, operation);
       }
+
+      previousApiId = apiResourceId;
     }
 
     for (const product of environment.products) {

test/basic-output.json

@@ -15,6 +15,7 @@
         "protocols": ["https"],
         "isCurrent": true
       },
+      "dependsOn": [],
       "resources": [],
       "apiVersion": "2019-01-01"
     },

test/environments-output.json

@@ -15,6 +15,7 @@
         "protocols": ["https"],
         "isCurrent": true
       },
+      "dependsOn": [],
       "resources": [
         {
           "type": "policies",

test/operations-output.json

@@ -15,6 +15,7 @@
         "protocols": ["https"],
         "isCurrent": true
       },
+      "dependsOn": [],
       "resources": [],
       "apiVersion": "2019-01-01"
     },
@@ -50,7 +51,8 @@
       "type": "Microsoft.ApiManagement/service/apis/operations",
       "name": "apim1/example-api-development/post-data",
       "dependsOn": [
-        "[resourceId('Microsoft.ApiManagement/service/apis', 'apim1', 'example-api-development')]"
+        "[resourceId('Microsoft.ApiManagement/service/apis', 'apim1', 'example-api-development')]",
+        "[resourceId('Microsoft.ApiManagement/service/apis/operations', 'apim1', 'example-api-development', 'get-data')]"
       ],
       "properties": {
         "displayName": "Post-Data",
@@ -62,9 +64,7 @@
             "description": "Param 1",
             "type": "number",
             "defaultValue": "1",
-            "values": [
-              "1"
-            ]
+            "values": ["1"]
           }
         ],
         "request": {

test/parameters-output.json

@@ -20,6 +20,7 @@
           "protocols": ["https"],
           "isCurrent": true
         },
+        "dependsOn": [],
         "resources": [],
         "apiVersion": "2019-01-01"
       },

test/properties-mixed-output.json

@@ -15,6 +15,7 @@
         "protocols": ["https"],
         "isCurrent": true
       },
+      "dependsOn": [],
       "resources": [
         {
           "type": "policies",

test/properties-reference-output.json

@@ -15,6 +15,7 @@
         "protocols": ["https"],
         "isCurrent": true
       },
+      "dependsOn": [],
       "resources": [
         {
           "type": "policies",

test/reference-variables-output.json

@@ -12,18 +12,15 @@
         "apiRevision": "1",
         "subscriptionRequired": true,
         "path": "example-api",
-        "protocols": [
-          "https"
-        ],
+        "protocols": ["https"],
         "isCurrent": true
       },
+      "dependsOn": [],
       "resources": [
         {
           "type": "policies",
           "name": "policy",
-          "dependsOn": [
-            "example-api-development"
-          ],
+          "dependsOn": ["example-api-development"],
           "properties": {
             "value": "<policies><inbound><base />\n<set-backend-service base-url=\"https://dev.example.com\" />\n<authentication-basic username=\"{{ExampleApi-Development-Default-Username}}\" password=\"{{ExampleApi-Development-Default-Password}}\" />\n</inbound><backend><base /></backend><outbound><base /></outbound><on-error><base /></on-error></policies>",
             "format": "xml"
@@ -60,18 +57,15 @@
         "apiRevision": "1",
         "subscriptionRequired": true,
         "path": "example-api",
-        "protocols": [
-          "https"
-        ],
+        "protocols": ["https"],
         "isCurrent": true
       },
+      "dependsOn": [],
       "resources": [
         {
           "type": "policies",
           "name": "policy",
-          "dependsOn": [
-            "example-api-quality"
-          ],
+          "dependsOn": ["example-api-quality"],
           "properties": {
             "value": "<policies><inbound><base />\n<set-backend-service base-url=\"https://example.com\" />\n<authentication-basic username=\"{{ExampleApi-Quality-Default-Username}}\" password=\"{{ExampleApi-Quality-Default-Password}}\" />\n</inbound><backend><base /></backend><outbound><base /></outbound><on-error><base /></on-error></policies>",
             "format": "xml"