Apply suggestions from code review

Co-authored-by: Juan Escalada <[email protected]>
Signed-off-by: Kris West <[email protected]>

Author: kriswest

src/db/file/pushes.ts

@@ -6,9 +6,6 @@ import { PushQuery } from '../types';
 
 const COMPACTION_INTERVAL = 1000 * 60 * 60 * 24; // once per day
 
-// these don't get coverage in tests as they have already been run once before the test
-/* istanbul ignore if */
-
 // export for testing purposes
 export let db: Datastore;
 if (process.env.NODE_ENV === 'test') {

test/proxy.test.ts

@@ -102,8 +102,6 @@ describe.skip('Proxy Module TLS Certificate Loading', () => {
       close: vi.fn(),
     } as any);
 
-    process.env.NODE_ENV = 'test';
-    process.env.GIT_PROXY_HTTPS_SERVER_PORT = '8443';
 
     const ProxyClass = (await import('../src/proxy/index')).Proxy;
     proxyModule = new ProxyClass();