You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I wonder if it's worth reworking this file so that it has less duplication of fields and so that it exercises the consolidation logic thoroughly if testing of dark visitors update is added?
The tests should be expanded to cover the logic for the dark visitors update, suitably refactored so that the dark visitors input is parameterised. The robots.json test fixture should also be reworked to:
ensure good coverage of the dark visitors logic.
reduce duplication.
(This issue is a follow-on to PR 51, should that be merged.)
The text was updated successfully, but these errors were encountered:
glyn
changed the title
Test dark visitors update and improve code/test_files/robots.jsonn
Test dark visitors update and improve code/test_files/robots.json
Oct 20, 2024
Originally posted by @glyn in #51 (comment)
The tests should be expanded to cover the logic for the dark visitors update, suitably refactored so that the dark visitors input is parameterised. The robots.json test fixture should also be reworked to:
(This issue is a follow-on to PR 51, should that be merged.)
The text was updated successfully, but these errors were encountered: