Skip to content

Commit

Permalink
Fix test failures.
Browse files Browse the repository at this point in the history
This was caused by incorrect module names for mocking. They needed to be
updated to the new module and class for `CachingFeatureFlagRouter`.

Related #92
  • Loading branch information
aholmes committed Aug 7, 2024
1 parent 963e031 commit 0daebfe
Showing 1 changed file with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -177,10 +177,10 @@ def test__feature_is_enabled__checks_cache(
):
session_mock = mocker.patch("sqlalchemy.orm.session.Session")
feature_is_enabled_mock = mocker.patch(
"BL_Python.platform.feature_flag.feature_flag_router.FeatureFlagRouter.feature_is_enabled"
"BL_Python.platform.feature_flag.caching_feature_flag_router.CachingFeatureFlagRouter.feature_is_enabled"
)
_ = mocker.patch(
"BL_Python.platform.feature_flag.feature_flag_router.FeatureFlagRouter.set_feature_is_enabled"
"BL_Python.platform.feature_flag.caching_feature_flag_router.CachingFeatureFlagRouter.set_feature_is_enabled"
)

logger = logging.getLogger("FeatureFlagLogger")
Expand All @@ -199,10 +199,10 @@ def test__feature_is_enabled__sets_cache(
):
session_mock = mocker.patch("sqlalchemy.orm.session.Session")
feature_is_enabled_mock = mocker.patch(
"BL_Python.platform.feature_flag.feature_flag_router.FeatureFlagRouter.feature_is_enabled"
"BL_Python.platform.feature_flag.caching_feature_flag_router.CachingFeatureFlagRouter.feature_is_enabled"
)
set_feature_is_enabled_mock = mocker.patch(
"BL_Python.platform.feature_flag.feature_flag_router.FeatureFlagRouter.set_feature_is_enabled"
"BL_Python.platform.feature_flag.caching_feature_flag_router.CachingFeatureFlagRouter.set_feature_is_enabled"
)
feature_is_enabled_mock.return_value = True

Expand Down

0 comments on commit 0daebfe

Please sign in to comment.