Skip to content

Commit 0710500

Browse files
aniketpaluntkathole
authored andcommitted
Minor refactoring
Signed-off-by: Aniket Paluskar <[email protected]>
1 parent 8b585dc commit 0710500

File tree

1 file changed

+27
-10
lines changed

1 file changed

+27
-10
lines changed

sdk/python/tests/unit/online_store/test_online_writes.py

Lines changed: 27 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -222,8 +222,11 @@ def test_empty_dataframe_warns(self):
222222
# Check that our specific warning message is present
223223
warning_messages = [str(w.message) for w in warning_list]
224224
self.assertTrue(
225-
any("Cannot write empty dataframe to online store" in msg for msg in warning_messages),
226-
f"Expected warning not found. Actual warnings: {warning_messages}"
225+
any(
226+
"Cannot write empty dataframe to online store" in msg
227+
for msg in warning_messages
228+
),
229+
f"Expected warning not found. Actual warnings: {warning_messages}",
227230
)
228231

229232
def test_empty_dataframe_async_warns(self):
@@ -242,8 +245,11 @@ async def test_async_empty():
242245
# Check that our specific warning message is present
243246
warning_messages = [str(w.message) for w in warning_list]
244247
self.assertTrue(
245-
any("Cannot write empty dataframe to online store" in msg for msg in warning_messages),
246-
f"Expected warning not found. Actual warnings: {warning_messages}"
248+
any(
249+
"Cannot write empty dataframe to online store" in msg
250+
for msg in warning_messages
251+
),
252+
f"Expected warning not found. Actual warnings: {warning_messages}",
247253
)
248254

249255
asyncio.run(test_async_empty())
@@ -271,8 +277,12 @@ def test_dataframe_with_empty_feature_columns_warns(self):
271277
# Check that our specific warning message is present
272278
warning_messages = [str(w.message) for w in warning_list]
273279
self.assertTrue(
274-
any("Cannot write dataframe with empty feature columns to online store" in msg for msg in warning_messages),
275-
f"Expected warning not found. Actual warnings: {warning_messages}"
280+
any(
281+
"Cannot write dataframe with empty feature columns to online store"
282+
in msg
283+
for msg in warning_messages
284+
),
285+
f"Expected warning not found. Actual warnings: {warning_messages}",
276286
)
277287

278288
def test_dataframe_with_empty_feature_columns_async_warns(self):
@@ -301,8 +311,12 @@ async def test_async_empty_features():
301311
# Check that our specific warning message is present
302312
warning_messages = [str(w.message) for w in warning_list]
303313
self.assertTrue(
304-
any("Cannot write dataframe with empty feature columns to online store" in msg for msg in warning_messages),
305-
f"Expected warning not found. Actual warnings: {warning_messages}"
314+
any(
315+
"Cannot write dataframe with empty feature columns to online store"
316+
in msg
317+
for msg in warning_messages
318+
),
319+
f"Expected warning not found. Actual warnings: {warning_messages}",
306320
)
307321

308322
asyncio.run(test_async_empty_features())
@@ -391,8 +405,11 @@ def test_empty_inputs_dict_warns(self):
391405
# Check that our specific warning message is present
392406
warning_messages = [str(w.message) for w in warning_list]
393407
self.assertTrue(
394-
any("Cannot write empty dataframe to online store" in msg for msg in warning_messages),
395-
f"Expected warning not found. Actual warnings: {warning_messages}"
408+
any(
409+
"Cannot write empty dataframe to online store" in msg
410+
for msg in warning_messages
411+
),
412+
f"Expected warning not found. Actual warnings: {warning_messages}",
396413
)
397414

398415
def test_inputs_dict_with_empty_features_warns(self):

0 commit comments

Comments
 (0)