Skip to content

Commit 98c883f

Browse files
committed
test: TT-180 remove unused imports, fix quotes
1 parent 690af81 commit 98c883f

File tree

2 files changed

+3
-5
lines changed

2 files changed

+3
-5
lines changed

tests/time_tracker_api/activities/activities_model_test.py

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
11
from unittest.mock import Mock, patch
22
import pytest
3-
from faker import Faker
4-
from typing import List
53

64
from commons.data_access_layer.database import EventContext
75
from time_tracker_api.activities.activities_model import (
@@ -28,7 +26,7 @@ def test_validate_list(
2826
@pytest.mark.parametrize(
2927
"id_list,expected_result",
3028
[
31-
(["id1"], '("id1")'),
29+
(["id1"], "('id1')"),
3230
(["id1", "id2"], "('id1', 'id2')"),
3331
(["id1", "id2", "id3", "id4"], "('id1', 'id2', 'id3', 'id4')"),
3432
],
@@ -48,7 +46,7 @@ def test_convert_list_to_tuple_string(
4846
@pytest.mark.parametrize(
4947
"id_list,expected_result",
5048
[
51-
(["id1"], 'c.id IN ("id1")'),
49+
(["id1"], "c.id IN ('id1')"),
5250
(["id1", "id2"], "c.id IN ('id1', 'id2')"),
5351
(["id1", "id2", "id3", "id4"], "c.id IN ('id1', 'id2', 'id3', 'id4')"),
5452
],

time_tracker_api/activities/activities_model.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ def create_sql_in_condition(self, id_list):
5858
def convert_list_to_tuple_string(self, id_list):
5959
self.validate_list(id_list)
6060
id_value = (
61-
f'("{id_list[0]}")' if len(id_list) == 1 else str(tuple(id_list))
61+
f"('{id_list[0]}')" if len(id_list) == 1 else str(tuple(id_list))
6262
)
6363
return id_value
6464

0 commit comments

Comments
 (0)