Fix: should load settings with empty/null value (#9)

Author: Eskibear

src/AzureAppConfigurationImpl.ts

@@ -40,7 +40,7 @@ export class AzureAppConfigurationImpl extends Map<string, unknown> implements A
             });
 
             for await (const setting of settings) {
-                if (setting.key && setting.value) {
+                if (setting.key) {
                     const [key, value] = await this.processAdapters(setting);
                     const trimmedKey = this.keyWithPrefixesTrimmed(key);
                     keyValues.push([trimmedKey, value]);

test/load.test.js

@@ -41,6 +41,24 @@ const mockedKVs = [{
     tags: {},
     etag: 'GdmsLWq3mFjFodVEXUYRmvFr3l_qRiKAW_KdpFbxZKk',
     isReadOnly: false
+}, {
+    value: null,
+    key: 'KeyForNullValue',
+    label: '',
+    contentType: '',
+    lastModified: '2023-05-04T04:32:56.000Z',
+    tags: {},
+    etag: 'GdmsLWq3mFjFodVEXUYRmvFr3l_qRiKAW_KdpFbxZKk',
+    isReadOnly: false
+}, {
+    value: "",
+    key: 'KeyForEmptyValue',
+    label: '',
+    contentType: '',
+    lastModified: '2023-05-04T04:32:56.000Z',
+    tags: {},
+    etag: 'GdmsLWq3mFjFodVEXUYRmvFr3l_qRiKAW_KdpFbxZKk',
+    isReadOnly: false
 }];
 
 describe("load", function () {
@@ -121,4 +139,14 @@ describe("load", function () {
         expect(settings.has("Key")).eq(true);
         expect(settings.get("Key")).eq("TestValue");
     });
+
+    it("should support null/empty value", async () => {
+        const connectionString = createMockedConnectionString();
+        const settings = await load(connectionString);
+        expect(settings).not.undefined;
+        expect(settings.has("KeyForNullValue")).eq(true);
+        expect(settings.get("KeyForNullValue")).eq(null);
+        expect(settings.has("KeyForEmptyValue")).eq(true);
+        expect(settings.get("KeyForEmptyValue")).eq("");
+    });
 })