Skip to content

Commit 50c06f9

Browse files
authored
Remove test globals (#3264)
* Remove assert from globals * Remove Client from globals * Remove global test function * Remove MemoryStream from globals * Require assert in SASL integration tests * Attempt to use a postgres with ssl? * Use latest image * Remove connection tests - they test internals that are better covered by testint the client
1 parent f7e484e commit 50c06f9

File tree

81 files changed

+174
-917
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

81 files changed

+174
-917
lines changed

.github/workflows/ci.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ jobs:
2525
needs: lint
2626
services:
2727
postgres:
28-
image: postgres:11
28+
image: ghcr.io/railwayapp-templates/postgres-ssl
2929
env:
3030
POSTGRES_USER: postgres
3131
POSTGRES_PASSWORD: postgres

packages/pg/test/integration/client/api-tests.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
'use strict'
22
var helper = require('../test-helper')
33
var pg = helper.pg
4+
const assert = require('assert')
45

56
var suite = new helper.Suite()
67

packages/pg/test/integration/client/appname-tests.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
'use strict'
22
var helper = require('./test-helper')
33
var Client = helper.Client
4+
const assert = require('assert')
45

56
var suite = new helper.Suite()
67

packages/pg/test/integration/client/array-tests.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
'use strict'
22
var helper = require('./test-helper')
33
var pg = helper.pg
4+
const assert = require('assert')
45

56
var suite = new helper.Suite()
67

packages/pg/test/integration/client/big-simple-query-tests.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
'use strict'
22
var helper = require('./test-helper')
33
var Query = helper.pg.Query
4+
const assert = require('assert')
45

56
const suite = new helper.Suite()
67

packages/pg/test/integration/client/configuration-tests.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,8 @@
11
'use strict'
22
var helper = require('./test-helper')
33
var pg = helper.pg
4+
const assert = require('assert')
5+
const { Client } = helper
46

57
var suite = new helper.Suite()
68

packages/pg/test/integration/client/connection-timeout-tests.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
const net = require('net')
33
const buffers = require('../../test-buffers')
44
const helper = require('./test-helper')
5+
const assert = require('assert')
56

67
const suite = new helper.Suite()
78

packages/pg/test/integration/client/custom-types-tests.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
const helper = require('./test-helper')
33
const Client = helper.pg.Client
44
const suite = new helper.Suite()
5+
const assert = require('assert')
56

67
const customTypes = {
78
getTypeParser: () => () => 'okay!',

packages/pg/test/integration/client/empty-query-tests.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
'use strict'
22
var helper = require('./test-helper')
33
const suite = new helper.Suite()
4+
const assert = require('assert')
45

56
suite.test('empty query message handling', function (done) {
67
const client = helper.client()

packages/pg/test/integration/client/error-handling-tests.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
var helper = require('./test-helper')
44

55
var pg = helper.pg
6+
const assert = require('assert')
67
const Client = pg.Client
78
const DatabaseError = pg.DatabaseError
89

0 commit comments

Comments
 (0)