tests: skip modules whose optional extra is not installed

tests/test_emailobject.py

@@ -10,12 +10,21 @@
 from typing import TypeVar
 from zipfile import ZipFile
 
-from pymisp.tools import EMailObject
 from pymisp.exceptions import InvalidMISPObject
 
+# EMailObject is only exported when the email extra is installed. Importing it
+# unguarded makes this module a collection error rather than a skip, which
+# fails the whole run for anyone who installed pymisp without extras.
+try:
+    from pymisp.tools import EMailObject
+    HAS_EMAIL_EXTRA = True
+except ImportError:
+    HAS_EMAIL_EXTRA = False
+
 T = TypeVar('T', bound='TestEmailObject')
 
 
+@unittest.skipUnless(HAS_EMAIL_EXTRA, "requires the email extra")
 class TestEmailObject(unittest.TestCase):
 
     eml_1: BytesIO

tests/test_fileobject.py

@@ -5,10 +5,12 @@
 import unittest
 import json
 from pymisp.tools import FileObject
+from pymisp.tools.fileobject import HAS_MAGIC
 import pathlib
 
 
 class TestFileObject(unittest.TestCase):
+    @unittest.skipUnless(HAS_MAGIC, "requires the fileobjects extra (libmagic)")
     def test_mimeType(self) -> None:
         file_object = FileObject(filepath=pathlib.Path(__file__))
         attributes = json.loads(file_object.to_json())['Attribute']

tests/test_openioc.py

@@ -14,6 +14,7 @@
     pass
 
 
+
 # Two children of an AND-indicator whose combined search-context pair is NOT a
 # defined composite (RegistryItem/Value + FileItem/Md5sum). These must come back
 # as their two individual attributes, not a merged composite.
@@ -54,6 +55,10 @@
 </ioc>"""
 
 
+# Every test here calls load_openioc, which raises when bs4 is absent. The
+# import guard above already anticipates bs4 being missing; without this the
+# tests fail instead of skipping.
+@unittest.skipUnless(openioc.has_bs4, "requires the openioc extra (bs4)")
 class TestOpenIOC(unittest.TestCase):
 
     def test_non_composite_and_indicator_not_merged(self) -> None: