Fixes the missing exception throw action when an api request is executed, like BucketExistsAsync, etc.

Minio.Functional.Tests/FunctionalTest.cs

@@ -371,21 +371,29 @@ internal static async Task BucketExists_Test(IMinioClient minio)
                 await minio.RemoveBucketAsync(rbArgs).ConfigureAwait(false);
 
             var found = await minio.BucketExistsAsync(beArgs).ConfigureAwait(false);
-            Assert.IsFalse(found);
+            if (found)
+                throw new Exception("BucketExistsAsync api failed to return false for a non-existing bucket");
+
             await minio.MakeBucketAsync(mbArgs).ConfigureAwait(false);
             found = await minio.BucketExistsAsync(beArgs).ConfigureAwait(false);
-            Assert.IsTrue(found);
-            new MintLogger(nameof(BucketExists_Test), bucketExistsSignature, "Tests whether BucketExists passes",
+
+            if (!found)
+                throw new Exception("BucketExistsAsync api failed to return true for an existing bucket");
+
+            new MintLogger(nameof(BucketExists_Test), bucketExistsSignature,
+                "Tests whether BucketExistsAsync api passes",
                 TestStatus.PASS, DateTime.Now - startTime, args: args).Log();
         }
         catch (NotImplementedException ex)
         {
-            new MintLogger(nameof(BucketExists_Test), bucketExistsSignature, "Tests whether BucketExists passes",
+            new MintLogger(nameof(BucketExists_Test), bucketExistsSignature,
+                "Tests whether BucketExistsAsync api passes",
                 TestStatus.NA, DateTime.Now - startTime, ex.Message, ex.ToString(), args: args).Log();
         }
         catch (Exception ex)
         {
-            new MintLogger(nameof(BucketExists_Test), bucketExistsSignature, "Tests whether BucketExists passes",
+            new MintLogger(nameof(BucketExists_Test), bucketExistsSignature,
+                "Tests whether BucketExistsAsync api passes",
                 TestStatus.FAIL, DateTime.Now - startTime, ex.Message, ex.ToString(), args: args).Log();
             throw;
         }
@@ -586,17 +594,17 @@ internal static async Task TearDown(IMinioClient minio, string bucketName)
     {
         var beArgs = new BucketExistsArgs()
             .WithBucket(bucketName);
-        var bktExists = await minio.BucketExistsAsync(beArgs).ConfigureAwait(false);
-        if (!bktExists)
-            return;
         var getVersions = false;
-        // Get Versioning/Retention Info.
-        var lockConfigurationArgs =
-            new GetObjectLockConfigurationArgs()
-                .WithBucket(bucketName);
-        ObjectLockConfiguration lockConfig = null;
+        var lockConfig = new ObjectLockConfiguration();
         try
         {
+            var bktExists = await minio.BucketExistsAsync(beArgs).ConfigureAwait(false);
+            if (!bktExists)
+                return;
+            // Get Versioning/Retention Info.
+            var lockConfigurationArgs =
+                new GetObjectLockConfigurationArgs()
+                    .WithBucket(bucketName);
             var versioningConfig = await minio.GetVersioningAsync(new GetVersioningArgs()
                 .WithBucket(bucketName)
                 .WithVersions(true)).ConfigureAwait(false);

Minio/RequestExtensions.cs

@@ -144,11 +144,9 @@ await requestMessageBuilder.ResponseWriter(responseResult.ContentStream, cancell
             if (ex.Message.Equals("ThrowBucketNotFoundException", StringComparison.Ordinal))
                 throw new BucketNotFoundException();
 
-            if (responseResult is not null)
-                responseResult.Exception = ex;
-            else
-                responseResult = new ResponseResult(request, ex);
-            return responseResult;
+            if (responseResult is not null) responseResult.Exception = ex;
+            else return new ResponseResult(request, ex);
+            throw;
         }
     }