Location: php:php-checks/src/test/java/org/sonar/php/checks/CharacterBeforeOpeningPHPTagCheckTest.java:32

Why is this an issue?

Superfluous exceptions within throws clauses have negative effects on the readability and maintainability of the code. An exception in a throws clause is superfluous if it is:

What changed

Removes the superfluous 'throws Exception' declaration from the 'ok()' method, since no exception is actually thrown from the method's body. This addresses the code smell where a throws clause declares an exception that cannot be thrown by any execution path of the method.

--- a/php-checks/src/test/java/org/sonar/php/checks/CharacterBeforeOpeningPHPTagCheckTest.java
+++ b/php-checks/src/test/java/org/sonar/php/checks/CharacterBeforeOpeningPHPTagCheckTest.java
@@ -32,1 +32,1 @@ class CharacterBeforeOpeningPHPTagCheckTest {
-  void ok() throws Exception {
+  void ok() {

Location: php:php-checks/src/test/java/org/sonar/php/checks/NoPaddingRsaCheckTest.java:25

Why is this an issue?

Superfluous exceptions within throws clauses have negative effects on the readability and maintainability of the code. An exception in a throws clause is superfluous if it is:

What changed

Removes the superfluous 'throws Exception' declaration from the test method 'test()' in NoPaddingRsaCheckTest.java. The method body does not actually throw any checked exception, so declaring 'throws Exception' was unnecessary and flagged as a code smell. By removing the throws clause, the method signature accurately reflects that no checked exceptions are thrown.

--- a/php-checks/src/test/java/org/sonar/php/checks/NoPaddingRsaCheckTest.java
+++ b/php-checks/src/test/java/org/sonar/php/checks/NoPaddingRsaCheckTest.java
@@ -25,1 +25,1 @@ class NoPaddingRsaCheckTest {
-  void test() throws Exception {
+  void test() {

Location: php:php-checks/src/test/java/org/sonar/php/checks/WrongAssignmentOperatorCheckTest.java:25

Why is this an issue?

Superfluous exceptions within throws clauses have negative effects on the readability and maintainability of the code. An exception in a throws clause is superfluous if it is:

What changed

Removes the superfluous 'throws Exception' declaration from the test method 'test()' in WrongAssignmentOperatorCheckTest.java. The method body does not actually throw any checked exception, so the 'throws Exception' clause was unnecessary. This directly addresses the code smell about declaring a thrown exception that cannot be thrown from the method's body.

--- a/php-checks/src/test/java/org/sonar/php/checks/WrongAssignmentOperatorCheckTest.java
+++ b/php-checks/src/test/java/org/sonar/php/checks/WrongAssignmentOperatorCheckTest.java
@@ -25,1 +25,1 @@ class WrongAssignmentOperatorCheckTest {
-  void test() throws Exception {
+  void test() {

Location: php:php-checks/src/test/java/org/sonar/php/checks/regex/EmptyStringRepetitionCheckTest.java:25

Why is this an issue?

Superfluous exceptions within throws clauses have negative effects on the readability and maintainability of the code. An exception in a throws clause is superfluous if it is:

What changed

Removes the superfluous 'throws Exception' declaration from the test method 'test()' in EmptyStringRepetitionCheckTest.java. The method body does not actually throw any checked exception, so the 'throws Exception' clause was unnecessary and flagged as a code smell. By removing it, the method signature accurately reflects that no checked exceptions are thrown.

--- a/php-checks/src/test/java/org/sonar/php/checks/regex/EmptyStringRepetitionCheckTest.java
+++ b/php-checks/src/test/java/org/sonar/php/checks/regex/EmptyStringRepetitionCheckTest.java
@@ -25,1 +25,1 @@ class EmptyStringRepetitionCheckTest {
-  void test() throws Exception {
+  void test() {

Location: php:php-checks/src/test/java/org/sonar/php/checks/regex/RegexLookaheadCheckTest.java:25

Why is this an issue?

Superfluous exceptions within throws clauses have negative effects on the readability and maintainability of the code. An exception in a throws clause is superfluous if it is:

What changed

Removes the superfluous 'throws Exception' declaration from the test method 'test()' in RegexLookaheadCheckTest.java. The method body does not actually throw any checked exception, so the 'throws Exception' clause was unnecessary. This directly addresses the code smell about declaring thrown exceptions that cannot be thrown from the method's body.

--- a/php-checks/src/test/java/org/sonar/php/checks/regex/RegexLookaheadCheckTest.java
+++ b/php-checks/src/test/java/org/sonar/php/checks/regex/RegexLookaheadCheckTest.java
@@ -25,1 +25,1 @@ class RegexLookaheadCheckTest {
-  void test() throws Exception {
+  void test() {