Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Minimal test to reproduce issue #160 #161

Closed

Conversation

reillysiemens
Copy link
Contributor

@reillysiemens reillysiemens commented Oct 29, 2017

Here is a minimal test to reproduce the problem describe in #160.

Test output:

[13:02:26] tucker@khanti:~/Projects/gutenberg/components/rendering git:(test_case_issue_160) $ cargo test can_render_body_shortcode_and_paragraph_after
    Finished dev [unoptimized + debuginfo] target(s) in 0.0 secs
     Running /home/tucker/Projects/gutenberg/target/debug/deps/rendering-587d7974cf425a61

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 9 filtered out

     Running /home/tucker/Projects/gutenberg/target/debug/deps/markdown-7ea55b9ed89c348a

running 1 test
test can_render_body_shortcode_and_paragraph_after ... FAILED

failures:

---- can_render_body_shortcode_and_paragraph_after stdout ----
        ("<p>This is a figure caption.</p>\n\n", [])
thread 'can_render_body_shortcode_and_paragraph_after' panicked at 'assertion failed: `(left == right)`
  left: `"<p>This is a figure caption.</p>\n\n"`,
 right: `"<p>This is a figure caption.</p>\n<p>Here is another paragraph.</p>\n"`', tests/markdown.rs:164:4
note: Run with `RUST_BACKTRACE=1` for a backtrace.


failures:
    can_render_body_shortcode_and_paragraph_after

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

error: test failed, to rerun pass '--test markdown'

@reillysiemens
Copy link
Contributor Author

This is superseded by #162 which provides a fix as well.

@reillysiemens reillysiemens deleted the test_case_issue_160 branch November 5, 2017 22:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant