Dm/clean up test directories (vertica#145)

* Restructuring v-pool test directory * Updating package.json and tsconfig.json based on recent structure changes
DMickens · May 14, 2024 · 62173ea · 62173ea
1 parent 2f62918
commit 62173ea
Show file tree

Hide file tree

Showing 8 changed files with 42 additions and 10 deletions.
diff --git a/packages/v-protocol/package.json b/packages/v-protocol/package.json
@@ -16,11 +16,12 @@
     "typescript": "^4.0.3"
   },
   "scripts": {
-    "test": "node_modules/.bin/mocha dist/**/*.test.js",
+    "test": "tsc -p ./test/tsconfig.json; node_modules/.bin/mocha test/dist/test/*.test.js",
     "build": "tsc",
     "build:watch": "tsc --watch",
     "prepublish": "yarn build",
-    "pretest": "yarn build"
+    "pretest": "yarn build",
+    "clean": "rm -rf dist; rm -rf test/dist"
   },
   "repository": {
     "type": "git",

diff --git a/...ges/v-protocol/src/testing/buffer-list.ts → packages/v-protocol/test/buffer-list.ts b/...ges/v-protocol/src/testing/buffer-list.ts → packages/v-protocol/test/buffer-list.ts
diff --git a/...ges/v-protocol/src/inbound-parser.test.ts → ...es/v-protocol/test/inbound-parser.test.ts b/...ges/v-protocol/src/inbound-parser.test.ts → ...es/v-protocol/test/inbound-parser.test.ts
@@ -16,12 +16,12 @@
  * =============================================================================
  */
 
-import buffers from './testing/test-buffers'
-import BufferList from './testing/buffer-list'
-import { parse } from '.'
+import buffers from './test-buffers'
+import BufferList from './buffer-list'
+import { parse } from '../src'
 import assert from 'assert'
 import { PassThrough } from 'stream'
-import { BackendMessage } from './backend-messages'
+import { BackendMessage } from '../src/backend-messages'
 
 var authOkBuffer = buffers.authenticationOk()
 var paramStatusBuffer = buffers.parameterStatus('client_encoding', 'UTF8')

diff --git a/...-protocol/src/outbound-serializer.test.ts → ...protocol/test/outbound-serializer.test.ts b/...-protocol/src/outbound-serializer.test.ts → ...protocol/test/outbound-serializer.test.ts
@@ -17,8 +17,8 @@
  */
 
 import assert from 'assert'
-import { serialize } from './serializer'
-import BufferList from './testing/buffer-list'
+import { serialize } from '../src/serializer'
+import BufferList from './buffer-list'
 
 describe('serializer', () => {
   /*it('builds startup message', function () {

diff --git a/...es/v-protocol/src/testing/test-buffers.ts → packages/v-protocol/test/test-buffers.ts b/...es/v-protocol/src/testing/test-buffers.ts → packages/v-protocol/test/test-buffers.ts
diff --git a/packages/v-protocol/test/tsconfig.json b/packages/v-protocol/test/tsconfig.json
@@ -0,0 +1,27 @@
+{
+    "compilerOptions": {
+      "module": "commonjs",
+      "esModuleInterop": true,
+      "allowSyntheticDefaultImports": true,
+      "strict": true,
+      "target": "es6",
+      "noImplicitAny": true,
+      "moduleResolution": "node",
+      "sourceMap": true,
+      "outDir": "dist",
+      "incremental": true,
+      "baseUrl": ".",
+      "declaration": true,
+      "paths": {
+        "*": [
+          "node_modules/*",
+          "src/types/*"
+        ]
+      }
+    },
+    "include": [
+        "outbound-serializer.test.ts",
+        "inbound-parser.test.ts"
+    ]
+  }
+
diff --git a/packages/vertica-nodejs/Makefile b/packages/vertica-nodejs/Makefile
@@ -21,7 +21,7 @@ update-npm:
 	@npm i npm --global
 
 bench:
-	@find benchmark -name "*-bench.js" | $(node-command)
+	@find test/benchmark -name "*bench.js" | $(node-command)
 
 test-unit:
 	@printf "\n***TESTING VERTICA-NODEJS UNIT TESTS***\n\n"

diff --git a/packages/vertica-nodejs/bench.js → ...es/vertica-nodejs/test/benchmark/bench.js b/packages/vertica-nodejs/bench.js → ...es/vertica-nodejs/test/benchmark/bench.js
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-const vertica = require('./lib')
+const vertica = require('../../lib')
 
 const params = {
   text:
@@ -50,6 +50,9 @@ const bench = async (client, q, time) => {
 }
 
 const run = async () => {
+  console.log("Benchmarking currently does not work for vertica-nodejs and has not worked since being ported from node-postgres.")
+  console.log("In the future, consider fixing, or removing.")
+  /*
   const client = new vertica.Client()
   await client.connect()
   console.log('start')
@@ -96,6 +99,7 @@ const run = async () => {
 
   await client.end()
   await client.end()
+  */
 }
 
 run().catch((e) => console.error(e) || process.exit(-1))