Relax query time tests

Author: gionatamettifogo

test/connection.test.ts

@@ -5,8 +5,7 @@
 import { SQLiteCloudError } from '../src/index'
 import { SQLiteCloudConnection, anonimizeCommand } from '../src/connection'
 import { parseConnectionString } from '../src/utilities'
-import { CHINOOK_DATABASE_URL, TESTING_DATABASE_URL, LONG_TIMEOUT, getTestingConfig, getChinookConfig, getChinookConnection } from './shared'
-import { Database } from 'sqlite3'
+import { CHINOOK_DATABASE_URL, LONG_TIMEOUT, getTestingConfig, getChinookConfig, getChinookConnection, WARN_SPEED_MS, EXPECT_SPEED_MS } from './shared'
 
 describe('connection', () => {
   let chinook: SQLiteCloudConnection
@@ -363,8 +362,10 @@ describe('connection', () => {
             expect(results).toBe('Hello World, this is a test string.')
             if (++completed >= numQueries) {
               const queryMs = (Date.now() - startTime) / numQueries
-              console.log(`${numQueries}x test string, ${queryMs.toFixed(0)}ms per query`)
-              expect(queryMs).toBeLessThan(2000)
+              if (queryMs > WARN_SPEED_MS) {
+                console.log(`${numQueries}x test string, ${queryMs.toFixed(0)}ms per query`)
+                expect(queryMs).toBeLessThan(EXPECT_SPEED_MS)
+              }
               done()
             }
           })
@@ -386,8 +387,10 @@ describe('connection', () => {
             expect(results.numberOfRows).toBe(4)
             if (++completed >= numQueries) {
               const queryMs = (Date.now() - startTime) / numQueries
-              console.log(`${numQueries}x individual selects, ${queryMs.toFixed(0)}ms per query`)
-              expect(queryMs).toBeLessThan(2000)
+              if (queryMs > WARN_SPEED_MS) {
+                console.log(`${numQueries}x individual selects, ${queryMs.toFixed(0)}ms per query`)
+                expect(queryMs).toBeLessThan(EXPECT_SPEED_MS)
+              }
               done()
             }
           })
@@ -412,8 +415,10 @@ describe('connection', () => {
               expect(results.numberOfRows).toBe(4)
               if (++completed >= numQueries) {
                 const queryMs = (Date.now() - startTime) / numQueries
-                console.log(`${numQueries}x batched selects, ${queryMs.toFixed(0)}ms per query`)
-                expect(queryMs).toBeLessThan(5000)
+                if (queryMs > WARN_SPEED_MS) {
+                  console.log(`${numQueries}x batched selects, ${queryMs.toFixed(0)}ms per query`)
+                  expect(queryMs).toBeLessThan(EXPECT_SPEED_MS)
+                }
                 done()
               }
             }

test/shared.ts

@@ -15,6 +15,11 @@ dotenv.config()
 export const LONG_TIMEOUT = 100 * 1000 // 100 seconds
 export const EXTRA_LONG_TIMEOUT = 60 * 60 * 1000 // 1 hour
 
+/** Will warn if a query or other basic operation is slower than this */
+export const WARN_SPEED_MS = 500
+/** Will except queries to be quicker than this */
+export const EXPECT_SPEED_MS = 6 * 1000
+
 /** Number of times or size of stress (when repeated in sequence) */
 export const SEQUENCE_TEST_SIZE = 75
 /** Concurrency size for multiple connection tests */

test/stress.test.ts

@@ -3,10 +3,16 @@
  */
 
 import { Database, SQLiteCloudRowset } from '../src'
-import { getChinookDatabase, getTestingDatabaseAsync, removeDatabaseAsync, SEQUENCE_TEST_SIZE, SIMULTANEOUS_TEST_SIZE, EXTRA_LONG_TIMEOUT } from './shared'
-
-const WARN_SPEED_MS = 500 // will warn if slower than this
-const EXPECT_SPEED_MS = 6 * 1000 // will throw error if slower than this
+import {
+  getChinookDatabase,
+  getTestingDatabaseAsync,
+  removeDatabaseAsync,
+  SEQUENCE_TEST_SIZE,
+  SIMULTANEOUS_TEST_SIZE,
+  EXTRA_LONG_TIMEOUT,
+  WARN_SPEED_MS,
+  EXPECT_SPEED_MS
+} from './shared'
 
 describe('stress testing', () => {
   it(