Add tests for connection-string and fix cloudflare module type and esm compat

Author: brianc

packages/pg-cloudflare/esm/index.js

@@ -4,23 +4,12 @@
   "description": "A socket implementation that can run on Cloudflare Workers using native TCP connections.",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
+  "type": "module",
   "license": "MIT",
   "devDependencies": {
     "ts-node": "^8.5.4",
     "typescript": "^4.0.3"
   },
-  "exports": {
-    ".": {
-      "import": "./esm/worker.js",
-      "require": "./dist/index.js",
-      "default": "./dist/index.js"
-    },
-    "./worker": {
-      "import": "./esm/worker.js",
-      "require": "./dist/index.js",
-      "default": "./dist/index.js"
-    }
-  },
   "scripts": {
     "build": "tsc",
     "build:watch": "tsc --watch",

packages/pg-cloudflare/esm/worker.js

@@ -3,6 +3,5 @@ import connectionString from '../index.js'
 
 // Re-export the parse function
 export const parse = connectionString.parse
-
-// Re-export the default
-export default connectionString
+export const toClientConfig = connectionString.toClientConfig
+export const parseIntoClientConfig = connectionString.parseIntoClientConfig

packages/pg-cloudflare/package.json

@@ -7,7 +7,7 @@
   "exports": {
     ".": {
       "types": "./index.d.ts",
-      "import": "./esm/index.js",
+      "import": "./esm/index.mjs",
       "require": "./index.js",
       "default": "./index.js"
     }

…ckages/pg-connection-string/esm/index.js …kages/pg-connection-string/esm/index.mjspackages/pg-connection-string/esm/index.js renamed to packages/pg-connection-string/esm/index.mjs packages/pg-connection-string/esm/index.js renamed to packages/pg-connection-string/esm/index.mjs

@@ -13,10 +13,11 @@
     "esm",
     "test"
   ],
-  "dependencies": {
+  "devDependencies": {
     "pg": "^8.14.1",
     "pg-pool": "^3.8.0",
-    "pg-native": "^3.3.0"
+    "pg-native": "^3.3.0",
+    "pg-cloudflare": "^1.1.1"
   },
   "author": "Brian M. Carlson <brian.m.carlson@gmail.com>",
   "license": "MIT"

packages/pg-connection-string/package.json

@@ -0,0 +1,9 @@
+import assert from 'node:assert'
+import { describe, it } from 'node:test'
+import { CloudflareSocket } from 'pg-cloudflare'
+
+describe('pg-pool', () => {
+  it('should export CloudflareSocket constructor', () => {
+    assert.ok(new CloudflareSocket())
+  })
+})

packages/pg-esm-test/package.json

@@ -0,0 +1,17 @@
+import assert from 'node:assert'
+import { describe, it } from 'node:test'
+import { parse, toClientConfig, parseIntoClientConfig } from 'pg-connection-string'
+
+describe('pg-connection-string', () => {
+  it('should export parse function', () => {
+    assert.strictEqual(typeof parse, 'function')
+  })
+
+  it('should export toClientConfig function', () => {
+    assert.strictEqual(typeof toClientConfig, 'function')
+  })
+
+  it('should export parseIntoClientConfig function', () => {
+    assert.strictEqual(typeof parseIntoClientConfig, 'function')
+  })
+})

packages/pg-esm-test/pg-cloudflare.test.js

@@ -1,4 +1,3 @@
-// test/my-module.test.js
 import assert from 'node:assert'
 import { describe, it } from 'node:test'
 import Client from 'pg-native'

packages/pg-esm-test/pg-connection-string.test.js

@@ -1,4 +1,3 @@
-// test/my-module.test.js
 import assert from 'node:assert'
 import { describe, it } from 'node:test'
 import Pool from 'pg-pool'