doc test marked with ignore gets compiled and executed

[japaric]

STR

$ cargo new foo
$ cd foo
$ cat src/lib.rs
/// TEXT GOES HERE
/// ``` ignore,
/// #extern crate foo;
/// #
/// #fn main() {
///     assert!(false);
/// #}
///
fn foo() {}

$ cargo test
     Running target/debug/foo-0201ccfee7015899

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured

   Doc-tests foo

running 1 test
test foo_0 ... FAILED

failures:

---- foo_0 stdout ----
        thread 'foo_0' panicked at 'test executable failed:

thread '<main>' panicked at 'assertion failed: false', <anon>:2
note: Run with `RUST_BACKTRACE=1` for a backtrace.
', ../src/librustdoc/test.rs:311
note: Run with `RUST_BACKTRACE=1` for a backtrace.


failures:
    foo_0

test result: FAILED. 0 passed; 1 failed; 0 ignored; 0 measured

error: test failed

But, if we add an empty line between the start of the doc test fence and TEXT GOES HERE, the doc test is ignored.

 /// TEXT GOES HERE
+///
 /// ``` ignore,
 /// #extern crate foo;
 /// #

$ cargo test
     Running target/debug/foo-0201ccfee7015899

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured

   Doc-tests foo

running 1 test
test foo_0 ... ignored

test result: ok. 0 passed; 0 failed; 1 ignored; 0 measured

Meta

$ rustc -V
rustc 1.9.0-nightly (d5a91e695 2016-03-26)
$ cargo -V
cargo 0.10.0-nightly (25e1301 2016-03-25)

cc @azerupi

[steveklabnik]

I believe this is a duplicate, but can't remember the original...

[GuillaumeGomez]

I'll take a look.

[GuillaumeGomez]

The issue seems to be in the hoedown library. I'll take a deeper look later.

[pmarcelll]

@steveklabnik you are right: #28712

[hadronized]

I just experienced the bug as well now. Is there a workaround?

[GuillaumeGomez]

Adding an empty line before the code block and hoping that the common-mark pull request gets merged some day.

[Phlosioneer]

Triage: This is fixed. Probably because common-mark got merged.

[steveklabnik]

That's correct! nice catch, @Phlosioneer