Sage: Ticket #30268: Use `./sage -t`, not `sage -t` in output of tests ran from `SAGE_ROOT`
https://trac.sagemath.org/ticket/30268
<p>
This is to modify output of tests in the case where
<code>SAGE_ROOT</code> exists and tests are ran from <code>SAGE_ROOT</code>.
</p>
<p>
For example when running <code>make test</code> when building
Sage in an extracted source tarball or a git clone.
</p>
<p>
Before this ticket, failures and timeouts show with <code>sage -t</code>:
</p>
<pre class="wiki">sage -t --random-seed=0 src/sage/misc/latex.py # 1 doctest failed
sage -t --random-seed=0 src/sage/plot/animate.py # Timed out
</pre><p>
After this ticket, failures and timeouts show with <code>./sage -t</code>:
</p>
<pre class="wiki">./sage -t --random-seed=0 src/sage/misc/latex.py # 1 doctest failed
./sage -t --random-seed=0 src/sage/plot/animate.py # Timed out
</pre><p>
This makes the testing process less error-prone.
</p>
<p>
Indeed, the natural next action is to run each test
again by itself, typically by copying and pasting the
lines from the above report. A developer might have
several versions of Sage installed, and the <code>sage</code>
in their <code>PATH</code> might not be the one currently being
tested, so one could easy inadvertently retest with
the wrong Sage and draw wrong conclusions. Since the
paths to tested files in the report are local, the
copy-pasted line must be run from the <code>SAGE_ROOT</code>
directory, so using <code>./sage</code> is perfectly sensible.
</p>
en-usSagehttps://trac.sagemath.org/chrome/site/logo_sagemath_trac.png
https://trac.sagemath.org/ticket/30268
Trac 1.1.6fbisseySun, 02 Aug 2020 08:27:26 GMT
https://trac.sagemath.org/ticket/30268#comment:1
https://trac.sagemath.org/ticket/30268#comment:1
<p>
<code>sage -t</code> is perfectly normal and natural for sage-on-distros. In fact <code>./sage</code> doesn't make any sense there since <code>SAGE_ROOT</code> doesn't exist.
</p>
<p>
If you want to display this, make it dependent on <code>SAGE_ROOT</code> not being undefined please.
</p>
TicketslelievreSun, 02 Aug 2020 10:36:38 GMTdescription, summary changed
https://trac.sagemath.org/ticket/30268#comment:2
https://trac.sagemath.org/ticket/30268#comment:2
<ul>
<li><strong>description</strong>
modified (<a href="/ticket/30268?action=diff&version=2">diff</a>)
</li>
<li><strong>summary</strong>
changed from <em>Use "./sage -t", not "sage -t" in test output</em> to <em>Use `./sage -t`, not `sage -t` in output of tests ran from `SAGE_ROOT`</em>
</li>
</ul>
<p>
Good point. I modified the summary and description accordingly.
</p>
TicketmkoeppeSat, 24 Oct 2020 20:15:01 GMTmilestone changed
https://trac.sagemath.org/ticket/30268#comment:3
https://trac.sagemath.org/ticket/30268#comment:3
<ul>
<li><strong>milestone</strong>
changed from <em>sage-9.2</em> to <em>sage-9.3</em>
</li>
</ul>
Ticket