Skip to content

Commit 2ffd220

Browse files
committed
Test all linux platforms defined in utils_pypi
Use the list of linux platforms as defined in utils_pypi for testing, instead of redefining the list in the test. This ensures that newly added platforms are tested and fixes the problem that the list in the test was already outdated. Signed-off-by: Martin Nonnenmacher <[email protected]>
1 parent af03795 commit 2ffd220

File tree

1 file changed

+1
-10
lines changed

1 file changed

+1
-10
lines changed

tests/test_utils_pypi.py

Lines changed: 1 addition & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -140,15 +140,6 @@ def check(self, using=Distribution):
140140
),
141141
]
142142

143-
linux_platforms = [
144-
"linux_x86_64",
145-
"manylinux1_x86_64",
146-
"manylinux2010_x86_64",
147-
"manylinux2014_x86_64",
148-
"manylinux_2_27_x86_64",
149-
"manylinux_2_28_x86_64",
150-
]
151-
152143

153144
@pytest.mark.parametrize("dist_test", sdist_tests + wheel_tests)
154145
def test_Distribution_from_filename(dist_test):
@@ -165,7 +156,7 @@ def test_Wheel_from_filename(dist_test):
165156
dist_test.check(using=Wheel)
166157

167158

168-
@pytest.mark.parametrize("linux_platform", linux_platforms)
159+
@pytest.mark.parametrize("linux_platform", utils_pypi.PLATFORMS_BY_OS["linux"])
169160
def test_PypiPackage_get_supported_wheels(linux_platform):
170161
whl = Wheel.from_filename(f"onnxruntime-1.19.2-cp311-cp311-{linux_platform}.whl")
171162
pkg = PypiPackage.package_from_dists(dists=[whl])

0 commit comments

Comments
 (0)