Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Bob Evans <[email protected]>
  • Loading branch information
jsumners-nr and bizob2828 authored Oct 4, 2024
1 parent d9ab77b commit a564237
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions test/versioned/fastify/add-hook.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -116,7 +116,7 @@ test('non-error hooks', async (t) => {
await fastify.listen({ port: 0 })
const address = fastify.server.address()
const result = await common.makeRequest(address, '/add-hook')
match(result, { hello: 'world' })
assert.deepEqual(result, { hello: 'world' })

// verify every hook was called after response
for (const [hookName, isOk] of Object.entries(ok)) {
Expand Down Expand Up @@ -177,7 +177,7 @@ test('error hook', async function errorHookTest(t) {
const address = fastify.server.address()
const result = await common.makeRequest(address, '/error')
assert.ok(ok)
match(result, {
assert.deepEqual(result, {
statusCode: 500,
error: 'Internal Server Error',
message: 'test onError hook'
Expand Down
2 changes: 1 addition & 1 deletion test/versioned/fastify/code-level-metrics-hooks.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@ async function performTest(t) {
const address = fastify.server.address()
const result = await common.makeRequest(address, '/add-hook')

match(result, { hello: 'world' })
assert.deepEqual(result, { hello: 'world' })

assert.equal(txPassed, true, 'transactionFinished assertions passed')
}
Expand Down
2 changes: 1 addition & 1 deletion test/versioned/fastify/naming-common.js
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ module.exports = async function runTests(t, getExpectedSegments) {
await fastify.listen()
const address = fastify.server.address()
const result = await makeRequest(address, '/params/id/parent/edit')
match(result, { id: 'id', parent: 'parent' })
assert.deepEqual(result, { id: 'id', parent: 'parent' })

assert.equal(txPassed, true, 'transactionFinished assertions passed')
})
Expand Down

0 comments on commit a564237

Please sign in to comment.