diff --git a/plugins/InspectionGadgets/test/com/siyeh/igtest/bugs/assert_with_side_effects/AssertWithSideEffects.java b/plugins/InspectionGadgets/test/com/siyeh/igtest/bugs/assert_with_side_effects/AssertWithSideEffects.java index 4b3e71f94833..57fba33b43df 100644 --- a/plugins/InspectionGadgets/test/com/siyeh/igtest/bugs/assert_with_side_effects/AssertWithSideEffects.java +++ b/plugins/InspectionGadgets/test/com/siyeh/igtest/bugs/assert_with_side_effects/AssertWithSideEffects.java @@ -30,12 +30,4 @@ public class AssertWithSideEffects { void jdbc(ResultSet rs) throws SQLException { assert rs.last(); } - - void assertMutation(Set set) { - assert set.add("foo"); - - assert new HashSet<>().add("bar"); - - assert (set.isEmpty() ? new TreeSet<>() : new HashSet<>()).add("baz"); - } } \ No newline at end of file diff --git a/plugins/InspectionGadgets/test/com/siyeh/igtest/bugs/assert_with_side_effects/expected.xml b/plugins/InspectionGadgets/test/com/siyeh/igtest/bugs/assert_with_side_effects/expected.xml index b518c936f5ae..93d780851750 100644 --- a/plugins/InspectionGadgets/test/com/siyeh/igtest/bugs/assert_with_side_effects/expected.xml +++ b/plugins/InspectionGadgets/test/com/siyeh/igtest/bugs/assert_with_side_effects/expected.xml @@ -20,11 +20,4 @@ 'assert' statement with side effects <code>assert</code> has side effects #loc - - - AssertWithSideEffects.java - 35 - 'assert' statement with side effects - <code>assert</code> has side effects #loc - \ No newline at end of file diff --git a/plugins/InspectionGadgets/test/com/siyeh/igtest/bugs/assert_with_side_effects8/AssertWithSideEffects.java b/plugins/InspectionGadgets/test/com/siyeh/igtest/bugs/assert_with_side_effects8/AssertWithSideEffects.java index 800615b5abf8..d50c08eb95bf 100644 --- a/plugins/InspectionGadgets/test/com/siyeh/igtest/bugs/assert_with_side_effects8/AssertWithSideEffects.java +++ b/plugins/InspectionGadgets/test/com/siyeh/igtest/bugs/assert_with_side_effects8/AssertWithSideEffects.java @@ -1,5 +1,6 @@ package com.siyeh.igtest.bugs.assert_with_side_effects; import java.util.regex.*; +import java.util.*; public class AssertWithSideEffects { void assertMutation() { Matcher m = Pattern.compile("foobar").matcher("foo"); @@ -7,4 +8,12 @@ public class AssertWithSideEffects { assert Pattern.compile("foobar").matcher("foo").matches(); } + + void assertMutation(Set set) { + assert set.add("foo"); + + assert new HashSet<>().add("bar"); + + assert (set.isEmpty() ? new TreeSet<>() : new HashSet<>()).add("baz"); + } } \ No newline at end of file diff --git a/plugins/InspectionGadgets/test/com/siyeh/igtest/bugs/assert_with_side_effects8/expected.xml b/plugins/InspectionGadgets/test/com/siyeh/igtest/bugs/assert_with_side_effects8/expected.xml index 739f5ee0feaf..009fbe70bd77 100644 --- a/plugins/InspectionGadgets/test/com/siyeh/igtest/bugs/assert_with_side_effects8/expected.xml +++ b/plugins/InspectionGadgets/test/com/siyeh/igtest/bugs/assert_with_side_effects8/expected.xml @@ -2,7 +2,13 @@ AssertWithSideEffects.java - 6 + 7 + 'assert' statement with side effects + <code>assert</code> has side effects #loc + + + AssertWithSideEffects.java + 13 'assert' statement with side effects <code>assert</code> has side effects #loc