Merge pull request #987 from adamtheturtle/update-mypy

Fix strict equality checks

Author: adamtheturtle

dev-requirements.txt

@@ -8,7 +8,7 @@ flake8-commas==2.0.0  # Require silicon valley commas
 flake8-quotes==1.0.0  # Require single quotes
 flake8==3.7.7  # Lint
 isort==4.3.16  # Lint imports
-mypy-mypyc==0.670  # Type checking
+mypy==0.700  # Type checking
 pip_check_reqs==2.0.3
 pydocstyle==3.0.0  # Lint docstrings
 pyenchant==2.0.0  # Bindings for a spellchecking sytem

setup.cfg

@@ -73,6 +73,7 @@ disallow_untyped_defs = True
 follow_imports = silent
 ignore_missing_imports = True
 no_implicit_optional = True
+strict_equality = True
 strict_optional = True
 warn_no_return = True
 warn_redundant_casts = True

tests/test_vws.py

@@ -191,7 +191,7 @@ def test_get_target_summary_report(
             'current_month_recos',
             'previous_month_recos',
         }
-        assert result.keys() == expected_keys
+        assert set(result.keys()) == expected_keys
 
 
 class TestGetDatabaseSummaryReport:
@@ -220,7 +220,7 @@ def test_get_target(self, client: VWS) -> None:
             'total_recos',
             'transaction_id',
         }
-        assert report.keys() == expected_keys
+        assert set(report.keys()) == expected_keys
 
 
 class TestGetTargetRecord:
@@ -252,7 +252,7 @@ def test_get_target_record(
             'tracking_rating',
             'reco_rating',
         }
-        assert result.keys() == expected_keys
+        assert set(result.keys()) == expected_keys
 
 
 class TestWaitForTargetProcessed: