Skip to content

Commit

Permalink
Merge pull request #830 from rollbar/wj-ignoredMessages-regex-test
Browse files Browse the repository at this point in the history
Test ignoredMessages with regex including escaped literal
  • Loading branch information
waltjones authored Mar 23, 2020
2 parents 2a8ec70 + bf65a86 commit 3792739
Showing 1 changed file with 39 additions and 0 deletions.
39 changes: 39 additions & 0 deletions test/predicates.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -377,6 +377,45 @@ describe('messageIsIgnored', function() {
};
expect(p.messageIsIgnored(logger)(item, settings)).to.not.be.ok();
});
it('false if ignoredMessages regex match', function() {
var item = {
level: 'critical',
body: {
message: {body: 'This is an ignored message'}
}
};
var settings = {
reportLevel: 'debug',
ignoredMessages: ['^This is an .{7} message$']
};
expect(p.messageIsIgnored(logger)(item, settings)).to.not.be.ok();
});
it('false if ignoredMessages literal match', function() {
var item = {
level: 'critical',
body: {
message: {body: '{"data":{"messages":[{"message":"Unauthorized"}]}}'}
}
};
var settings = {
reportLevel: 'debug',
ignoredMessages: ['{"data":{"messages":\\[']
};
expect(p.messageIsIgnored(logger)(item, settings)).to.not.be.ok();
});
it('false if ignoredMessages more literal regex matches', function() {
var item = {
level: 'critical',
body: {
message: {body: 'Match these characters: (*+?)'}
}
};
var settings = {
reportLevel: 'debug',
ignoredMessages: ['\\(\\*\\+\\?\\)']
};
expect(p.messageIsIgnored(logger)(item, settings)).to.not.be.ok();
});
it('true if both trace and body message but ignoredMessages only match body', function() {
var item = {
level: 'critical',
Expand Down

0 comments on commit 3792739

Please sign in to comment.