Add code check for framework.ExpectEqual()

This adds code check for using framework.ExpectEqual(foo, bar) in
e2e tests for keeping the test code simple.
This commit is contained in:
Kenichi Omichi 2019-07-31 03:30:55 +00:00
parent bdde11a664
commit ad988085f6

View File

@ -42,6 +42,15 @@ do
fi
done
errors_expect_equal=()
for file in "${all_e2e_files[@]}"
do
if grep -E "Expect\(.*\)\.To\((gomega\.Equal|Equal)" "${file}" > /dev/null
then
errors_expect_equal+=( "${file}" )
fi
done
if [ ${#errors_expect_no_error[@]} -ne 0 ]; then
{
echo "Errors:"
@ -70,4 +79,18 @@ if [ ${#errors_expect_error[@]} -ne 0 ]; then
exit 1
fi
if [ ${#errors_expect_equal[@]} -ne 0 ]; then
{
echo "Errors:"
for err in "${errors_expect_equal[@]}"; do
echo "$err"
done
echo
echo 'The above files need to use framework.ExpectEqual(foo, bar) instead of '
echo 'Expect(foo).To(Equal(bar)) or gomega.Expect(foo).To(gomega.Equal(bar))'
echo
} >&2
exit 1
fi
echo 'Congratulations! All e2e test source files are valid.'