From 8244eb8ef88cc1b64d27991d6a5c7c69586b8a13 Mon Sep 17 00:00:00 2001 From: David Peter Date: Sun, 22 Aug 2021 16:31:30 +0200 Subject: [PATCH] Experiment: remove CARGO_TEST_OPTIONS In principle, this *could* work. `cargo cross` runs the tests via QEMU. For integration tests, the only difficulty is that we run `bat` from within the tests. But maybe this is handled by assert_cmd. --- .github/workflows/CICD.yml | 11 +---------- 1 file changed, 1 insertion(+), 10 deletions(-) diff --git a/.github/workflows/CICD.yml b/.github/workflows/CICD.yml index 16fdb2e6..028f5ac5 100644 --- a/.github/workflows/CICD.yml +++ b/.github/workflows/CICD.yml @@ -194,21 +194,12 @@ jobs: echo ::set-output name=BIN_PATH::${BIN_PATH} echo ::set-output name=BIN_NAME::${BIN_NAME} - - name: Set testing options - id: test-options - shell: bash - run: | - # test only library unit tests and binary for arm-type targets - unset CARGO_TEST_OPTIONS - unset CARGO_TEST_OPTIONS ; case ${{ matrix.job.target }} in arm-* | aarch64-*) CARGO_TEST_OPTIONS="--lib --bin ${PROJECT_NAME}" ;; esac; - echo ::set-output name=CARGO_TEST_OPTIONS::${CARGO_TEST_OPTIONS} - - name: Run tests uses: actions-rs/cargo@v1 with: use-cross: ${{ matrix.job.use-cross }} command: test - args: --locked --target=${{ matrix.job.target }} ${{ steps.test-options.outputs.CARGO_TEST_OPTIONS}} + args: --locked --target=${{ matrix.job.target }} - name: Run bat uses: actions-rs/cargo@v1