Skip to content

Commit 0b9117f

Browse files
authored
Merge pull request #6045 from thc202/client/tests/tidy
client: prevent ZAP dirs usage/init in tests
2 parents 6b96a7b + fd34de4 commit 0b9117f

File tree

3 files changed

+5
-8
lines changed

3 files changed

+5
-8
lines changed

addOns/client/src/test/java/org/zaproxy/addon/client/internal/ClientMapUnitTest.java

-2
Original file line numberDiff line numberDiff line change
@@ -295,8 +295,6 @@ void shouldAddSiteLevelFragmentNodes() {
295295
assertThat(root.getChildAt(0).getChildAt(2).getChildCount(), is(1));
296296
assertThat(
297297
root.getChildAt(0).getChildAt(2).getChildAt(0).getUserObject().getName(), is("#"));
298-
System.out.println(root.getChildAt(0).getChildAt(2).getChildAt(0).getUserObject().getUrl());
299-
System.out.println("https://www.example.com/?p2=v3&p1=v4#/");
300298
assertThat(
301299
root.getChildAt(0).getChildAt(2).getChildAt(0).getUserObject().getUrl(),
302300
is("https://www.example.com/?p2=v3&p1=v4#"));

addOns/client/src/test/java/org/zaproxy/addon/client/spider/UrlTableModelUnitTest.java

+4-5
Original file line numberDiff line numberDiff line change
@@ -23,21 +23,20 @@
2323
import static org.hamcrest.Matchers.contains;
2424
import static org.hamcrest.Matchers.hasSize;
2525

26-
import java.util.Locale;
2726
import org.junit.jupiter.api.BeforeAll;
2827
import org.junit.jupiter.api.BeforeEach;
2928
import org.junit.jupiter.api.Test;
30-
import org.parosproxy.paros.Constant;
31-
import org.zaproxy.zap.utils.I18N;
29+
import org.zaproxy.addon.client.ExtensionClientIntegration;
30+
import org.zaproxy.zap.testutils.TestUtils;
3231

3332
/** Unit test for {@link UrlTableModel}. */
34-
class UrlTableModelUnitTest {
33+
class UrlTableModelUnitTest extends TestUtils {
3534

3635
private UrlTableModel model;
3736

3837
@BeforeAll
3938
static void setupAll() {
40-
Constant.messages = new I18N(Locale.ROOT);
39+
mockMessages(new ExtensionClientIntegration());
4140
}
4241

4342
@BeforeEach

addOns/client/src/test/java/org/zaproxy/addon/client/zest/ExtensionClientZestTest.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ class ExtensionClientZestTest {
4545
void setUp() {
4646
extensionLoader =
4747
mock(ExtensionLoader.class, withSettings().strictness(Strictness.LENIENT));
48-
Control.initSingletonForTesting(Model.getSingleton(), extensionLoader);
48+
Control.initSingletonForTesting(mock(Model.class), extensionLoader);
4949

5050
ExtensionZest extensionZest = mock(ExtensionZest.class);
5151
given(extensionLoader.getExtension(ExtensionZest.class)).willReturn(extensionZest);

0 commit comments

Comments
 (0)