Skip to content

Commit 1ac353d

Browse files
committed
fix has-matching-changes to reflect latest script changes
1 parent 30ade3c commit 1ac353d

File tree

1 file changed

+5
-17
lines changed

1 file changed

+5
-17
lines changed

buildspec.yml

Lines changed: 5 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -46,23 +46,11 @@ phases:
4646
tox -e py27,py36,stats -- tests/
4747

4848
# spark integration tests
49-
- cd $CODEBUILD_SRC_DIR/sagemaker-spark-sdk
50-
- HAS_MATCHING_CHANGES_OUTPUT=$(has-matching-changes "src/" "test/" "build.sbt")
51-
- |
52-
if [ "$HAS_MATCHING_CHANGES" = "Changes Found" ] ; then
53-
cd $CODEBUILD_SRC_DIR/integration-tests/sagemaker-spark-sdk
54-
sbt -Dsbt.log.noformat=true it:test
55-
else
56-
echo "skipping spark integration tests"
57-
fi
49+
- cd $CODEBUILD_SRC_DIR/integration-tests/sagemaker-spark-sdk
50+
- test_cmd="sbt -Dsbt.log.noformat=true it:test"
51+
- execute-command-if-has-matching-changes "$test_cmd" "src/" "test/" "build.sbt"
5852

5953
# pyspark integration tests
6054
- cd $CODEBUILD_SRC_DIR/sagemaker-pyspark-sdk
61-
- HAS_MATCHING_CHANGES_OUTPUT=$(has-matching-changes "src/" "tests/" "setup.*" "requirements.txt" "tox.ini)
62-
- |
63-
if [ "$HAS_MATCHING_CHANGES" = "Changes Found" ] ; then
64-
IGNORE_COVERAGE=- \
65-
tox -e py36 -- $CODEBUILD_SRC_DIR/integration-tests/sagemaker-pyspark-sdk/tests/ -n 10 --boxed --reruns 2
66-
else
67-
echo "skipping pyspark integration tests"
68-
fi
55+
- test_cmd="IGNORE_COVERAGE=- && tox -e py36 -- $CODEBUILD_SRC_DIR/integration-tests/sagemaker-pyspark-sdk/tests/ -n 10 --boxed --reruns 2"
56+
- execute-command-if-has-matching-changes "$test_cmd" "src/" "tests/" "setup.*" "requirements.txt" "tox.ini"

0 commit comments

Comments
 (0)