Commit f0d53c7d authored by Qinusty's avatar Qinusty Committed by Francisco Redondo Marchena
Browse files

tests/sources: deb, tar and zip all test for retry behaviour 

This is following 5d6418a06a3a0262df5f6b9e6e1a36578515fd4a where
DownloadableFileSource was modified to raise 'temporary'
errors when relating to file downloading.
parent 66d341d2

tests/sources/deb.py

+2 −1
Original line number Diff line number Diff line
@@ -45,7 +45,7 @@ def test_no_ref(cli, tmpdir, datafiles): 
    assert cli.get_element_state(project, 'target.bst') == 'no reference'





# Test that when I fetch a nonexistent URL, errors are handled gracefully.

# Test that when I fetch a nonexistent URL, errors are handled gracefully and a retry is performed.

@pytest.mark.skipif(HAVE_ARPY is False, reason="arpy is not available")

@pytest.mark.datafiles(os.path.join(DATA_DIR, 'fetch'))

def test_fetch_bad_url(cli, tmpdir, datafiles):
@@ -56,6 +56,7 @@ def test_fetch_bad_url(cli, tmpdir, datafiles): 
    result = cli.run(project=project, args=[

        'fetch', 'target.bst'

    ])

    assert "Try #" in result.stderr

    result.assert_main_error(ErrorDomain.STREAM, None)

    result.assert_task_error(ErrorDomain.SOURCE, None)

tests/sources/tar.py

+2 −1
Original line number Diff line number Diff line
@@ -56,7 +56,7 @@ def test_no_ref(cli, tmpdir, datafiles): 
    assert cli.get_element_state(project, 'target.bst') == 'no reference'





# Test that when I fetch a nonexistent URL, errors are handled gracefully.

# Test that when I fetch a nonexistent URL, errors are handled gracefully and a retry is performed.

@pytest.mark.datafiles(os.path.join(DATA_DIR, 'fetch'))

def test_fetch_bad_url(cli, tmpdir, datafiles):

    project = os.path.join(datafiles.dirname, datafiles.basename)
@@ -66,6 +66,7 @@ def test_fetch_bad_url(cli, tmpdir, datafiles): 
    result = cli.run(project=project, args=[

        'fetch', 'target.bst'

    ])

    assert "Try #" in result.stderr

    result.assert_main_error(ErrorDomain.STREAM, None)

    result.assert_task_error(ErrorDomain.SOURCE, None)

tests/sources/zip.py

+2 −1
Original line number Diff line number Diff line
@@ -43,7 +43,7 @@ def test_no_ref(cli, tmpdir, datafiles): 
    assert cli.get_element_state(project, 'target.bst') == 'no reference'





# Test that when I fetch a nonexistent URL, errors are handled gracefully.

# Test that when I fetch a nonexistent URL, errors are handled gracefully and a retry is performed.

@pytest.mark.datafiles(os.path.join(DATA_DIR, 'fetch'))

def test_fetch_bad_url(cli, tmpdir, datafiles):

    project = os.path.join(datafiles.dirname, datafiles.basename)
@@ -53,6 +53,7 @@ def test_fetch_bad_url(cli, tmpdir, datafiles): 
    result = cli.run(project=project, args=[

        'fetch', 'target.bst'

    ])

    assert "Try #" in result.stderr

    result.assert_main_error(ErrorDomain.STREAM, None)

    result.assert_task_error(ErrorDomain.SOURCE, None)