Skip to content

Commit 137a4e8

Browse files
richardlaujoyeecheung
authored andcommitted
test: add tests for release commits (#53)
1 parent fccec73 commit 137a4e8

File tree

2 files changed

+56
-1
lines changed

2 files changed

+56
-1
lines changed

test/rules/reviewers.js

Lines changed: 28 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,5 +35,33 @@ This is a test`
3535
Rule.validate(context)
3636
})
3737

38+
t.test('skip for release commit', (tt) => {
39+
tt.plan(2)
40+
const v = new Validator()
41+
const context = new Commit({
42+
sha: 'e7c077c610afa371430180fbd447bfef60ebc5ea'
43+
, author: {
44+
name: 'Evan Lucas'
45+
, email: '[email protected]'
46+
, date: '2016-04-12T19:42:23Z'
47+
}
48+
, message: `2016-04-12, Version x.y.z
49+
50+
This is a test`
51+
}, v)
52+
53+
context.report = (opts) => {
54+
tt.pass('called report')
55+
tt.strictSame(opts, {
56+
id: 'reviewers'
57+
, message: 'skipping reviewers for release commit'
58+
, string: ''
59+
, level: 'skip'
60+
})
61+
}
62+
63+
Rule.validate(context)
64+
})
65+
3866
t.end()
3967
})

test/rules/subsystem.js

Lines changed: 28 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,34 @@ test('rule: subsystem', (t) => {
3333
Rule.validate(context, {options: {subsystems: Rule.defaults.subsystems}})
3434
})
3535

36+
t.test('skip for release commit', (tt) => {
37+
tt.plan(2)
38+
39+
const v = new Validator()
40+
const context = new Commit({
41+
sha: 'e7c077c610afa371430180fbd447bfef60ebc5ea'
42+
, author: {
43+
name: 'Evan Lucas'
44+
, email: '[email protected]'
45+
, date: '2016-04-12T19:42:23Z'
46+
}
47+
, message: '2016-04-12, Version x.y.z'
48+
}, v)
49+
50+
context.report = (opts) => {
51+
tt.pass('called report')
52+
tt.strictSame(opts, {
53+
id: 'subsystem'
54+
, message: 'Release commits do not have subsystems'
55+
, string: ''
56+
, level: 'skip'
57+
})
58+
tt.end()
59+
}
60+
61+
Rule.validate(context, {options: {subsystems: Rule.defaults.subsystems}})
62+
})
63+
3664
t.test('valid', (tt) => {
3765
tt.plan(2)
3866

@@ -60,6 +88,5 @@ test('rule: subsystem', (t) => {
6088

6189
Rule.validate(context, {options: {subsystems: Rule.defaults.subsystems}})
6290
})
63-
6491
t.end()
6592
})

0 commit comments

Comments
 (0)