feat: allow assets bucket name to be provided via config

lib/__tests__/addAssetsBucketForDeployment.test.js

@@ -72,6 +72,26 @@ describe("addAssetsBucketForDeployment", () => {
     });
   });
 
+  it("should log when a bucket is going to be provisioned from plugin config", () => {
+    expect.assertions(1);
+
+    parseNextConfiguration.mockReturnValueOnce(
+      parsedNextConfigurationFactory()
+    );
+
+    plugin = new ServerlessPluginBuilder()
+      .withNextCustomConfig({
+        assetsBucketName: "my-assets"
+      })
+      .build();
+
+    return addAssetsBucketForDeployment.call(plugin).then(() => {
+      expect(logger.log).toBeCalledWith(
+        expect.stringContaining(`Found bucket "my-assets"`)
+      );
+    });
+  });
+
   it("should update coreCloudFormationTemplate with static assets bucket", () => {
     expect.assertions(2);
 

lib/addAssetsBucketForDeployment.js

@@ -16,15 +16,23 @@ const getCFTemplatesWithBucket = async function(bucketName) {
 };
 
 module.exports = async function() {
-  const { staticAssetsBucket } = parseNextConfiguration(
+  let { staticAssetsBucket } = parseNextConfiguration(
     this.getPluginConfigValue("nextConfigDir")
   );
 
+  const bucketNameFromConfig = this.getPluginConfigValue("assetsBucketName");
+
+  if (bucketNameFromConfig) {
+    // bucket name provided via user config takes precendence
+    // over parsed value from assetPrefix
+    staticAssetsBucket = bucketNameFromConfig;
+  }
+
   if (!staticAssetsBucket) {
     return;
   }
 
-  logger.log(`Found bucket "${staticAssetsBucket}" in assetPrefix!`);
+  logger.log(`Found bucket "${staticAssetsBucket}"`);
 
   const [
     compiledCfWithBucket,