ploomber / soorgeon

Convert monolithic Jupyter notebooks 📙 into maintainable Ploomber pipelines. 📊
https://ploomber.io
Apache License 2.0
78 stars 20 forks source link

Test fails CI due to 404 #58

Closed idomic closed 2 years ago

idomic commented 2 years ago

One of our tests fails due to the kagle notebook api call


path = PosixPath('/home/runner/work/soorgeon/soorgeon/tests/../_kaggle/intro-to-time-series-forecasting')
    @pytest.mark.parametrize('path', path_to_nbs, ids=list(index))
    def test_notebooks(tmp_empty, path):
        name = Path(path).name
        download_fn = index[name]['partial']
>       download_fn()
/home/runner/work/soorgeon/soorgeon/tests/test_sample_notebooks.py:[23](https://github.com/ploomber/soorgeon/runs/6747373533?check_suite_focus=true#step:7:24):

if not 200 <= r.status <= 299:
>           raise ApiException(http_resp=r)
E           kaggle.rest.ApiException: (401)
E           Reason: Unauthorized```
edublancas commented 2 years ago

This error is intermittent; I've seen it fails only sometimes. probably because the Kaggle API isn't very reliable. I'll open an issue to follow-up on this