Skip to content

Commit e6feb69

Browse files
caberoscaberos
authored andcommitted
fix the team code review comments
1 parent d3e3790 commit e6feb69

File tree

3 files changed

+10
-9
lines changed

3 files changed

+10
-9
lines changed

SoftLayer/CLI/account/licenses.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
"""Show the all account licenses."""
1+
"""Show all licenses."""
22
# :license: MIT, see LICENSE for more details.
33
import click
44
from SoftLayer import utils
@@ -11,7 +11,7 @@
1111
@click.command()
1212
@environment.pass_env
1313
def cli(env):
14-
"""return the control panel and VMWare licenses"""
14+
"""Show all licenses."""
1515

1616
manager = AccountManager(env.client)
1717

SoftLayer/fixtures/SoftLayer_Account.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1104,7 +1104,7 @@
11041104
getActiveAccountLicenses = [{
11051105
"accountId": 123456,
11061106
"capacity": "4",
1107-
"key": "M02A5-6CJ8L-J8R9H-000R0-CDR70",
1107+
"key": "Y8GNS-7QRNG-OUIJO-MATEI-5GJRM",
11081108
"units": "CPU",
11091109
"billingItem": {
11101110
"allowCancellationFlag": 1,
@@ -1141,7 +1141,7 @@
11411141
{
11421142
"accountId": 123456,
11431143
"capacity": "4",
1144-
"key": "4122M-ABXC05-K829T-098HP-00QJM",
1144+
"key": "TSZES-SJF85-04GLD-AXA64-8O1EO",
11451145
"units": "CPU",
11461146
"billingItem": {
11471147
"allowCancellationFlag": 1,
@@ -1179,7 +1179,7 @@
11791179
getActiveVirtualLicenses = [{
11801180
"id": 12345,
11811181
"ipAddress": "192.168.23.78",
1182-
"key": "PLSK.06866259.0000",
1182+
"key": "TEST.60220734.0000",
11831183
"billingItem": {
11841184
"categoryCode": "control_panel",
11851185
"description": "Plesk Onyx (Linux) - (Unlimited) - VPS "

SoftLayer/managers/account.py

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -307,8 +307,8 @@ def get_network_message_delivery_accounts(self):
307307
def get_active_virtual_licenses(self):
308308
"""Gets all active virtual licenses account.
309309
310-
:returns: active virtual licenses account
311-
"""
310+
:returns: active virtual licenses account
311+
"""
312312

313313
_mask = """billingItem[categoryCode,createDate,description],
314314
key,id,ipAddress,
@@ -320,8 +320,9 @@ def get_active_virtual_licenses(self):
320320
def get_active_account_licenses(self):
321321
"""Gets all active account licenses.
322322
323-
:returns: Active account Licenses
324-
"""
323+
:returns: Active account Licenses
324+
"""
325+
325326
_mask = """billingItem,softwareDescription"""
326327

327328
return self.client['SoftLayer_Account'].getActiveAccountLicenses(mask=_mask)

0 commit comments

Comments
 (0)