Fork us on GitHub Follow us on Facebook Follow us on Twitter

Opened 3 years ago

Last modified 6 months ago

#811 new defect

PCUT should not prefix test case identifiers with "test_"

Reported by: Jiri Svoboda Owned by:
Priority: major Milestone:
Component: helenos/lib/other Version: mainline
Keywords: Cc:
Blocker for: Depends on:
See also:


Declaring something like

int test_foo;


fails to compile because foo is mangled to test_foo. Perhaps test case identifiers should be mangled to something that the programmer is less likely to use accidentally, such as _test_ or _pcut_, for example.

Change History (3)

comment:1 Changed 3 years ago by Jiří Zárevúcky

I don't see why that matters. Test case names are decorative only and in the unlikely case there is a conflict, you can just rename the testcase. Also, since PCUT is maintained in its own repo, this issue should be taken there.

comment:2 Changed 3 years ago by Jiri Svoboda

It happened to me several times so it's not unlikely. The fact that you can work around it does not mean it should not be fixed.

I created issue on Github

comment:3 Changed 6 months ago by Jiri Svoboda

This was fixed in PCUT upstream, but PCUT in HelenOS needs updating.

Note: See TracTickets for help on using tickets.