diff options
author | Lin Jen-Shin <godfat@godfat.org> | 2016-09-19 08:12:45 +0300 |
---|---|---|
committer | Lin Jen-Shin <godfat@godfat.org> | 2016-09-19 08:12:45 +0300 |
commit | 31e592647aaab8923922a316e442c404ef70b8df (patch) | |
tree | 9d9d96486e47ce54f5ebb915f9b0bc15d0d596d0 /spec/features/projects/branches/download_buttons_spec.rb | |
parent | 1d51bc7dfd04886fa5af69a60bb509691d697813 (diff) |
Fix download artifacts button link:
Rails ignored unused arguments and ref_name_and_path is the last
path argument, therefore we need to concatenate that parts ourselves.
Otherwise, 'download' won't be shown at all.
Diffstat (limited to 'spec/features/projects/branches/download_buttons_spec.rb')
-rw-r--r-- | spec/features/projects/branches/download_buttons_spec.rb | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/spec/features/projects/branches/download_buttons_spec.rb b/spec/features/projects/branches/download_buttons_spec.rb index 04058300570..92028c19361 100644 --- a/spec/features/projects/branches/download_buttons_spec.rb +++ b/spec/features/projects/branches/download_buttons_spec.rb @@ -33,7 +33,11 @@ feature 'Download buttons in branches page', feature: true do end scenario 'shows download artifacts button' do - expect(page).to have_link "Download '#{build.name}'" + href = latest_succeeded_namespace_project_artifacts_path( + project.namespace, project, 'binary-encoding/download', + job: 'build') + + expect(page).to have_link "Download '#{build.name}'", href: href end end end |