Skip to content

Commit 77629a8

Browse files
committed
run black formatter
1 parent 3aad65f commit 77629a8

File tree

6 files changed

+24
-17
lines changed

6 files changed

+24
-17
lines changed

devcycle_python_sdk/local_client.py

Lines changed: 11 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,11 @@
1616
from devcycle_python_sdk.models.bucketed_config import BucketedConfig
1717
from devcycle_python_sdk.models.eval_hook import EvalHook
1818
from devcycle_python_sdk.models.eval_hook_context import HookContext
19-
from devcycle_python_sdk.models.eval_reason import DefaultReasonDetails, EvalReason, EvalReasons
19+
from devcycle_python_sdk.models.eval_reason import (
20+
DefaultReasonDetails,
21+
EvalReason,
22+
EvalReasons,
23+
)
2024
from devcycle_python_sdk.models.event import DevCycleEvent, EventType
2125
from devcycle_python_sdk.models.feature import Feature
2226
from devcycle_python_sdk.models.platform_data import default_platform_data
@@ -139,7 +143,9 @@ def variable(self, user: DevCycleUser, key: str, default_value: Any) -> Variable
139143
logger.warning(
140144
f"DevCycle: Unable to track AggVariableDefaulted event for Variable {key}: {e}"
141145
)
142-
return Variable.create_default_variable(key, default_value, DefaultReasonDetails.MISSING_CONFIG)
146+
return Variable.create_default_variable(
147+
key, default_value, DefaultReasonDetails.MISSING_CONFIG
148+
)
143149

144150
context = HookContext(key, user, default_value)
145151
variable = Variable.create_default_variable(
@@ -162,7 +168,7 @@ def variable(self, user: DevCycleUser, key: str, default_value: Any) -> Variable
162168
else:
163169
variable.eval = EvalReason(
164170
reason=EvalReasons.DEFAULT,
165-
details=DefaultReasonDetails.USER_NOT_TARGETED
171+
details=DefaultReasonDetails.USER_NOT_TARGETED,
166172
)
167173

168174
if before_hook_error is None:
@@ -171,17 +177,15 @@ def variable(self, user: DevCycleUser, key: str, default_value: Any) -> Variable
171177
raise before_hook_error
172178
except Exception as e:
173179
variable.eval = EvalReason(
174-
reason=EvalReasons.DEFAULT,
175-
details=DefaultReasonDetails.ERROR
180+
reason=EvalReasons.DEFAULT, details=DefaultReasonDetails.ERROR
176181
)
177182

178183
if isinstance(e, BeforeHookError):
179184
self.eval_hooks_manager.run_error(context, e)
180185
elif isinstance(e, AfterHookError):
181186
self.eval_hooks_manager.run_error(context, e)
182187
else:
183-
logger.warning(
184-
f"DevCycle: Error retrieving variable for user: {e}")
188+
logger.warning(f"DevCycle: Error retrieving variable for user: {e}")
185189

186190
return variable
187191
finally:

devcycle_python_sdk/models/eval_reason.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,11 +4,13 @@
44

55
class EvalReasons:
66
"""Evaluation reasons constants"""
7+
78
DEFAULT = "DEFAULT"
89

910

1011
class DefaultReasonDetails:
1112
"""Default reason details constants"""
13+
1214
MISSING_CONFIG = "Missing Config"
1315
USER_NOT_TARGETED = "User Not Targeted"
1416
TYPE_MISMATCH = "Variable Type Mismatch"

devcycle_python_sdk/models/variable.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -66,12 +66,13 @@ def from_json(cls, data: dict) -> "Variable":
6666
)
6767

6868
@staticmethod
69-
def create_default_variable(key: str, default_value: Any, default_reason_detail: str = None) -> "Variable":
69+
def create_default_variable(
70+
key: str, default_value: Any, default_reason_detail: str = None
71+
) -> "Variable":
7072
var_type = determine_variable_type(default_value)
7173
if default_reason_detail is not None:
7274
eval_reason = EvalReason(
73-
reason=EvalReasons.DEFAULT,
74-
details=default_reason_detail
75+
reason=EvalReasons.DEFAULT, details=default_reason_detail
7576
)
7677
else:
7778
eval_reason = None

devcycle_python_sdk/protobuf/utils.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,7 @@ def create_eval_reason_from_pb(eval_reason_pb: pb2.EvalReason_PB) -> EvalReason:
9494

9595
def create_variable(sdk_variable: pb2.SDKVariable_PB, default_value: Any) -> Variable: # type: ignore
9696
eval_reason_obj = None
97-
if sdk_variable.HasField('eval'):
97+
if sdk_variable.HasField("eval"):
9898
eval_reason_obj = create_eval_reason_from_pb(sdk_variable.eval)
9999

100100
if sdk_variable.type == pb2.VariableType_PB.Boolean: # type: ignore

test/api/test_local_bucketing.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -157,9 +157,9 @@ def test_generate_bucketed_config(self):
157157
)
158158
}
159159
expected_eval = EvalReason(
160-
reason='TARGETING_MATCH',
161-
details='All Users',
162-
target_id='63125321d31c601f992288bc'
160+
reason="TARGETING_MATCH",
161+
details="All Users",
162+
target_id="63125321d31c601f992288bc",
163163
)
164164
expected_variables = {
165165
"a-cool-new-feature": Variable(

test/test_local_client.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -296,9 +296,9 @@ def test_all_variables(self):
296296

297297
result = self.client.all_variables(user)
298298
expected_eval = EvalReason(
299-
reason='TARGETING_MATCH',
300-
details='All Users',
301-
target_id='63125321d31c601f992288bc'
299+
reason="TARGETING_MATCH",
300+
details="All Users",
301+
target_id="63125321d31c601f992288bc",
302302
)
303303
expected_variables = {
304304
"a-cool-new-feature": Variable(

0 commit comments

Comments
 (0)