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

1 optional doctest failing in cmdline.py related to pandoc #27172

Closed
seblabbe opened this issue Jan 29, 2019 · 9 comments
Closed

1 optional doctest failing in cmdline.py related to pandoc #27172

seblabbe opened this issue Jan 29, 2019 · 9 comments

Comments

@seblabbe
Copy link
Contributor

As reported in 8.7.beta0 on sage-release:

sage -t --long --optional=sage,pandoc src/sage/tests/cmdline.py

yields:

**********************************************************************
File "src/sage/tests/cmdline.py", line 820, in sage.tests.cmdline.test_executable
Failed example:
    print(open(output, 'r').read() == t)          # optional - pandoc
Expected:
    True
Got:
    False
**********************************************************************
1 item had failures:
   1 of 253 in sage.tests.cmdline.test_executable
    [252 tests, 1 failure, 41.02 s]

The reason is that:

sage: open(output, 'r').read()
'.. escape-backslashes\n.. default-role:: math\n\n\n::\n\n    sage: 1+1\n    2\n\n\n\n\n'
sage: t
'.. escape-backslashes\n.. default-role:: math\n\n\n::\n\n    sage: 1+1\n    2\n\n\n'

CC: @sagetrac-tmonteil

Component: packages: optional

Author: Sébastien Labbé

Branch/Commit: f2579fd

Reviewer: Travis Scrimshaw

Issue created by migration from https://trac.sagemath.org/ticket/27172

@seblabbe

This comment has been minimized.

@seblabbe

This comment has been minimized.

@seblabbe
Copy link
Contributor Author

Branch: u/slabbe/27172

@seblabbe
Copy link
Contributor Author

Commit: f2579fd

@seblabbe
Copy link
Contributor Author

Author: Sébastien Labbé

@seblabbe
Copy link
Contributor Author

New commits:

f2579fd27172: fixing optional pandoc doctest

@tscrim
Copy link
Collaborator

tscrim commented Jan 30, 2019

Reviewer: Travis Scrimshaw

@tscrim
Copy link
Collaborator

tscrim commented Jan 30, 2019

comment:5

LGTM.

@vbraun
Copy link
Member

vbraun commented Feb 4, 2019

Changed branch from u/slabbe/27172 to f2579fd

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants