Skip to content

Commit

Permalink
Revert "Tests: only check for successful submission in stager test"
Browse files Browse the repository at this point in the history
This reverts commit 511b8a8.
  • Loading branch information
Radu Carpa committed Jan 15, 2024
1 parent 655bfb3 commit 33a1651
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
1 change: 1 addition & 0 deletions etc/docker/dev/fts/entrypoint.sh
Original file line number Diff line number Diff line change
Expand Up @@ -41,5 +41,6 @@ update-ca-trust

# startup the FTS services
/usr/sbin/fts_server # main FTS server daemonizes
/usr/sbin/fts_qos # for the stager tests
/usr/sbin/fts_msg_bulk # daemon to send messages to activemq
/usr/sbin/httpd -DFOREGROUND # FTS REST frontend & FTSMON
5 changes: 2 additions & 3 deletions tests/test_conveyor.py
Original file line number Diff line number Diff line change
Expand Up @@ -823,7 +823,6 @@ def test_non_deterministic_dst(did_factory, did_client, root_account, vo, caches


@skip_rse_tests_with_accounts
@pytest.mark.dirty(reason="Submits a transfer to FTS which cannot be ever executed in this test env.")
@pytest.mark.noparallel(groups=[NoParallelGroups.STAGER, NoParallelGroups.POLLER, NoParallelGroups.FINISHER])
def test_stager(rse_factory, did_factory, root_account, replica_client):
"""
Expand Down Expand Up @@ -861,8 +860,8 @@ def test_stager(rse_factory, did_factory, root_account, replica_client):
'requested_at': datetime.utcnow()}])
stager(once=True, rses=[{'id': rse_id} for rse_id in all_rses])

request = request_core.get_request_by_did(rse_id=dst_rse_id, **did)
assert request['state'] == RequestState.SUBMITTED
replica = __wait_for_replica_transfer(dst_rse_id=dst_rse_id, max_wait_seconds=2 * MAX_POLL_WAIT_SECONDS, **did)
assert replica['state'] == ReplicaState.AVAILABLE


@pytest.mark.noparallel(groups=[NoParallelGroups.SUBMITTER, NoParallelGroups.FINISHER])
Expand Down

0 comments on commit 33a1651

Please sign in to comment.