Skip to content

Commit 023fde6

Browse files
committed
test: replace xfail with light_client_7393 marker for nightly exclusion
Aligns with the convention introduced in #7412: tag the LIGHT_CLIENT parametrize entry with a named, issue-tied marker and exclude it from the reliability nightly via the workflow selector instead of using xfail strict=False. Single grep target for cleanup once #7393 lands. - Register light_client_7393 marker in tests-functional/pytest.ini - Update test-reliability.yml: -m "reliability and not light_client_7393" - Replace xfail mark with light_client_7393 across 11 test files
1 parent f045e94 commit 023fde6

13 files changed

Lines changed: 14 additions & 13 deletions

.github/workflows/test-reliability.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ jobs:
5757
run: |
5858
pytest --reruns 2 \
5959
-n auto \
60-
-m "reliability" \
60+
-m "reliability and not light_client_7393" \
6161
-c tests-functional/pytest.ini \
6262
--junit-xml=pytest_results.xml \
6363
--tb=short \

tests-functional/pytest.ini

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,4 +24,5 @@ markers =
2424
assets
2525
benchmark
2626
connector
27-
ens
27+
ens
28+
light_client_7393: temporary marker for status-go#7393 filter subscription race; remove when fixed

tests-functional/tests/reliability/test_contact_request.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
"waku_light_client",
1414
[
1515
pytest.param(False, id=FULL_NODE),
16-
pytest.param(True, id=LIGHT_CLIENT, marks=pytest.mark.xfail(reason="status-go#7393 filter subscription race", strict=False)),
16+
pytest.param(True, id=LIGHT_CLIENT, marks=pytest.mark.light_client_7393),
1717
],
1818
indirect=True,
1919
)

tests-functional/tests/reliability/test_create_private_groups.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
"waku_light_client",
1212
[
1313
pytest.param(False, id=FULL_NODE),
14-
pytest.param(True, id=LIGHT_CLIENT, marks=pytest.mark.xfail(reason="status-go#7393 filter subscription race", strict=False)),
14+
pytest.param(True, id=LIGHT_CLIENT, marks=pytest.mark.light_client_7393),
1515
],
1616
indirect=True,
1717
)

tests-functional/tests/reliability/test_one_to_one_messages.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
"waku_light_client",
1212
[
1313
pytest.param(False, id=FULL_NODE),
14-
pytest.param(True, id=LIGHT_CLIENT, marks=pytest.mark.xfail(reason="status-go#7393 filter subscription race", strict=False)),
14+
pytest.param(True, id=LIGHT_CLIENT, marks=pytest.mark.light_client_7393),
1515
],
1616
indirect=True,
1717
)

tests-functional/tests/reliability/test_private_groups_messages.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
"waku_light_client",
1212
[
1313
pytest.param(False, id=FULL_NODE),
14-
pytest.param(True, id=LIGHT_CLIENT, marks=pytest.mark.xfail(reason="status-go#7393 filter subscription race", strict=False)),
14+
pytest.param(True, id=LIGHT_CLIENT, marks=pytest.mark.light_client_7393),
1515
],
1616
indirect=True,
1717
)

tests-functional/tests/test_wakuext_contact_requests.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
"waku_light_client",
1414
[
1515
pytest.param(False, id=FULL_NODE),
16-
pytest.param(True, id=LIGHT_CLIENT, marks=pytest.mark.xfail(reason="status-go#7393 filter subscription race", strict=False)),
16+
pytest.param(True, id=LIGHT_CLIENT, marks=pytest.mark.light_client_7393),
1717
],
1818
indirect=True,
1919
)

tests-functional/tests/test_wakuext_contact_verification.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
"waku_light_client",
1515
[
1616
pytest.param(False, id=FULL_NODE),
17-
pytest.param(True, id=LIGHT_CLIENT, marks=pytest.mark.xfail(reason="status-go#7393 filter subscription race", strict=False)),
17+
pytest.param(True, id=LIGHT_CLIENT, marks=pytest.mark.light_client_7393),
1818
],
1919
indirect=True,
2020
)

tests-functional/tests/test_wakuext_group_chats.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
"waku_light_client",
1111
[
1212
pytest.param(False, id=FULL_NODE),
13-
pytest.param(True, id=LIGHT_CLIENT, marks=pytest.mark.xfail(reason="status-go#7393 filter subscription race", strict=False)),
13+
pytest.param(True, id=LIGHT_CLIENT, marks=pytest.mark.light_client_7393),
1414
],
1515
indirect=True,
1616
)

tests-functional/tests/test_wakuext_message_reactions.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ async def receiver(self, async_backend_new_profile, waku_light_client):
2525
"waku_light_client",
2626
[
2727
pytest.param(False, id=FULL_NODE),
28-
pytest.param(True, id=LIGHT_CLIENT, marks=pytest.mark.xfail(reason="status-go#7393 filter subscription race", strict=False)),
28+
pytest.param(True, id=LIGHT_CLIENT, marks=pytest.mark.light_client_7393),
2929
],
3030
indirect=True,
3131
)

0 commit comments

Comments
 (0)