Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Download referentiedata op basis van release tag #3

Merged

Conversation

sTomerG
Copy link
Collaborator

@sTomerG sTomerG commented Jul 12, 2024

Beschrijving:
Deze pull request introduceert belangrijke wijzigingen en verbeteringen in de VERA testframework codebase, voornamelijk met betrekking tot de behandeling van referentiedata. De volgende veranderingen zijn doorgevoerd:

  1. Toevoeging van release_tag Parameter:

    • Een nieuwe parameter release_tag is toegevoegd aan de ReferentiedataTest klasse. Hiermee kan de specifieke versie van de referentiedata worden opgehaald. Standaard wordt de nieuwste versie (latest) gebruikt, maar een specifieke versie kan ook worden opgegeven.
  2. Caching Mechanisme:

    • Een caching mechanisme is geïmplementeerd voor referentiedata in de klasse ReferentiedataTest. Dit zorgt ervoor dat referentiedata slechts één keer per release-tag wordt gedownload en hergebruikt bij volgende verzoeken, wat de efficiëntie van het systeem verhoogt.
  3. Nieuwe Hulpfunctie:

    • Een nieuwe hulpfunctie get_csv_from_release is toegevoegd in utils.py. Deze functie downloadt een CSV-bestand van een specifieke release op GitHub en verwerkt deze.
    • Een aanvullende functie get_latest_release_tag is toegevoegd om de nieuwste release-tag van een GitHub-repository op te halen.
  4. Aanpassingen in tutorial.ipynb:

    • De tutorial notebook is bijgewerkt om het gebruik van de nieuwe release_tag parameter te demonstreren.

Overige Wijzigingen:

  • Verwijdering van de verouderde CSV-inleeslogica in referentiedata.py.
  • Verwijdering van src/vera_testframework/data/Referentiedata.csv
  • Toevoegen van test met invalide release tag

@github-actions github-actions bot added the enhancement New feature or request label Jul 12, 2024
@sTomerG sTomerG merged commit 7702286 into main Jul 12, 2024
6 checks passed
@sTomerG sTomerG deleted the feature/download-referentiedata-op-basis-van-release-tag branch July 12, 2024 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant