Add counter tests

Author: khiga8

README.md

@@ -55,17 +55,12 @@ linters:
 
 - [GitHub::Accessibility::AvoidBothDisabledAndAriaDisabledCounter](./docs/rules/accessibility/avoid-both-disabled-and-aria-disabled-counter.md)
 - [GitHub::Accessibility::AvoidGenericLinkTextCounter](./docs/rules/accessibility/avoid-generic-link-text-counter.md)
-<<<<<<< HEAD
 - [GitHub::Accessibility::DisabledAttributeCounter](./docs/rules/accessibility/disabled-attribute-counter.md)
-- [GitHub::Accessibility::IframeHasTitle](./docs/rules/accessibility/iframe-has-title.md)
 - [GitHub::Accessibility::LandmarkHasLabelCounter](./docs/rules/accessibility/landmark-has-label-counter.md)
-- [GitHub::Accessibility::ImageHasAlt](./docs/rules/accessibility/image-has-alt.md)
 - [GitHub::Accessibility::LinkHasHrefCounter](./docs/rules/accessibility/link-has-href-counter.md)
 - [GitHub::Accessibility::NestedInteractiveElementsCounter](./docs/rules/accessibility/nested-interactive-elements-counter.md)
-=======
 - [GitHub::Accessibility::IframeHasTitleCounter](./docs/rules/accessibility/iframe-has-title-counter.md)
 - [GitHub::Accessibility::ImageHasAltCounter](./docs/rules/accessibility/image-has-alt-counter.md)
->>>>>>> 3ac20b8 (Migrate remaining rules to counters)
 - [GitHub::Accessibility::NoAriaLabelMisuseCounter](./docs/rules/accessibility/no-aria-label-misuse-counter.md)
 - [GitHub::Accessibility::NoPositiveTabIndexCounter](./docs/rules/accessibility/no-positive-tab-index-counter.md)
 - [GitHub::Accessibility::NoRedundantImageAltCounter](./docs/rules/accessibility/no-redundant-image-alt-counter.md)

test/linters/accessibility/avoid_both_disabled_and_aria_disabled_counter_test.rb

@@ -35,4 +35,36 @@ def test_does_not_warn_if_only_aria_disabled_attribute_is_set
 
     assert_empty @linter.offenses
   end
+
+  def test_does_not_raise_when_ignore_comment_with_correct_count
+    @file = <<~ERB
+      <%# erblint:counter GitHub::Accessibility::AvoidBothDisabledAndAriaDisabledCounter 1 %>
+      <button disabled aria-disabled="true">Some text</span>
+    ERB
+
+    @linter.run(processed_source)
+    assert_empty @linter.offenses
+  end
+
+  def test_does_not_autocorrect_when_ignores_are_correct
+    @file = <<~ERB
+      <%# erblint:counter GitHub::Accessibility::AvoidBothDisabledAndAriaDisabledCounter 1 %>
+      <button disabled aria-disabled="true">Some text</button>
+    ERB
+
+    assert_equal @file, corrected_content
+  end
+
+  def test_does_autocorrect_when_ignores_are_not_correct
+    @file = <<~ERB
+      <button disabled aria-disabled="true">Some text</button>
+    ERB
+    refute_equal @file, corrected_content
+
+    expected_content = <<~ERB
+      <%# erblint:counter GitHub::Accessibility::AvoidBothDisabledAndAriaDisabledCounter 1 %>
+      <button disabled aria-disabled="true">Some text</button>
+    ERB
+    assert_equal expected_content, corrected_content
+  end
 end

test/linters/accessibility/iframe_has_title_counter_test.rb

@@ -27,4 +27,36 @@ def test_does_not_warn_if_iframe_has_title_set_to_string
 
     assert_empty @linter.offenses
   end
+
+  def test_does_not_raise_when_ignore_comment_with_correct_count
+    @file = <<~ERB
+      <%# erblint:counter GitHub::Accessibility::IframeHasTitleCounter 1 %>
+      <iframe></iframe>
+    ERB
+
+    @linter.run(processed_source)
+    assert_empty @linter.offenses
+  end
+
+  def test_does_not_autocorrect_when_ignores_are_correct
+    @file = <<~ERB
+      <%# erblint:counter GitHub::Accessibility::IframeHasTitleCounter 1 %>
+      <iframe></iframe>
+    ERB
+
+    assert_equal @file, corrected_content
+  end
+
+  def test_does_autocorrect_when_ignores_are_not_correct
+    @file = <<~ERB
+      <iframe></iframe>
+    ERB
+    refute_equal @file, corrected_content
+
+    expected_content = <<~ERB
+      <%# erblint:counter GitHub::Accessibility::IframeHasTitleCounter 1 %>
+      <iframe></iframe>
+    ERB
+    assert_equal expected_content, corrected_content
+  end
 end

test/linters/accessibility/image_has_alt_counter_test.rb

@@ -27,4 +27,36 @@ def test_does_not_warn_if_image_has_alt_attribute_set_to_string
 
     assert_empty @linter.offenses
   end
+
+  def test_does_not_raise_when_ignore_comment_with_correct_count
+    @file = <<~ERB
+      <%# erblint:counter GitHub::Accessibility::ImageHasAltCounter 1 %>
+      <img></img>
+    ERB
+
+    @linter.run(processed_source)
+    assert_empty @linter.offenses
+  end
+
+  def test_does_not_autocorrect_when_ignores_are_correct
+    @file = <<~ERB
+      <%# erblint:counter GitHub::Accessibility::ImageHasAltCounter 1 %>
+      <img></img>
+    ERB
+
+    assert_equal @file, corrected_content
+  end
+
+  def test_does_autocorrect_when_ignores_are_not_correct
+    @file = <<~ERB
+      <img></img>
+    ERB
+    refute_equal @file, corrected_content
+
+    expected_content = <<~ERB
+      <%# erblint:counter GitHub::Accessibility::ImageHasAltCounter 1 %>
+      <img></img>
+    ERB
+    assert_equal expected_content, corrected_content
+  end
 end

test/linters/accessibility/no_positive_tab_index_counter_test.rb

@@ -27,4 +27,36 @@ def test_does_not_warn_if_zero_tabindex_is_used
 
     assert_empty @linter.offenses
   end
+
+  def test_does_not_raise_when_ignore_comment_with_correct_count
+    @file = <<~ERB
+      <%# erblint:counter GitHub::Accessibility::NoPositiveTabIndexCounter 1 %>
+      <button tabindex='1'></button>
+    ERB
+
+    @linter.run(processed_source)
+    assert_empty @linter.offenses
+  end
+
+  def test_does_not_autocorrect_when_ignores_are_correct
+    @file = <<~ERB
+      <%# erblint:counter GitHub::Accessibility::NoPositiveTabIndexCounter 1 %>
+      <button tabindex='1'></button>
+    ERB
+
+    assert_equal @file, corrected_content
+  end
+
+  def test_does_autocorrect_when_ignores_are_not_correct
+    @file = <<~ERB
+      <button tabindex='1'></button>
+    ERB
+    refute_equal @file, corrected_content
+
+    expected_content = <<~ERB
+      <%# erblint:counter GitHub::Accessibility::NoPositiveTabIndexCounter 1 %>
+      <button tabindex='1'></button>
+    ERB
+    assert_equal expected_content, corrected_content
+  end
 end

test/linters/accessibility/no_redundant_image_alt_counter_test.rb

@@ -28,7 +28,7 @@ def test_does_not_warn_if_alt_contains_no_redundant_text
     assert_empty @linter.offenses
   end
 
-  def test_does_not_warn_if_linter_is_disabled_in_file
+  def test_does_not_raise_when_ignore_comment_with_correct_count
     @file = <<~ERB
       <%# erblint:counter GitHub::Accessibility::NoRedundantImageAltCounter 1 %>
       <img alt='image of an octopus'></img>
@@ -37,4 +37,26 @@ def test_does_not_warn_if_linter_is_disabled_in_file
     @linter.run(processed_source)
     assert_empty @linter.offenses
   end
+
+  def test_does_not_autocorrect_when_ignores_are_correct
+    @file = <<~ERB
+      <%# erblint:counter GitHub::Accessibility::NoRedundantImageAltCounter 1 %>
+      <img alt='image of an octopus'></img>
+    ERB
+
+    assert_equal @file, corrected_content
+  end
+
+  def test_does_autocorrect_when_ignores_are_not_correct
+    @file = <<~ERB
+      <img alt='image of an octopus'></img>
+    ERB
+    refute_equal @file, corrected_content
+
+    expected_content = <<~ERB
+      <%# erblint:counter GitHub::Accessibility::NoRedundantImageAltCounter 1 %>
+      <img alt='image of an octopus'></img>
+    ERB
+    assert_equal expected_content, corrected_content
+  end
 end