[Add][Test] Suspend

Signed-off-by: Mark Van de Vyver <[email protected]>
tikv · Sep 13, 2023 · 628f170 · 628f170
1 parent 23422fb
commit 628f170
Show file tree

Hide file tree

Showing 6 changed files with 82 additions and 7 deletions.
diff --git a/minitrace-macro/tests/expand.rs b/minitrace-macro/tests/expand.rs
@@ -1,25 +1,32 @@
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 pub fn expand_defaults_dev() {
     // To generate macro result files
     macrotest::expand("tests/expand/defaults/*.rs");
 }
 
 #[test]
 #[cfg(feature = "ci")]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 pub fn expand_defaults_ci() {
     // To test generated macro result files
     macrotest::expand_without_refresh("tests/expand/defaults/*.rs");
 }
 
 #[test]
 #[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 pub fn expand_non_defaults_dev() {
     // To generate macro result files
     macrotest::expand("tests/expand/non-defaults/*.rs");
 }
 
 #[test]
 #[cfg(feature = "ci")]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 pub fn expand_non_defaults_ci() {
     // To test generated macro result files
     macrotest::expand_without_refresh("tests/expand/non-defaults/*.rs");

diff --git a/minitrace-macro/tests/spans.rs b/minitrace-macro/tests/spans.rs
@@ -1,4 +1,6 @@
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn spans() {
     let t = trybuild::TestCases::new();
     t.pass("tests/spans/*.rs");

diff --git a/minitrace-macro/tests/trace-dev.rs b/minitrace-macro/tests/trace-dev.rs
@@ -1,12 +1,14 @@
 // Useful while working on specific test cases
 #[test]
 #[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn trace_err_dev() {
     let t = trybuild::TestCases::new();
     t.compile_fail("tests/trace/ui/err/006-has-too-many-arguments.rs");
 }
 #[test]
 #[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn trace_ok_dev() {
     let t = trybuild::TestCases::new();
     t.pass("tests/trace/ui/ok/00-has-no-arguments.rs");

diff --git a/minitrace-macro/tests/trace.rs b/minitrace-macro/tests/trace.rs
@@ -1,157 +1,219 @@
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn errors() {
     let t = trybuild::TestCases::new();
     t.compile_fail("tests/trace/ui/err/*.rs");
 }
 
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn error_001() {
     let t = trybuild::TestCases::new();
     t.compile_fail("tests/trace/ui/err/001-*.rs");
 }
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn error_002() {
     let t = trybuild::TestCases::new();
     t.compile_fail("tests/trace/ui/err/002-*.rs");
 }
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn error_003() {
     let t = trybuild::TestCases::new();
     t.compile_fail("tests/trace/ui/err/003-*.rs");
 }
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn error_004() {
     let t = trybuild::TestCases::new();
     t.compile_fail("tests/trace/ui/err/004-*.rs");
 }
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn error_005() {
     let t = trybuild::TestCases::new();
     t.compile_fail("tests/trace/ui/err/005-*.rs");
 }
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn error_006() {
     let t = trybuild::TestCases::new();
     t.compile_fail("tests/trace/ui/err/006-*.rs");
 }
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn error_007() {
     let t = trybuild::TestCases::new();
     t.compile_fail("tests/trace/ui/err/007-*.rs");
 }
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn error_008() {
     let t = trybuild::TestCases::new();
     t.compile_fail("tests/trace/ui/err/008-*.rs");
 }
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn error_009() {
     let t = trybuild::TestCases::new();
     t.compile_fail("tests/trace/ui/err/009-*.rs");
 }
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn error_010() {
     let t = trybuild::TestCases::new();
     t.compile_fail("tests/trace/ui/err/010-*.rs");
 }
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn error_011() {
     let t = trybuild::TestCases::new();
     t.compile_fail("tests/trace/ui/err/011-*.rs");
 }
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn error_012() {
     let t = trybuild::TestCases::new();
     t.compile_fail("tests/trace/ui/err/012-*.rs");
 }
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn error_013() {
     let t = trybuild::TestCases::new();
     t.compile_fail("tests/trace/ui/err/013-*.rs");
 }
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn error_014() {
     let t = trybuild::TestCases::new();
     t.compile_fail("tests/trace/ui/err/014-*.rs");
 }
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn error_015() {
     let t = trybuild::TestCases::new();
     t.compile_fail("tests/trace/ui/err/015-*.rs");
 }
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn error_016() {
     let t = trybuild::TestCases::new();
     t.compile_fail("tests/trace/ui/err/016-*.rs");
 }
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn error_017() {
     let t = trybuild::TestCases::new();
     t.compile_fail("tests/trace/ui/err/017-*.rs");
 }
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn error_018() {
     let t = trybuild::TestCases::new();
     t.compile_fail("tests/trace/ui/err/018-*.rs");
 }
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn error_019() {
     let t = trybuild::TestCases::new();
     t.compile_fail("tests/trace/ui/err/019-*.rs");
 }
 
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn oks() {
     let t = trybuild::TestCases::new();
     t.pass("tests/trace/ui/ok/*.rs");
 }
 
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn ok_001() {
     let t = trybuild::TestCases::new();
     t.pass("tests/trace/ui/ok/001-*.rs");
 }
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn ok_002() {
     let t = trybuild::TestCases::new();
     t.pass("tests/trace/ui/ok/002-*.rs");
 }
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn ok_003() {
     let t = trybuild::TestCases::new();
     t.pass("tests/trace/ui/ok/003-*.rs");
 }
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn ok_004() {
     let t = trybuild::TestCases::new();
     t.pass("tests/trace/ui/ok/004-*.rs");
 }
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn ok_005() {
     let t = trybuild::TestCases::new();
     t.pass("tests/trace/ui/ok/005-*.rs");
 }
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn ok_006() {
     let t = trybuild::TestCases::new();
     t.pass("tests/trace/ui/ok/006-*.rs");
 }
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn ok_007() {
     let t = trybuild::TestCases::new();
     t.pass("tests/trace/ui/ok/007-*.rs");
 }
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn ok_008() {
     let t = trybuild::TestCases::new();
     t.pass("tests/trace/ui/ok/008-*.rs");
 }
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn ok_009() {
     let t = trybuild::TestCases::new();
     t.pass("tests/trace/ui/ok/009-*.rs");
 }
 #[test]
+#[ignore]
+// Integration tests to be migrated once PR #127 lands and closes #113
 fn ok_010() {
     let t = trybuild::TestCases::new();
     t.pass("tests/trace/ui/ok/010-*.rs");

diff --git a/minitrace-tests/Cargo.toml b/minitrace-tests/Cargo.toml
@@ -33,13 +33,15 @@ inventory = "0.2.2"
 # NOTE:
 # Features are only recognized via the command line in the following tests.
 #
-# cargo test --features minitrace-tests/tk --manifest-path minitrace-tests/Cargo.toml indev-tokio -- --nocapture
+# cargo test --features "minitrace-tests/default minitrace-tests/tk" \
+#            --manifest-path minitrace-tests/Cargo.toml \
+#            indev-tokio -- --nocapture
 #
-[[test]]
-name = "indev-tokio"
-path = "integration/indev.rs"
-harness = false
-required-features = ["default", "tk"]
+# [[test]]
+# name = "indev-tokio"
+# path = "integration/indev.rs"
+# harness = false
+# required-features = ["default", "tk"]
 
 # [[test]]
 # name = "dev-tokio"

diff --git a/minitrace-tests/README.md b/minitrace-tests/README.md
@@ -37,7 +37,7 @@ Thanks to the following for [how to do this](https://www.infinyon.com/blog/2021/
 4. Run the single `indev` test case:
 
     ```bash
-    cargo test --features "default minitrace-tests/tk" --manifest-path minitrace-tests/Cargo.toml \
+    cargo test --features "minitrace-tests/default minitrace-tests/tk" --manifest-path minitrace-tests/Cargo.toml \
                indev-tokio \
                -- --nocapture
     ```