Fix failing doctest in src/sage/interfaces/tachyon.py

[kliem]

sage -t src/sage/interfaces/tachyon.py
**********************************************************************
File "src/sage/interfaces/tachyon.py", line 175, in sage.interfaces.tachyon.TachyonRT.usage
Failed example:
    t.usage(use_pager=False)
Expected:
    Tachyon Parallel/Multiprocessor Ray Tracer   Version...
Got:
    Usage: tachyon modelfile [options] 
    <BLANKLINE>
    Valid options and accepted formats are as follows (** denotes default behaviour).
    <BLANKLINE>
    Model file formats supported:
...

This seems to have been introduced by using the systems tachyon. It appears that in some cases (depending on the version probably) tachyon does not print out the first lines

Tachyon Parallel/Multiprocessor Ray Tracer   Version 0.98.9   
Copyright 1994-2010,    John E. Stone <john.stone@gmail.com> 
------------------------------------------------------------ 

We fix this by testing for the following output:

...
Usage: 
  tachyon modelfile [options]...
<BLANKLINE>
Model file formats supported:
  filename.dat ...

CC: @mkoeppe

Component: interfaces

Keywords: tachyon

Author: Jonathan Kliem

Branch/Commit: 871df4b

Reviewer: Frédéric Chapoton, Matthias Koeppe

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

[kliem]

New commits:

7c25dcc

fix tachyon interface error

[kliem]

Commit: 7c25dcc

[kliem]

Branch: public/29442

[fchapoton]

Reviewer: Frédéric Chapoton

[fchapoton]

comment:2

ok, thanks

[kliem]

comment:3

I should have double checked.

Usage: tachyon modelfile [options] 
    <BLANKLINE>
    Valid options and accepted formats are as follows (** denotes default behaviour).
    <BLANKLINE>
    Model file formats supported:

[sagetrac-git]

Changed commit from 7c25dcc to 80c1d75

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

80c1d75

more flexible test

[kliem]

comment:5

Sorry, but the ticket as it was would have been pointless.

---

New commits:

80c1d75

more flexible test

---

New commits:

80c1d75

more flexible test

[mkoeppe]

comment:6

I don't think doctest results starting with "..." work because they are mistaken for continuation prompts.
See https://github.com/mkoeppe/sage/runs/553984065

File "src/sage/interfaces/tachyon.py", line 175, in sage.interfaces.tachyon.TachyonRT.usage
Failed example:
    t.usage(use_pager=False)
Expected:
    ...
    Usage:
      tachyon modelfile [options]...
    Model file formats supported:
      filename.dat ...
Got:
    Usage: tachyon modelfile [options] 
    <BLANKLINE>
    Valid options and accepted formats are as follows (** denotes default behaviour).
    <BLANKLINE>

[kliem]

comment:7

Ok. Now, I'm testing internally the following and it seems it passes:

            sage: from sage.interfaces.tachyon import TachyonRT
            sage: t = TachyonRT()
            sage: t.usage(use_pager=False)
            ...
              tachyon modelfile [options]...
            <BLANKLINE>
            Model file formats supported:
              filename.dat ...
            sage: print("Usage: tachyon modelfile [options]\n\nValid options and accepted formats are as follows (** denotes default behaviour).\n\nModel file formats supported    +++:\n  filename.dat        the model files originated with this package")
            ...
              tachyon modelfile [options]...
            Model file formats supported:
              filename.dat ...

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

871df4b

doctest should work with both versions now

[sagetrac-git]

Changed commit from 80c1d75 to 871df4b

[kliem]

comment:10

This seems to work now

https://github.com/kliem/sage-test-27122/actions/runs/70559675

There are many clean runs now.

[mkoeppe]

Changed reviewer from Frédéric Chapoton to Frédéric Chapoton, Matthias Koeppe

[mkoeppe]

comment:12

Looks good to me.

[vbraun]

Changed branch from public/29442 to 871df4b