Przetestowanie metody, która przyjmuje Scanner i zwraca obiekt.

0

Cześć, tak jak w temacie wątku, potrzebuje przetestować metode, która przyjmuje Scanner input, a zwraca mi obiekt.
Metoda wygląda tak:

public RentingACar rentACar(Scanner input) {
        RentingACar rentingACar = new RentingACar();

        System.out.print("Brand: ");
        rentingACar.setBrand(input.next());
        System.out.print("Name: ");
        rentingACar.setName(input.next());
        System.out.print("Surname: ");
        rentingACar.setSurname(input.next());
        System.out.print("Rent Date: ");
        rentingACar.setRentDate(input.next());
        System.out.print("Client number: ");
        rentingACar.setClientNumber(input.nextInt());

        return rentingACar;
    }

Próbowałem mokować jakos scanner, ale nieudolnie. Wywala NPE przy Client client = clientDataGetter.createClient(scanner);

@Mock
    Scanner scanner;

    @Before
    public void setup() {
        MockitoAnnotations.initMocks(this);
    }

    @Test
    void rentACar() {
        ClientDataGetter clientDataGetter = new ClientDataGetter();
        RentingACar rentingACar = clientDataGetter.rentACar(scanner);

        assertNotNull(rentingACar);
    }

1

Więcej zachodu z mockami via fraemworki niż utworzeniem obiektu... Nie możesz sobie zrobić np. tak ?

        Scanner myScanner = new Scanner("foo\n\bar\nbaz\n123");
        System.out.println("Next="+myScanner.next());
        System.out.println("Next="+myScanner.next());
        System.out.println("Next="+myScanner.next());
        System.out.println("Next="+myScanner.nextInt());

1

Można stworzyć interfejs Tokenizer:

interface Tokenizer {
  String next();
  int nextInt();
}

Podłożyć go tutaj:

public RentingACar rentACar(Tokenizer input) {

Zrobić 2 implementacje: prawdziwą i testową. Testowa przyjmowałaby w konstruktorze listę obiektów i potem je zwracała przy wywołaniach next. Prawdziwa pobierała by z kontruktora Scanner i później go używała (kompozycja). Takie rozwiązanie bardzo czytelnie by wypadało w testach.

A przy pomocy Mockito też możesz, tylko dodaj zachowania mocka (stuby):

Mockito.when(scanner.next())
  .thenReturn("aaa")
  .thenReturn("bbb");

NPE które dostajesz też trzeba dokładniej obejrzeć, cały stack trace. Dokładnie który obiekt jest nullem? Trzeba to ustalić, potem zaradzić, pewnie jakimś stubem.

0

jarekczek napisał(a):

Można stworzyć interfejs Tokenizer:
interface Tokenizer {
  String next();
  int nextInt();
}
Podłożyć go tutaj:
public RentingACar rentACar(Tokenizer input) {
Zrobić 2 implementacje: prawdziwą i testową. Testowa przyjmowałaby w konstruktorze listę obiektów i potem je zwracała przy wywołaniach next. Prawdziwa pobierała by z kontruktora Scanner i później go używała (kompozycja). Takie rozwiązanie bardzo czytelnie by wypadało w testach.

A przy pomocy Mockito też możesz, tylko dodaj zachowania mocka (stuby):
Mockito.when(scanner.next())
  .thenReturn("aaa")
  .thenReturn("bbb");
NPE które dostajesz też trzeba dokładniej obejrzeć, cały stack trace. Dokładnie który obiekt jest nullem? Trzeba to ustalić, potem zaradzić, pewnie jakimś stubem.

Osobiście wolałbym rozwiązanie z Mockiem, bo jest on mi bliższy. Może pownienem zrobić w takim razie tak jak Ty, czyli:

Mockito.when(scanner.next())
  .thenReturn("aaa")
  .thenReturn("bbb");

Z tym, że pododawać tyle odpowiedzi ile Scanner wymaga?

Oto co wywala w konsoli do poprzedniego rozwiązania:

java.lang.NullPointerException
	at car.rental.data.getters.ClientDataGetter.rentACar(ClientDataGetter.java:42)
	at car.rental.data.getters.ClientDataGetterTest.rentACar(ClientDataGetterTest.java:46)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/java.lang.reflect.Method.invoke(Method.java:564)
	at org.junit.platform.commons.util.ReflectionUtils.invokeMethod(ReflectionUtils.java:515)
	at org.junit.jupiter.engine.execution.ExecutableInvoker.invoke(ExecutableInvoker.java:115)
	at org.junit.jupiter.engine.descriptor.TestMethodTestDescriptor.lambda$invokeTestMethod$6(TestMethodTestDescriptor.java:170)
	at org.junit.jupiter.engine.execution.ThrowableCollector.execute(ThrowableCollector.java:40)
	at org.junit.jupiter.engine.descriptor.TestMethodTestDescriptor.invokeTestMethod(TestMethodTestDescriptor.java:166)
	at org.junit.jupiter.engine.descriptor.TestMethodTestDescriptor.execute(TestMethodTestDescriptor.java:113)
	at org.junit.jupiter.engine.descriptor.TestMethodTestDescriptor.execute(TestMethodTestDescriptor.java:58)
	at org.junit.platform.engine.support.hierarchical.NodeTestTask.lambda$executeRecursively$5(NodeTestTask.java:134)
	at org.junit.platform.engine.support.hierarchical.SingleTestExecutor.executeSafely(SingleTestExecutor.java:66)
	at org.junit.platform.engine.support.hierarchical.NodeTestTask.executeRecursively(NodeTestTask.java:128)
	at org.junit.platform.engine.support.hierarchical.NodeTestTask.execute(NodeTestTask.java:109)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1378)
	at org.junit.platform.engine.support.hierarchical.SameThreadHierarchicalTestExecutorService.invokeAll(SameThreadHierarchicalTestExecutorService.java:38)
	at org.junit.platform.engine.support.hierarchical.NodeTestTask.lambda$executeRecursively$5(NodeTestTask.java:138)
	at org.junit.platform.engine.support.hierarchical.SingleTestExecutor.executeSafely(SingleTestExecutor.java:66)
	at org.junit.platform.engine.support.hierarchical.NodeTestTask.executeRecursively(NodeTestTask.java:128)
	at org.junit.platform.engine.support.hierarchical.NodeTestTask.execute(NodeTestTask.java:109)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1378)
	at org.junit.platform.engine.support.hierarchical.SameThreadHierarchicalTestExecutorService.invokeAll(SameThreadHierarchicalTestExecutorService.java:38)
	at org.junit.platform.engine.support.hierarchical.NodeTestTask.lambda$executeRecursively$5(NodeTestTask.java:138)
	at org.junit.platform.engine.support.hierarchical.SingleTestExecutor.executeSafely(SingleTestExecutor.java:66)
	at org.junit.platform.engine.support.hierarchical.NodeTestTask.executeRecursively(NodeTestTask.java:128)
	at org.junit.platform.engine.support.hierarchical.NodeTestTask.execute(NodeTestTask.java:109)
	at org.junit.platform.engine.support.hierarchical.SameThreadHierarchicalTestExecutorService.submit(SameThreadHierarchicalTestExecutorService.java:32)
	at org.junit.platform.engine.support.hierarchical.HierarchicalTestExecutor.execute(HierarchicalTestExecutor.java:49)
	at org.junit.platform.engine.support.hierarchical.HierarchicalTestEngine.execute(HierarchicalTestEngine.java:47)
	at org.junit.platform.launcher.core.DefaultLauncher.execute(DefaultLauncher.java:184)
	at org.junit.platform.launcher.core.DefaultLauncher.lambda$execute$5(DefaultLauncher.java:152)
	at org.junit.platform.launcher.core.DefaultLauncher.withInterceptedStreams(DefaultLauncher.java:166)
	at org.junit.platform.launcher.core.DefaultLauncher.execute(DefaultLauncher.java:145)
	at org.junit.platform.launcher.core.DefaultLauncher.execute(DefaultLauncher.java:92)
	at com.intellij.junit5.JUnit5IdeaTestRunner.startRunnerWithArgs(JUnit5IdeaTestRunner.java:74)
	at com.intellij.rt.execution.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:47)
	at com.intellij.rt.execution.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:242)
	at com.intellij.rt.execution.junit.JUnitStarter.main(JUnitStarter.java:70)

42 linijka z ClientDataGetter jest to ta linijka rentingACar.setBrand(input.next()); czyli pierwszy Scanner.
Natomiast 46 linijka z testu to RentingACar rentingACar = clientDataGetter.rentACar(scanner);.

Z logicznego punktu widzenia, fajnie by było gdyby scanner zwracał dowolną wartość, gdy ta jest potrzebna.

1

Te tokeny jakoś sensownie nazwij pod kątem testu..

@Test
    void rentACar() {
        ClientDataGetter clientDataGetter = new ClientDataGetter();
		Scanner scanner = new Scanner("brand\nname\nsurname\ndate\n1234");
        RentingACar rentingACar = clientDataGetter.rentACar(scanner);
        assertNotNull(rentingACar);

      //TODO: weryfikacja czy gettery zwracają co trzeba?
    }

edited : Używasz nextInt(), więc dane wejściowe powinny być zgodne z oczekiwaną operacją na scannerze, stąd clientNumber np. 1234

Przetestowanie metody, która przyjmuje Scanner i zwraca obiekt.

jarekczek napisał(a):

1 użytkowników online, w tym zalogowanych: 0, gości: 1

Praca dla programistów

Forum dyskusyjne

Sprawy administracyjne

O nas

Skontaktuj się z nami