Merge pull request #29 from packethost/moar-tests

Moar tests

Author: mmlb

.drone.yml

@@ -0,0 +1,21 @@
+matrix:
+  PYTHON_VERSION:
+    - 2.7
+    - 3.3
+    - 3.4
+    - 3.5
+    - 3.6
+
+pipeline:
+  fmt_and_lint:
+    image: python:alpine
+    commands:
+      - pip install pylama yapf
+      - yapf -dr packet | (! grep '.')
+      - pylama packet test setup.py
+
+  test:
+    image: python:${PYTHON_VERSION}-alpine
+    commands:
+      - pip install tox
+      - tox

.style.yapf

@@ -0,0 +1,4 @@
+[style]
+based_on_style = chromium
+column_limit = 120
+indent_width = 4

.travis.yml

@@ -1,6 +1,10 @@
 language: python
+python:
+  - "2.7"
+  - "3.3"
+  - "3.4"
+  - "3.5"
+  - "3.6"
 
-install:
-  - pip install .
-  - pip install requests coverage flake8
-script: test/tests.sh
+install: pip install tox
+script: tox

packet/Facility.py

@@ -2,6 +2,7 @@
 
 
 class Facility():
+
     def __init__(self, data):
         self.id = data['id']
         self.code = data['code']

packet/Manager.py

@@ -11,11 +11,12 @@
 
 
 class Manager(BaseAPI):
+
     def __init__(self, auth_token, consumer_token=None):
         super(Manager, self).__init__(auth_token, consumer_token)
 
     def call_api(self, method, type='GET', params=None):
-        return super(Manager, self).call_api(method, type, params)  # pragma: no cover
+        return super(Manager, self).call_api(method, type, params)
 
     def get_user(self):
         return self.call_api("user")
@@ -70,10 +71,20 @@ def list_devices(self, project_id, params={}):
             devices.append(device)
         return devices
 
-    def create_device(self, project_id, hostname, plan, facility,
-                      operating_system, billing_cycle='hourly', userdata='',
-                      locked=False, tags={}, features={}, ipxe_script_url='',
-                      always_pxe=False, public_ipv4_subnet_size=31):
+    def create_device(self,
+                      project_id,
+                      hostname,
+                      plan,
+                      facility,
+                      operating_system,
+                      billing_cycle='hourly',
+                      userdata='',
+                      locked=False,
+                      tags={},
+                      features={},
+                      ipxe_script_url='',
+                      always_pxe=False,
+                      public_ipv4_subnet_size=31):
 
         params = {
             'hostname': hostname,
@@ -152,13 +163,7 @@ def get_capacity(self):
     def validate_capacity(self, servers):
         params = {'servers': []}
         for server in servers:
-            params['servers'].append(
-                {
-                    'facility': server[0],
-                    'plan': server[1],
-                    'quantity': server[2]
-                }
-            )
+            params['servers'].append({'facility': server[0], 'plan': server[1], 'quantity': server[2]})
 
         try:
             self.call_api('/capacity', 'POST', params)

packet/OperatingSystem.py

@@ -2,6 +2,7 @@
 
 
 class OperatingSystem(object):
+
     def __init__(self, data):
         self.slug = data['slug']
         self.name = data['name']

packet/Plan.py

@@ -2,6 +2,7 @@
 
 
 class Plan():
+
     def __init__(self, data):
         self.id = data['id']
         self.name = data['name']

packet/Project.py

@@ -19,9 +19,7 @@ def __init__(self, data, manager):
         self.ssh_keys = data['ssh_keys']
 
     def update(self):
-        params = {
-            "name": self.name
-        }
+        params = {"name": self.name}
 
         return self.manager.call_api("projects/%s" % self.id, type='PATCH', params=params)
 

packet/Volume.py

@@ -28,12 +28,7 @@ def __init__(self, data, manager):
             self.attached_to = None
 
     def update(self):
-        params = {
-            "description": self.description,
-            "size": self.size,
-            "plan": self.plan,
-            "locked": self.locked
-        }
+        params = {"description": self.description, "size": self.size, "plan": self.plan.slug, "locked": self.locked}
 
         return self.manager.call_api("storage/%s" % self.id, type='PATCH', params=params)
 

packet/__init__.py

@@ -7,7 +7,6 @@
 __license__ = "LGPL v3"
 __copyright__ = "Copyright (c) 2015, Aaron Welch and Packet"
 
-
 from .Device import Device  # noqa
 from .Facility import Facility  # noqa
 from .OperatingSystem import OperatingSystem  # noqa