Skip to content

Conversation

jlahovnik
Copy link
Collaborator

the method _stream_download_dict in eodag might return a RedirectResponse -> add handling for this case

@jlahovnik jlahovnik self-assigned this Jul 24, 2025
Copy link

github-actions bot commented Jul 24, 2025

Test Results

  4 files  ±0    4 suites  ±0   43s ⏱️ -2s
 82 tests ±0   82 ✅ ±0  0 💤 ±0  0 ❌ ±0 
328 runs  ±0  328 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 6947a99. ± Comparison against base commit bcd81f1.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Jul 24, 2025

badge

Code Coverage

Filename                                 Stmts    Miss  Cover    Missing
-------------------------------------  -------  ------  -------  ------------------------------------------------------------------------------------------------------------------------
eodag/__init__.py                            0       0  100.00%
eodag/api.py                                10      10  0.00%    20-37
eodag/app.py                                77      16  79.22%   82, 149, 163-170, 181-183, 232-244, 248
eodag/client.py                             25       0  100.00%
eodag/config.py                             18       0  100.00%
eodag/constants.py                           6       0  100.00%
eodag/core.py                              282      59  79.08%   73-81, 130-131, 141, 267-268, 345, 465-470, 480-481, 502, 517-535, 598-599, 623, 639-643, 657-661, 663-667, 672, 684-716
eodag/cql_evaluate.py                       51      29  43.14%   30-32, 43, 48, 53, 58, 74-99, 104-106, 111
eodag/dag.py                                59       2  96.61%   35-37
eodag/errors.py                            100      15  85.00%   47-49, 146, 163-166, 182, 205-215
eodag/logs.py                               47       8  82.98%   40-49
eodag/middlewares.py                        32       2  93.75%   20, 24
eodag/telemetry.py                          53      53  0.00%    20-150
eodag/utils.py                              39      15  61.54%   29-31, 43, 67-72, 85, 89-97, 114
eodag/eodag_types/__init__.py                0       0  100.00%
eodag/eodag_types/queryables.py             42      12  71.43%   45-51, 61, 77-81
eodag/eodag_types/search.py                 27      27  0.00%    20-69
eodag/extensions/collection_order.py        65       3  95.38%   83-84, 95
eodag/extensions/data_download.py          113      39  65.49%   60-72, 93, 111-112, 119, 134-141, 146-147, 154-182, 193
eodag/extensions/ecmwf.py                   69       0  100.00%
eodag/extensions/filter.py                  55       2  96.36%   196-197
eodag/extensions/offset_pagination.py       12       0  100.00%
eodag/extensions/pagination.py              15       0  100.00%
eodag/extensions/stac.py                   132       0  100.00%
eodag/models/__init__.py                     0       0  100.00%
eodag/models/links.py                      133      13  90.23%   36, 90-92, 185, 203, 235, 282, 331-342
eodag/models/stac_metadata.py              186       8  95.70%   98, 181, 183, 196-203, 261
TOTAL                                     1648     313  81.01%

Diff against main

Filename                             Stmts    Miss  Cover
---------------------------------  -------  ------  -------
eodag/extensions/data_download.py       +2      +1  -0.28%
TOTAL                                   +2      +1  -0.03%

Results for commit: 6947a99

Minimum allowed coverage is 70%

♻️ This comment has been updated with latest results

@jlahovnik jlahovnik force-pushed the redirect-response branch from 5c465bc to 6947a99 Compare July 24, 2025 12:25
@jlahovnik jlahovnik requested a review from sbrunato July 24, 2025 12:31
@sbrunato sbrunato marked this pull request as draft July 28, 2025 12:47
@jlahovnik
Copy link
Collaborator Author

replaced by #42

@jlahovnik jlahovnik closed this Jul 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant