Use explicit re-raise chaining in util modules

sanitizers · Sep 20, 2020 · 45a0e3d · 45a0e3d
1 parent c7a3e2d
commit 45a0e3d
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/octomachinery/github/utils/event_utils.py b/octomachinery/github/utils/event_utils.py
@@ -18,8 +18,8 @@ def _probe_yaml(event_file_fd):
             ),
             3,
         )
-    except yaml.parser.ParserError:
-        raise ValueError('YAML file is not valid')
+    except yaml.parser.ParserError as yaml_err:
+        raise ValueError('YAML file is not valid') from yaml_err
     finally:
         event_file_fd.seek(0)
 
@@ -108,8 +108,8 @@ def validate_http_headers(headers):
     x_gh_delivery_exc = ValueError('X-GitHub-Delivery must be of type UUID4')
     try:
         x_gh_delivery_uuid = UUID(headers['x-github-delivery'])
-    except ValueError:
-        raise x_gh_delivery_exc
+    except ValueError as val_err:
+        raise x_gh_delivery_exc from val_err
     if x_gh_delivery_uuid.version != 4:
         raise x_gh_delivery_exc
 

diff --git a/octomachinery/runtime/utils.py b/octomachinery/runtime/utils.py
@@ -32,7 +32,9 @@ def __getattr__(self, name):
         try:
             return self.__map__[name].get()
         except LookupError:
-            raise ContextLookupError(f'No `{name}` present in the context')
+            raise ContextLookupError(
+                f'No `{name}` present in the context',
+            ) from None
 
     def __setattr__(self, name, value):
         if name in ('__map__', '__token_map__'):