[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

Re: [tor-bugs] #6205 [Stem]: integration tests fail on Windows



#6205: integration tests fail on Windows
----------------------+-----------------------------------------------------
 Reporter:  reganeet  |          Owner:  reganeet      
     Type:  defect    |         Status:  needs_revision
 Priority:  normal    |      Milestone:                
Component:  Stem      |        Version:                
 Keywords:            |         Parent:                
   Points:            |   Actualpoints:                
----------------------+-----------------------------------------------------
Changes (by atagar):

  * status:  needs_information => needs_revision


Comment:

 > Yes we do; os.getcwd() will provide an absolute path with the drive
 letter.

 Ahh, gotcha.

 > Can we set the default value of "is_extra_tor_running" to False, get rid
 of "if self.is_extra_tor_running is None" in setUp()

 Simplest is probably just to have an 'is_windows()' check and hardcode it
 to False in that case. It would be nice if we could later check this on
 Windows, but not at all vital.

 Pushed your changes with some minor tweaks...
 https://gitweb.torproject.org/stem.git/commitdiff/6f626f3

 Would you mind exercising these new Windows expand_path() use cases in the
 unit tests? If you look in 'test/unit/util/system.py' you'll find a
 'test_expand_path_unix' test. It would be great to have a windows
 counterpart for this. :)

-- 
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/6205#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