Opened 13 days ago

Last modified 3 days ago

#31909 needs_review defect

Write a stem test to detect #31495 regression

Reported by: nickm Owned by: nickm
Priority: Medium Milestone: Tor: 0.4.2.x-final
Component: Core Tor/Stem Version:
Severity: Normal Keywords: 042-should
Cc: atagar Actual Points:
Parent ID: Points:
Reviewer: atagar Sponsor: Sponsor31-can

Description

We should make a test that tries to do SETCONF UseBridges=1 Bridge="127.0.0.1:1234" to make sure that the setconf is not rejected.

Atagar, are the stem tests a good place to put tor regression tests? If so, what would be the best place in the test hierarchy?

Child Tickets

Change History (7)

comment:1 Changed 13 days ago by nickm

Owner: changed from atagar to nickm
Status: newaccepted

comment:2 Changed 13 days ago by nickm

Status: acceptedneeds_information

comment:3 Changed 7 days ago by atagar

Hi Nick. Below test_set_conf() would be a good spot...

https://gitweb.torproject.org/stem.git/tree/test/integ/control/controller.py#n714

comment:4 Changed 6 days ago by nickm

Branch is ticket31909; PR in https://github.com/torproject/stem/pull/23

comment:5 Changed 6 days ago by nickm

Status: needs_informationneeds_review

comment:6 Changed 5 days ago by nickm

Reviewer: atagar

comment:7 Changed 3 days ago by atagar

Thanks Nick! I'm digging into another of asn's big hidden service branches (#31823) so it'll probably be a couple weeks before I merge this. Sorry in advance!

Note: See TracTickets for help on using tickets.