Include ActionView::Context in the spec

The include shouldn't have ever been in the helper, Rails will already have it
x-govuk · Jul 7, 2021 · 91f8ffc · 91f8ffc
1 parent 6729080
commit 91f8ffc
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/app/helpers/govuk_skip_link_helper.rb b/app/helpers/govuk_skip_link_helper.rb
@@ -1,7 +1,4 @@
 module GovukSkipLinkHelper
-  include ActionView::Helpers::UrlHelper
-  include ActionView::Context
-
   def govuk_skip_link(text: 'Skip to main content', href: '#main-content', classes: [], **html_attributes, &block)
     link_classes = Array.wrap(classes).append('govuk-skip-link')
 

diff --git a/spec/helpers/govuk_skip_link_helper_spec.rb b/spec/helpers/govuk_skip_link_helper_spec.rb
@@ -1,6 +1,7 @@
 require 'spec_helper'
 
 RSpec.describe(GovukSkipLinkHelper, type: 'helper') do
+  include ActionView::Context
   include ActionView::Helpers::UrlHelper
 
   let(:custom_link_text) { 'Get straight to the important stuff' }