Skip to content

Commit b2a1289

Browse files
committed
Update to SpotBugs 6.2.4
1 parent c0c812b commit b2a1289

3 files changed

Lines changed: 2 additions & 13 deletions

File tree

build.gradle

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ plugins {
77
id 'java-library'
88
id 'maven-publish'
99
id 'checkstyle'
10-
id 'com.github.spotbugs' version '6.2.3'
10+
id 'com.github.spotbugs' version '6.2.4'
1111
id 'com.github.ben-manes.versions' version '0.52.0'
1212
}
1313

@@ -97,7 +97,7 @@ dependencies {
9797

9898
testImplementation platform('org.junit:junit-bom:5.13.4')
9999
testImplementation 'org.junit.jupiter:junit-jupiter-api'
100-
testImplementation 'org.mockito:mockito-junit-jupiter:5.18.0'
100+
testImplementation 'org.mockito:mockito-junit-jupiter:5.19.0'
101101
testImplementation 'org.assertj:assertj-core:3.27.4'
102102

103103
testRuntimeOnly 'org.junit.platform:junit-platform-launcher'

src/test/java/org/jivesoftware/openfire/plugin/passwordreset/Fixtures.java

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@
66
import static org.mockito.Mockito.mock;
77
import static org.mockito.Mockito.withSettings;
88

9-
import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
109
import java.io.File;
1110
import java.net.URL;
1211
import lombok.SneakyThrows;
@@ -72,9 +71,6 @@ public static void clearExistingProperties() {
7271
*
7372
* @return a mock XMPPServer
7473
*/
75-
@SuppressFBWarnings(
76-
value = "RV_RETURN_VALUE_IGNORED_NO_SIDE_EFFECT",
77-
justification = "False positive")
7874
public static XMPPServer mockXmppServer() {
7975
final XMPPServer xmppServer = mock(
8076
XMPPServer.class,

src/test/java/org/jivesoftware/openfire/plugin/passwordreset/servlet/client/PasswordResetChangePasswordServletTest.java

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@
1010
import static org.mockito.Mockito.never;
1111
import static org.mockito.Mockito.verify;
1212

13-
import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
1413
import java.sql.SQLException;
1514
import java.util.Optional;
1615
import javax.servlet.RequestDispatcher;
@@ -162,9 +161,6 @@ void postWithNothingWillRedirect() {
162161

163162
@SneakyThrows
164163
@Test
165-
@SuppressFBWarnings(
166-
value = "RV_RETURN_VALUE_IGNORED_NO_SIDE_EFFECT",
167-
justification = "False positive")
168164
void validFormWillUpdatePasswordAndRedirect() {
169165

170166
givenValidFormSubmission();
@@ -298,9 +294,6 @@ void badTokenWillRedirectToBadTokenPage() {
298294

299295
@SneakyThrows
300296
@Test
301-
@SuppressFBWarnings(
302-
value = "RV_RETURN_VALUE_IGNORED_NO_SIDE_EFFECT",
303-
justification = "False positive")
304297
void differentUserIdWillRedirectToBadTokenPage() {
305298

306299
givenValidFormSubmission();

0 commit comments

Comments
 (0)