enforce to use semicolon

Author: zhiyuanliang-ms

.eslintrc

@@ -56,6 +56,7 @@
             {
                 "max": 1 
             }
-        ]
+        ],
+        "semi": ["error", "always"]
     }
 }

src/AzureAppConfigurationImpl.ts

@@ -481,7 +481,7 @@ export class AzureAppConfigurationImpl implements AzureAppConfiguration {
             if (index >= 0) {
                 this.#onRefreshListeners.splice(index, 1);
             }
-        }
+        };
         return new Disposable(remove);
     }
 

src/JsonKeyValueAdapter.ts

@@ -54,4 +54,4 @@ function isJsonContentType(contentTypeValue: string): boolean {
     }
 
     return typeParts[1].split("+").includes("json");
-}
+}

src/load.ts

@@ -94,7 +94,7 @@ function getClientOptions(options?: AzureAppConfigurationOptions): AppConfigurat
     const defaultRetryOptions = {
         maxRetries: MaxRetries,
         maxRetryDelayInMs: MaxRetryDelayInMs,
-    }
+    };
     const retryOptions = Object.assign({}, defaultRetryOptions, options?.clientOptions?.retryOptions);
 
     return Object.assign({}, options?.clientOptions, {

src/requestTracing/utils.ts

@@ -40,7 +40,7 @@ export function listConfigurationSettingsWithTrace(
             customHeaders: {
                 [CORRELATION_CONTEXT_HEADER_NAME]: createCorrelationContextHeader(appConfigOptions, initialLoadCompleted)
             }
-        }
+        };
     }
 
     return client.listConfigurationSettings(actualListOptions);
@@ -64,7 +64,7 @@ export function getConfigurationSettingWithTrace(
             customHeaders: {
                 [CORRELATION_CONTEXT_HEADER_NAME]: createCorrelationContextHeader(appConfigOptions, initialLoadCompleted)
             }
-        }
+        };
     }
 
     return client.getConfigurationSetting(configurationSettingId, actualGetOptions);

test/clientOptions.test.ts

@@ -37,7 +37,7 @@ describe("custom client options", function () {
 
     afterEach(() => {
         nock.restore();
-    })
+    });
 
     it("should retry 2 times by default", async () => {
         const countPolicy = new HttpRequestCountPolicy();
@@ -49,7 +49,7 @@ describe("custom client options", function () {
                         position: "perRetry"
                     }]
                 }
-            })
+            });
         };
         let error;
         try {
@@ -74,7 +74,7 @@ describe("custom client options", function () {
                         maxRetries
                     }
                 }
-            })
+            });
         };
 
         let error;
@@ -109,7 +109,7 @@ describe("custom client options", function () {
                         position: "perRetry"
                     }]
                 }
-            })
+            });
         };
         let error;
         try {
@@ -120,4 +120,4 @@ describe("custom client options", function () {
         expect(error).not.undefined;
         expect(countPolicy.count).eq(3);
     });
-})
+});

test/featureFlag.test.ts

@@ -65,7 +65,8 @@ describe("feature flags", function () {
 
     after(() => {
         restoreMocks();
-    })
+    });
+    
     it("should load feature flags if enabled", async () => {
         const connectionString = createMockedConnectionString();
         const settings = await load(connectionString, {

test/json.test.ts

@@ -17,7 +17,7 @@ describe("json", function () {
 
     afterEach(() => {
         restoreMocks();
-    })
+    });
 
     it("should load and parse if content type is application/json", async () => {
         mockAppConfigurationClientListConfigurationSettings([jsonKeyValue]);
@@ -86,4 +86,4 @@ describe("json", function () {
         expect(settings.get("json.settings.emptyString")).eq("", "is empty string");
         expect(settings.get("json.settings.illegalString")).eq("[unclosed", "is illegal string");
     });
-})
+});

test/keyvault.test.ts

@@ -111,4 +111,4 @@ describe("key vault reference", function () {
         expect(settings.get("TestKey")).eq("SecretValue");
         expect(settings.get("TestKey2")).eq("SecretValue2");
     });
-})
+});

test/load.test.ts

@@ -85,7 +85,8 @@ describe("load", function () {
 
     after(() => {
         restoreMocks();
-    })
+    });
+    
     it("should load data from config store with connection string", async () => {
         const connectionString = createMockedConnectionString();
         const settings = await load(connectionString);

test/refresh.test.ts

@@ -33,12 +33,12 @@ describe("dynamic refresh", function () {
             { value: "30", key: "app.settings.fontSize", label: "prod" }
         ].map(createMockedKeyValue);
         mockAppConfigurationClientListConfigurationSettings(mockedKVs);
-        mockAppConfigurationClientGetConfigurationSetting(mockedKVs)
+        mockAppConfigurationClientGetConfigurationSetting(mockedKVs);
     });
 
     afterEach(() => {
         restoreMocks();
-    })
+    });
 
     it("should throw error when refresh is not enabled but refresh is called", async () => {
         const connectionString = createMockedConnectionString();
@@ -331,14 +331,14 @@ describe("dynamic refresh feature flags", function () {
 
     afterEach(() => {
         restoreMocks();
-    })
+    });
 
     it("should refresh feature flags when enabled", async () => {
         mockedKVs = [
             createMockedFeatureFlag("Beta", { enabled: true })
         ];
         mockAppConfigurationClientListConfigurationSettings(mockedKVs);
-        mockAppConfigurationClientGetConfigurationSetting(mockedKVs)
+        mockAppConfigurationClientGetConfigurationSetting(mockedKVs);
 
         const connectionString = createMockedConnectionString();
         const settings = await load(connectionString, {

test/requestTracing.test.ts

@@ -42,7 +42,7 @@ describe("request tracing", function () {
     });
 
     after(() => {
-    })
+    });
 
     it("should have correct user agent prefix", async () => {
         try {

test/utils/testHelper.ts

@@ -35,7 +35,7 @@ function _filterKVs(unfilteredKvs: ConfigurationSetting[], listOptions: any) {
             labelMatched = kv.label === labelFilter;
         }
         return keyMatched && labelMatched;
-    })
+    });
 }
 
 /**
@@ -86,7 +86,7 @@ function mockAppConfigurationClientListConfigurationSettings(...pages: Configura
                             });
                         }
                     }
-                }
+                };
             }
         };
 
@@ -126,7 +126,7 @@ function mockSecretClientGetSecret(uriValueList: [string, string][]) {
             name: secretName,
             value: dict.get(url.toString())
         } as KeyVaultSecret;
-    })
+    });
 }
 
 function restoreMocks() {
@@ -139,11 +139,11 @@ const createMockedConnectionString = (endpoint = createMockedEndpoint(), secret
     const toEncodeAsBytes = Buffer.from(secret);
     const returnValue = toEncodeAsBytes.toString("base64");
     return `Endpoint=${endpoint};Id=${id};Secret=${returnValue}`;
-}
+};
 
 const createMockedTokenCredential = (tenantId = TEST_TENANT_ID, clientId = TEST_CLIENT_ID, clientSecret = TEST_CLIENT_SECRET) => {
     return new ClientSecretCredential(tenantId, clientId, clientSecret);
-}
+};
 
 const createMockedKeyVaultReference = (key: string, vaultUri: string): ConfigurationSetting => ({
     // https://${vaultName}.vault.azure.net/secrets/${secretName}
@@ -210,4 +210,4 @@ export {
     createMockedFeatureFlag,
 
     sleepInMs
-}
+};