comparison tests/test-contrib-check-commit.t @ 27783:1d095371de47

check-commit: sort errors by line number
author timeless <timeless@mozdev.org>
date Tue, 12 Jan 2016 08:34:38 +0000
parents 7291c8165e33
children e529b5f1b9e3
comparison
equal deleted inserted replaced
27782:7291c8165e33 27783:1d095371de47
53 > + raise error.Abort(_('no source URL, branch revision, or revision ' 53 > + raise error.Abort(_('no source URL, branch revision, or revision '
54 > 'list provided')) 54 > 'list provided'))
55 > if opts.get('all'): 55 > if opts.get('all'):
56 > EOF 56 > EOF
57 $ cat patch-with-long-header.diff | $TESTDIR/../contrib/check-commit 57 $ cat patch-with-long-header.diff | $TESTDIR/../contrib/check-commit
58 1: username is not an email address
59 # User timeless
60 7: summary keyword should be most user-relevant one-word command or topic
61 transplant/foo: this summary is way too long use Oxford comma (bc) (bug123) (issue 244)
58 7: (BC) needs to be uppercase 62 7: (BC) needs to be uppercase
59 transplant/foo: this summary is way too long use Oxford comma (bc) (bug123) (issue 244)
60 7: no space allowed between issue and number
61 transplant/foo: this summary is way too long use Oxford comma (bc) (bug123) (issue 244) 63 transplant/foo: this summary is way too long use Oxford comma (bc) (bug123) (issue 244)
62 7: use (issueDDDD) instead of bug 64 7: use (issueDDDD) instead of bug
63 transplant/foo: this summary is way too long use Oxford comma (bc) (bug123) (issue 244) 65 transplant/foo: this summary is way too long use Oxford comma (bc) (bug123) (issue 244)
64 1: username is not an email address 66 7: no space allowed between issue and number
65 # User timeless
66 7: summary keyword should be most user-relevant one-word command or topic
67 transplant/foo: this summary is way too long use Oxford comma (bc) (bug123) (issue 244) 67 transplant/foo: this summary is way too long use Oxford comma (bc) (bug123) (issue 244)
68 7: summary line too long (limit is 78) 68 7: summary line too long (limit is 78)
69 transplant/foo: this summary is way too long use Oxford comma (bc) (bug123) (issue 244) 69 transplant/foo: this summary is way too long use Oxford comma (bc) (bug123) (issue 244)
70 [1] 70 [1]
71 71
94 > 94 >
95 > EOF 95 > EOF
96 $ cat patch-with-long-header.diff | $TESTDIR/../contrib/check-commit 96 $ cat patch-with-long-header.diff | $TESTDIR/../contrib/check-commit
97 1: username is not an email address 97 1: username is not an email address
98 # User timeless 98 # User timeless
99 7: don't capitalize summary lines
100 This has no topic and ends with a period.
99 7: summary line doesn't start with 'topic: ' 101 7: summary line doesn't start with 'topic: '
100 This has no topic and ends with a period.
101 7: don't capitalize summary lines
102 This has no topic and ends with a period. 102 This has no topic and ends with a period.
103 7: don't add trailing period on summary line 103 7: don't add trailing period on summary line
104 This has no topic and ends with a period. 104 This has no topic and ends with a period.
105 19: adds double empty line
106 +
107 15: adds double empty line 105 15: adds double empty line
108 + 106 +
109 16: adds a function with foo_bar naming 107 16: adds a function with foo_bar naming
110 + def blah_blah(x): 108 + def blah_blah(x):
109 19: adds double empty line
110 +
111 [1] 111 [1]