Skip to content

Commit b6aa506

Browse files
committed
cleanup
1 parent a5a97e6 commit b6aa506

File tree

2 files changed

+19
-3
lines changed

2 files changed

+19
-3
lines changed

src/test/java/org/htmlunit/BrowserVersionFeaturesTest.java

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -24,14 +24,13 @@
2424
import java.util.ArrayList;
2525
import java.util.LinkedList;
2626
import java.util.List;
27+
import java.util.Objects;
2728

2829
import org.apache.commons.io.FileUtils;
2930
import org.htmlunit.javascript.configuration.BrowserFeature;
3031
import org.htmlunit.javascript.configuration.SupportedBrowser;
3132
import org.junit.jupiter.api.Test;
3233

33-
import com.tngtech.archunit.thirdparty.com.google.common.base.Objects;
34-
3534
/**
3635
* Tests for {@link BrowserVersionFeatures}.
3736
*
@@ -89,7 +88,7 @@ public void unusedFeatures() throws Exception {
8988
for (final SupportedBrowser annotatedBrowser : browserFeature.value()) {
9089
boolean inUse = false;
9190
for (final BrowserVersion supportedBrowserVersion : browsers) {
92-
if (Objects.equal(supportedBrowser(supportedBrowserVersion), annotatedBrowser)) {
91+
if (Objects.equals(supportedBrowser(supportedBrowserVersion), annotatedBrowser)) {
9392
inUse = true;
9493
continue;
9594
}

src/test/java/org/htmlunit/archunit/Architecture2Test.java

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -214,4 +214,21 @@ public void check(final JavaMethod method, final ConditionEvents events) {
214214
.that().areAnnotatedWith(Test.class)
215215
.and().areNotDeclaredIn("org.htmlunit.junit.annotation.AnnotationUtilsTest")
216216
.should(haveConsistentTestAnnotations);
217+
218+
/**
219+
* Do not use archunit outside of this.
220+
*/
221+
@ArchTest
222+
public static final ArchRule archunitPackageRule = noClasses()
223+
.that()
224+
.resideOutsideOfPackage("org.htmlunit.archunit..")
225+
226+
.should().dependOnClassesThat().resideInAnyPackage("com.tngtech.archunit..");
227+
228+
/**
229+
* Do not use google commons.
230+
*/
231+
@ArchTest
232+
public static final ArchRule googleCommonPackageRule = noClasses()
233+
.should().dependOnClassesThat().resideInAnyPackage("com.google.common..");
217234
}

0 commit comments

Comments
 (0)