Skip to content

Commit d6830a4

Browse files
committed
test: Simplify scripts to ensure exit on first error
1 parent 3442b61 commit d6830a4

File tree

2 files changed

+19
-24
lines changed

2 files changed

+19
-24
lines changed

test/Docker/test-serializer.sh

Lines changed: 13 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,12 @@
11
#!/bin/bash
22

3+
set -ex
4+
35
# This is a script to build the project and run the test suite in the base
46
# Docker container.
57

6-
die() {
7-
echo "Error: $@" 1>&2
8-
exit 1;
9-
}
10-
8+
# jsoncpp
9+
cd /usr/src
1110
git clone git://github.com/Slicer/jsoncpp.git
1211
mkdir jsoncpp-build && cd $_
1312
cmake -G Ninja \
@@ -19,30 +18,29 @@ cmake -G Ninja \
1918
-DJSONCPP_WITH_CMAKE_PACKAGE:BOOL=ON \
2019
-DBUILD_SHARED_LIBS:BOOL=ON \
2120
-DBUILD_STATIC_LIBS:BOOL=OFF \
22-
../jsoncpp || die "JsonCpp: CMake configuration failed"
23-
ninja install || die "JsonCpp: Build failed"
21+
../jsoncpp
22+
ninja install
2423

24+
# ParameterSerializer
2525
cd /usr/src
26-
2726
git clone git://github.com/jcfr/ParameterSerializer.git -b conditonally-add-subdirectory
2827
mkdir ParameterSerializer-build && cd $_
2928
cmake \
3029
-G Ninja \
3130
-DBUILD_TESTING:BOOL=OFF \
3231
-DCMAKE_BUILD_TYPE:STRING=Release \
33-
/usr/src/ParameterSerializer || die "ParameterSerializer: CMake configuration failed"
34-
ninja || die "ParameterSerializer: Build failed"
35-
36-
cd /usr/src/SlicerExecutionModel-build || die "Could not cd into the build directory"
32+
/usr/src/ParameterSerializer
33+
ninja
3734

35+
# SlicerExecutionModel
36+
mkdir -p /usr/src/SlicerExecutionModel-build && cd $_
3837
BUILDNAME=sem_use_serializer-on_$1
39-
4038
cmake \
4139
-G Ninja \
4240
-DCMAKE_BUILD_TYPE:STRING=Release \
4341
-DBUILDNAME:STRING=$BUILDNAME \
4442
-DParameterSerializer_DIR:PATH=/usr/src/ParameterSerializer-build \
4543
-DSlicerExecutionModel_USE_SERIALIZER:BOOL=ON \
4644
-DSlicerExecutionModel_USE_JSONCPP:BOOL=ON \
47-
/usr/src/SlicerExecutionModel || die "SlicerExecutionModel: CMake configuration failed"
48-
ctest -VV -D Experimental || die "SlicerExecutionModel: ctest failed"
45+
/usr/src/SlicerExecutionModel
46+
ctest -VV -D Experimental

test/Docker/test.sh

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,15 @@
11
#!/bin/bash
22

3+
set -ex
4+
35
# This is a script to build the project and run the test suite in the base
46
# Docker container.
57

6-
die() {
7-
echo "Error: $@" 1>&2
8-
exit 1;
9-
}
10-
11-
cd /usr/src/SlicerExecutionModel-build || die "Could not cd into the build directory"
12-
8+
# SlicerExecutionModel
9+
mkdir -p /usr/src/SlicerExecutionModel-build && cd $_
1310
cmake \
1411
-G Ninja \
1512
-DCMAKE_BUILD_TYPE:STRING=Release \
1613
-DBUILDNAME:STRING=$1 \
17-
/usr/src/SlicerExecutionModel || die "CMake configuration failed"
18-
ctest -VV -D Experimental || die "ctest failed"
14+
/usr/src/SlicerExecutionModel
15+
ctest -VV -D Experimental

0 commit comments

Comments
 (0)