Skip to content

Commit f667db5

Browse files
committed
refactor: update tests to use private module name
1 parent bd6d88f commit f667db5

1 file changed

Lines changed: 11 additions & 11 deletions

File tree

packages/google-api-core/tests/unit/test_feature_gating_helpers.py

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,8 @@
1313
# limitations under the License.
1414

1515
import pytest
16-
from google.api_core import feature_gating_helpers
17-
from google.api_core.feature_gating_helpers import (
16+
from google.api_core import _feature_gating_helpers
17+
from google.api_core._feature_gating_helpers import (
1818
_get_env_bool,
1919
_strtobool,
2020
)
@@ -72,7 +72,7 @@ def test_resolve_feature_flags_ga_enabled_via_env(monkeypatch):
7272
monkeypatch.setenv("GOOGLE_SDK_PYTHON_TRACING_ENABLED", "true")
7373

7474
# Action
75-
result = feature_gating_helpers.resolve_feature_flags(
75+
result = _feature_gating_helpers.resolve_feature_flags(
7676
env_var="GOOGLE_SDK_PYTHON_TRACING_ENABLED",
7777
feature_key="tracer_provider",
7878
configuration=None,
@@ -112,7 +112,7 @@ def test_resolve_feature_flags_exp_enabled_with_provider(monkeypatch):
112112
monkeypatch.setenv("GOOGLE_SDK_EXPERIMENTAL_PYTHON_TRACING_ENABLED", "true")
113113
configuration = {"tracer_provider": object()}
114114

115-
result = feature_gating_helpers.resolve_feature_flags(
115+
result = _feature_gating_helpers.resolve_feature_flags(
116116
env_var="GOOGLE_SDK_EXPERIMENTAL_PYTHON_TRACING_ENABLED",
117117
feature_key="tracer_provider",
118118
configuration=configuration,
@@ -124,7 +124,7 @@ def test_resolve_feature_flags_exp_enabled_without_provider(monkeypatch):
124124
"""Verify that experimental feature is enabled if the experimental environment variable is enabled and NO provider is provided."""
125125
monkeypatch.setenv("GOOGLE_SDK_EXPERIMENTAL_PYTHON_TRACING_ENABLED", "true")
126126

127-
result = feature_gating_helpers.resolve_feature_flags(
127+
result = _feature_gating_helpers.resolve_feature_flags(
128128
env_var="GOOGLE_SDK_EXPERIMENTAL_PYTHON_TRACING_ENABLED",
129129
feature_key="tracer_provider",
130130
configuration=None,
@@ -136,7 +136,7 @@ def test_resolve_feature_flags_exp_disabled_without_provider(monkeypatch):
136136
"""Verify that experimental feature is disabled if the experimental environment variable is disabled and NO provider is provided."""
137137
monkeypatch.setenv("GOOGLE_SDK_EXPERIMENTAL_PYTHON_TRACING_ENABLED", "false")
138138

139-
result = feature_gating_helpers.resolve_feature_flags(
139+
result = _feature_gating_helpers.resolve_feature_flags(
140140
env_var="GOOGLE_SDK_EXPERIMENTAL_PYTHON_TRACING_ENABLED",
141141
feature_key="tracer_provider",
142142
configuration=None,
@@ -150,7 +150,7 @@ def test_resolve_feature_flags_ga_enabled_via_provider(monkeypatch):
150150
monkeypatch.setenv("GOOGLE_SDK_PYTHON_TRACING_ENABLED", "false")
151151
configuration = {"tracer_provider": object()}
152152

153-
result = feature_gating_helpers.resolve_feature_flags(
153+
result = _feature_gating_helpers.resolve_feature_flags(
154154
env_var="GOOGLE_SDK_PYTHON_TRACING_ENABLED",
155155
feature_key="tracer_provider",
156156
configuration=configuration,
@@ -167,7 +167,7 @@ def test_resolve_feature_flags_ga_fallback_to_false(monkeypatch, env_val):
167167
monkeypatch.setenv("GOOGLE_SDK_PYTHON_TRACING_ENABLED", env_val)
168168
else:
169169
monkeypatch.delenv("GOOGLE_SDK_PYTHON_TRACING_ENABLED", raising=False)
170-
result = feature_gating_helpers.resolve_feature_flags(
170+
result = _feature_gating_helpers.resolve_feature_flags(
171171
env_var="GOOGLE_SDK_PYTHON_TRACING_ENABLED",
172172
feature_key="tracer_provider",
173173
configuration=None,
@@ -191,7 +191,7 @@ def __init__(self):
191191
def test_resolve_feature_flags_options_without_key(configuration):
192192
"""Verify behavior when configuration is present but missing the provider key."""
193193
# GA Path: should fall through to env var / fallback
194-
result = feature_gating_helpers.resolve_feature_flags(
194+
result = _feature_gating_helpers.resolve_feature_flags(
195195
env_var="GOOGLE_SDK_PYTHON_TRACING_ENABLED",
196196
feature_key="tracer_provider",
197197
configuration=configuration,
@@ -205,7 +205,7 @@ def test_resolve_feature_flags_rejects_dunder_keys(monkeypatch):
205205
# If the guardrail is missing, getattr might return it and return True
206206
configuration = {"__class__": "some_value"}
207207

208-
result = feature_gating_helpers.resolve_feature_flags(
208+
result = _feature_gating_helpers.resolve_feature_flags(
209209
env_var="GOOGLE_SDK_PYTHON_TRACING_ENABLED",
210210
feature_key="__class__",
211211
configuration=configuration,
@@ -215,7 +215,7 @@ def test_resolve_feature_flags_rejects_dunder_keys(monkeypatch):
215215
class MockWithClass:
216216
__class__ = "some_value"
217217

218-
result = feature_gating_helpers.resolve_feature_flags(
218+
result = _feature_gating_helpers.resolve_feature_flags(
219219
env_var="GOOGLE_SDK_PYTHON_TRACING_ENABLED",
220220
feature_key="__class__",
221221
configuration=MockWithClass(),

0 commit comments

Comments
 (0)