Merge pull request #11867 from mkllnk/tos-spec

DRY Terms of Service spec to save time

spec/system/admin/configuration/terms_of_service_files_spec.rb

@@ -10,33 +10,16 @@ describe "Terms of Service files" do
 
     before { login_as_admin }
 
-    it "can be reached via Configuration" do
+    it "can be uploaded and deleted" do
       visit spree.edit_admin_general_settings_path
       click_link "Terms of Service"
 
       expect(page).to have_content "No terms of services have been uploaded yet."
-    end
 
-    it "can be uploaded" do
-      visit admin_terms_of_service_files_path
       attach_file "Attachment", Rails.root.join(test_file_path)
       click_button "Create Terms of service file"
 
       expect(page).to have_link "Terms of Service"
-      expect(page).to have_link "Delete file"
-    end
-
-    it "provides Rails' standard action for a new file" do
-      visit new_admin_terms_of_service_files_path
-      expect(page).to have_button "Create Terms of service file"
-    end
-
-    it "can delete the current file" do
-      attachment = File.open(Rails.root.join(test_file_path))
-      file = Rack::Test::UploadedFile.new(attachment, "application/pdf")
-      TermsOfServiceFile.create!(attachment: file)
-
-      visit admin_terms_of_service_files_path
 
       accept_alert { click_link "Delete" }