Skip to content

Commit

Permalink
Fix i18n test that doesn't fill placeholders
Browse files Browse the repository at this point in the history
  • Loading branch information
schwma committed Oct 7, 2024
1 parent 53a8a80 commit aff200e
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
3 changes: 2 additions & 1 deletion test/resources/error-handling/i18n/messages_en.properties
Original file line number Diff line number Diff line change
@@ -1 +1,2 @@
ORDER_EXCEEDS_STOCK=The order of {0} books exceeds the stock by {1}
ORDER_EXCEEDS_STOCK=The order of {0} books exceeds the stock by {1}
ORDER_EXCEEDS_STOCK_NO_ARGS=The order of books exceeds the stock
2 changes: 1 addition & 1 deletion test/resources/error-handling/srv/custom-handler-errors.js
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ module.exports = srv => {

if (id === 1) req.reject(400, message, args)
if (id === 2) req.reject(code, message, args)
req.reject(message)
req.reject('ORDER_EXCEEDS_STOCK_NO_ARGS')
}
})
}
2 changes: 1 addition & 1 deletion test/tests/error-handling-dev.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -469,7 +469,7 @@ describe('graphql - error handling in development', () => {
`
const errors = [
{
message: 'The order of NULL books exceeds the stock by NULL',
message: 'The order of books exceeds the stock',
extensions: {
code: '500',
numericSeverity: 4,
Expand Down

0 comments on commit aff200e

Please sign in to comment.