Opened 7 years ago

Closed 6 years ago

#11859 closed Bugs (fixed)

Wrong handling of "," in Run-Parameters

Reported by: usenet@… Owned by: Raffi Enficiaud
Milestone: Boost 1.62.0 Component: test
Version: Boost 1.60.0 Severity: Optimization
Keywords: test parameter run_test filter Cc:

Description

Documentation reads

patern_list ::= (pattern ',')* pattern

but if I use the ',' like in

--run_test=suite_1/test_1_1,suite_1/suite_1_2/test_1_2_1

respectively variable

BOOST_TEST_RUN_FILTERS=suite_1/test_1_1,suite1_2/test_1_2_1

the result is "Test setup error: no test cases matching filter or all test cases were disabled". With multiple --run_test= all is fine but to use them one has to manually run the Test-Program if one want to state different tests for some rounds to next some rounds.

Change History (5)

comment:1 by Raffi Enficiaud, 7 years ago

Owner: changed from Gennadiy Rozental to Raffi Enficiaud
Status: newassigned

comment:2 by Raffi Enficiaud, 6 years ago

Milestone: To Be DeterminedBoost 1.62.0

comment:3 by Raffi Enficiaud, 6 years ago

The pattern as described in the documentation is correct, however completely misleading. The coma operator is meant to indicate test units that are siblings:

--run_test=suite_1/test_1_1,suite_1/suite_1_2/test_1_2_1

should run the tests under suite1:

suite_1/test_1_1
suite_1/suite_1/suite_1_2/test_1_2_1

which is not what you want.

In order to specify several filters that are not indicating siblings test units, I just added the ':' in order to separate those filters. This also enables specifying several filters from the environment variable.

This should be in develop in a couple of hours. Let's see if it goes to 1.62.

comment:4 by Raffi Enficiaud, 6 years ago

This has been merged to master (rev a8fa08b9b78516b78458ecb8b005d520a0c173f7).

comment:5 by Raffi Enficiaud, 6 years ago

Resolution: fixed
Status: assignedclosed

Released in 1.62

Note: See TracTickets for help on using tickets.