Opened 9 months ago

Closed 9 months ago

#32450 closed enhancement (implemented)

test/parseconf: Document include behaviour, and add basic parsing tests

Reported by: teor Owned by: teor
Priority: Medium Milestone: Tor: 0.4.3.x-final
Component: Core Tor/Tor Version:
Severity: Normal Keywords: tor-config, network-team-roadmap-november
Cc: Actual Points: 0.1
Parent ID: #29211 Points: 0.1
Reviewer: nickm Sponsor: Sponsor31-can

Description

We don't document the "%include" behaviour in test_parseconf.sh.

We also don't test the basic script behaviour, without any options.

Child Tickets

Change History (3)

comment:1 Changed 9 months ago by teor

Status: assignedneeds_review

See my PR:

It's a really quick fix. Almost merged it without review :-)

comment:2 Changed 9 months ago by teor

Summary: config: Document include behaviour, and add basic parsing teststest/parseconf: Document include behaviour, and add basic parsing tests

comment:3 Changed 9 months ago by nickm

Resolution: implemented
Status: needs_reviewclosed

Looks fine; merged it.

Note: See TracTickets for help on using tickets.