Merge pull request #69 from namecheap/perf/intl

fix: provide compatibility when running code in vm.runInNewContext()

Author: stas-nc

src/app/utils/TtlCache.ts

@@ -61,6 +61,14 @@ export class TtlCache<K, V> {
     }
 
     private scheduleCleanup(): void {
+        /**
+         * Required due to source code evaluation in vm context compatibility
+         * Temporary workaround until for internal needs
+         * Should be removed soon
+         */
+        if (typeof setTimeout !== 'function') {
+            return;
+        }
         this.timeoutId = setTimeout(() => this.cleanup(), this.cleanupInterval);
         this.timeoutId.unref?.();
     }

test/app/TtlCache.spec.ts

@@ -4,13 +4,16 @@ import sinon from 'sinon';
 
 describe('TtlCache', () => {
     let clock: sinon.SinonFakeTimers;
+    let setTimeoutBackup: typeof setTimeout;
 
     beforeEach(() => {
+        setTimeoutBackup = global.setTimeout;
         clock = sinon.useFakeTimers();
     });
 
     afterEach(() => {
         clock.restore();
+        global.setTimeout = setTimeoutBackup;
     });
 
     it('should set and get a value', () => {
@@ -68,4 +71,13 @@ describe('TtlCache', () => {
         clock.tick(500); // Trigger cleanup
         expect(cache.get('key1')).to.be.undefined;
     });
+
+    it('should not schedule cleanup if setTimeout is not available', () => {
+        const cache = new TtlCache<string, number>({ ttl: 1000, cleanupInterval: 500 });
+        global.setTimeout = undefined as any;
+        cache.set('key1', 123);
+        expect(cache.get('key1')).to.equal(123);
+        clock.tick(1001);
+        expect(cache.get('key1')).to.equal(123);
+    });
 });