Skip to content

Commit 449bc49

Browse files
committed
Remove || true from tests
1 parent 3d432e3 commit 449bc49

File tree

2 files changed

+7
-7
lines changed

2 files changed

+7
-7
lines changed

.github/workflows/ci-aarch64.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -108,14 +108,14 @@ jobs:
108108
run: |
109109
echo "::warning::This step is designed to always pass, please look for more details"
110110
# timeout added due to hanging enable_thread_pinning example, and should be removed when fixed
111-
timeout 12m .ci/scripts/test.sh --test-kind examples --build-dir __release_lnx_${{ matrix.os.cxx }} --compiler ${{ matrix.os.cxx }} --interface daal/cpp --build-system cmake --backend ref || true
111+
timeout 12m .ci/scripts/test.sh --test-kind examples --build-dir __release_lnx_${{ matrix.os.cxx }} --compiler ${{ matrix.os.cxx }} --interface daal/cpp --build-system cmake --backend ref
112112
env:
113113
CMAKE_GENERATOR: Ninja
114114

115115
- name: oneapi/cpp examples
116116
run: |
117117
echo "::warning::This step is designed to always pass, please look for more details"
118-
.ci/scripts/test.sh --test-kind examples --build-dir __release_lnx_${{ matrix.os.cxx }} --compiler ${{ matrix.os.cxx }} --interface oneapi/cpp --build-system cmake --backend ref || true
118+
.ci/scripts/test.sh --test-kind examples --build-dir __release_lnx_${{ matrix.os.cxx }} --compiler ${{ matrix.os.cxx }} --interface oneapi/cpp --build-system cmake --backend ref
119119
env:
120120
CMAKE_GENERATOR: Ninja
121121

@@ -185,15 +185,15 @@ jobs:
185185
run: |
186186
echo "::warning::This step is designed to always pass, please check logs for more details"
187187
# timeout added due to hanging enable_thread_pinning example, and should be removed when fixed
188-
timeout 12m .ci/scripts/test.sh --test-kind examples --build-dir __release_lnx_${{ matrix.os.cxx }} --compiler ${{ matrix.os.cxx }} --interface daal/cpp --build-system cmake --backend ref --rng-backend openrng || true
188+
timeout 12m .ci/scripts/test.sh --test-kind examples --build-dir __release_lnx_${{ matrix.os.cxx }} --compiler ${{ matrix.os.cxx }} --interface daal/cpp --build-system cmake --backend ref --rng-backend openrng
189189
env:
190190
CMAKE_GENERATOR: Ninja
191191

192192
- name: oneapi/cpp examples
193193
run: |
194194
echo "::warning::This step is designed to always pass, please check logs for more details"
195195
# excludes examples from EXCLUDE_LIST present in examples/daal/cpp/CMakeLists.txt
196-
.ci/scripts/test.sh --test-kind examples --build-dir __release_lnx_${{ matrix.os.cxx }} --compiler ${{ matrix.os.cxx }} --interface oneapi/cpp --build-system cmake --backend ref --rng-backend openrng || true
196+
.ci/scripts/test.sh --test-kind examples --build-dir __release_lnx_${{ matrix.os.cxx }} --compiler ${{ matrix.os.cxx }} --interface oneapi/cpp --build-system cmake --backend ref --rng-backend openrng
197197
env:
198198
CMAKE_GENERATOR: Ninja
199199

dev/bazel/tests/mkl_linkage_test.sh

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -56,11 +56,11 @@ fi
5656
# Static lib must NOT contain MKL archive object files
5757
# (libmkl_core.a, libmkl_intel_ilp64.a, libmkl_tbb_thread.a objects)
5858
AR_CONTENTS=$(ar t "$STATIC_LIB") || { echo "ERROR: ar failed on $STATIC_LIB"; exit 1; }
59-
MKL_ARCHIVE_OBJS=$(echo "$AR_CONTENTS" | grep -c '^_mkl_\|^mkl_blas\|^mkl_vml\|^libmkl' || true)
59+
MKL_ARCHIVE_OBJS=$(echo "$AR_CONTENTS" | grep -c '^_mkl_\|^mkl_blas\|^mkl_vml\|^libmkl)
6060
check "Static lib: no MKL archive objects merged in" "$MKL_ARCHIVE_OBJS"
6161
6262
# Static lib must NOT have defined MKL symbols (only undefined refs are ok)
63-
MKL_DEFINED=$(nm "$STATIC_LIB" 2>/dev/null | grep -c ' [Tt] mkl_' || true)
63+
MKL_DEFINED=$(nm "$STATIC_LIB" 2>/dev/null | grep -c ' [Tt] mkl_')
6464
check "Static lib: no defined MKL symbols (U refs are ok)" "$MKL_DEFINED"
6565
6666
echo ""
@@ -74,7 +74,7 @@ fi
7474
7575
# Dynamic lib must NOT export MKL symbols (they must be hidden).
7676
# Use --defined-only to exclude undefined (U) imports — only defined exports matter.
77-
MKL_EXPORTED=$(nm -D --defined-only "$DYNAMIC_LIB" 2>/dev/null | grep -c ' mkl_' || true)
77+
MKL_EXPORTED=$(nm -D --defined-only "$DYNAMIC_LIB" 2>/dev/null | grep -c ' mkl_')
7878
check "Dynamic lib: no exported MKL symbols (symbols hidden via --exclude-libs)" "$MKL_EXPORTED"
7979
8080
echo ""

0 commit comments

Comments
 (0)