feat(storage): use aws-sdk-s3 gem instead of aws-sdk

Closes #190
platanus · Aug 3, 2019 · c55c536 · c55c536
1 parent f0ea208
commit c55c536
Show file tree

Hide file tree

Showing 5 changed files with 11 additions and 11 deletions.
diff --git a/lib/potassium/recipes/aws_sdk.rb b/lib/potassium/recipes/aws_sdk.rb
diff --git a/lib/potassium/recipes/file_storage.rb b/lib/potassium/recipes/file_storage.rb
@@ -63,6 +63,7 @@ def add_active_storage
   end
 
   def common_setup
+    gather_gem 'aws-sdk-s3', '~> 1.0'
     add_readme_section :internal_dependencies, get(:storage)
     append_to_file '.env.development', "S3_BUCKET=\n"
   end

diff --git a/lib/potassium/templates/application.rb b/lib/potassium/templates/application.rb
@@ -49,7 +49,6 @@
   create :ruby
   create :yarn
   create :editorconfig
-  create :aws_sdk
   create :mailer
   create :background_processor
   create :schedule

diff --git a/spec/features/file_storage_spec.rb b/spec/features/file_storage_spec.rb
@@ -8,6 +8,11 @@
       create_dummy_project(storage: :active_storage)
     end
 
+    it "adds the aws-sdk-s3 gem to Gemfile" do
+      gemfile_content = IO.read("#{project_path}/Gemfile")
+      expect(gemfile_content).to include("gem 'aws-sdk-s3'")
+    end
+
     it "customizes config file" do
       content = IO.read("#{project_path}/config/storage.yml")
       expect(content).to include("bucket: <%= ENV['S3_BUCKET'] %>")
@@ -41,6 +46,11 @@
       expect(gemfile_content).to include("gem 'paperclip'")
     end
 
+    it "adds the aws-sdk-s3 gem to Gemfile" do
+      gemfile_content = IO.read("#{project_path}/Gemfile")
+      expect(gemfile_content).to include("gem 'aws-sdk-s3'")
+    end
+
     it "adds brief to README file" do
       content = IO.read("#{project_path}/README.md")
       expect(content).to include("Paperclip")

diff --git a/spec/features/new_project_spec.rb b/spec/features/new_project_spec.rb
@@ -26,11 +26,6 @@
     expect(gemfile).to include %{gem 'pg'}
   end
 
-  it "configures aws" do
-    gemfile_content = IO.read("#{project_path}/Gemfile")
-    expect(gemfile_content).to include("'aws-sdk', '~> 3'")
-  end
-
   it "configures the correct ruby version" do
     ruby_version_file = IO.read("#{project_path}/.ruby-version")