Mercurial > hg
changeset 12743:4c4aeaab2339
check-code: add 'no tab indent' check for unified tests
and fix the offending tests accordingly
author | Adrian Buehlmann <adrian@cadifra.com> |
---|---|
date | Sat, 16 Oct 2010 18:09:01 +0200 |
parents | 6ab4a7d3c179 |
children | 0793d763e413 |
files | contrib/check-code.py tests/test-convert-filemap.t tests/test-convert-git.t tests/test-hgweb-no-path-info.t tests/test-hgweb-no-request-uri.t tests/test-hgweb-non-interactive.t tests/test-inherit-mode.t tests/test-newcgi.t tests/test-parentrevspec.t tests/test-pull-http.t tests/test-push-http.t tests/test-rename-merge2.t |
diffstat | 12 files changed, 90 insertions(+), 89 deletions(-) [+] |
line wrap: on
line diff
--- a/contrib/check-code.py Sun Oct 17 04:14:06 2010 +0200 +++ b/contrib/check-code.py Sat Oct 16 18:09:01 2010 +0200 @@ -72,6 +72,7 @@ ] uprefix = r"^ \$ " +uprefixc = r"^ > " utestpats = [ (uprefix + r'.*\|\s*sed', "use regex test output patterns instead of sed"), (uprefix + r'(true|exit 0)', "explicit zero exit unnecessary"), @@ -79,6 +80,7 @@ (uprefix + r'.*\|\| echo.*(fail|error)', "explicit exit code checks unnecessary"), (uprefix + r'set -e', "don't use set -e"), + (uprefixc + r'( *)\t', "don't use tabs to indent"), ] for p, m in testpats:
--- a/tests/test-convert-filemap.t Sun Oct 17 04:14:06 2010 +0200 +++ b/tests/test-convert-filemap.t Sat Oct 16 18:09:01 2010 +0200 @@ -95,7 +95,7 @@ > fmap="$prefix.fmap" > repo="$prefix.repo" > for i in $files; do - > echo "include $i" >> "$fmap" + > echo "include $i" >> "$fmap" > done > hg -q convert $opts --filemap "$fmap" --datesort source "$repo" > hg up -q -R "$repo"
--- a/tests/test-convert-git.t Sun Oct 17 04:14:06 2010 +0200 +++ b/tests/test-convert-git.t Sat Oct 16 18:09:01 2010 +0200 @@ -116,7 +116,7 @@ > fmap="$prefix.fmap" > repo="$prefix.repo" > for i in $files; do - > echo "include $i" >> "$fmap" + > echo "include $i" >> "$fmap" > done > hg -q convert $opts --filemap "$fmap" --datesort git-repo2 "$repo" > hg up -q -R "$repo"
--- a/tests/test-hgweb-no-path-info.t Sun Oct 17 04:14:06 2010 +0200 +++ b/tests/test-hgweb-no-path-info.t Sat Oct 16 18:09:01 2010 +0200 @@ -24,26 +24,26 @@ > input = StringIO() > > def startrsp(status, headers): - > print '---- STATUS' - > print status - > print '---- HEADERS' - > print [i for i in headers if i[0] != 'ETag'] - > print '---- DATA' - > return output.write + > print '---- STATUS' + > print status + > print '---- HEADERS' + > print [i for i in headers if i[0] != 'ETag'] + > print '---- DATA' + > return output.write > > env = { - > 'wsgi.version': (1, 0), - > 'wsgi.url_scheme': 'http', - > 'wsgi.errors': errors, - > 'wsgi.input': input, - > 'wsgi.multithread': False, - > 'wsgi.multiprocess': False, - > 'wsgi.run_once': False, - > 'REQUEST_METHOD': 'GET', - > 'SCRIPT_NAME': '', - > 'SERVER_NAME': '127.0.0.1', - > 'SERVER_PORT': os.environ['HGPORT'], - > 'SERVER_PROTOCOL': 'HTTP/1.0' + > 'wsgi.version': (1, 0), + > 'wsgi.url_scheme': 'http', + > 'wsgi.errors': errors, + > 'wsgi.input': input, + > 'wsgi.multithread': False, + > 'wsgi.multiprocess': False, + > 'wsgi.run_once': False, + > 'REQUEST_METHOD': 'GET', + > 'SCRIPT_NAME': '', + > 'SERVER_NAME': '127.0.0.1', + > 'SERVER_PORT': os.environ['HGPORT'], + > 'SERVER_PROTOCOL': 'HTTP/1.0' > } > > def process(app):
--- a/tests/test-hgweb-no-request-uri.t Sun Oct 17 04:14:06 2010 +0200 +++ b/tests/test-hgweb-no-request-uri.t Sat Oct 16 18:09:01 2010 +0200 @@ -24,35 +24,34 @@ > input = StringIO() > > def startrsp(status, headers): - > print '---- STATUS' - > print status - > print '---- HEADERS' - > print [i for i in headers if i[0] != 'ETag'] - > print '---- DATA' - > return output.write + > print '---- STATUS' + > print status + > print '---- HEADERS' + > print [i for i in headers if i[0] != 'ETag'] + > print '---- DATA' + > return output.write > > env = { - > 'wsgi.version': (1, 0), - > 'wsgi.url_scheme': 'http', - > 'wsgi.errors': errors, - > 'wsgi.input': input, - > 'wsgi.multithread': False, - > 'wsgi.multiprocess': False, - > 'wsgi.run_once': False, - > 'REQUEST_METHOD': 'GET', - > 'SCRIPT_NAME': '', - > 'SERVER_NAME': '127.0.0.1', - > 'SERVER_PORT': os.environ['HGPORT'], - > 'SERVER_PROTOCOL': 'HTTP/1.0' + > 'wsgi.version': (1, 0), + > 'wsgi.url_scheme': 'http', + > 'wsgi.errors': errors, + > 'wsgi.input': input, + > 'wsgi.multithread': False, + > 'wsgi.multiprocess': False, + > 'wsgi.run_once': False, + > 'REQUEST_METHOD': 'GET', + > 'SCRIPT_NAME': '', + > 'SERVER_NAME': '127.0.0.1', + > 'SERVER_PORT': os.environ['HGPORT'], + > 'SERVER_PROTOCOL': 'HTTP/1.0' > } > > def process(app): - > content = app(env, startrsp) - > sys.stdout.write(output.getvalue()) - > sys.stdout.write(''.join(content)) - > print '---- ERRORS' - > print errors.getvalue() - > + > content = app(env, startrsp) + > sys.stdout.write(output.getvalue()) + > sys.stdout.write(''.join(content)) + > print '---- ERRORS' + > print errors.getvalue() > > output = StringIO() > env['PATH_INFO'] = '/'
--- a/tests/test-hgweb-non-interactive.t Sun Oct 17 04:14:06 2010 +0200 +++ b/tests/test-hgweb-non-interactive.t Sat Oct 16 18:09:01 2010 +0200 @@ -34,28 +34,28 @@ > output = StringIO() > > def startrsp(status, headers): - > print '---- STATUS' - > print status - > print '---- HEADERS' - > print [i for i in headers if i[0] != 'ETag'] - > print '---- DATA' - > return output.write + > print '---- STATUS' + > print status + > print '---- HEADERS' + > print [i for i in headers if i[0] != 'ETag'] + > print '---- DATA' + > return output.write > > env = { - > 'wsgi.version': (1, 0), - > 'wsgi.url_scheme': 'http', - > 'wsgi.errors': errors, - > 'wsgi.input': input, - > 'wsgi.multithread': False, - > 'wsgi.multiprocess': False, - > 'wsgi.run_once': False, - > 'REQUEST_METHOD': 'GET', - > 'SCRIPT_NAME': '', - > 'PATH_INFO': '', - > 'QUERY_STRING': '', - > 'SERVER_NAME': '127.0.0.1', - > 'SERVER_PORT': os.environ['HGPORT'], - > 'SERVER_PROTOCOL': 'HTTP/1.0' + > 'wsgi.version': (1, 0), + > 'wsgi.url_scheme': 'http', + > 'wsgi.errors': errors, + > 'wsgi.input': input, + > 'wsgi.multithread': False, + > 'wsgi.multiprocess': False, + > 'wsgi.run_once': False, + > 'REQUEST_METHOD': 'GET', + > 'SCRIPT_NAME': '', + > 'PATH_INFO': '', + > 'QUERY_STRING': '', + > 'SERVER_NAME': '127.0.0.1', + > 'SERVER_PORT': os.environ['HGPORT'], + > 'SERVER_PROTOCOL': 'HTTP/1.0' > } > > i = hgweb('.')
--- a/tests/test-inherit-mode.t Sun Oct 17 04:14:06 2010 +0200 +++ b/tests/test-inherit-mode.t Sat Oct 16 18:09:01 2010 +0200 @@ -17,12 +17,12 @@ > isdir = {} > for root, dirs, files in os.walk(sys.argv[1]): > for d in dirs: - > name = os.path.join(root, d) - > isdir[name] = 1 - > allnames.append(name) + > name = os.path.join(root, d) + > isdir[name] = 1 + > allnames.append(name) > for f in files: - > name = os.path.join(root, f) - > allnames.append(name) + > name = os.path.join(root, f) + > allnames.append(name) > allnames.sort() > for name in allnames: > suffix = name in isdir and '/' or ''
--- a/tests/test-newcgi.t Sun Oct 17 04:14:06 2010 +0200 +++ b/tests/test-newcgi.t Sat Oct 16 18:09:01 2010 +0200 @@ -16,7 +16,7 @@ > from mercurial.hgweb.request import wsgiapplication > > def make_web_app(): - > return hgweb("test", "Empty test repository") + > return hgweb("test", "Empty test repository") > > wsgicgi.launch(wsgiapplication(make_web_app)) > HGWEB @@ -42,7 +42,7 @@ > from mercurial.hgweb.request import wsgiapplication > > def make_web_app(): - > return hgwebdir("hgweb.config") + > return hgwebdir("hgweb.config") > > wsgicgi.launch(wsgiapplication(make_web_app)) > HGWEBDIR
--- a/tests/test-parentrevspec.t Sun Oct 17 04:14:06 2010 +0200 +++ b/tests/test-parentrevspec.t Sat Oct 16 18:09:01 2010 +0200 @@ -6,11 +6,11 @@ > p2=$3 > > if [ "$p1" ]; then - > hg up -qC $p1 + > hg up -qC $p1 > fi > > if [ "$p2" ]; then - > HGMERGE=true hg merge -q $p2 + > HGMERGE=true hg merge -q $p2 > fi > > echo >> foo @@ -41,8 +41,8 @@ $ lookup() > { > for rev in "$@"; do - > printf "$rev: " - > hg id -nr $rev + > printf "$rev: " + > hg id -nr $rev > done > true > }
--- a/tests/test-pull-http.t Sun Oct 17 04:14:06 2010 +0200 +++ b/tests/test-pull-http.t Sat Oct 16 18:09:01 2010 +0200 @@ -29,12 +29,12 @@ $ cat errors.log $ req() { - > hg serve -p $HGPORT -d --pid-file=hg.pid -E errors.log - > cat hg.pid >> $DAEMON_PIDS - > hg --cwd ../test pull http://localhost:$HGPORT/ - > kill `cat hg.pid` - > echo % serve errors - > cat errors.log + > hg serve -p $HGPORT -d --pid-file=hg.pid -E errors.log + > cat hg.pid >> $DAEMON_PIDS + > hg --cwd ../test pull http://localhost:$HGPORT/ + > kill `cat hg.pid` + > echo % serve errors + > cat errors.log > } expect error, pulling not allowed
--- a/tests/test-push-http.t Sun Oct 17 04:14:06 2010 +0200 +++ b/tests/test-push-http.t Sat Oct 16 18:09:01 2010 +0200 @@ -13,12 +13,12 @@ $ echo a >> a $ hg ci -mb $ req() { - > hg serve -p $HGPORT -d --pid-file=hg.pid -E errors.log - > cat hg.pid >> $DAEMON_PIDS - > hg --cwd ../test2 push http://localhost:$HGPORT/ - > kill `cat hg.pid` - > echo % serve errors - > cat errors.log + > hg serve -p $HGPORT -d --pid-file=hg.pid -E errors.log + > cat hg.pid >> $DAEMON_PIDS + > hg --cwd ../test2 push http://localhost:$HGPORT/ + > kill `cat hg.pid` + > echo % serve errors + > cat errors.log > } $ cd ../test
--- a/tests/test-rename-merge2.t Sun Oct 17 04:14:06 2010 +0200 +++ b/tests/test-rename-merge2.t Sat Oct 16 18:09:01 2010 +0200 @@ -66,8 +66,8 @@ > cp rev $1 > hg add $1 2> /dev/null > if [ "$2" != "" ] ; then - > cp rev $2 - > hg add $2 2> /dev/null + > cp rev $2 + > hg add $2 2> /dev/null > fi > } $ uc() { up $1; hg cp $1 $2; } # update + copy