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

feat: 539 - taxonomy for Nova groups #645

Merged
merged 2 commits into from
Dec 17, 2022

Conversation

monsieurtanuki
Copy link
Contributor

New files:

  • api_getTaxonomyNovaServer_test.dart: Integration test about nova.
  • TaxonomyNova.dart: A JSON-serializable version of a Nova taxonomy result.
  • TaxonomyNova.g.dart: generated

Impacted files:

  • api_searchProducts_test.dart: unrelated refactoring in order to make that test more re-testable
  • LICENSE: unrelated minor fix
  • openfoodfacts.dart: new method getTaxonomyNova
  • TagType.dart: added an entry for "nova"

What

  • New method getTaxonomyNova, with of course limited outputs (only 4 items)

Part of

New files:
* `api_getTaxonomyNovaServer_test.dart`: Integration test about nova.
* `TaxonomyNova.dart`: A JSON-serializable version of a Nova taxonomy result.
* `TaxonomyNova.g.dart`: generated

Impacted files:
* `api_searchProducts_test.dart`: unrelated refactoring in order to make that test more re-testable
* `LICENSE`: unrelated minor fix
* `openfoodfacts.dart`: new method `getTaxonomyNova`
* `TagType.dart`: added an entry for "nova"
@codecov-commenter
Copy link

Codecov Report

Merging #645 (c360bf0) into master (dc2dfc3) will decrease coverage by 0.47%.
The diff coverage is 78.26%.

@@            Coverage Diff             @@
##           master     #645      +/-   ##
==========================================
- Coverage   68.58%   68.10%   -0.48%     
==========================================
  Files         185      197      +12     
  Lines        7598     7833     +235     
==========================================
+ Hits         5211     5335     +124     
- Misses       2387     2498     +111     
Impacted Files Coverage Δ
lib/model/ProductResult.dart 0.00% <ø> (-60.00%) ⬇️
lib/utils/TagType.dart 0.00% <ø> (ø)
test/api_getRobotoff_test.dart 68.25% <ø> (+3.87%) ⬆️
.../api_getTaxonomyPackagingRecyclingServer_test.dart 100.00% <ø> (ø)
lib/model/ProductImage.dart 13.51% <10.00%> (-34.11%) ⬇️
test/api_saveProduct_test.dart 32.77% <15.38%> (-4.24%) ⬇️
lib/model/TaxonomyNova.g.dart 23.07% <23.07%> (ø)
lib/model/ProductResultField.g.dart 41.66% <41.66%> (ø)
lib/model/ProductResultField.dart 42.85% <42.85%> (ø)
lib/model/ProductResultFieldAnswer.dart 42.85% <42.85%> (ø)
... and 49 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Impacted file:
* `api_getToBeCompletedProducts_test.dart`
@monsieurtanuki monsieurtanuki merged commit 5bf2959 into openfoodfacts:master Dec 17, 2022
@monsieurtanuki
Copy link
Contributor Author

Thank you @teolemon!

@teolemon teolemon added 🧬 Taxonomies NOVA groups NOVA is an indicator for food ultra-processing https://world.openfoodfacts.org/nova labels Jan 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NOVA groups NOVA is an indicator for food ultra-processing https://world.openfoodfacts.org/nova 🧬 Taxonomies
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants