Merge pull request #300 from andrey-yantsen/fix-tests

Fix tests

Author: Hellowlol

.travis.yml

@@ -1,9 +1,11 @@
 language: python
 
 stages:
-  - test
+  - syntax
+  - name: test
+    if: type != pull_request
   - name: deploy
-    if: tag IS present
+    if: tag = present
 
 sudo: required
 services:
@@ -43,11 +45,8 @@ after_script:
 
 jobs:
   include:
-    - python: 3.6
-      env:
-        - PLEX_CONTAINER_TAG=1.3.2.3112-1751929
-        - TEST_ACCOUNT_ONCE=1
-    - python: 3.6
+    - stage: syntax
+      python: 3.6
       name: "Flake8"
       install:
         - pip install -r requirements_dev.txt
@@ -56,6 +55,11 @@ jobs:
       after_script: true
       env:
         - PLEX_CONTAINER_TAG=latest
+    - stage: test
+      python: 3.6
+      env:
+        - PLEX_CONTAINER_TAG=1.3.2.3112-1751929
+        - TEST_ACCOUNT_ONCE=1
     - stage: deploy
       name: "Deploy to PyPi"
       python: 3.6

plexapi/photo.py

@@ -2,7 +2,6 @@
 from plexapi import media, utils
 from plexapi.base import PlexPartialObject
 from plexapi.exceptions import NotFound, BadRequest
-from plexapi.exceptions import NotFound
 from plexapi.compat import quote_plus
 
 

tests/test_utils.py

@@ -10,7 +10,7 @@ def test_utils_toDatetime():
 
 
 def test_utils_threaded():
-    def _squared(num, results, i):
+    def _squared(num, results, i, job_is_done_event=None):
         time.sleep(0.5)
         results[i] = num * num
     starttime = time.time()