From 9dd807344c55751654174f4685ea1b8dc957b3f2 Mon Sep 17 00:00:00 2001 From: sharkdp Date: Sat, 24 Oct 2020 14:45:13 +0200 Subject: [PATCH] Ignore 'all_themes_are_present' unit test by default --- .github/workflows/CICD.yml | 5 +++++ tests/assets.rs | 2 ++ 2 files changed, 7 insertions(+) diff --git a/.github/workflows/CICD.yml b/.github/workflows/CICD.yml index 5ff77927..19d4e927 100644 --- a/.github/workflows/CICD.yml +++ b/.github/workflows/CICD.yml @@ -58,6 +58,11 @@ jobs: with: command: test args: --release + - name: Run ignored-by-default unit tests with new syntaxes and themes + uses: actions-rs/cargo@v1 + with: + command: test + args: --release -- --ignored - name: Syntax highlighting regression test run: tests/syntax-tests/regression_test.sh - name: List of languages diff --git a/tests/assets.rs b/tests/assets.rs index ff61850b..55b5399e 100644 --- a/tests/assets.rs +++ b/tests/assets.rs @@ -1,7 +1,9 @@ use bat::assets::HighlightingAssets; /// This test ensures that we are not accidentally removing themes due to submodule updates. +/// It is 'ignore'd by default because it requires themes.bin to be up-to-date. #[test] +#[ignore] fn all_themes_are_present() { let assets = HighlightingAssets::from_binary();