Skip to content

Commit

Permalink
Return directly as list
Browse files Browse the repository at this point in the history
Return typing was changed from tuple to sequence, so
no need to conver to tuple anymore as List is also a sequence
  • Loading branch information
stveit committed Sep 12, 2023
1 parent 45f67c6 commit 9638692
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions python/nav/portadmin/napalm/juniper.py
Original file line number Diff line number Diff line change
Expand Up @@ -448,8 +448,7 @@ def raise_if_not_configurable(self):
raise DeviceNotConfigurableError("Device has no NAPALM profile")

def get_poe_state_options(self) -> Sequence[PoeState]:
options_list = [self.POE_ENABLED, self.POE_DISABLED]
return tuple(options_list)
return [self.POE_ENABLED, self.POE_DISABLED]

Check warning on line 451 in python/nav/portadmin/napalm/juniper.py

View check run for this annotation

Codecov / codecov/patch

python/nav/portadmin/napalm/juniper.py#L451

Added line #L451 was not covered by tests

@wrap_unhandled_rpc_errors
def set_poe_state(self, interface: manage.Interface, state: PoeState):
Expand Down

0 comments on commit 9638692

Please sign in to comment.