fix basic auth

Author: CamDavidsonPilon

core/pioreactor/mureq.py

@@ -345,7 +345,10 @@ def _prepare_outgoing_headers(
 ) -> HTTPMessage:
     from pioreactor.config import config
 
-    DEFAULT_AUTH = f'Basic {config.get("ui_basic_auth", "api_key", fallback="")}'
+    if config.get("ui_basic_auth", "api_key", fallback=""):
+        default_auth = f'Basic {config.get("ui_basic_auth", "api_key", fallback="")}'
+    else:
+        default_auth = ""
 
     if headers is None:
         headers = HTTPMessage()
@@ -359,7 +362,8 @@ def _prepare_outgoing_headers(
             new_headers[k] = v
         headers = new_headers
     _setdefault_header(headers, "User-Agent", DEFAULT_UA)
-    _setdefault_header(headers, "Authorization", DEFAULT_AUTH)
+    if default_auth:
+        _setdefault_header(headers, "Authorization", default_auth)
     return headers
 
 

core/tests/test_dosing_automation.py

@@ -1004,7 +1004,9 @@ def test_AltMediaFractionCalculator() -> None:
         timestamp=default_datetime_for_pioreactor(1),
         source_of_event="test",
     )
-    assert ac.update(add_alt_media_event, 0.0, current_volume_ml) == round(1 / (current_volume_ml + 1), 10)
+    assert round(ac.update(add_alt_media_event, 0.0, current_volume_ml), 10) == round(
+        1 / (current_volume_ml + 1), 10
+    )
 
     alt_media_added = 2.0
     add_alt_media_event = DosingEvent(

core/tests/test_execute_experiment_profile.py

@@ -25,6 +25,7 @@
 from pioreactor.experiment_profiles.profile_struct import Stop
 from pioreactor.experiment_profiles.profile_struct import Update
 from pioreactor.experiment_profiles.profile_struct import When
+from pioreactor.mureq import HTTPErrorStatus
 from pioreactor.pubsub import collect_all_logs_of_level
 from pioreactor.pubsub import publish
 from pioreactor.pubsub import subscribe_and_callback
@@ -752,12 +753,14 @@ def test_profiles_in_github_repo() -> None:
     # Set the API endpoint URL
     owner = "Pioreactor"
     repo = "experiment_profile_examples"
-    path = ""  # Top level directory
-    api_url = f"https://api.github.com/repos/{owner}/{repo}/contents/{path}"
+    api_url = f"https://api.github.com/repos/{owner}/{repo}/contents"
 
-    # Make a GET request to the GitHub API
     response = get(api_url)
-    response.raise_for_status()  # Will raise an HTTPError if the HTTP request returned an unsuccessful status code
+    try:
+        response.raise_for_status()
+    except HTTPErrorStatus as e:
+        # raise HTTPErrorStatus(f"HTTP error: Failed to fetch repository contents from {api_url}: saw {str(e)}")
+        raise e
 
     # Check for YAML files
     yaml_files = [