Logging issue (#129)

* Stop changing the root logger since it also effect other modules

Author: roywilly

src/sumo/wrapper/_auth.py

@@ -6,11 +6,6 @@
 import os
 import logging
 
-logging.basicConfig(
-    format="%(asctime)s %(levelname)-8s %(message)s",
-    level=logging.INFO,
-    datefmt="%Y-%m-%d %H:%M:%S",
-)
 
 logger = logging.getLogger("sumo.wrapper")
 
@@ -31,7 +26,7 @@ def __init__(
         writeback=False,
         verbosity="CRITICAL",
     ):
-
+        logger.setLevel(verbosity)
         logger.debug("Initialize Auth")
         self.client_id = client_id
         logger.debug("client_id is %s", self.client_id)
@@ -213,7 +208,6 @@ def _read_cache(self):
             self.cache.deserialize(file.read())
 
     def _get_cache(self):
-
         logger.debug("_get_cache")
         self.cache = msal.SerializableTokenCache()
 

src/sumo/wrapper/_call_azure_api.py

@@ -41,7 +41,6 @@ def __init__(
         writeback=False,
         verbosity="CRITICAL",
     ):
-
         logger.setLevel(level=verbosity)
 
         self.resource_id = resource_id

src/sumo/wrapper/_call_sumo_api.py

@@ -378,7 +378,6 @@ def _post_objects(
         """
 
         if not url:
-
             url = f"{self.base_url}/objects"
 
             if object_id:

tests/test_sumo_thin_client.py

@@ -154,7 +154,7 @@ def test_upload_search_delete_ensemble_child(token):
     result = _delete_object(C=C, object_id=case_id)
     assert result == "Accepted"
 
-    sleep(4)
+    sleep(40)
 
     # Search for ensemble
     search_results = C.api.get("/searchroot", query=query, select=["_source"])
@@ -189,6 +189,11 @@ def test_upload_duplicate_ensemble(token):
 
     with open("tests/testdata/case.yml", "r") as stream:
         fmu_metadata2 = yaml.safe_load(stream)
+    
+    case_uuid = str(uuid.uuid4())
+    fmu_metadata1["fmu"]["case"]["uuid"] = case_uuid
+    fmu_metadata2["fmu"]["case"]["uuid"] = case_uuid
+
 
     # upload case metadata, get object_id
     response1 = _upload_parent_object(C=C, json=fmu_metadata1)
@@ -206,10 +211,11 @@ def test_upload_duplicate_ensemble(token):
     assert get_result["_id"] == case_id1
 
     # Delete Ensemble
+    sleep(5)
     result = _delete_object(C=C, object_id=case_id1)
     assert result == "Accepted"
 
-    sleep(5)
+    sleep(50)
 
     # Search for ensemble
     with pytest.raises(Exception):

tests/testdata/surface.yml

@@ -133,9 +133,11 @@ data: # The data block describes the actual data (e.g. surface). Only present in
     zmin: 1244.039
     zmax: 2302.683
   time:
-    - value: '2020-10-28T14:28:02'
+    t0:
+      value: '2020-10-28T14:28:02'
       label: "some label"
-    - value: '2020-10-28T14:28:02'
+    t1:
+      value: '2020-10-28T14:33:02'
       label: "some other label"
   is_prediction: true # A mechanism for separating pure QC output from actual predictions
   is_observation: true # Used for 4D data currently but also valid for other data?