[tor-bugs] #14804 [Stem]: Confusing arguments to run_tests.py

[Tor Bug Tracker & Wiki]

#14804: Confusing arguments to run_tests.py
---------------------------+--------------------
     Reporter:  Sebastian  |      Owner:  atagar
         Type:  defect     |     Status:  new
     Priority:  normal     |  Milestone:
    Component:  Stem       |    Version:
   Resolution:             |   Keywords:
Actual Points:             |  Parent ID:
       Points:             |
---------------------------+--------------------

Comment (by atagar):

 On reflection the '-t' alias isn't related at all to what you're talking
 about. Anyway,
 [https://gitweb.torproject.org/stem.git/commit/?id=34a900b5affc967f6d9717e527d32ad63a56d1c9
 fixed].

 You're actually highlighting a couple more interesting usability issue.
 Those are now fixed too...

  *
 [https://gitweb.torproject.org/stem.git/commit/?id=7753976dc7477a86fa8bc923292af3a09da7d660
 Balk if provided unrecognized arguments].
  *
 [https://gitweb.torproject.org/stem.git/commit/?id=c9dcc48de7222f8e1819fdef4e324ac7d9176839
 Keep the default run target if none were provided].

 Stem's integration test targets are split into two camps...

  * **Run targets**, which all start with 'RUN_'. Think of these as a
 torrc. Each one you provide means we do a run of our integration tests, so
 '--target RUN_COOKIE,RUN_PASSWORD' would run our tests twice. You can use
 'RUN_ALL' to do all of them.

  * **Attribute targets** like 'ONLINE' and 'CHROOT'. These change
 characteristics of the test runs but aren't a separate run of their own.

 When you provided '--target ONLINE' it replaced our default run target,
 which is why the tests became a no-op. If only attribute targets are
 provided we now keep the default so this should behave and you expected.

 Thanks for the catches!

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/14804#comment:2>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online