hhursev / recipe-scraper / 7025259615 / 1

96%

master: 96%

push

github

Data driven tests refactor (#944)

* Convert existing testshtml and scraper test files to data driven format.

* Update generate.py to support new test framework.
This creates the appropriate directory in the tests/test_data/ directory and saves the testhtml and a template testjson there.

* Update README and scraper development guide to be consistent with data driven test framework.

Also fix the python version in the README.

* Remove test_scraper template

* Include the tests in the tests/library folder.

* Add missing test for weightwatcherspublic

* Fix typo in scraper development guide and change notes to tips

* Convert abuelascounter test to data driven format following merge of main

* Remove ratings from usapears.testjson as the scraper doesn't support that function.

* Remove test for deleted duplicate 101cookbooks scraper and update remaining testjson from update testhtml

* Migrate tests for new scraper for williams-sonoma to data driven architecture

* Replace tasteau_2 testhtml with a different recipe so the test is successful.
The previous test would fail because the json+ld ingredient had metric units but the html ingredients had imperial units, so the grouping function wouldn't work perfectly. The new recipes uses metric units in the html so this isn't a problem.

* Specify encoding when loading testjson and testhtml files to resolve errors when running tests on Windows. Hopefully...

* Minor fixes to documentation

* Change .testjson extension to .json

* Restore tests for scrapers that use additional networks requests to obtain the recipe data.
These tests use the old testing architecture since the data driven approach does not support tests needing additional network requests.

* Add pre-commit hook to pretty format json files

* Replace relative import in legacy tests to absolute

* Change name of generated test function for each scraper to be the path to the test json file.

* Change ex... (continued)

1677 of 1905 branches covered (0.0%)

8531 of 8806 relevant lines covered (96.88%)

0.97 hits per line