Skip to content

Commit e737202

Browse files
committed
Run pyupgrade and flynt
1 parent 97ea738 commit e737202

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

41 files changed

+99
-272
lines changed

test/conftest.py

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,3 @@
1-
from __future__ import unicode_literals
2-
from __future__ import print_function
3-
from __future__ import division
4-
from __future__ import absolute_import
51
import json
62
import os
73
import pytest
@@ -27,11 +23,11 @@ def manager():
2723
def read_from_file(filename):
2824
filename = filename.replace("/", "_")
2925
cwd = os.path.dirname(__file__)
30-
f = open(cwd + '/json_data/'+filename, 'r')
26+
f = open(cwd + '/json_data/'+filename)
3127
return f.read()
3228

3329

34-
class Mock(object):
30+
class Mock:
3531
base_url = 'https://api.upcloud.com/1.3'
3632

3733
@staticmethod

test/helpers/infra.py

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,3 @@
1-
from __future__ import print_function
2-
from __future__ import unicode_literals
3-
from __future__ import division
4-
from __future__ import absolute_import
5-
61
from upcloud_api import CloudManager, Storage, FirewallRule, Tag, IPAddress
72
from upcloud_api.server import Server, login_user_block
83

test/helpers/infra_helpers.py

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,3 @@
1-
from __future__ import print_function
2-
from __future__ import unicode_literals
3-
from __future__ import division
4-
from __future__ import absolute_import
5-
61
from upcloud_api import Tag
72

83

test/test_apidoc/test_10_ip_addresses.py

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,9 @@
1-
from __future__ import unicode_literals
2-
from __future__ import print_function
3-
from __future__ import division
4-
from __future__ import absolute_import
5-
6-
71
from upcloud_api import IPAddress, Server
82
from conftest import Mock, read_from_file
93
import json
104
import responses
115

12-
class TestIP(object):
6+
class TestIP:
137

148
def test_ip_in_server_creation(self):
159
"""IPAddress in server creation.
@@ -36,7 +30,7 @@ def test_ip_in_server_details(self):
3630
data = read_from_file('server_00798b85-efdc-41ca-8021-f6ef457b8531.json')
3731
s = Server(**json.loads(data))
3832
for ip in s.ip_addresses:
39-
assert set(ip.to_dict().keys()) == set(['address', 'family', 'access'])
33+
assert set(ip.to_dict().keys()) == {'address', 'family', 'access'}
4034

4135
def test_ip_details(self):
4236
"""IPAdress LIST/GET.

test/test_cloud_manager.py

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,9 @@
1-
from __future__ import unicode_literals
2-
from __future__ import print_function
3-
from __future__ import division
4-
from __future__ import absolute_import
5-
61
from conftest import Mock
72
import json
83
import responses
94

105

11-
class TestCloudManagerBasic(object):
6+
class TestCloudManagerBasic:
127
@responses.activate
138
def test_get_account(self, manager):
149
data = Mock.mock_get("account")

test/test_firewall.py

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,3 @@
1-
from __future__ import unicode_literals
2-
from __future__ import print_function
3-
from __future__ import division
4-
from __future__ import absolute_import
5-
61
from upcloud_api import FirewallRule
72

83
from conftest import Mock
@@ -30,7 +25,7 @@ def check_fields(body):
3025
"""
3126
for field in required_fields:
3227
if field not in request_body['firewall_rule']:
33-
raise Exception('missing required field: {0}. Body was:{1}'.format(field, request_body))
28+
raise Exception(f'missing required field: {field}. Body was:{request_body}')
3429

3530
if isinstance(request_body, list):
3631
for body in request_body:
@@ -42,7 +37,7 @@ def check_fields(body):
4237

4338

4439

45-
class TestFirewall(object):
40+
class TestFirewall:
4641
@responses.activate
4742
def test_add_firewall_rule(self, manager):
4843
Mock.mock_get('server/00798b85-efdc-41ca-8021-f6ef457b8531')

test/test_host.py

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,8 @@
1-
from __future__ import unicode_literals
2-
from __future__ import print_function
3-
from __future__ import division
4-
from __future__ import absolute_import
5-
61
from conftest import Mock
72
import responses
83

94

10-
class TestHost(object):
5+
class TestHost:
116
@responses.activate
127
def test_get_hosts(self, manager):
138
data = Mock.mock_get('host')

test/test_integration/test_integration_test.py

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,3 @@
1-
from __future__ import unicode_literals
2-
from __future__ import print_function
3-
from __future__ import division
4-
from __future__ import absolute_import
51
import os
62
import pytest
73
import multiprocessing
@@ -52,11 +48,11 @@ def teardown_module(module):
5248
pool.map(destroy_server, manager.get_servers())
5349
pool.map(delete_tag, manager.get_tags())
5450
else:
55-
print('removing {0}'.format(CREATED_SERVERS))
51+
print(f'removing {CREATED_SERVERS}')
5652
for server in CREATED_SERVERS:
5753
server.stop_and_destroy()
5854

59-
print('removing {0}'.format(CREATED_TAGS))
55+
print(f'removing {CREATED_TAGS}')
6056
for tag in CREATED_TAGS:
6157
manager.delete_tag(tag)
6258

@@ -97,7 +93,7 @@ def test_infra_ops():
9793
found = True
9894

9995
if not found:
100-
raise Exception('server {0} not found in all_servers'.format(cs.uuid))
96+
raise Exception(f'server {cs.uuid} not found in all_servers')
10197

10298
# assert servers' states
10399
# TODO(elnygren): add more assertions here

test/test_ip_manager.py

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,7 @@
1-
from __future__ import unicode_literals
2-
from __future__ import print_function
3-
from __future__ import division
4-
from __future__ import absolute_import
5-
61
from conftest import Mock
72
import responses
83

9-
class TestIP(object):
4+
class TestIP:
105
@responses.activate
116
def test_get_ip(self, manager):
127
data = Mock.mock_get('ip_address/10.1.0.101')

test/test_network.py

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,8 @@
1-
from __future__ import unicode_literals
2-
from __future__ import print_function
3-
from __future__ import division
4-
from __future__ import absolute_import
5-
61
from conftest import Mock
72
import responses
83

94

10-
class TestNetwork(object):
5+
class TestNetwork:
116
@responses.activate
127
def test_get_networks(self, manager):
138
data = Mock.mock_get('network')

0 commit comments

Comments
 (0)