diff options
| author |  Alexander Batischev <eual.jp@gmail.com>
| 2024-11-18 20:22:26 +0300 |
|---|---|---|
| committer | Alexander Batischev <eual.jp@gmail.com>
| 2024-11-18 20:22:26 +0300 |
| commit | f1b880fb5b698d9f047c9d74227d40c3279f882c (patch) | |
| tree | de1373f4afed7aabfb4d4d7dc1d1c72e69388821 | |
| parent | 27e67df0c4f93e207dd1d06aa393fb32acda6321 (diff) | |
| download | newsboat-f1b880fb5b698d9f047c9d74227d40c3279f882c.tar.gz newsboat-f1b880fb5b698d9f047c9d74227d40c3279f882c.tar.zst newsboat-f1b880fb5b698d9f047c9d74227d40c3279f882c.zip | |
Coveralls: write profiles at known-good location
This avoids problems with the two tests that `chdir()` to a non-existent
directory:
getcwd_value_depends_on_current_directory
getcwd_returns_an_error_if_current_directory_doesnt_exist
Even though Clang documentation[1] claims that any intermediate
directories will get created, they apparently aren't, and the tests fail
to write their profiles:
2024-11-17T10:48:24.0635477Z �[1m�[32m Running�[0m tests/getcwd_value_depends_on_current_directory.rs (target/debug/deps/getcwd_value_depends_on_current_directory-4d1a2bc9de6599ab)
2024-11-17T10:48:24.0650449Z
2024-11-17T10:48:24.0651388Z running 1 test
2024-11-17T10:48:24.0654636Z test t_getcwd_value_depends_on_current_directory ... ok
2024-11-17T10:48:24.0655742Z LLVM Profile Error: Failed to write file "fv-az1921-214_2166150040734416821_0.profraw": No such file or directory
2024-11-17T10:48:24.0580399Z �[1m�[32m Running�[0m tests/getcwd_returns_an_error_if_current_directory_doesnt_exist.rs (target/debug/deps/getcwd_returns_an_error_if_current_directory_doesnt_exist-9788325f32678866)
2024-11-17T10:48:24.0593638Z
2024-11-17T10:48:24.0593817Z running 1 test
2024-11-17T10:48:24.0597256Z test t_getcwd_returns_an_error_if_current_directory_doesnt_exist ... ok
2024-11-17T10:48:24.0597646Z
2024-11-17T10:48:24.0598507Z LLVM Profile Error: Failed to write file "fv-az1921-214_17658522863853593920_0.prof
Cf. https://github.com/newsboat/newsboat/pull/2917#issuecomment-2481191986
| -rw-r--r-- | .github/workflows/coveralls.yaml | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/.github/workflows/coveralls.yaml b/.github/workflows/coveralls.yaml index 77e1a8fc..4f1a7591 100644 --- a/.github/workflows/coveralls.yaml +++ b/.github/workflows/coveralls.yaml @@ -23,7 +23,7 @@ jobs: # This prevents our tests from hogging too much of the CPU and failing # due to races. RUST_TEST_THREADS: 2 - LLVM_PROFILE_FILE: '%h_%m.profraw' + LLVM_PROFILE_FILE: '/tmp/newsboat-profiles/%h_%m.profraw' steps: - name: Install dependencies @@ -67,7 +67,7 @@ jobs: run: make --jobs=3 NEWSBOAT_RUN_IGNORED_TESTS=1 ci-check - name: Calculate test coverage - run: grcov . --source-dir . --binary-path . --ignore-not-existing --ignore='/*' --ignore='3rd-party/*' --ignore='doc/*' --ignore='test/*' --ignore='target/*' --ignore='newsboat.cpp' --ignore='podboat.cpp' -t lcov -o coverage.lcov + run: grcov /tmp/newsboat-profiles/ --source-dir . --binary-path . --ignore-not-existing --ignore='/*' --ignore='3rd-party/*' --ignore='doc/*' --ignore='test/*' --ignore='target/*' --ignore='newsboat.cpp' --ignore='podboat.cpp' -t lcov -o coverage.lcov - name: Submit coverage to Coveralls uses: coverallsapp/github-action@v2 |