Add the ability to have comments in matrix tests

[ollien]

On a project of mine, I found a case where it was useful to have more than one test_matrix invocation on a test. However, there was no way from the test output to differentiate them, so I thought I'd add comment support to identify the individual matrices :)

Example:

  #[test_matrix(
      2,
      [1, 3, 5]
      ; "multiplied by two"
  )]
  #[test_matrix(
      [4, 6, 8],
      [1, 3, 5]
      ; "multiplied by an even"
  )]
  fn multiplication_tests(x: i32, y: i32) {
      assert_eq!(0, (x * y) % 2)
  }

[~/Documents/code/test-case-example] nick@cubert$ cargo t
   Compiling tmp v0.1.0 (/home/nick/Documents/code/test-case-example)
    Finished test [unoptimized + debuginfo] target(s) in 0.24s
     Running unittests src/lib.rs (target/debug/deps/tmp-eca7087e9424c00f)

running 12 tests
test tests::multiplication_tests::multiplied_by_an_even_4_1_expects ... ok
test tests::multiplication_tests::multiplied_by_an_even_4_3_expects ... ok
test tests::multiplication_tests::multiplied_by_an_even_6_5_expects ... ok
test tests::multiplication_tests::multiplied_by_an_even_6_1_expects ... ok
test tests::multiplication_tests::multiplied_by_an_even_4_5_expects ... ok
test tests::multiplication_tests::multiplied_by_an_even_8_1_expects ... ok
test tests::multiplication_tests::multiplied_by_an_even_6_3_expects ... ok
test tests::multiplication_tests::multiplied_by_an_even_8_3_expects ... ok
test tests::multiplication_tests::multiplied_by_an_even_8_5_expects ... ok
test tests::multiplication_tests::multiplied_by_two_2_1_expects ... ok
test tests::multiplication_tests::multiplied_by_two_2_3_expects ... ok
test tests::multiplication_tests::multiplied_by_two_2_5_expects ... ok

test result: ok. 12 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

   Doc-tests tmp

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

[ollien]

I wanted to remove this trait implementation, but it's part of the public interface and I didn't think it was worth forcing a major bump of the crate version. Nothing else in the crate uses it

[ollien]

Looks like the new panic formatting in 1.73 blew up some of the testcases. I'll get this fixed tonight

[luke-biel]

It seems that the tests failed on different target than the one you tried. There's a ./scripts/test_all.sh script that you can use to have complete test suite ran.

[ollien]

Ah yep, looks like nightly needed to be updated (I ran it on stable). Should be fixed now :)