Merge branch 'test-skip-dependency' into 'master'

CI: Skip test if the emoji dep is not here

See merge request poezio/slixmpp!96

1 files changed, 6 insertions, 5 deletions

diff --git a/tests/test_stanza_xep_0444.py b/tests/test_stanza_xep_0444.py
index 276460c5..0fe3f6fc 100644
--- a/tests/test_stanza_xep_0444.py
+++ b/tests/test_stanza_xep_0444.py
@@ -13,6 +13,11 @@ from slixmpp.plugins.xep_0444 import XEP_0444
 import slixmpp.plugins.xep_0444.stanza as stanza
 from slixmpp.xmlstream import register_stanza_plugin
 
+try:
+    import emoji
+except ImportError:
+    emoji = None
+
 
 class TestReactions(SlixTest):
 
@@ -41,13 +46,9 @@ class TestReactions(SlixTest):
         self.assertEqual({'😃', '🤗'}, msg['reactions']['values'])
 
 
+    @unittest.skipIf(emoji is None, 'Emoji package not installed')
     def testCreateReactionsUnrestricted(self):
         """Testing creating Reactions with the extra all_chars arg."""
-        try:
-            import emoji
-        except ImportError:
-            # No emoji package: this test does not make sense
-            return
         xmlstring = """
           <message>
               <reactions xmlns="urn:xmpp:reactions:0" id="abcd">