From 005d69d1a9f3eb355901b135c854a4c4984e3540 Mon Sep 17 00:00:00 2001 From: Takashi Kajinami Date: Wed, 11 Jun 2025 12:02:40 +0900 Subject: [PATCH] Drop remaining skip check for Python < 3 ... because Python 2.x is no longer supported. Change-Id: I3167a539b3e26ceb35976fbd7a2356ba59d4a5e4 --- test/unit/common/ring/test_builder.py | 6 ------ test/unit/common/ring/test_ring.py | 3 --- test/unit/common/ring/test_utils.py | 4 ---- 3 files changed, 13 deletions(-) diff --git a/test/unit/common/ring/test_builder.py b/test/unit/common/ring/test_builder.py index 4398b6328b..4b58b8f4f9 100644 --- a/test/unit/common/ring/test_builder.py +++ b/test/unit/common/ring/test_builder.py @@ -2492,9 +2492,6 @@ class TestRingBuilder(unittest.TestCase): (0, 0, '127.0.0.1', 3): [0, 256, 0, 0], }) - @unittest.skipIf(sys.version_info < (3,), - "Seed-specific tests don't work well between python " - "versions. This test is now PY3 only") def test_undispersable_zone_converge_on_balance(self): rb = ring.RingBuilder(8, 6, 0) dev_id = 0 @@ -2550,9 +2547,6 @@ class TestRingBuilder(unittest.TestCase): self.assertEqual(rb.get_balance(), 14.453125) self.assertEqual(rb.dispersion, 16.6015625) - @unittest.skipIf(sys.version_info < (3,), - "Seed-specific tests don't work well between python " - "versions. This test is now PY3 only") def test_undispersable_server_converge_on_balance(self): rb = ring.RingBuilder(8, 6, 0) dev_id = 0 diff --git a/test/unit/common/ring/test_ring.py b/test/unit/common/ring/test_ring.py index 962c41c5a9..2bf97aff97 100644 --- a/test/unit/common/ring/test_ring.py +++ b/test/unit/common/ring/test_ring.py @@ -531,9 +531,6 @@ class TestRing(TestRingBase): self.ring.devs.append(new_dev) self.ring._rebuild_tier_data() - @unittest.skipIf(sys.version_info < (3,), - "Seed-specific tests don't work well between python " - "versions. This test is now PY3 only") def test_get_more_nodes(self): # Yes, these tests are deliberately very fragile. We want to make sure # that if someone changes the results the ring produces, they know it. diff --git a/test/unit/common/ring/test_utils.py b/test/unit/common/ring/test_utils.py index 6f8ec9f6d1..881f62a877 100644 --- a/test/unit/common/ring/test_utils.py +++ b/test/unit/common/ring/test_utils.py @@ -13,7 +13,6 @@ # See the License for the specific language governing permissions and # limitations under the License. -import sys import unittest from collections import defaultdict @@ -664,9 +663,6 @@ class TestUtils(unittest.TestCase): } self.assertEqual(device, expected) - @unittest.skipIf(sys.version_info < (3,), - "Seed-specific tests don't work well between python " - "versions. This test is now PY3 only") def test_dispersion_report(self): rb = ring.RingBuilder(8, 3, 0) rb.add_dev({'id': 0, 'region': 1, 'zone': 0, 'weight': 100,