Simpler test for I18n namespace conflict

ddnexus · Apr 8, 2021 · 3eef450 · 3eef450
1 parent b2aaa01
commit 3eef450
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 13 deletions.
diff --git a/test/mock_helpers/application_helper.rb b/test/mock_helpers/application_helper.rb
diff --git a/test/mock_helpers/view.rb b/test/mock_helpers/view.rb
@@ -8,6 +8,10 @@ def initialize(url='http://example.com:3000/foo?page=2')
     @url = url
   end
 
+  def test_i18n_call
+    I18n.t('test')
+  end
+
   def request
     Rack::Request.new(Rack::MockRequest.env_for(@url))
   end

diff --git a/test/pagy/extras/i18n_test.rb b/test/pagy/extras/i18n_test.rb
@@ -7,16 +7,14 @@
 
 SimpleCov.command_name 'i18n' if ENV['RUN_SIMPLECOV'] == 'true'
 
-require_relative '../../mock_helpers/application_helper'
-include ApplicationHelper
-
 describe Pagy::Frontend do
 
   let(:view) { MockView.new }
 
   describe "#pagy_t with I18n" do
-    it 'works with an included Module' do
-      ApplicationHelper.any_method_name()
+
+    it 'does not conflict with the I18n gem namespace' do
+      view.test_i18n_call
     end
 
     it 'pluralizes' do