The-Academic-Observatory / oaebu-workflows

Telescopes, Workflows and Data Services for the 'Book Analytics Dashboard Project (2022-2025)', building upon the project 'Developing a Pilot Data Trust for Open Access eBook Usage (2020-2022)'
https://documentation.book-analytics.org/
Apache License 2.0
5 stars 0 forks source link

Fixed GA partition date and added tests #159

Closed keegansmith21 closed 1 year ago

keegansmith21 commented 1 year ago

The partition date was incorrectly set as the data_interval_end rather than the data_interval start. This caused the GA telescope's release date to be one month ahead of what it was supposed to be. I added some test files and an assertion to avoid scenarios like this in the future.

codecov[bot] commented 1 year ago

Codecov Report

Patch and project coverage have no change.

Comparison is base (26d4231) 95.09% compared to head (13c4ee5) 95.09%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #159 +/- ## ======================================== Coverage 95.09% 95.09% ======================================== Files 16 16 Lines 2408 2408 Branches 317 317 ======================================== Hits 2290 2290 Misses 73 73 Partials 45 45 ``` | [Impacted Files](https://app.codecov.io/gh/The-Academic-Observatory/oaebu-workflows/pull/159?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The-Academic-Observatory) | Coverage Δ | | |---|---|---| | [...\_workflows/workflows/google\_analytics\_telescope.py](https://app.codecov.io/gh/The-Academic-Observatory/oaebu-workflows/pull/159?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The-Academic-Observatory#diff-b2FlYnVfd29ya2Zsb3dzL3dvcmtmbG93cy9nb29nbGVfYW5hbHl0aWNzX3RlbGVzY29wZS5weQ==) | `90.90% <ø> (ø)` | | | [oaebu\_workflows/workflows/onix\_workflow.py](https://app.codecov.io/gh/The-Academic-Observatory/oaebu-workflows/pull/159?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The-Academic-Observatory#diff-b2FlYnVfd29ya2Zsb3dzL3dvcmtmbG93cy9vbml4X3dvcmtmbG93LnB5) | `96.38% <ø> (ø)` | |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.