From ff5124ed6ecb2f20841e535b3aaa3b0d230f920f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Matija=20=C4=8Cupi=C4=87?= <matteeyah@gmail.com>
Date: Sat, 6 Jan 2018 04:11:41 +0100
Subject: [PATCH] Use anchor instead of form for reset caches button

---
 .../javascripts/pipelines/components/nav_controls.vue  | 10 +++++++---
 spec/javascripts/pipelines/nav_controls_spec.js        |  2 +-
 2 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/app/assets/javascripts/pipelines/components/nav_controls.vue b/app/assets/javascripts/pipelines/components/nav_controls.vue
index f2d5f76c0ce..f31a91c3403 100644
--- a/app/assets/javascripts/pipelines/components/nav_controls.vue
+++ b/app/assets/javascripts/pipelines/components/nav_controls.vue
@@ -50,9 +50,13 @@ export default {
       Get started with Pipelines
     </a>
 
-    <form :action="resetCachePath" method="post">
-      <input type="submit" class="btn btn-default" value="Clear runner caches" />
-    </form>
+    <a
+      data-method="post"
+      rel="nofollow"
+      :href="resetCachePath"
+      class="btn btn-default">
+      Clear runner caches
+    </a>
 
     <a
       :href="ciLintPath"
diff --git a/spec/javascripts/pipelines/nav_controls_spec.js b/spec/javascripts/pipelines/nav_controls_spec.js
index cf9bbee2b13..09a0c14d96c 100644
--- a/spec/javascripts/pipelines/nav_controls_spec.js
+++ b/spec/javascripts/pipelines/nav_controls_spec.js
@@ -58,7 +58,7 @@ describe('Pipelines Nav Controls', () => {
     }).$mount();
 
     expect(component.$el.querySelectorAll('.btn-default')[0].textContent).toContain('Clear runner caches');
-    expect(component.$el.querySelector('form').getAttribute('action')).toEqual(mockData.resetCachePath);
+    expect(component.$el.querySelectorAll('.btn-default')[0].getAttribute('href')).toEqual(mockData.resetCachePath);
   });
 
   it('should render link for CI lint', () => {