Skip to content

Commit dd91749

Browse files
committed
Sonarcloud smell resolves continued for Generic
1 parent 8ca36b8 commit dd91749

File tree

1 file changed

+15
-10
lines changed

1 file changed

+15
-10
lines changed

tests/test_generic.py

Lines changed: 15 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -19,40 +19,44 @@ async def test_fail_legacy_system(self):
1919
self.smile_setup = "faulty_stretch"
2020
try:
2121
_server, _smile, _client = await self.connect_wrapper()
22-
assert False # pragma: no cover
22+
setup_result = False # pragma: no cover
2323
except pw_exceptions.InvalidXMLError:
24-
assert True
24+
setup_result = True
25+
assert setup_result
2526

2627
@pytest.mark.asyncio
2728
async def test_fail_anna_connected_to_adam(self):
2829
"""Test erroneous adam with anna system."""
2930
self.smile_setup = "anna_connected_to_adam"
3031
try:
3132
_server, _smile, _client = await self.connect_wrapper()
32-
assert False # pragma: no cover
33+
setup_result = False # pragma: no cover
3334
except pw_exceptions.InvalidSetupError:
34-
assert True
35+
setup_result = True
36+
assert setup_result
3537

3638
@pytest.mark.asyncio
3739
async def test_invalid_credentials(self):
3840
"""Test P1 with invalid credentials setup."""
3941
self.smile_setup = "p1v4"
4042
try:
4143
await self.connect_wrapper(fail_auth=True)
42-
assert False # pragma: no cover
44+
setup_result = False # pragma: no cover
4345
except pw_exceptions.InvalidAuthentication:
4446
_LOGGER.debug("InvalidAuthentication raised successfully")
45-
assert True
47+
setup_result = True
48+
assert setup_result
4649

4750
@pytest.mark.asyncio
4851
async def test_connect_fail_firmware(self):
4952
"""Test a P1 non existing firmware setup."""
5053
self.smile_setup = "fail_firmware"
5154
try:
5255
await self.connect_wrapper()
53-
assert False # pragma: no cover
56+
setup_result = False # pragma: no cover
5457
except pw_exceptions.UnsupportedDeviceError:
55-
assert True
58+
setup_result = True
59+
assert setup_result
5660

5761
# Test connect for timeout
5862
@patch(
@@ -70,6 +74,7 @@ async def test_connect_timeout(self, timeout_test):
7074
smile,
7175
client,
7276
) = await self.connect_wrapper()
73-
assert False # pragma: no cover
77+
setup_result = False # pragma: no cover
7478
except pw_exceptions.PlugwiseException:
75-
assert True
79+
setup_result = True
80+
assert setup_result

0 commit comments

Comments
 (0)