[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
Re: [tor-bugs] #32613 [Core Tor/Tor]: Simple tests for checkSpaces.pl
#32613: Simple tests for checkSpaces.pl
--------------------------+------------------------------------
Reporter: nickm | Owner: nickm
Type: task | Status: needs_revision
Priority: Medium | Milestone: Tor: 0.4.3.x-final
Component: Core Tor/Tor | Version:
Severity: Normal | Resolution:
Keywords: | Actual Points: .1
Parent ID: #32522 | Points: .1
Reviewer: teor | Sponsor:
--------------------------+------------------------------------
Changes (by teor):
* status: needs_review => needs_revision
Comment:
Replying to [comment:1 nickm]:
> I have a test as ticket32613 with PR at
https://github.com/torproject/tor/pull/1569
Looks fine, please fix a typo in a function name.
> This isn't automated, since we only need to run it when we change our
checkSpace.pl script.
I think we should run it in "make check" (CI), and the commit hook for
these reasons:
* it's fast
* if the CI environment breaks, it's important to have known-good tests
for diagnostics
* CI helps us make sure that any changes work on Linux and macOS, across a
variety of environments
* we're going to change this script at least three times in the next few
months, in #32610, #29226 (perhaps multiple times), and when we replace it
with python
* developers don't usually remember to run extra tests :-)
And skipping Windows is easy, we have plenty of scripts that do that.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/32613#comment:4>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs