-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
More thorough tests of regex #51
Conversation
Remove unneded character in regex
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Other than the let
this looks fantastic! Thank you for your contributions @benzvan!
@@ -46,12 +46,29 @@ describe('ticketFinder', () => { | |||
}); | |||
|
|||
it('should find Jira ticket when there are tickets in the branch name only', async () => { | |||
const tickets = await ticketFinder({ body: mockBodyWithNothing, head: { ref: 'therynamo/JIRA-123' } }); | |||
// ticket id only | |||
var tickets = await ticketFinder({ body: mockBodyWithNothing, head: { ref: 'JIRA-123' } }); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
var tickets = await ticketFinder({ body: mockBodyWithNothing, head: { ref: 'JIRA-123' } }); | |
let tickets = await ticketFinder({ body: mockBodyWithNothing, head: { ref: 'JIRA-123' } }); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The linter should have caught this, but usually we want to "prefer let
".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Me too. Fixed it.
* More thorough tests of regex Remove unneded character in regex * let for var
Remove unneded character in regex