remove unnecessary code in tests

[jameslamb]

Contributes to #161.

Changes

• removes one instance of duplicated test test_tags_invalid_value (code link)
• removes unused variable in one Dask test

These were detected by flake8.

./tests/test_function_modifiers.py:744:1: F811 redefinition of unused 'test_tags_invalid_value' from line 739
./graph_adapter_tests/h_dask/test_h_dask.py:49:5: F841 local variable 'index' is assigned to but never used

Testing

pre-commit run --all-files

Notes

Thanks for your time and consideration.

Checklist

• PR has an informative and human-readable title (this will be pulled into the release notes)
• Changes are limited to a single goal (no scope creep)
• Code can be automatically merged (no conflicts)
• Code passed the pre-commit check & code is left cleaner/nicer than when first encountered.
• Passes all existing automated tests
• Any change in functionality is tested
• New functions are documented (with a description, list of inputs, and expected output)
• Placeholder code is flagged / future TODOs are captured in comments
• Project documentation has been updated if adding/changing functionality.
• Reviewers requested with the Reviewers tool ➡️

Testing checklist

Python - local testing

• python 3.6
• python 3.7