diff --git a/plugins/InspectionGadgets/src/com/siyeh/ig/asserttoif/IfCanBeAssertionInspection.java b/plugins/InspectionGadgets/src/com/siyeh/ig/asserttoif/IfCanBeAssertionInspection.java index 0330b4aa8427..02e6779c7b37 100644 --- a/plugins/InspectionGadgets/src/com/siyeh/ig/asserttoif/IfCanBeAssertionInspection.java +++ b/plugins/InspectionGadgets/src/com/siyeh/ig/asserttoif/IfCanBeAssertionInspection.java @@ -78,7 +78,9 @@ public class IfCanBeAssertionInspection extends BaseInspection { @Override public void visitIfStatement(PsiIfStatement statement) { super.visitIfStatement(statement); - if (statement.getElseBranch() == null && getThrownNewException(statement.getThenBranch()) != null) { + if (statement.getCondition() != null && + statement.getElseBranch() == null && + getThrownNewException(statement.getThenBranch()) != null) { registerStatementError(statement); } } diff --git a/plugins/InspectionGadgets/src/inspectionDescriptions/IfCanBeAssertion.html b/plugins/InspectionGadgets/src/inspectionDescriptions/IfCanBeAssertion.html index 2519f12a6c0e..75224616cdff 100644 --- a/plugins/InspectionGadgets/src/inspectionDescriptions/IfCanBeAssertion.html +++ b/plugins/InspectionGadgets/src/inspectionDescriptions/IfCanBeAssertion.html @@ -1,6 +1,6 @@
-This inspection reports if statements (with no else branch) throwingjava.lang.Throwable.java.lang.Throwable.if (param == 2) throw new Exception();
diff --git a/plugins/InspectionGadgets/test/com/siyeh/igfixes/asserttoif/if_to_assert/NoCondition.java b/plugins/InspectionGadgets/test/com/siyeh/igfixes/asserttoif/if_to_assert/NoCondition.java
new file mode 100644
index 000000000000..b06e622d9183
--- /dev/null
+++ b/plugins/InspectionGadgets/test/com/siyeh/igfixes/asserttoif/if_to_assert/NoCondition.java
@@ -0,0 +1,7 @@
+class NoCondition {
+ void m(Object o) {
+