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

Doctest failure if $PYTHONPATH is set #23613

Open
koffie opened this issue Aug 11, 2017 · 11 comments
Open

Doctest failure if $PYTHONPATH is set #23613

koffie opened this issue Aug 11, 2017 · 11 comments

Comments

@koffie
Copy link

koffie commented Aug 11, 2017

./sage -t src/sage/tests/cmdline.py
Running doctests with ID 2017-08-11-19-16-39-40d33b31.
Git branch: test_external_package
Using --optional=ccache,mpir,openssl,python2,sage
Doctesting 1 file.
sage -t --warn-long 45.3 src/sage/tests/cmdline.py
**********************************************************************
File "src/sage/tests/cmdline.py", line 208, in sage.tests.cmdline.test_executable
Failed example:
    print(out)
Expected:
    Found local metadata for sqlite-...
    = SQLite =
    ...
    SQLite is a software library that implements a self-contained,
    serverless, zero-configuration, transactional SQL database engine.
    ...
Got:
    Cleaning up PYTHONPATH:
      Old: ":/Users/mderickx/code:/Users/mderickx/code/lmfdb"
      New: "/Users/mderickx/code:/Users/mderickx/code/lmfdb"
    Found local metadata for sqlite-autoconf-3170000
    = SQLite =
    <BLANKLINE>
    == Description ==
    <BLANKLINE>
    SQLite is a software library that implements a self-contained,
    serverless, zero-configuration, transactional SQL database engine.
    <BLANKLINE>
    == License ==
    <BLANKLINE>

Component: scripts

Author: Maarten Derickx

Branch/Commit: u/mderickx/23613 @ db40f22

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

@koffie koffie added this to the sage-8.1 milestone Aug 11, 2017
@koffie
Copy link
Author

koffie commented Aug 11, 2017

comment:1

I thought this would be a quick fix, namely just add

...

to the doctests in order to account for the error. But it seems that the meaning of ... is overloaded in doctests, and that the doctest framework sees ... as an input continuation marker like ....:.

@koffie
Copy link
Author

koffie commented Aug 11, 2017

Branch: u/mderickx/trac_23613

@koffie
Copy link
Author

koffie commented Aug 11, 2017

New commits:

d1222c8Fix doctest that fails when $PYTHONPATH is set

@koffie
Copy link
Author

koffie commented Aug 11, 2017

Commit: d1222c8

@koffie
Copy link
Author

koffie commented Aug 13, 2017

Author: Maarten Derickx

@koffie
Copy link
Author

koffie commented Aug 13, 2017

comment:4

The workaround apparently doesn't work in combination with -tp

@koffie
Copy link
Author

koffie commented Aug 14, 2017

Changed branch from u/mderickx/trac_23613 to u/mderickx/23613

@koffie
Copy link
Author

koffie commented Aug 14, 2017

Changed commit from d1222c8 to db40f22

@koffie
Copy link
Author

koffie commented Aug 14, 2017

New commits:

db40f22Fix doctest that fails when $PYTHONPATH is set: trac #23613

@kevinywlui
Copy link
Mannequin

kevinywlui mannequin commented Aug 23, 2019

comment:8

Merge conflict with #27635

@kevinywlui kevinywlui mannequin added s: needs work and removed s: needs review labels Aug 23, 2019
@DaveWitteMorris
Copy link
Member

comment:9

Removing the "beginner" tag from old tickets. Some could be returned to beginner-friendly status by adding a comment about what needs to be done. Some others might be easy for an experienced developer to finish.

@mkoeppe mkoeppe removed this from the sage-8.1 milestone Dec 29, 2022
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