Skip to content

Commit 28506ea

Browse files
committed
Update fixture docstrings
1 parent b86aea6 commit 28506ea

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

reframe/core/fixtures.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,9 @@ def add(self, fixture, variant_num, parent_name, partitions, prog_envs):
114114
The rationale is as follows for the different scopes:
115115
- session: Only one environment+partition combination per fixture.
116116
This kind of fixture may be shared across all tests. The name
117-
for fixtures with this scope is not mangled by this registry.
117+
for fixtures with this scope is mangled with the system name.
118+
This is necesssary to avoid name conflicts with classes that
119+
are used both as regular tests and fixtures with session scope.
118120
- partition: Only one environment per partition. This kind of fixture
119121
may be shared amongst all the tests running on the same
120122
partition. The name is mangled to include the partition where
@@ -175,7 +177,7 @@ def add(self, fixture, variant_num, parent_name, partitions, prog_envs):
175177

176178
# Register the fixture
177179
if scope == 'session':
178-
# The name is just the class name
180+
# The name is mangled with the system name
179181
name = f'{fname}~{self._sys_name}'
180182

181183
# Select a valid environment supported by a partition

0 commit comments

Comments
 (0)