update @azure/core-rest-pipeline to 1.12.2 (#25)

Author: Eskibear

package-lock.json

@@ -28,12 +28,12 @@ class HttpRequestCountPolicy {
 
 describe("custom client options", function () {
     const fakeEndpoint = "https://azure.azconfig.io";
-    before(() => {
+    beforeEach(() => {
         // Thus here mock it to reply 500, in which case the retry mechanism works.
         nock(fakeEndpoint).persist().get(() => true).reply(500);
     });
 
-    after(() => {
+    afterEach(() => {
         nock.restore();
     })
 
@@ -96,9 +96,26 @@ describe("custom client options", function () {
         expect(countPolicy.count).eq(2);
     });
 
-    // Note:
-    // core-rest-pipeline skips the retry throwing `RestError: getaddrinfo ENOTFOUND azure.azconfig.io`
-    // Probably would be fixed in upstream libs.
-    // See https://github.com/Azure/azure-sdk-for-js/issues/27037
-    it("should retry on DNS failure");
+    it("should retry on DNS failure", async () => {
+        nock.restore(); // stop mocking with 500 error but sending real requests which will fail with ENOTFOUND
+        const countPolicy = new HttpRequestCountPolicy();
+        const loadPromise = () => {
+            return load(createMockedConnectionString(fakeEndpoint), {
+                clientOptions: {
+                    additionalPolicies: [{
+                        policy: countPolicy,
+                        position: "perRetry"
+                    }]
+                }
+            })
+        };
+        let error;
+        try {
+            await loadPromise();
+        } catch (e) {
+            error = e;
+        }
+        expect(error).not.undefined;
+        expect(countPolicy.count).eq(3);
+    });
 })