Missing doc tests

[josecelano]

$ cargo pretty-test
thread 'main' panicked at /home/josecelano/.cargo/registry/src/index.crates.io-6f17d22bba15001f/cargo-pretty-test-0.2.0/src/parsing.rs:430:5:
assertion `left == right` failed: the parsed amount of running tests [265, 0, 96, 31] should equal to the number in stats.total [265, 0, 96, 0]
  left: [265, 0, 96, 31]
 right: [265, 0, 96, 0]
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

The whole cargo test output I'm using:

cargo_test_ouput.txt

[zjp-CN]

Is there any reproducible code?

The error shows the amount of last type test is 31 directly from the raw output, but zero tests are actually computed afterr parsing.

Or try the patch to see if it works as expected:
cargo install cargo-pretty-test --git https://github.com/zjp-CN/pretty-test.git --branch experimental

[zjp-CN]

I can reproduce it now. But it's late now. I'll check it out tomorrow.

[josecelano]

I can reproduce it now. But it's late now. I'll check it out tomorrow.

OK. No worries. I would do it, but I have to review the whole code first. You have done a lot!

[zjp-CN]

Found the crux: the regex for parsing doc test items is the literally doc, but should be a \S+ pattern instead.

test src/servers/http/v1/responses/announce.rs - servers::http::v1::responses::announce::NonCompact (line 21) ... ok

And it seems to be an additional part - compile for no_run doc tests.

test src/tracker/peer.rs - tracker::peer::Peer (line 48) - compile ... ok

cargo-pretty-test/src/regex.rs

Line 48 in fde2b43

tree: Regex::new(r"(?m)^test (?P<split>\S+( - should panic)?( - doc \(line \d+\))?) \.\.\. (?P<status>\S+(, .*)?)$").expect(RE_ERROR),

[josecelano]

Found the crux: the regex for parsing doc test items is the literally doc, but should be a \S+ pattern instead.

test src/servers/http/v1/responses/announce.rs - servers::http::v1::responses::announce::NonCompact (line 21) ... ok

And it seems to be an additional part - compile for no_run doc tests.

test src/tracker/peer.rs - tracker::peer::Peer (line 48) - compile ... ok

cargo-pretty-test/src/regex.rs

Line 48 in fde2b43

tree: Regex::new(r"(?m)^test (?P<split>\S+( - should panic)?( - doc \(line \d+\))?) \.\.\. (?P<status>\S+(, .*)?)$").expect(RE_ERROR),

Good catch! That regex deserves a crate for itself :-)

[zjp-CN]

#20 already contains full rustdoc attributes tests

88726b6#diff-b2c08f496e9c45b70893a9e0fc514f87cd9483de279fa6d1114c26f693b172a6R43

so I think the regex for doc tests is tested.