Remove index param from attach new file helper test function

This commit is contained in:
Bertocq
2018-01-03 15:30:42 +01:00
parent fb1f45ba6b
commit 5bf16eb1ac

View File

@@ -85,7 +85,7 @@ shared_examples "nested documentable" do |login_as_name, documentable_factory_na
login_as user_to_login login_as user_to_login
visit send(path, arguments) visit send(path, arguments)
documentable_attach_new_file(0, "spec/fixtures/files/empty.pdf") documentable_attach_new_file("spec/fixtures/files/empty.pdf")
expect_document_has_title(0, "empty.pdf") expect_document_has_title(0, "empty.pdf")
end end
@@ -109,7 +109,7 @@ shared_examples "nested documentable" do |login_as_name, documentable_factory_na
login_as user_to_login login_as user_to_login
visit send(path, arguments) visit send(path, arguments)
documentable_attach_new_file(0, "spec/fixtures/files/empty.pdf") documentable_attach_new_file("spec/fixtures/files/empty.pdf")
expect(page).to have_css ".loading-bar.complete" expect(page).to have_css ".loading-bar.complete"
end end
@@ -118,7 +118,7 @@ shared_examples "nested documentable" do |login_as_name, documentable_factory_na
login_as user_to_login login_as user_to_login
visit send(path, arguments) visit send(path, arguments)
documentable_attach_new_file(0, "spec/fixtures/files/logo_header.png", false) documentable_attach_new_file("spec/fixtures/files/logo_header.png", false)
expect(page).to have_css ".loading-bar.errors" expect(page).to have_css ".loading-bar.errors"
end end
@@ -127,7 +127,7 @@ shared_examples "nested documentable" do |login_as_name, documentable_factory_na
login_as user_to_login login_as user_to_login
visit send(path, arguments) visit send(path, arguments)
documentable_attach_new_file(0, "spec/fixtures/files/empty.pdf") documentable_attach_new_file("spec/fixtures/files/empty.pdf")
expect_document_has_cached_attachment(0, ".pdf") expect_document_has_cached_attachment(0, ".pdf")
end end
@@ -136,7 +136,7 @@ shared_examples "nested documentable" do |login_as_name, documentable_factory_na
login_as user_to_login login_as user_to_login
visit send(path, arguments) visit send(path, arguments)
documentable_attach_new_file(0, "spec/fixtures/files/logo_header.png", false) documentable_attach_new_file("spec/fixtures/files/logo_header.png", false)
expect_document_has_cached_attachment(0, "") expect_document_has_cached_attachment(0, "")
end end
@@ -157,7 +157,7 @@ shared_examples "nested documentable" do |login_as_name, documentable_factory_na
login_as user_to_login login_as user_to_login
visit send(path, arguments) visit send(path, arguments)
documentable_attach_new_file(0, "spec/fixtures/files/empty.pdf") documentable_attach_new_file("spec/fixtures/files/empty.pdf")
click_link "Remove document" click_link "Remove document"
expect(page).not_to have_css("#nested-documents .document") expect(page).not_to have_css("#nested-documents .document")
@@ -178,7 +178,7 @@ shared_examples "nested documentable" do |login_as_name, documentable_factory_na
visit send(path, arguments) visit send(path, arguments)
send(fill_resource_method_name) if fill_resource_method_name send(fill_resource_method_name) if fill_resource_method_name
documentable_attach_new_file(0, "spec/fixtures/files/empty.pdf") documentable_attach_new_file("spec/fixtures/files/empty.pdf")
click_on submit_button click_on submit_button
expect(page).to have_content documentable_success_notice expect(page).to have_content documentable_success_notice
@@ -189,7 +189,7 @@ shared_examples "nested documentable" do |login_as_name, documentable_factory_na
visit send(path, arguments) visit send(path, arguments)
send(fill_resource_method_name) if fill_resource_method_name send(fill_resource_method_name) if fill_resource_method_name
documentable_attach_new_file(0, "spec/fixtures/files/empty.pdf") documentable_attach_new_file("spec/fixtures/files/empty.pdf")
click_on submit_button click_on submit_button
documentable_redirected_to_resource_show_or_navigate_to documentable_redirected_to_resource_show_or_navigate_to
@@ -282,9 +282,9 @@ rescue
return return
end end
def documentable_attach_new_file(index, path, success = true) def documentable_attach_new_file(path, success = true)
click_link "Add new document" click_link "Add new document"
document = all(".document")[index] document = all("#new_document").last
document_input = document.find("input[type=file]", visible: false) document_input = document.find("input[type=file]", visible: false)
attach_file(document_input[:id], path, make_visible: true) attach_file(document_input[:id], path, make_visible: true)
within document do within document do