Resolve Failing Tests: Correction to adanced_usage=True (#1074)

* Revert change: `pprint` to `print` ... `pprint` is valid and the correct thing for this example (pretty-print-usage)

Signed-off-by: Martyn Bristow <martyn.bristow@gmail.com>

* Resolution of `advanced_mode=True` breaks existing functionality by returning raw response

Return None on 404. Preserve Existing Functionality

Signed-off-by: Martyn Bristow <martyn.bristow@gmail.com>

Signed-off-by: Martyn Bristow <martyn.bristow@gmail.com>

Author: martynbristow

atlassian/bitbucket/cloud/repositories/pipelines.py

@@ -243,11 +243,13 @@ def log(self, start=None, end=None):
         response = None
         try:
             response = self.get("log", headers=headers, advanced_mode=True)
+            response.raise_for_status()
         except HTTPError as e:
             # A 404 indicates that no log is present.
             if not e.response.status_code == 404:
                 # Rethrow the exception
                 raise
+            return None
 
         if response is None:
             if start is None:

tests/test_bitbucket_cloud_oo.py

@@ -92,6 +92,7 @@ def test_get_pipeline_step(self):
         assert result["uuid"] == "{PipelineStep1Uuid}", "Result of [get_pipeline_step(...)]"
 
     def test_get_pipeline_step_log_1(self):
+        """Test Not Found"""
         result = BITBUCKET.get_pipeline_step_log(
             "TestWorkspace1", "testrepository1", "{PipelineUuid}", "{PipelineStep1Uuid}"
         )