lourenco/mgmt
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

travis: more strict commit messages tests

Browse Source 
Signed-off-by: Julien Pivotto <roidelapluie@inuits.eu>
This commit is contained in:
Julien Pivotto
2017-02-09 16:18:01 +01:00
parent 72873abe05
commit 90147f3dfb
1 changed files with 9 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
test/test-commit-message.sh
View File

@@ -1,6 +1,6 @@
#!/bin/bash -e
travis_regex='^[a-z0-9]\(\(, \)\|[a-z0-9]\)\+[a-z0-9]: '
travis_regex='^\([a-z0-9]\(\(, \)\|[a-z0-9]\)\+[a-z0-9]: \)\+[^:]\+$'
# Testing the regex itself.
@@ -9,6 +9,8 @@ travis_regex='^[a-z0-9]\(\(, \)\|[a-z0-9]\)\+[a-z0-9]: '
[[ $(echo "foo: bar" | grep -c "$travis_regex") -eq 1 ]]
[[ $(echo "f1oo, b2ar: bar" | grep -c "$travis_regex") -eq 1 ]]
[[ $(echo "2foo: bar" | grep -c "$travis_regex") -eq 1 ]]
[[ $(echo "foo: bar: barfoo" | grep -c "$travis_regex") -eq 1 ]]
[[ $(echo "foo: bar, foo: barfoo" | grep -c "$travis_regex") -eq 1 ]]
# Space required after :
[[ $(echo "foo:bar" | grep -c "$travis_regex") -eq 0 ]]
@@ -28,6 +30,12 @@ travis_regex='^[a-z0-9]\(\(, \)\|[a-z0-9]\)\+[a-z0-9]: '
# More than one char is required before :
[[ $(echo "a: bar" | grep -c "$travis_regex") -eq 0 ]]
# Run checks agains multiple :.
[[ $(echo "a: bar:" | grep -c "$travis_regex") -eq 0 ]]
[[ $(echo "a: bar, fooX: barfoo" | grep -c "$travis_regex") -eq 0 ]]
[[ $(echo "a: bar, foo: barfoo foo: nope" | grep -c "$travis_regex") -eq 0 ]]
[[ $(echo "nope a: bar, foo: barfoofoo: nope" | grep -c "$travis_regex") -eq 0 ]]
test_commit_message() {
echo Testing commit message $1
if ! git log --format=%s $1 | head -n 1 | grep -q "$travis_regex"