From cbce87dab11adf8a91c60c307470b18cb2d44b41 Mon Sep 17 00:00:00 2001 From: Andrew Hlynskyi Date: Mon, 28 Aug 2023 14:16:36 +0300 Subject: [PATCH] chore: add an extra notes for hang tests --- cli/src/tests/parser_hang_test.rs | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/cli/src/tests/parser_hang_test.rs b/cli/src/tests/parser_hang_test.rs index b8274804..02a2689c 100644 --- a/cli/src/tests/parser_hang_test.rs +++ b/cli/src/tests/parser_hang_test.rs @@ -15,7 +15,12 @@ use tree_sitter::Parser; // because they don't work well with subprocesses _(it's an assumption)_. // // Bellow are two alternative examples of how to disable tests for some arches -// if a way with excluding the whole mod from compilation would work well. +// if a way with excluding the whole mod from compilation wouldn't work well. +// +// XXX: Also may be it makes sense to keep such tests as ignored by default +// to omit surprises and enable them on CI by passing an extra option explicitly: +// +// > cargo test -- --include-ignored // // #[cfg(all(any(target_arch = "x86_64", target_arch = "x86"), not(sanitizing)))] // #[cfg_attr(not(all(any(target_arch = "x86_64", target_arch = "x86"), not(sanitizing))), ignore)]