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

fix: fixed unit tests #755

Merged
merged 1 commit into from
Jul 12, 2023
Merged

Conversation

monsieurtanuki
Copy link
Contributor

What

  • Fixed some unit tests.

Files

New file:

  • api_get_save_product_test.dart: moved there read-write tests from api_get_product_test.dart

Impacted files:

  • api_add_product_image_test.dart: skipped tests on unreliable TEST env
  • api_get_product_test.dart: now we're focused on read-only tests in PROD; we moved the read-write tests to new file api_get_save_product_test.dart
  • api_get_robotoff_test.dart: fixed special cases with specific insight types
  • api_get_to_be_completed_products_test.dart: added a filter on stores for less demanding queries
  • api_ocr_ingredients_test.dart: fixed async tests with no await
  • api_search_products_test.dart: added a filter in order to make the query less server intensive - and less prone to server crash

New file:
* `api_get_save_product_test.dart`: moved there read-write tests from `api_get_product_test.dart`

Impacted files:
* `api_add_product_image_test.dart`: skipped tests on unreliable TEST env
* `api_get_product_test.dart`: now we're focused on read-only tests in PROD; we moved the read-write tests to new file `api_get_save_product_test.dart`
* `api_get_robotoff_test.dart`: fixed special cases with specific insight types
* `api_get_to_be_completed_products_test.dart`: added a filter on stores for less demanding queries
* `api_ocr_ingredients_test.dart`: fixed async tests with no await
* `api_search_products_test.dart`: added a filter in order to make the query less server intensive - and less prone to server crash
@monsieurtanuki monsieurtanuki requested a review from a team as a code owner July 11, 2023 06:43
@monsieurtanuki monsieurtanuki mentioned this pull request Jul 12, 2023
@g123k g123k merged commit 60bd0ee into openfoodfacts:master Jul 12, 2023
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants