Opened 5 years ago

Closed 5 years ago

#7872 closed defect (fixed)

fix re-attach listeners test

Reported by: robinson Owned by: atagar
Priority: Medium Milestone:
Component: Core Tor/Stem Version:
Severity: Keywords:
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

Damian,

Unfortunately, the recent fix[0] for password authentication reduces the three authentication tests to three copies of the same test. This patch[1] restores the three different tests with optional password use.

RUN_COOKIE and RUN_SCOOKIE targets are still broken. I think the way to fix this is to use different call semantics based on the test target. I'll look into that.

[0]: https://gitweb.torproject.org/stem.git/commit/19d9a1ccde704ea05bd8e4b4231820a0b28df974

[1]: https://gitorious.org/stem-robinson/stem-robinson/commits/fix-reattach-listeners-test

Child Tickets

Change History (1)

comment:1 Changed 5 years ago by atagar

Resolution: fixed
Status: newclosed

Ahhh, I forgot the reason we were using multiple methods of authentication. Thanks!

RUN_COOKIE and RUN_SCOOKIE targets are still broken. I think the way to fix this is to use different call semantics based on the test target. I'll look into that.

Included a fix for this...

https://gitweb.torproject.org/stem.git/commitdiff/c482e68ebff140bdde174e15566b6be05f5b5dbd

Note: See TracTickets for help on using tickets.