Skip to content

Commit 581531c

Browse files
committed
Merge remote-tracking branch 'origin/pr/331'
* origin/pr/331: tests: Account for argparse bug fix in Python 3.13
2 parents 4e4fa79 + 6461531 commit 581531c

File tree

1 file changed

+14
-1
lines changed

1 file changed

+14
-1
lines changed

qubesadmin/tests/tools/qvm_service.py

Lines changed: 14 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@
2020

2121
# pylint: disable=missing-docstring
2222

23+
import sys
2324
import unittest
2425

2526
import qubesadmin.tests
@@ -100,7 +101,7 @@ def test_003_enable_opt(self):
100101
0)
101102
self.assertAllCalled()
102103

103-
@unittest.expectedFailure
104+
@unittest.skipIf(sys.version_info.minor < 13, reason="argparse bug")
104105
def test_004_enable_opt_mixed(self):
105106
self.app.expected_calls[
106107
('dom0', 'admin.vm.List', None, None)] = \
@@ -116,6 +117,18 @@ def test_004_enable_opt_mixed(self):
116117
0)
117118
self.assertAllCalled()
118119

120+
@unittest.skipIf(
121+
sys.version_info.minor >= 13, reason="argparse works correctly"
122+
)
123+
def test_004_enable_opt_mixed_broken(self):
124+
with self.assertRaises(SystemExit):
125+
self.assertEqual(
126+
qubesadmin.tools.qvm_service.main(
127+
['some-vm', '--enable', 'service1'],
128+
app=self.app),
129+
0)
130+
self.assertAllCalled()
131+
119132
def test_005_disable(self):
120133
self.app.expected_calls[
121134
('dom0', 'admin.vm.List', None, None)] = \

0 commit comments

Comments
 (0)