Skip to content

Commit fb2813d

Browse files
committed
skip instead of returning.
1 parent 474a248 commit fb2813d

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

tests/models/test_modeling_common.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2108,7 +2108,7 @@ def test_hotswapping_compiled_model_linear(self, rank0, rank1):
21082108
@parameterized.expand([(11, 11), (7, 13), (13, 7)]) # important to test small to large and vice versa
21092109
def test_hotswapping_compiled_model_conv2d(self, rank0, rank1):
21102110
if "unet" not in self.model_class.__name__.lower():
2111-
return
2111+
pytest.skip("Test only applies to UNet.")
21122112

21132113
# It's important to add this context to raise an error on recompilation
21142114
target_modules = ["conv", "conv1", "conv2"]
@@ -2118,7 +2118,7 @@ def test_hotswapping_compiled_model_conv2d(self, rank0, rank1):
21182118
@parameterized.expand([(11, 11), (7, 13), (13, 7)]) # important to test small to large and vice versa
21192119
def test_hotswapping_compiled_model_both_linear_and_conv2d(self, rank0, rank1):
21202120
if "unet" not in self.model_class.__name__.lower():
2121-
return
2121+
pytest.skip("Test only applies to UNet.")
21222122

21232123
# It's important to add this context to raise an error on recompilation
21242124
target_modules = ["to_q", "conv"]

0 commit comments

Comments
 (0)