NavaThread exception (#53)

* fix : NavaThread run method updated

* fix : error_test.py updated

* fix : error_test.py updated

* fix : NavaThread exception section updated

* fix : error_test.py updated

* fix : error_test.py updated

* revert error_test.py

* fix : autopep8

* doc : CHANGELOG.md updated

CHANGELOG.md

@@ -10,6 +10,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 - `engine` parameter added to `NavaThread` class
 - `README.md` modified
 - Test system modified
+- `NavaThread` exception bug fixed
 - `__play_win` function renamed to `__play_winsound`
 - `__play_mac` function renamed to `__play_afplay`
 - `__play_linux` function renamed to `__play_alsa`

nava/functions.py

@@ -302,7 +302,7 @@ def play(sound_path, async_mode=False, loop=False, engine=Engine.AUTO):
             return __play_afplay(sound_path=sound_path, async_mode=async_mode, loop=loop)
         elif engine == Engine.ALSA:
             return __play_alsa(sound_path=sound_path, async_mode=async_mode, loop=loop)
-    except Exception:  # pragma: no cover
+    except Exception:
         raise NavaBaseError(SOUND_FILE_PLAY_ERROR)
 
 

nava/thread.py

@@ -2,7 +2,8 @@
 """Nava thread."""
 
 import threading
-from .params import Engine
+from .params import Engine, SOUND_FILE_PLAY_ERROR
+from .errors import NavaBaseError
 
 
 class NavaThread(threading.Thread):
@@ -25,22 +26,27 @@ def __init__(self, loop, engine, *args, **kwargs):
         self._play_process = None
         self._loop = loop
         self._engine = engine
+        self._nava_exception = None
 
     def run(self):
         """
         Run target function.
 
         :return: None
         """
-        if self._target is not None:
-            if self._engine == Engine.WINSOUND:
-                self._play_process = self._target(*self._args, **self._kwargs)
-            else:
-                while True:
+        try:
+            if self._target is not None:
+                if self._engine == Engine.WINSOUND:
                     self._play_process = self._target(*self._args, **self._kwargs)
-                    self._play_process.wait()
-                    if not self._loop:
-                        break
+                else:
+                    while True:
+                        self._play_process = self._target(*self._args, **self._kwargs)
+                        self._play_process.wait()
+                        if not self._loop:
+                            break
+        except Exception:  # pragma: no cover
+            self._nava_exception = SOUND_FILE_PLAY_ERROR
+            raise NavaBaseError(SOUND_FILE_PLAY_ERROR)
 
     def stop(self):
         """

test/error_test.py

@@ -1,7 +1,8 @@
 # -*- coding: utf-8 -*-
 """
 >>> import os
->>> from nava import play, stop
+>>> import sys
+>>> from nava import play, stop, Engine
 >>> test_sound_path = os.path.join("others", "test.wav")
 >>> play("test.wav")
 Traceback (most recent call last):
@@ -23,6 +24,16 @@
 Traceback (most recent call last):
     ...
 nava.errors.NavaBaseError: `engine` type must be `Engine` enum.
+>>> sys_platform = sys.platform
+>>> if sys_platform == "win32":
+...     sound_id = play(test_sound_path, async_mode=False, engine=Engine.AFPLAY)
+... elif sys_platform == "darwin":
+...     sound_id = play(test_sound_path, async_mode=False, engine=Engine.WINSOUND)
+... else:
+...     sound_id = play(test_sound_path, async_mode=False, engine=Engine.WINSOUND)
+Traceback (most recent call last):
+    ...
+nava.errors.NavaBaseError: Sound can not play due to some issues.
 >>> import nava
 >>> nava.functions.play_cli("test2.wav")
 Error: Given sound file doesn't exist.