Clear cache on benchmark test

Author: sylhare

.github/workflows/main.yml

@@ -12,7 +12,9 @@ jobs:
           cache: 'yarn'
       - name: Install dependencies
         run: yarn install --frozen-lockfile
-      - name: Run tests
-        run: yarn test
+      - name: Run unit tests
+        run: yarn run test:unit
+      - name: Run benchmark tests
+        run: yarn run test:benchmark
       - name: Build application
         run: yarn run build

package.json

@@ -15,6 +15,8 @@
     "postbuild": "node scripts/stamp.js < dest/simple-jekyll-search.min.js > dest/simple-jekyll-search.min.js.tmp && mv dest/simple-jekyll-search.min.js.tmp dest/simple-jekyll-search.min.js && yarn run copy-example-code",
     "copy-example-code": "cp dest/simple-jekyll-search.min.js docs/assets/js/",
     "test": "vitest run --coverage",
+    "test:unit": "vitest run --exclude '**/performance/**' --coverage",
+    "test:benchmark": "NODE_OPTIONS='--max-old-space-size=4096' vitest run tests/performance/",
     "test:watch": "vitest",
     "start": "cd docs; jekyll serve",
     "start:docs": "cd docs && bundle exec jekyll serve"

src/SearchStrategies/types.ts

@@ -24,10 +24,14 @@ export class SearchStrategy implements Matcher {
   private readonly cache: SearchCache<CachedResult>;
 
   constructor(
-    findMatchesFunction: (text: string, criteria: string) => MatchInfo[]
+    findMatchesFunction: (text: string, criteria: string) => MatchInfo[],
+    cacheOptions?: { maxSize?: number; ttl?: number }
   ) {
     this.findMatchesFunction = findMatchesFunction;
-    this.cache = new SearchCache<CachedResult>({ maxSize: 500, ttl: 60000 });
+    this.cache = new SearchCache<CachedResult>({ 
+      maxSize: cacheOptions?.maxSize ?? 100, 
+      ttl: cacheOptions?.ttl ?? 60000 
+    });
   }
 
   matches(text: string | null, criteria: string): boolean {

tests/SearchStrategies/SearchStrategy.caching.test.ts

@@ -1,4 +1,4 @@
-import { describe, expect, it, beforeEach } from 'vitest';
+import { describe, expect, it, beforeEach, afterEach } from 'vitest';
 import { SearchStrategy } from '../../src/SearchStrategies/types';
 
 describe('SearchStrategy Caching', () => {
@@ -29,6 +29,12 @@ describe('SearchStrategy Caching', () => {
     strategy = new SearchStrategy(findMatchesFunction);
   });
 
+  afterEach(() => {
+    if (strategy && strategy.clearCache) {
+      strategy.clearCache();
+    }
+  });
+
   describe('matches() caching', () => {
     it('should cache matches() results', () => {
       strategy.matches('hello world', 'hello');

tests/performance/benchmark.test.ts

@@ -1,4 +1,4 @@
-import { describe, it, expect, beforeEach } from 'vitest';
+import { describe, it, expect, beforeEach, afterEach } from 'vitest';
 import { SearchStrategy } from '../../src/SearchStrategies/types';
 import { PerformanceMonitor } from '../utils/PerformanceMonitor';
 
@@ -20,10 +20,6 @@ describe('Performance Benchmarks', () => {
   let monitor: PerformanceMonitor;
 
   beforeEach(() => {
-    const matchFunction = (text: string, criteria: string) => {
-      return text.toLowerCase().includes(criteria.toLowerCase());
-    };
-
     const findMatchesFunction = (text: string, criteria: string) => {
       const lowerText = text.toLowerCase();
       const lowerCriteria = criteria.toLowerCase();
@@ -47,10 +43,19 @@ describe('Performance Benchmarks', () => {
       return matches;
     };
 
-    strategy = new SearchStrategy(matchFunction, findMatchesFunction);
+    strategy = new SearchStrategy(findMatchesFunction);
     monitor = new PerformanceMonitor();
   });
 
+  afterEach(() => {
+    if (strategy && strategy.clearCache) {
+      strategy.clearCache();
+    }
+    if (monitor) {
+      monitor.reset();
+    }
+  });
+
   describe('Cache Hit Rate', () => {
     it('should achieve high cache hit rate on repeated searches', () => {
       for (let i = 0; i < 100; i++) {

vite.config.ts

@@ -25,7 +25,7 @@ export default defineConfig({
   test: {
     coverage: {
       provider: 'v8',
-      reporter: ['text', 'json', 'html', 'lcov'],
+      reporter: ['text', 'lcov'],
       include: ['src/**/*.ts'],
       exclude: [
         '**/*.d.ts',