Skip to content

Commit

Permalink
Specify collation when relied on in tests
Browse files Browse the repository at this point in the history
Fixes #189.
  • Loading branch information
charmander authored and brianc committed Nov 17, 2017
1 parent 9da3a85 commit 0b80aba
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion test/integration/client/api-tests.js
Expand Up @@ -26,7 +26,7 @@ suite.test('callback API', done => {
client.query(config)
client.query('INSERT INTO peep(name) VALUES ($1)', ['aaron'])

client.query('SELECT * FROM peep ORDER BY name', (err, res) => {
client.query('SELECT * FROM peep ORDER BY name COLLATE "C"', (err, res) => {
assert(!err)
assert.equal(res.rowCount, 3)
assert.deepEqual(res.rows, [
Expand Down
4 changes: 2 additions & 2 deletions test/integration/client/prepared-statement-tests.js
Expand Up @@ -122,7 +122,7 @@ var suite = new helper.Suite()
suite.test('with small row count', function (done) {
var query = client.query(new Query({
name: 'get names',
text: 'SELECT name FROM zoom ORDER BY name',
text: 'SELECT name FROM zoom ORDER BY name COLLATE "C"',
rows: 1
}, done))

Expand All @@ -132,7 +132,7 @@ var suite = new helper.Suite()
suite.test('with large row count', function (done) {
var query = client.query(new Query({
name: 'get names',
text: 'SELECT name FROM zoom ORDER BY name',
text: 'SELECT name FROM zoom ORDER BY name COLLATE "C"',
rows: 1000
}, done))
checkForResults(query)
Expand Down
4 changes: 2 additions & 2 deletions test/integration/client/simple-query-tests.js
Expand Up @@ -6,7 +6,7 @@ var Query = helper.pg.Query
test('simple query interface', function () {
var client = helper.client()

var query = client.query(new Query('select name from person order by name'))
var query = client.query(new Query('select name from person order by name collate "C"'))

client.on('drain', client.end.bind(client))

Expand Down Expand Up @@ -43,7 +43,7 @@ test('prepared statements do not mutate params', function () {

var params = [1]

var query = client.query(new Query('select name from person where $1 = 1 order by name', params))
var query = client.query(new Query('select name from person where $1 = 1 order by name collate "C"', params))

assert.deepEqual(params, [1])

Expand Down
2 changes: 1 addition & 1 deletion test/native/evented-api-tests.js
Expand Up @@ -64,7 +64,7 @@ test('parameterized queries', function () {

test('multiple parameters', function () {
var client = setupClient()
var q = client.query(new Query('SELECT name FROM boom WHERE name = $1 or name = $2 ORDER BY name', ['Aaron', 'Brian']))
var q = client.query(new Query('SELECT name FROM boom WHERE name = $1 or name = $2 ORDER BY name COLLATE "C"', ['Aaron', 'Brian']))
assert.emits(q, 'row', function (row) {
assert.equal(row.name, 'Aaron')
assert.emits(q, 'row', function (row) {
Expand Down

0 comments on commit 0b80aba

Please sign in to comment.