Skip to content

Commit 29a91cd

Browse files
committed
Merge branch 'readable-test-fixtures'
* readable-test-fixtures: Make test fixture easier to read
2 parents 2a46458 + 6e5f1ce commit 29a91cd

File tree

8 files changed

+2311
-33
lines changed

8 files changed

+2311
-33
lines changed

src/functions/github-webhook-handler/lib/event-types.test.js

+7-4
Original file line numberDiff line numberDiff line change
@@ -2,18 +2,21 @@
22

33
let assert = require('assert')
44

5-
let pushedEvent = JSON.parse(require('../../../../test/fixtures/pushed').Records[0].Sns.Message)
6-
let prCreatedEvent = JSON.parse(require('../../../../test/fixtures/pr-created').Records[0].Sns.Message)
5+
let pushedEvent = require('../../../../test/fixtures/pushed')
6+
let prCreatedEvent = require('../../../../test/fixtures/pr-created')
77

8+
let snsnize = require('./snsnizer').snsnize
89
let ghEventType = require('./event-types').ghEventType
910

1011
describe('event-types', function() {
1112
describe('ghEventType', function() {
1213
it('should return "push" when the event is pushed event', function() {
13-
assert.equal('push', ghEventType(pushedEvent))
14+
const message = JSON.parse(snsnize(pushedEvent).Records[0].Sns.Message)
15+
assert.equal('push', ghEventType(message))
1416
})
1517
it('should return "pr" when the event is pr-created event', function() {
16-
assert.equal('pr', ghEventType(prCreatedEvent))
18+
const message = JSON.parse(snsnize(prCreatedEvent).Records[0].Sns.Message)
19+
assert.equal('pr', ghEventType(message))
1720
})
1821
})
1922
})
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
'use strict'
2+
3+
// Returns a SNS-styled JSON obj which has `jsonToMessageBody` as `Message` value.
4+
// This method expects to be used as a unit-test helper.
5+
exports.snsnize = (jsonToMessageBody) => {
6+
return {
7+
Records: [{
8+
Sns: {
9+
Message: JSON.stringify(jsonToMessageBody)
10+
}
11+
}]
12+
}
13+
}

test/fixtures/pr-closed.json

+420-5
Large diffs are not rendered by default.

test/fixtures/pr-created.json

+420-5
Large diffs are not rendered by default.

test/fixtures/pr-merged.json

+438-5
Large diffs are not rendered by default.

test/fixtures/pr-reopened.json

+420-5
Large diffs are not rendered by default.

test/fixtures/pr-synchronized.json

+422-5
Large diffs are not rendered by default.

test/fixtures/pushed.json

+171-4
Large diffs are not rendered by default.

0 commit comments

Comments
 (0)