Skip to content

Conversation

@cauriol
Copy link
Contributor

@cauriol cauriol commented Jul 10, 2025

Following CS-SI/eodag#1745, fixes actual pagination with a new mechanism using token parameter for next page link

@cauriol cauriol self-assigned this Jul 10, 2025
@github-actions
Copy link

github-actions bot commented Jul 10, 2025

Test Results

  4 files  ± 0    4 suites  ±0   44s ⏱️ +2s
110 tests +16  110 ✅ +16  0 💤 ±0  0 ❌ ±0 
440 runs  +64  440 ✅ +64  0 💤 ±0  0 ❌ ±0 

Results for commit 6d1b30e. ± Comparison against base commit f9f6e60.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Jul 10, 2025

badge

Code Coverage

Filename                                 Stmts    Miss  Cover    Missing
-------------------------------------  -------  ------  -------  ------------------------------------------------------------------------------------------------------------------------------------
eodag/__init__.py                            0       0  100.00%
eodag/api.py                                 8       8  0.00%    20-37
eodag/app.py                                75      15  80.00%   149, 163-170, 181-183, 232-244, 248
eodag/client.py                             25       0  100.00%
eodag/config.py                             19       0  100.00%
eodag/constants.py                           5       0  100.00%
eodag/core.py                              293      58  80.20%   138-139, 149, 195, 282-283, 363, 484-489, 499-500, 521, 536-554, 617-618, 642, 658-662, 676-680, 682-686, 691, 703-735, 749, 765-767
eodag/cql_evaluate.py                       48      27  43.75%   43, 48, 53, 58, 74-99, 104-106, 111
eodag/dag.py                                53       0  100.00%
eodag/errors.py                             94      12  87.23%   143, 160-163, 179, 202-212
eodag/logs.py                               62       9  85.48%   40-49, 105
eodag/middlewares.py                        32       2  93.75%   20, 24
eodag/telemetry.py                          44       0  100.00%
eodag/utils.py                              41      13  68.29%   44, 68-73, 86, 90-98, 134
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                 25      25  0.00%    20-69
eodag/extensions/collection_order.py        72       1  98.61%   108
eodag/extensions/data_download.py          121      42  65.29%   61-73, 94, 112-113, 120, 135-142, 147-148, 155-187, 195-198
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/stac.py                   132       0  100.00%
eodag/models/__init__.py                     0       0  100.00%
eodag/models/links.py                      141      13  90.78%   39, 93-95, 202, 220, 252, 299, 348-359
eodag/models/stac_metadata.py              196      12  93.88%   94, 162, 175-182, 240-243, 245, 259
TOTAL                                     1664     251  84.92%

Diff against main

Filename                 Stmts    Miss  Cover
---------------------  -------  ------  -------
eodag/core.py              +15      +5  -0.74%
eodag/models/links.py       +8       0  +0.55%
TOTAL                      +23      +5  -0.22%

Results for commit: 6d1b30e

Minimum allowed coverage is 70%

♻️ This comment has been updated with latest results

@bjgiraudon bjgiraudon force-pushed the main branch 2 times, most recently from 12a9c03 to c541296 Compare October 24, 2025 09:15
@sbrunato sbrunato marked this pull request as ready for review November 5, 2025 19:11
@sbrunato sbrunato merged commit 2381bbd into main Nov 5, 2025
11 checks passed
@sbrunato sbrunato deleted the pagination_next branch November 5, 2025 19:13
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.

3 participants