Skip to content

Commit

Permalink
Testing: Activate XRD archive download test on integration. rucio#2311
Browse files Browse the repository at this point in the history
The dependency commit was merged.
  • Loading branch information
rcarpa committed Apr 26, 2021
1 parent b7102af commit a9b4d86
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
2 changes: 2 additions & 0 deletions .github/workflows/integration_tests.yml
Expand Up @@ -74,6 +74,8 @@ jobs:
docker exec -t dev_rucio_1 tools/run_tests_docker.sh -ir
- name: File Upload/Download Test
run: docker exec -t dev_rucio_1 tools/pytest.sh -v --tb=short test_rucio_server.py
- name: Archive Upload/Download Test
run: docker exec -t dev_rucio_1 tools/pytest.sh -v --tb=short test_download.py::test_download_from_archive_on_xrd
- name: Test Protocol XrootD
run: docker exec -t dev_rucio_1 tools/pytest.sh -v --tb=short test_rse_protocol_xrootd.py
- name: Stop containers
Expand Down
3 changes: 2 additions & 1 deletion lib/rucio/tests/test_download.py
Expand Up @@ -37,6 +37,7 @@
from rucio.core import did as did_core
from rucio.rse import rsemanager as rsemgr
from rucio.rse.protocols.posix import Default as PosixProtocol
from rucio.tests.common import skip_rse_tests_with_accounts


@pytest.fixture
Expand Down Expand Up @@ -220,9 +221,9 @@ def test_download_multiple(rse_factory, did_factory, download_client):
)


@pytest.mark.xfail(reason='XRD1 must be initialized https://github.com/rucio/rucio/pull/4165/')
@pytest.mark.dirty
@pytest.mark.noparallel(reason='uses pre-defined XRD1 RSE, may fails when run in parallel') # TODO: verify if it really fails
@skip_rse_tests_with_accounts
def test_download_from_archive_on_xrd(did_factory, download_client, did_client):
scope = 'test'
rse = 'XRD1'
Expand Down

0 comments on commit a9b4d86

Please sign in to comment.