98448dce44
Added the bashate script style checker to the pep8 check target in tox.ini. It actually found two valid issues - a bad function declaration and a local variable issue, but mostly just indentation noise. Fixed all the complaints. Change-Id: I43b60e7dcf53acf259c8a52b248fbb8c63d3c8d4
69 lines
2.3 KiB
Bash
Executable File
69 lines
2.3 KiB
Bash
Executable File
#! /bin/sh
|
|
|
|
# Copyright (C) 2014 VA Linux Systems Japan K.K.
|
|
# Copyright (C) 2014 YAMAMOTO Takashi <yamamoto at valinux co jp>
|
|
# All Rights Reserved.
|
|
#
|
|
# Licensed under the Apache License, Version 2.0 (the "License"); you may
|
|
# not use this file except in compliance with the License. You may obtain
|
|
# a copy of the License at
|
|
#
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
#
|
|
# Unless required by applicable law or agreed to in writing, software
|
|
# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
|
|
# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
|
|
# License for the specific language governing permissions and limitations
|
|
# under the License.
|
|
|
|
TMPDIR=`mktemp -d /tmp/${0##*/}.XXXXXX` || exit 1
|
|
export TMPDIR
|
|
trap "rm -rf $TMPDIR" EXIT
|
|
|
|
FAILURES=$TMPDIR/failures
|
|
|
|
|
|
check_pot_files_errors () {
|
|
find octavia -type f -regex '.*\.pot?' \
|
|
-print0|xargs -0 -n 1 --no-run-if-empty msgfmt \
|
|
--check-format -o /dev/null
|
|
if [ "$?" -ne 0 ]; then
|
|
echo "PO files syntax is not correct!" >>$FAILURES
|
|
fi
|
|
}
|
|
|
|
check_identical_policy_files () {
|
|
# For unit tests, we maintain their own policy.json file to make test suite
|
|
# independent of whether it's executed from the octavia source tree or from
|
|
# site-packages installation path. We don't want two copies of the same
|
|
# file to diverge, so checking that they are identical
|
|
diff etc/policy.json octavia/tests/etc/policy.json 2>&1 > /dev/null
|
|
if [ "$?" -ne 0 ]; then
|
|
echo "policy.json files must be identical!" >>$FAILURES
|
|
fi
|
|
}
|
|
|
|
check_no_duplicate_api_test_idempotent_ids() {
|
|
# For API tests, an idempotent ID is assigned to each single API test,
|
|
# those IDs should be unique
|
|
output=$(check-uuid --package octavia.tests.tempest)
|
|
if [ "$?" -ne 0 ]; then
|
|
echo "There are duplicate idempotent ids in the API tests" >>$FAILURES
|
|
echo "please, assign unique uuids to each API test:" >>$FAILURES
|
|
echo "$output" >>$FAILURES
|
|
fi
|
|
}
|
|
|
|
# Add your checks here...
|
|
check_pot_files_errors
|
|
# TODO(johnsom) Uncomment when we have policies to test
|
|
# check_identical_policy_files
|
|
# TODO(johnsom) Uncomment when we have idempotent ids
|
|
# check_no_duplicate_api_test_idempotent_ids
|
|
|
|
# Fail, if there are emitted failures
|
|
if [ -f $FAILURES ]; then
|
|
cat $FAILURES
|
|
exit 1
|
|
fi
|