Merge pull request #956 from hackingmaterials/mp

xfail tests pending rewrite to current MP API client

Author: esoteric-ephemera

tests/data_retrieval/test_retrieve_MP.py

@@ -1,5 +1,6 @@
 import unittest
 
+import pytest
 from pymatgen.core import SETTINGS
 from pymatgen.electronic_structure.bandstructure import (
     BandStructure,
@@ -10,7 +11,8 @@
 from matminer.data_retrieval.retrieve_MP import MPDataRetrieval
 
 
-@unittest.skipIf(not SETTINGS.get("PMG_MAPI_KEY", ""), "PMG_MAPI_KEY not in environment variables.")
+@pytest.mark.skipif(not SETTINGS.get("PMG_MAPI_KEY", ""), reason="PMG_MAPI_KEY not in environment variables.")
+@pytest.mark.xfail(reason="Needs updates to use current MP API client - flakey test.", strict=False)
 class MPDataRetrievalTest(unittest.TestCase):
     def setUp(self):
         self.mpdr = MPDataRetrieval()