Макет теста jUnit с использованием powermockito - PullRequest
0 голосов
/ 06 июля 2018

Я пытаюсь смоделировать следующую функциональность

public class ServerConnection{
        public ServerConnection(ClientConnection cn) {
            super(cn);
        }

    public void setUrl(String url) {      
                this.url = URLUtil.processURL(url);
                try {
                    URL dst = new URL(this.url);
                    InputStream is = dst.openStream();
                    Scanner scanner = new Scanner(is);

                    StringBuilder sb = new StringBuilder();

                    while(scanner.hasNextLine())
                        sb.append(scanner.nextLine()).append("\n");


                    if (validate(sb.toString())) {
                       --------
                    } else {    }
                    is.close();
                    scanner.close();
                } catch (Exception ex) {

                }        
        }

    private boolean validate(String content) {
            JSONParser parser = new JSONParser();
            Boolean isJsonValid = false;
            JSONObject json = null;
            try {
              --------
              //json validation goes here
            } catch (Exception e) {          
            }
            return isJsonValid;
        }

        public void setId(Integer id) {
        if(id == null)
        this.id = 0;
        else
        this.id = id;
        }
}

Код PowerMockito Junit

@RunWith(PowerMockRunner.class)
@PrepareForTest({PathTest.class })
public class URLTest {
 ServerConnection sc ;
 String URL = "http://test.com";

  @Before
     public void setUp() throws Throwable{              
             ClientConnection con =PathTest.getCon();  // Here getCon() is a static method
             sc = new ServerConnection(con);
             sc.setId(1000); 
     }

@Test
    public void testName() throws Throwable {       

        String expectedResponse = "test";
        URL url = PowerMockito.mock(URL.class);
        HttpURLConnection connection = PowerMockito.mock(HttpURLConnection.class);
        InputStream inputStream = PowerMockito.mock(InputStream.class);
        Scanner scanner = PowerMockito.mock(Scanner.class);

        PowerMockito.whenNew(URL.class).withArguments(URL).thenReturn(url);
        PowerMockito.whenNew(Scanner.class).withArguments(inputStream).thenReturn(scanner);
        PowerMockito.when(scanner.useDelimiter("\\A")).thenReturn(scanner);

        PowerMockito.when(url.openConnection()).thenReturn(connection);

        // Response code mocked here
        PowerMockito.when(connection.getResponseCode()).thenReturn(200);

        PowerMockito.when(connection.getInputStream()).thenReturn(inputStream);
        PowerMockito.when(scanner.hasNext()).thenReturn(true);
        PowerMockito.when(scanner.next()).thenReturn(expectedResponse);

        sc.setUrl(URL);
    }
}

Во время выполнения Iam я заметил следующее сообщение об ошибке

URLTest
com.objects.URLTest
testName(com.objects.URLTest)
java.lang.ExceptionInInitializerError

    at java.lang.Class.forName0(Native Method)

    at java.lang.Class.forName(Class.java:264)

    at javassist.runtime.Desc.getClassObject(Desc.java:43)

    at javassist.runtime.Desc.getClassType(Desc.java:152)

    at javassist.runtime.Desc.getType(Desc.java:122)

    at javassist.runtime.Desc.getType(Desc.java:78)

    at com.objects.PathTest.getCon(TargetPathTest.java:24)

    at com.objects.URLTest.setUp(URLTest.java:41)

    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)

    at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)

    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)

    at java.lang.reflect.Method.invoke(Method.java:498)

    at org.junit.internal.runners.MethodRoadie.runBefores(MethodRoadie.java:133)

    at org.junit.internal.runners.MethodRoadie.runBeforesThenTestThenAfters(MethodRoadie.java:96)

    at org.powermock.modules.junit4.internal.impl.PowerMockJUnit44RunnerDelegateImpl$PowerMockJUnit44MethodRunner.executeTest(PowerMockJUnit44RunnerDelegateImpl.java:310)

    at org.powermock.modules.junit4.internal.impl.PowerMockJUnit47RunnerDelegateImpl$PowerMockJUnit47MethodRunner.executeTestInSuper(PowerMockJUnit47RunnerDelegateImpl.java:131)

    at org.powermock.modules.junit4.internal.impl.PowerMockJUnit47RunnerDelegateImpl$PowerMockJUnit47MethodRunner.access$100(PowerMockJUnit47RunnerDelegateImpl.java:59)

    at org.powermock.modules.junit4.internal.impl.PowerMockJUnit47RunnerDelegateImpl$PowerMockJUnit47MethodRunner$TestExecutorStatement.evaluate(PowerMockJUnit47RunnerDelegateImpl.java:147)

    at org.powermock.modules.junit4.internal.impl.PowerMockJUnit47RunnerDelegateImpl$PowerMockJUnit47MethodRunner.evaluateStatement(PowerMockJUnit47RunnerDelegateImpl.java:107)

    at org.powermock.modules.junit4.internal.impl.PowerMockJUnit47RunnerDelegateImpl$PowerMockJUnit47MethodRunner.executeTest(PowerMockJUnit47RunnerDelegateImpl.java:82)

    at org.powermock.modules.junit4.internal.impl.PowerMockJUnit44RunnerDelegateImpl$PowerMockJUnit44MethodRunner.runBeforesThenTestThenAfters(PowerMockJUnit44RunnerDelegateImpl.java:298)

    at org.junit.internal.runners.MethodRoadie.runTest(MethodRoadie.java:87)

    at org.junit.internal.runners.MethodRoadie.run(MethodRoadie.java:50)

    at org.powermock.modules.junit4.internal.impl.PowerMockJUnit44RunnerDelegateImpl.invokeTestMethod(PowerMockJUnit44RunnerDelegateImpl.java:218)

    at org.powermock.modules.junit4.internal.impl.PowerMockJUnit44RunnerDelegateImpl.runMethods(PowerMockJUnit44RunnerDelegateImpl.java:160)

    at org.powermock.modules.junit4.internal.impl.PowerMockJUnit44RunnerDelegateImpl$1.run(PowerMockJUnit44RunnerDelegateImpl.java:134)

    at org.junit.internal.runners.ClassRoadie.runUnprotected(ClassRoadie.java:34)

    at org.junit.internal.runners.ClassRoadie.runProtected(ClassRoadie.java:44)

    at org.powermock.modules.junit4.internal.impl.PowerMockJUnit44RunnerDelegateImpl.run(PowerMockJUnit44RunnerDelegateImpl.java:136)

    at org.powermock.modules.junit4.common.internal.impl.JUnit4TestSuiteChunkerImpl.run(JUnit4TestSuiteChunkerImpl.java:121)

    at org.powermock.modules.junit4.common.internal.impl.AbstractCommonPowerMockRunner.run(AbstractCommonPowerMockRunner.java:57)

    at org.powermock.modules.junit4.PowerMockRunner.run(PowerMockRunner.java:59)

    at org.eclipse.jdt.internal.junit4.runner.JUnit4TestReference.run(JUnit4TestReference.java:50)

    at org.eclipse.jdt.internal.junit.runner.TestExecution.run(TestExecution.java:38)

    at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:459)

    at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:675)

    at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.run(RemoteTestRunner.java:382)

    at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.main(RemoteTestRunner.java:192)

Caused by: java.lang.NullPointerException   

    ... 38 more

1 Ответ

0 голосов
/ 08 июля 2018

С этим кодом так много проблем.

Первый технический ответ: вы, кажется, не знаете, что делаете. У вас есть @PrepareForTest({PathTest.class }), что означает, что вы намереваетесь смоделировать статический метод этого класса.

Но тогда вы не делаете то, что необходимо для насмешки над статическим методом в этом классе. Просто следуйте их документации шаг за шагом. Я также думаю, что URL-класс является окончательным, поэтому вам необходимо подготовить и аннотировать этот класс, чтобы его можно было использовать с whenNew ()!

Но тогда: вам следует избегать поддельных объектов URL или URL-соединений. Просто используйте какое-либо внедрение зависимостей и убедитесь, что вы можете передать какой-то макетированный экземпляр в тестируемый код, например, используя аннотацию @InjectMocks для Mockito. Оттуда вы также можете избежать использования static метода.

Короче говоря: ваш рабочий код написан плохо, а ваш модульный тест (честно) просто ужасен. Вы должны серьезно отойти назад и выбросить все это. Затем переосмыслите свой производственный код, чтобы не использовал new напрямую, и чтобы не полагаться на статические методы. Таким образом, вы можете избавиться от PowerMockito, вы идете с обычным Mockito (ну, вы должны включить насмешку над финальными классами). Но будьте уверены: использование PowerMock (ito) не имеет смысла, если в этом нет необходимости.

А потом: не "придумывайте" издевательский код. Прочитайте учебные пособия и следуйте им шаг за шагом. В вашем тестовом примере вы издеваетесь, как все , но вы должны делать прямо противоположное: вы только издеваетесь, когда совсем нет другого способа проверить ваш код. И вы абсолютно не делаете метод статичным, когда это лишает вас возможности проводить модульное тестирование кода.

(статика имеет свое место, но когда она встает у вас на пути, вы делаете не то!)

...