Skip to content

Commit

Permalink
add java tests (#53)
Browse files Browse the repository at this point in the history
* add java tests
  • Loading branch information
ievans committed Feb 14, 2020
1 parent 1ae947f commit 74ca790
Show file tree
Hide file tree
Showing 2 changed files with 53 additions and 4 deletions.
28 changes: 28 additions & 0 deletions java/basic-rules.java
Expand Up @@ -2,9 +2,37 @@ class Bar {
void main() {
boolean myBoolean;

//myBoolean == myBoolean;

// ruleid:assignment-comparison
if (myBoolean = true) {
continue;
}

// ruleid:hardcoded-eq-true-or-false
if (true) {
continue;
}

// todoruleid:hardcoded-eq-true-or-false
if (true && false) {
continue;
}

// ok
if (myBoolean) {

}

// ruleid:eqeq-is-bad
if (myBoolean == myBoolean) {
continue;
}

// ruleid:eqeq-is-bad
if (myBoolean != myBoolean) {
continue;
}

}
}
29 changes: 25 additions & 4 deletions java/basic-rules.yaml
@@ -1,8 +1,29 @@
rules:
- id: assignment-comparison
patterns:
- pattern: if($X=true) {}
- pattern: if($X=false) {}
message: "This exception is being ignored. The best practice is to handle exceptions, throw them and show an error dialogue, or add a message to the log."
- pattern-either:
- pattern: if ($X=true) { ... }
- pattern: if ($X=false) { ... }
message: "The value of `$X` is being ignored and will be used in the conditional test"
languages: [java]
severity: ERROR
severity: ERROR
- id: eqeq-is-bad
patterns:
- pattern-not-inside: assert $X;
- pattern-not-inside: |
assert $X : $Y;
- pattern-either:
- pattern: $X == $X
- pattern: $X != $X
- pattern-not: 1 == 1
message: "useless comparison operation `$X == $X` or `$X != $X`"
languages: [java]
severity: ERROR
- id: hardcoded-eq-true-or-false
patterns:
- pattern-either:
- pattern: if (true) { ... }
- pattern: if (false) { ... }
message: "useless if statement, always the same behavior"
languages: [java]
severity: ERROR

0 comments on commit 74ca790

Please sign in to comment.