Problem bei JUnit Test Aufgabe

[werdas34]

Hallo,

ich habe vom Prof 19 Java Dateien bekommen, in denen alle das selbe machen(Liste mit maximal einer Zahl in der Liste). Diese Dateien heißen alle exakt gleich "SoloList" und sind in verschiedenen Package "SoloList.p01" bis "SoloList.p19".
Jede SoloList ist anders implementiert, manche haben Fehler und manche nicht.

Mein Problem:
Wenn ich eine JUnit Test schreibe, kommt der Punkt, wo der Compiler von mir fordert aus welchen Package ich diese Klasse SoloList instanzieren soll.
Es soll halt irgendwie von allen 19 Dateien dies nutzen.
Denn meine klare Aufgabe lautet:
"Ihre Aufgabe ist es, eine JUnit Testklasse zu entwickeln, die die fehlerhaften Implementierungen erkennt und die Fehler ausweist."

Muss ich nun 19 Test schreiben oder geht das eleganter?

mfg
werdas34

 

[looparda]

Vielleicht sowas in der Art:

 YourList getYourListImplInstance(String name)
            throws InstantiationException, IllegalAccessException, ClassNotFoundException
    {
        Class<?> result = null;
        final String listImplClassName =
            "package.your.lists." + name+ "Postfix";
    
        LOG.debug("load List: " + listImplClassName);
        result = Class.forName(listImplClassName);
        return (YourList) result.newInstance();
    }

Da nicht wirklich klar ist wie ähnlich sich die Klassen im Namen sind und wie die Package-Struktur ist kann man jetzt nicht mehr dazu sagen. Vielleicht ist die Initialisierung auch bei jeder Liste anders, dann macht dieser Ansatz auch schon wieder keinen Sinn.

Nach erneutem aufmerksamen Lesen der Frage lässt es sich doch konkretisieren:

//pseudocode. Sinn sollte klar werden
fori 0...9
    YourList list = getYourListImplInstance("p0"+ i);
    // tests

YourList getYourListImplInstance(String subpackage)
        throws InstantiationException, IllegalAccessException, ClassNotFoundException
{
    Class<?> result = null;
    final String listImplClassName =
            "package.your.lists." + subpackage + ".SoloList";

    LOG.debug("load List: " + listImplClassName);
    result = Class.forName(listImplClassName);
    return (YourList) result.newInstance();
}

 

[werdas34]

Danke für deine Antwort.

Nur denke ich das es eine einfachere Lösung geben muss, vielleicht auch das Die Dateien an bestimmten Stellen abgelegt werden müssen o.ä..

Da der Prof schreibt das wir eine(!) Testklasse schreiben sollen und keiner von uns das von sich heraus so lösen würde.

 

[looparda]

Haben die Listen einen gemeinsamen Basistypen?

 

[mihe7]

Denn meine klare Aufgabe lautet:

So klar ist die Aufgabe nicht. Sollen alle Klassen mit einem JUnit-Test automatisch getestet werden oder darfst Du die Klassen einfach austauschen und neu übersetzen?

 

[werdas34]

Haben die Listen einen gemeinsamen Basistypen?

Der Basistyp ist Integer.

Stellt euch vor es wird eine Aufgabe gestellt und 19 verschiedene Studenten lösen sie.
Alle lösen die Aufgabe richtig. Jedoch jeder anders. (Bei mir gibt es eben Fälle, wo es eben Fehler gibt.)

So klar ist die Aufgabe nicht. Sollen alle Klassen mit einem JUnit-Test automatisch getestet werden oder darfst Du die Klassen einfach austauschen und neu übersetzen?

Es wird nichts erwähnt.
In der Aufgabe steht nur das die die Dateien alle vom Sinn her das selbe machen, nur anders implementiert sind bzw. paar fehlerhaft sind.
Und als Schlusssatz:
"Ihre Aufgabe ist es, eine JUnit Testklasse zu entwickeln, die die fehlerhaften Implementierungen erkennt und die Fehler ausweist."

Also das ist nicht definiert.
Natürlich wäre es angenehmer, wenn alle Klassen automatisch übersetzt werden.

 

[mihe7]

Haben die Quelltexte entsprechende, d. h. unterschiedliche Package-Header oder liegen die einfach nur in den Ordnern?

Der Basistyp ist Integer.

@looparda dürfte gemeint haben, ob die Klassen alle die gleiche Basisklasse erweitern oder das gleiche Interface implementieren.

 

[werdas34]

Haben die Quelltexte entsprechende, d. h. unterschiedliche Package-Header oder liegen die einfach nur in den Ordnern?

Wenn du mit Header meinst, ob und was in den einzelnen Klassen bei package steht. Sie haben und da muss ich mich zu vorhin korrigieren nicht SoloList.p01 bis SoloList.p19 stehen, sondern nur p01 bis p19.
Also ganz oben von der Klasse steht:

package p01;

import ...

public class SoloList extends AbstractList<Integer>{

@looparda dürfte gemeint haben, ob die Klassen alle die gleiche Basisklasse erweitern oder das gleiche Interface implementieren.

Sorry, mein Fehler. Ja sie erben von AbstractList<Integer>.

 

[looparda]

Das ist das eleganteste, das mir dazu einfällt. Da es einen Basistypen gibt wird es sehr einfach:

import org.junit.Assert;
import org.junit.Test;

import static org.hamcrest.CoreMatchers.equalTo;

public class TestLoading {

    @Test
    public void testExecution() throws IllegalAccessException, ClassNotFoundException, InstantiationException {

        for (int i = 1; i <= 2; i++) {
            final String subpackage = getSubpackage(i);
            final AbstractList<Integer> yourList = getListInstance(subpackage);
            final Integer result = yourList.get(0);

            // Replace with actual list implementation tests
            Assert.assertThat(result, equalTo(i));
            System.out.println(result);
        }
    }

    private String getSubpackage(int i) {
        return String.format("p%02d", i);
    }

    private AbstractList<Integer> getListInstance(String subpackage) throws InstantiationException , IllegalAccessException, ClassNotFoundException {

        Class<?> result;
        final String listImplClassName = getClass().getPackage().getName() + "." + subpackage + ".SoloList";

        System.out.println("loading List: " + listImplClassName);
        result = Class.forName(listImplClassName);
        return (AbstractList<Integer>) result.newInstance();
    }

}

public class SoloList extends AbstractList<Integer> {

    @Override
    public Integer get(int index) {
        return 1; //TODO
    }
}
public class SoloList extends AbstractList<Integer> {

    @Override
    public Integer get(int index) {
        return 2; //TODO
    }
}
...

 

[mrBrown]

Um welche Junit-Version gehts denn?

 

[mihe7]

In dem Fall solltest Du auch parametrisierte Tests durchführen können:

@RunWith(Parameterized.class)
public class MyJUnitTest {
    @Parameters
    public static Collection<List<Integer>> data() {
        return Arrays.asList(new p01.SoloList(), new p02.SoloList(),
                new p03.SoloList() // usw. 
        );
    }

    private List<Integer> listToTest;

    public MyJUnitTest(List<Integer> current) {
        listToTest = current;
    }

    @Test
    public void someTest() {
        // use listToTest
    }
}

 

[werdas34]

Das ist das eleganteste, das mir dazu einfällt. Da es einen Basistypen gibt wird es sehr einfach:

import org.junit.Assert;
import org.junit.Test;

import static org.hamcrest.CoreMatchers.equalTo;

public class TestLoading {

    @Test
    public void testExecution() throws IllegalAccessException, ClassNotFoundException, InstantiationException {

        for (int i = 1; i <= 2; i++) {
            final String subpackage = getSubpackage(i);
            final AbstractList<Integer> yourList = getListInstance(subpackage);
            final Integer result = yourList.get(0);

            // Replace with actual list implementation tests
            Assert.assertThat(result, equalTo(i));
            System.out.println(result);
        }
    }

    private String getSubpackage(int i) {
        return String.format("p%02d", i);
    }

    private AbstractList<Integer> getListInstance(String subpackage) throws InstantiationException , IllegalAccessException, ClassNotFoundException {

        Class<?> result;
        final String listImplClassName = getClass().getPackage().getName() + "." + subpackage + ".SoloList";

        System.out.println("loading List: " + listImplClassName);
        result = Class.forName(listImplClassName);
        return (AbstractList<Integer>) result.newInstance();
    }

}

public class SoloList extends AbstractList<Integer> {

    @Override
    public Integer get(int index) {
        return 1; //TODO
    }
}
public class SoloList extends AbstractList<Integer> {

    @Override
    public Integer get(int index) {
        return 2; //TODO
    }
}
...

Vielen Dank.
Sehe ich das richtig, das ich die ganzen Implementierungen am Ende hinzufügen müsste?

Um welche Junit-Version gehts denn?

JUnit 4.

In dem Fall solltest Du auch parametrisierte Tests durchführen können:

@RunWith(Parameterized.class)
public class MyJUnitTest {
    @Parameters
    public static Collection<List<Integer>> data() {
        return Arrays.asList(new p01.SoloList(), new p02.SoloList(),
                new p03.SoloList() // usw.
        );
    }

    private List<Integer> listToTest;

    public MyJUnitTest(List<Integer> current) {
        listToTest = current;
    }

    @Test
    public void someTest() {
        // use listToTest
    }
}

Die Idee gefällt mir besonders gut.
Was ich mich nur frage.
Es wird ein Test durchgeführt auf p01. Dann geht die Liste weiter auf p02. Nur wie starte ich dann den Test erneut?

 

[mihe7]

Nur wie starte ich dann den Test erneut?

Das macht JUnit für Dich Die Fehler werden mit Array-Indizes angezeigt, z. B.

Failed tests: someTest[2] ...

sagt Dir, dass der Fehler beim dritten Element des Arrays aufgetreten ist.

 

[werdas34]

Das macht JUnit für Dich Die Fehler werden mit Array-Indizes angezeigt, z. B.

Failed tests: someTest[2] ...

sagt Dir, dass der Fehler beim dritten Element des Arrays aufgetreten ist.

Ich wollte grade meinen Beitrag editieren, nachdem ich mich mit Parameterized.class beschäftigt habe.

Also erstell ich jetzt normal meine Test und dann läuft das ?

 

[mrBrown]

Also erstell ich jetzt normal meine Test und dann läuft das ?

Ja, du musst nur die entsprechenden Test-Methoden hinzufügen (und den Rest, der in @mihe7 s Code angemerkt ist), und den dann ausführen lassen.

 

[werdas34]

Super. Vielen Dank. Jetzt Funktioniert es so wie ich es vorhatte.

Vielen Dank an alle die mitgewirkt haben.