Release v0.14.4 (#1546)

Signed-off-by: Daniel Valdivia <18384552+dvaldivia@users.noreply.github.com>
2022-02-10 11:25:31 -08:00
parent 32a3094386
commit 822f063be3
313 changed files with 664 additions and 639 deletions
--- a/portal-ui/tests/permissions/bucketSpecific.ts
+++ b/portal-ui/tests/permissions/bucketSpecific.ts
@@ -18,13 +18,18 @@ import * as roles from "../utils/roles";
 import * as elements from "../utils/elements";
 import * as functions from "../utils/functions";
 import { bucketsElement, logoutItem } from "../utils/elements-menu";
-import { namedTestBucketBrowseButtonFor, namedManageButtonFor} from "../utils/functions";
+import {
+  namedTestBucketBrowseButtonFor,
+  namedManageButtonFor,
+} from "../utils/functions";
 import { Selector } from "testcafe";
 import * as constants from "../utils/constants";

 const TEST_BUCKET_NAME_SPECIFIC = "specific-bucket";

-fixture("For user with permissions that only allow specific Buckets").page("http://localhost:9090");
+fixture("For user with permissions that only allow specific Buckets").page(
+  "http://localhost:9090"
+);

 test("Buckets sidebar item exists", async (t) => {
  const bucketsExist = bucketsElement.with({ boundTestRun: t }).exists;
@@ -113,7 +118,9 @@ test
    await t
      .useRole(roles.bucketRead)
      .navigateTo("http://localhost:9090/buckets")
-      .expect(namedTestBucketBrowseButtonFor(`${TEST_BUCKET_NAME_SPECIFIC}-4`).exists)
+      .expect(
+        namedTestBucketBrowseButtonFor(`${TEST_BUCKET_NAME_SPECIFIC}-4`).exists
+      )
      .ok();
  })
  .after(async (t) => {
@@ -166,7 +173,10 @@ test
  })
  .after(async (t) => {
    // Cleanup created bucket and corresponding uploads
-    await functions.cleanUpNamedBucketAndUploads(t, `${TEST_BUCKET_NAME_SPECIFIC}-6`);
+    await functions.cleanUpNamedBucketAndUploads(
+      t,
+      `${TEST_BUCKET_NAME_SPECIFIC}-6`
+    );
  });

 // Bucket write tests
@@ -176,7 +186,9 @@ test
    // Create a bucket
    await functions.setUpNamedBucket(t, `${TEST_BUCKET_NAME_SPECIFIC}-7`);
  })("Browse button exists", async (t) => {
-    const testBucketBrowseButton = namedTestBucketBrowseButtonFor(`${TEST_BUCKET_NAME_SPECIFIC}-7`);
+    const testBucketBrowseButton = namedTestBucketBrowseButtonFor(
+      `${TEST_BUCKET_NAME_SPECIFIC}-7`
+    );
    await t
      .useRole(roles.bucketSpecific)
      .navigateTo("http://localhost:9090/buckets")
@@ -185,7 +197,10 @@ test
  })
  .after(async (t) => {
    // Cleanup created bucket and corresponding uploads
-    await functions.cleanUpNamedBucketAndUploads(t, `${TEST_BUCKET_NAME_SPECIFIC}-7`);
+    await functions.cleanUpNamedBucketAndUploads(
+      t,
+      `${TEST_BUCKET_NAME_SPECIFIC}-7`
+    );
  });

 test
@@ -208,7 +223,10 @@ test
  })
  .after(async (t) => {
    // Cleanup created bucket and corresponding uploads
-    await functions.cleanUpNamedBucketAndUploads(t, `${TEST_BUCKET_NAME_SPECIFIC}-8`);
+    await functions.cleanUpNamedBucketAndUploads(
+      t,
+      `${TEST_BUCKET_NAME_SPECIFIC}-8`
+    );
  });

 test
@@ -217,7 +235,9 @@ test
    await functions.setUpNamedBucket(t, `${TEST_BUCKET_NAME_SPECIFIC}-9`);
  })("Upload button exists", async (t) => {
    const uploadExists = elements.uploadButton.exists;
-    const testBucketBrowseButton = namedTestBucketBrowseButtonFor(`${TEST_BUCKET_NAME_SPECIFIC}-9`);
+    const testBucketBrowseButton = namedTestBucketBrowseButtonFor(
+      `${TEST_BUCKET_NAME_SPECIFIC}-9`
+    );
    await t
      .useRole(roles.bucketSpecific)
      .navigateTo("http://localhost:9090/buckets")
@@ -227,7 +247,10 @@ test
  })
  .after(async (t) => {
    // Cleanup created bucket and corresponding uploads
-    await functions.cleanUpNamedBucketAndUploads(t, `${TEST_BUCKET_NAME_SPECIFIC}-9`);
+    await functions.cleanUpNamedBucketAndUploads(
+      t,
+      `${TEST_BUCKET_NAME_SPECIFIC}-9`
+    );
  });

 test
@@ -235,7 +258,9 @@ test
    // Create a bucket
    await functions.setUpNamedBucket(t, `${TEST_BUCKET_NAME_SPECIFIC}-10`);
  })("Object can be uploaded to a bucket", async (t) => {
-    const testBucketBrowseButton = namedTestBucketBrowseButtonFor(`${TEST_BUCKET_NAME_SPECIFIC}-10`);
+    const testBucketBrowseButton = namedTestBucketBrowseButtonFor(
+      `${TEST_BUCKET_NAME_SPECIFIC}-10`
+    );
    await t
      .useRole(roles.bucketSpecific)
      .navigateTo("http://localhost:9090/buckets")
@@ -245,7 +270,10 @@ test
  })
  .after(async (t) => {
    // Cleanup created bucket and corresponding uploads
-    await functions.cleanUpNamedBucketAndUploads(t, `${TEST_BUCKET_NAME_SPECIFIC}-10`);
+    await functions.cleanUpNamedBucketAndUploads(
+      t,
+      `${TEST_BUCKET_NAME_SPECIFIC}-10`
+    );
  });

 test
@@ -262,5 +290,8 @@ test
  })
  .after(async (t) => {
    // Cleanup created bucket and corresponding uploads
-    await functions.cleanUpNamedBucketAndUploads(t, `${TEST_BUCKET_NAME_SPECIFIC}-11`);
+    await functions.cleanUpNamedBucketAndUploads(
+      t,
+      `${TEST_BUCKET_NAME_SPECIFIC}-11`
+    );
  });
--- a/portal-ui/tests/policies/bucketSpecific.json
+++ b/portal-ui/tests/policies/bucketSpecific.json
@@ -1,43 +1,39 @@
 {
  "Version": "2012-10-17",
  "Statement": [
-      {
-          "Effect": "Allow",
-          "Action": [
-              "s3:*"
-          ],
-          "Resource": [
-              "arn:aws:s3:::specific-bucket-1/*",
-              "arn:aws:s3:::specific-bucket-2/*",
-              "arn:aws:s3:::specific-bucket-3/*",
-              "arn:aws:s3:::specific-bucket-4/*",
-              "arn:aws:s3:::specific-bucket-5/*",
-              "arn:aws:s3:::specific-bucket-6/*",
-              "arn:aws:s3:::specific-bucket-7/*",
-              "arn:aws:s3:::specific-bucket-8/*",
-              "arn:aws:s3:::specific-bucket-9/*",
-              "arn:aws:s3:::specific-bucket-10/*",
-              "arn:aws:s3:::specific-bucket-11/*"
-          ]
-      },
-      {
-          "Effect": "Allow",
-          "Action": [
-              "s3:CreateBucket"
-          ],
-          "Resource": [
-              "arn:aws:s3:::specific-bucket-1",
-              "arn:aws:s3:::specific-bucket-2",
-              "arn:aws:s3:::specific-bucket-3",
-              "arn:aws:s3:::specific-bucket-4",
-              "arn:aws:s3:::specific-bucket-5",
-              "arn:aws:s3:::specific-bucket-6",
-              "arn:aws:s3:::specific-bucket-7",
-              "arn:aws:s3:::specific-bucket-8",
-              "arn:aws:s3:::specific-bucket-9",
-              "arn:aws:s3:::specific-bucket-10",
-              "arn:aws:s3:::specific-bucket-11"
-          ]
-      }
+    {
+      "Effect": "Allow",
+      "Action": ["s3:*"],
+      "Resource": [
+        "arn:aws:s3:::specific-bucket-1/*",
+        "arn:aws:s3:::specific-bucket-2/*",
+        "arn:aws:s3:::specific-bucket-3/*",
+        "arn:aws:s3:::specific-bucket-4/*",
+        "arn:aws:s3:::specific-bucket-5/*",
+        "arn:aws:s3:::specific-bucket-6/*",
+        "arn:aws:s3:::specific-bucket-7/*",
+        "arn:aws:s3:::specific-bucket-8/*",
+        "arn:aws:s3:::specific-bucket-9/*",
+        "arn:aws:s3:::specific-bucket-10/*",
+        "arn:aws:s3:::specific-bucket-11/*"
+      ]
+    },
+    {
+      "Effect": "Allow",
+      "Action": ["s3:CreateBucket"],
+      "Resource": [
+        "arn:aws:s3:::specific-bucket-1",
+        "arn:aws:s3:::specific-bucket-2",
+        "arn:aws:s3:::specific-bucket-3",
+        "arn:aws:s3:::specific-bucket-4",
+        "arn:aws:s3:::specific-bucket-5",
+        "arn:aws:s3:::specific-bucket-6",
+        "arn:aws:s3:::specific-bucket-7",
+        "arn:aws:s3:::specific-bucket-8",
+        "arn:aws:s3:::specific-bucket-9",
+        "arn:aws:s3:::specific-bucket-10",
+        "arn:aws:s3:::specific-bucket-11"
+      ]
+    }
  ]
-}
+}
--- a/portal-ui/tests/utils/functions.ts
+++ b/portal-ui/tests/utils/functions.ts
@@ -36,10 +36,7 @@ export const setUpNamedBucket = (t, name) => {
  });

  return new Promise((resolve, reject) => {
-    minioClient
-      .makeBucket(name, "us-east-1")
-      .then(resolve)
-      .catch(resolve);
+    minioClient.makeBucket(name, "us-east-1").then(resolve).catch(resolve);
  });
 };

@@ -49,7 +46,7 @@ export const namedManageButtonFor = (name) => {
    .parent(4)
    .find("button:enabled")
    .withText("Manage");
-}
+};

 export const manageButtonFor = (modifier) => {
  return namedManageButtonFor(`${constants.TEST_BUCKET_NAME}-${modifier}`);
@@ -80,7 +77,9 @@ export const namedTestBucketBrowseButtonFor = (name) => {
 };

 export const testBucketBrowseButtonFor = (modifier) => {
-  return namedTestBucketBrowseButtonFor(`${constants.TEST_BUCKET_NAME}-${modifier}`);
+  return namedTestBucketBrowseButtonFor(
+    `${constants.TEST_BUCKET_NAME}-${modifier}`
+  );
 };

 export const uploadFilesButton = () => {
@@ -114,7 +113,7 @@ export const cleanUpNamedBucketAndUploads = (t, bucket) => {

 export const cleanUpBucketAndUploads = (t, modifier) => {
  const bucket = `${constants.TEST_BUCKET_NAME}-${modifier}`;
-  return cleanUpNamedBucketAndUploads(t, bucket)
+  return cleanUpNamedBucketAndUploads(t, bucket);
 };

 export const createUser = (t) => {