Add some tests

- also fix that travis build

2 files changed, 41 insertions, 8 deletions

diff --git a/test/test_common.py b/test/test_common.py
index 36680328..315318bd 100644
--- a/test/test_common.py
+++ b/test/test_common.py
@@ -2,16 +2,17 @@
 Test the functions in the `common` module
 """
 
-import pytest
 import sys
 sys.path.append('src')
 
-from common import (datetime_tuple, get_utc_time, get_local_time, shell_split,
-                    find_argument_quoted, find_argument_unquoted,
-                    parse_str_to_secs, parse_secs_to_str)
 import time
-from datetime import timedelta
+import pytest
 import datetime
+from sleekxmpp import JID
+from datetime import timedelta
+from common import (datetime_tuple, get_utc_time, get_local_time, shell_split,
+                    find_argument_quoted, find_argument_unquoted,
+                    parse_str_to_secs, parse_secs_to_str, safeJID)
 
 def test_datetime_tuple():
     time.timezone = 0
@@ -39,8 +40,6 @@ def test_local_time():
     time.altzone = -3600
     assert get_local_time(d) == d - delta
 
-#def find_delayed_tag(message):
-
 def test_shell_split():
     assert shell_split('"sdf 1" "toto 2"') == ['sdf 1', 'toto 2']
     assert shell_split('toto "titi"') == ['toto', 'titi']
@@ -69,4 +68,12 @@ def test_parse_str_to_secs():
     assert parse_str_to_secs("1d3mfaiiiiil") == 0
 
 def test_parse_secs_to_str():
-    assert  parse_secs_to_str(3601) == '1h1s'
+    assert parse_secs_to_str(3601) == '1h1s'
+    assert parse_secs_to_str(0) == '0s'
+
+    with pytest.raises(TypeError):
+        parse_secs_to_str('toto')
+
+def test_safeJID():
+    assert safeJID('toto@titi/tata') == JID('toto@titi/tata')
+    assert safeJID('é_è') == JID('')
diff --git a/test/test_theming.py b/test/test_theming.py
new file mode 100644
index 00000000..9cdb4829
--- /dev/null
+++ b/test/test_theming.py
@@ -0,0 +1,26 @@
+"""
+Test the functions in the `theming` module
+"""
+
+import sys
+import pytest
+sys.path.append('src')
+
+from theming import dump_tuple, read_tuple
+
+def test_read_tuple():
+    assert read_tuple('1,-1,u') == ((1, -1), 'u')
+    assert read_tuple('1,2') == ((1, 2), None)
+
+    with pytest.raises(IndexError):
+        read_tuple('1')
+
+    with pytest.raises(ValueError):
+        read_tuple('toto')
+
+def test_dump_tuple():
+    assert dump_tuple((1, 2)) == '1,2'
+    assert dump_tuple((1, )) == '1'
+    assert dump_tuple((1, 2, 'u')) == '1,2,u'
+
+