Commit 747b7b1
committed
Mark test_distribution/test_layout_map_mesh_shapes as multi_device
test_layout_map_live_presets was the only test in this file carrying
@pytest.mark.multi_device, but it's also marked kaggle_key_required and
extra_large, so PR keras-team#2809's CI job (which filters on
"multi_device and not kaggle_key_required and not extra_large") was
excluding it too -- leaving zero real tests collected from this model.
Mark the two actual multi-device tests instead: test_distribution, and
test_layout_map_mesh_shapes (the Tier-2 CI-safe mesh-shape sweep).
For test_layout_map_mesh_shapes, the marker must go directly above the
def, below @parameterized.named_parameters, not above it: absl's
TestGeneratorMetaclass only forwards attributes set via functools.wraps
inside _ParameterizedTestIter.__iter__ on the original function; a mark
applied outermost lands on the _ParameterizedTestIter object itself and
is silently dropped when the class dict is rewritten, so none of the
12 expanded per-mesh-shape test cases would inherit it. Verified via
collect-only with the same "-m" filter CI uses: 13/19 tests selected
(test_distribution + 12 mesh-shape cases), with test_layout_map_live_presets
and the non-distribution tests correctly excluded either way.1 parent 53aa9d0 commit 747b7b1
1 file changed
Lines changed: 2 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
193 | 193 | | |
194 | 194 | | |
195 | 195 | | |
| 196 | + | |
196 | 197 | | |
197 | 198 | | |
198 | 199 | | |
| |||
253 | 254 | | |
254 | 255 | | |
255 | 256 | | |
| 257 | + | |
256 | 258 | | |
257 | 259 | | |
258 | 260 | | |
| |||
0 commit comments