Skip to content

Commit c573e18

Browse files
author
Roshan Jossy
committed
Add tests
1 parent 4edc1ec commit c573e18

File tree

3 files changed

+2509
-15
lines changed

3 files changed

+2509
-15
lines changed

api/merge.js

Lines changed: 11 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,13 +3,13 @@ const request = require('request')
33
exports.handleSubmit = (req, res) => {
44
const pullRequest = req.body.pull_request
55
if (shouldHandlePullRequestChange(req)) {
6-
const result = handlePuRequest(pullRequest)
6+
const result = handlePullRequest(pullRequest)
77
res.json({ message: result })
88
}
99
res.json({ message: "Can't handle this pull request" })
1010
}
1111

12-
const handlePuRequest = (pullRequest) => {
12+
const handlePullRequest = (pullRequest) => {
1313
request(pullRequest.diff_url, (error, response, body) => {
1414
logStatusAndErrors(error, response)
1515
if (isChangeInContributorsFile(body)) {
@@ -46,8 +46,8 @@ const isChangeInContributorsFile = diff =>
4646
(diff.match(/Contributors\.md/g) || []).length === 4
4747

4848
const isSingleLineChange = pullRequest =>
49-
(pullRequest.additions === 1 || pullRequest.additions === 2 &&
50-
pullRequest.additions - pullRequest.deletions === 1 ) &&
49+
(pullRequest.additions === 1 || pullRequest.additions === 2) &&
50+
pullRequest.additions - pullRequest.deletions === 1 &&
5151
pullRequest.changed_files === 1
5252

5353
const getPostRequestOptions = (user, url) => ({
@@ -74,3 +74,10 @@ const logStatusAndErrors = (error, response) => {
7474
console.error('statusCode:', response && response.statusCode) // Print the response status code if a response was received
7575
}
7676

77+
module.exports = {
78+
shouldHandlePullRequestChange,
79+
isChangeInContributorsFile,
80+
isSingleLineChange,
81+
getPostRequestOptions,
82+
getMergeMessage
83+
}

package.json

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,11 +4,13 @@
44
"scripts": {
55
"build": "rollup -c",
66
"dev": "rollup -c -w",
7-
"start": "sirv public"
7+
"start": "sirv public",
8+
"test": "jest"
89
},
910
"devDependencies": {
1011
"@rollup/plugin-commonjs": "^11.0.0",
1112
"@rollup/plugin-node-resolve": "^6.0.0",
13+
"jest": "^26.0.1",
1214
"rollup": "^1.20.0",
1315
"rollup-plugin-livereload": "^1.0.0",
1416
"rollup-plugin-svelte": "^5.0.3",

0 commit comments

Comments
 (0)