test: ttlConfig expiration logic

src/config/ttlConfig.ts

@@ -48,7 +48,7 @@ export class TTLConfig<T extends TTLConfig.Options, P extends JsonMap> extends C
     const date = new Date().getTime();
     // delete all the expired entries
     Object.entries(contents)
-      .filter(([, value]) => !this.isExpired(date, value))
+      .filter(([, value]) => this.isExpired(date, value))
       .map(([key]) => this.unset(key));
   }
 

test/unit/config/ttlConfigTest.ts

@@ -96,4 +96,31 @@ describe('TTLConfig', () => {
       expect(isExpired).to.be.false;
     });
   });
+
+  describe.only('filters expired keys on init', () => {
+    $$.setConfigStubContents('TestConfig', {
+      contents: {
+        old: {
+          value: 1,
+          timestamp: new Date().getTime() - Duration.days(7).milliseconds,
+        },
+        current: {
+          value: 2,
+          timestamp: new Date().getTime(),
+        },
+        future: {
+          value: 3,
+          timestamp: new Date().getTime() + Duration.days(7).milliseconds,
+        },
+      },
+    });
+
+    it('should omit expired keys', async () => {
+      const config = await TestConfig.create();
+      const keys = config.keys();
+      expect(keys).to.include('current');
+      expect(keys).to.include('future');
+      expect(keys).to.not.include('old');
+    });
+  });
 });