Skip to content

Commit 745fb70

Browse files
committed
fix: formatting issues in test files
1 parent dc75288 commit 745fb70

File tree

6 files changed

+24
-19
lines changed

6 files changed

+24
-19
lines changed

openwisp_radius/tests/test_consumers.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -258,7 +258,8 @@ async def test():
258258
def test_user_can_access_batch_with_invalid_uuid(self):
259259
_, user, _ = self._create_test_data()
260260
consumer = RadiusBatchConsumer()
261-
262-
result = consumer._user_can_access_batch(user, "00000000-0000-0000-0000-000000000000")
263-
self.assertFalse(result)
264261

262+
result = consumer._user_can_access_batch(
263+
user, "00000000-0000-0000-0000-000000000000"
264+
)
265+
self.assertFalse(result)

openwisp_radius/tests/test_counters/test_base_counter.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -157,7 +157,7 @@ def test_consumed_method(self):
157157
consumed = counter.consumed()
158158
self.assertEqual(consumed, 0)
159159
self.assertIsInstance(consumed, int)
160-
160+
161161
from .utils import _acct_data
162162

163163
self._create_radius_accounting(**_acct_data)
@@ -166,5 +166,4 @@ def test_consumed_method(self):
166166
self.assertIsInstance(consumed, int)
167167

168168

169-
170169
del BaseTransactionTestCase

openwisp_radius/tests/test_counters/test_exceptions.py

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,5 @@
1-
from datetime import datetime
21
from unittest.mock import MagicMock
32

4-
from freezegun import freeze_time
5-
63
from ...counters.exceptions import BaseException, MaxQuotaReached, SkipCheck
74
from ..mixins import BaseTestCase
85

openwisp_radius/tests/test_counters/test_resets.py

Lines changed: 15 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,16 @@
1-
from datetime import date, datetime, timedelta
1+
from datetime import date, datetime
22

3-
from dateutil.relativedelta import relativedelta
43
from freezegun import freeze_time
54

6-
from ...counters.resets import _daily, _monthly, _monthly_subscription, _never, _timestamp, _today, _weekly
5+
from ...counters.resets import (
6+
_daily,
7+
_monthly,
8+
_monthly_subscription,
9+
_never,
10+
_timestamp,
11+
_today,
12+
_weekly,
13+
)
714
from ..mixins import BaseTestCase
815

916

@@ -18,7 +25,7 @@ def test_timestamp_function(self):
1825
start = datetime(2021, 11, 3, 0, 0, 0)
1926
end = datetime(2021, 11, 4, 0, 0, 0)
2027
start_ts, end_ts = _timestamp(start, end)
21-
28+
2229
self.assertIsInstance(start_ts, int)
2330
self.assertIsInstance(end_ts, int)
2431
self.assertEqual(start_ts, int(start.timestamp()))
@@ -63,7 +70,7 @@ def test_monthly_subscription_same_month(self):
6370
user = self._get_user()
6471
user.date_joined = datetime.fromisoformat("2021-07-02 12:34:58")
6572
user.save(update_fields=["date_joined"])
66-
73+
6774
start, end = _monthly_subscription(user)
6875
self.assertIsInstance(start, int)
6976
self.assertIsInstance(end, int)
@@ -75,7 +82,7 @@ def test_monthly_subscription_prev_month(self):
7582
user = self._get_user()
7683
user.date_joined = datetime.fromisoformat("2021-07-22 12:34:58")
7784
user.save(update_fields=["date_joined"])
78-
85+
7986
start, end = _monthly_subscription(user)
8087
self.assertEqual(str(datetime.fromtimestamp(start)), "2021-10-22 00:00:00")
8188
self.assertEqual(str(datetime.fromtimestamp(start)), "2021-10-22 00:00:00")
@@ -86,7 +93,7 @@ def test_monthly_subscription_future_day(self):
8693
user = self._get_user()
8794
user.date_joined = datetime.fromisoformat("2021-07-25 12:34:58")
8895
user.save(update_fields=["date_joined"])
89-
96+
9097
start, end = _monthly_subscription(user)
9198
self.assertEqual(str(datetime.fromtimestamp(start)), "2021-10-25 00:00:00")
9299
self.assertEqual(str(datetime.fromtimestamp(end)), "2021-11-25 00:00:00")
@@ -96,7 +103,7 @@ def test_monthly_subscription_with_kwargs(self):
96103
user = self._get_user()
97104
user.date_joined = datetime.fromisoformat("2021-07-15 12:00:00")
98105
user.save(update_fields=["date_joined"])
99-
106+
100107
start, end = _monthly_subscription(user, counter=None)
101108
self.assertIsInstance(start, int)
102109
self.assertIsInstance(end, int)

openwisp_radius/tests/test_counters/test_sqlite_counters.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -147,11 +147,13 @@ def test_sqlite_counter_mixin_init(self):
147147
def test_sqlite_traffic_mixin_sql_property(self):
148148
from ...counters.sqlite import SqliteTrafficMixin
149149

150-
self.assertIn("SELECT SUM(acctinputoctets + acctoutputoctets)", SqliteTrafficMixin.sql)
150+
self.assertIn(
151+
"SELECT SUM(acctinputoctets + acctoutputoctets)",
152+
SqliteTrafficMixin.sql,
153+
)
151154
self.assertIn("FROM radacct", SqliteTrafficMixin.sql)
152155
self.assertIn("WHERE username=%s", SqliteTrafficMixin.sql)
153156
self.assertIn("AND organization_id=%s", SqliteTrafficMixin.sql)
154157

155158

156-
157159
del BaseTransactionTestCase

openwisp_radius/tests/test_radclient.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -158,4 +158,3 @@ def test_rad_client_repr(self):
158158
self.assertIsNotNone(client.client)
159159
self.assertEqual(client.client.server, "127.0.0.1")
160160
self.assertEqual(client.client.secret, b"testing")
161-

0 commit comments

Comments
 (0)