Skip to content

Commit 185b519

Browse files
author
Ilyas Gasanov
committed
[DOP-21350] revert some commits
1 parent 1438016 commit 185b519

File tree

3 files changed

+11
-7
lines changed

3 files changed

+11
-7
lines changed

.github/workflows/scheduler-tests.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ jobs:
4949
# This is important, as coverage is exported after receiving SIGTERM
5050
- name: Run Scheduler Tests
5151
run: |
52-
docker compose -f ./docker-compose.test.yml --profile worker exec -T worker coverage run -m pytest -vvv -s -m "scheduler"
52+
docker compose -f ./docker-compose.test.yml --profile worker exec -T worker coverage run -m pytest -vvv -s -m "worker and scheduler_integration"
5353
5454
- name: Dump worker logs on failure
5555
if: failure()
File renamed without changes.

tests/test_integration/test_scheduler/test_schedule.py renamed to tests/test_integration/test_scheduler/test_scheduler.py

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
from syncmaster.settings import Settings
1010
from tests.mocks import MockTransfer
1111

12-
pytestmark = [pytest.mark.asyncio, pytest.mark.worker]
12+
pytestmark = [pytest.mark.asyncio, pytest.mark.worker, pytest.mark.scheduler_integration]
1313

1414

1515
async def test_scheduler(
@@ -38,8 +38,12 @@ async def test_scheduler(
3838
assert run is not None
3939
assert run.status in [Status.CREATED, Status.STARTED]
4040

41-
await asyncio.sleep(2)
42-
await session.refresh(run)
43-
completed_run = await session.scalar(select(Run).filter_by(id=run.id))
44-
assert completed_run.status == Status.FINISHED
45-
assert completed_run.ended_at is not None
41+
for _ in range(3):
42+
await asyncio.sleep(2)
43+
await session.refresh(run)
44+
run = await session.scalar(select(Run, run.id))
45+
if run.status == Status.FINISHED:
46+
break
47+
48+
assert run.status == Status.FINISHED
49+
assert run.ended_at is not None

0 commit comments

Comments
 (0)