ladnerjonas / pgdp-tests-ws21-22 Goto Github PK

Homework Number W09H01

Citation of the homework instruction:
insert() Falls ein gleiches Tupel bereits vorhanden ist, wird das übergebene Tupel verworfen. Es soll das Tupel zurückgegeben werden, welches letztendlich im Pool gespeichert ist.

Additional Information:
If I understand assertEquals() correctly, it assesses the equality of the given tuples. For that, it uses our implementation of tuple.equals(other). What we want though, is an assessment of identity.

This should fail, but does not:
assertEquals(tuple0Duplicate, pool0.insert(tuple0Duplicate));

I suggest, we change the test according to this example:

assertEquals(tuple0, pool0.insert(tuple0Duplicate)); //old

/* assertEquals(expected, test of identity) */
assertEquals(true, pool0.insert(tuple0Duplicate) == tuple0);
assertEquals(false, pool0.insert(tuple0Duplicate) == tuple0Duplicate);

The provided Maze impossible-2 is actually solvable
The maze:

WEWWW
W W W
W W W
W W W
WXWWW

Homework Number (example: W04H01):
W12H01
Citation of the homework instruction:

Additional Information:
testMultiplication and addition are failing for me (the first ones) but the other pass.So i wanted to ask what do they test exactly

Homework Number (example: W06H01):

Citation of the homework instruction:
toString: TradeOrder: Die⎵Bestellung(Einzeln)⎵hat⎵ein⎵Gesamtgewicht⎵von⎵g⎵und⎵kostet⎵PD.

Der Wert eines PinguFood-Objekts berechnet sich durch weight*ppg

Additional Information:
Ich habe das so verstanden, dass totalCost nicht die Kosten pro Gramm sind, wie es in den Tests suggeriert wird, sondern der Preis der ganzen Bestellung, also für 200g mit einem Preis von 2, dann 400PD.

Englisch:
I understood it to mean that totalCost is not the cost per gram, as suggested in the tests, but the price of the entire order, i.e. for 200g with a price of 2, then 400PD.

edit: Der Wert eines PinguFood-Objekts berechnet sich durch weight*ppg

Homework Number: W08H01

Citation of the homework instruction: -

Additional Information:
Depending on your homework start date one line (number 61) in MazeParser has to be replaced:

Maze.TileState[][] tiles = new Maze.TileState[lines[0].length()][lines.length];

to

Maze.TileState[][] tiles = new Maze.TileState[lines.length][lines[0].length()];

Keep in mind, that you should not push these changes. No warranty is given.

If 2-3 tests fail, but the difference menu of your IDE shows something like that:

or even says "Contents are identical".

Possible Solution:

Please try to manually verify your output and as long as your method does fit the requirements the failing tests should be ignored.

Homework Number W06H01:

TradeOrderAndHeirsTest:
line 129 should expect true since an order should not be supplied if the order is already fulfilled, therefore adding order in line 127 should not make a difference.

Hi, ich verstehe nicht warum beim AuthorTest bei testEquals() beim letzten Testfall null für birthday angegeben wird. Das führt doch normalerweise zu einer NullPointerException.

assertFalse(AUTHOR.equals(new Author("Max", "Mustermann", "Pinguinstraße 1",
                "[email protected]", null)));

In der Angabe wird jedoch nirgends der Fall beschrieben, dass birthday (oder ein anderer Parameter) null ist, warum wird er dann hier auf so ein Fall getestet? Oder hab ich was falsch verstanden?

LG

Wenn man die Methode getAgeTest() in AuthorTest zwischen 0:00Uhr und 1:00Uhr ausführt, schägt diese, trotz richtiger Implementierung, fehl.

Dies liegt am zweiten Test, welcher wie folgt aussieht:

Author author2 = new Author("Max", "Mustermann", "Garching-City", "[email protected]", new Date(LocalDate.now().getDayOfMonth(), LocalDate.now().getMonthValue(), LocalDate.now().getYear()));
assertEquals(0, author2.getAge());

Bei diesem Test wird LocalDate.now() benutzt, um das aktuelle Datum zu bekommen. Allerdings wird dieses Datum anhand der lokalen Zeitzone gebildet (UTC+1). Die Klassen sollen allerdings in UTC +0 getestet werden.

Um das Problem zu beheben, schlage ich vor LocalDate.now() mit LocalDate.now(ZoneOffset.UTC) auszutauschen, sodass die Klasse so aussieht:

import java.time.ZoneOffset;
...
public class AuthorTest {
    ...
    Author author2 = new Author("Max", "Mustermann", "Garching-City", "[email protected]", new Date(LocalDate.now(ZoneOffset.UTC).getDayOfMonth(), LocalDate.now(ZoneOffset.UTC).getMonthValue(), LocalDate.now(ZoneOffset.UTC).getYear()));
    assertEquals(0, author2.getAge());
    ...
}

Alternativ könnte auch die Methode today() von der Klasse Date benutzt werden:

public class AuthorTest {
    ...
    Author author2 = new Author("Max", "Mustermann", "Garching-City", "[email protected]", new Date(Date.today().getDay(), Date.today().getMonth(), Date.today().getYear()));
    assertEquals(0, author2.getAge());
    ...
}

W12H01 tests:

Is it possible that the tests don't consider the following scenario? https://zulip.in.tum.de/#narrow/stream/893-PGdP-W12H01--.20Parallel.20PinguMath/topic/.E2.9C.94.20ComputeThreadCounter
It seems like testMinDim, testMultithreadingAddition, and testMultithreadingMultiplication expect the getComputerThreadCounter to be 4/8, but it's possible to have 5/9 if a thread has been started as described in the Zulip post.

Sorry if this has been mentioned somewhere else, and thanks for the tests :)

Homework Number W0701

line 109.
Meiner Meinung nach sollte size 5 und nicht 4 sein. Wir haben vor line 108 3 LinkedDocuments und 1 DummyLinked(Adresse abc) in der Sammlung. Allerdings entfernt die add Methode keine Dummy Elemente? Zumindest kann ich das der Aufgabenstellung nicht entnehmen. Folglich sollte also in der Collection einfach ein LinkedDocument eingefügt werden, welches die gleiche Addresse wie das DummyElement hat.

UnitTests W02H02, MarketEndDayPropertiesTest:
Assert.assertEquals("Der Laden der Saleuine Claudia und Karl-Heinz hat am 2. Tag 85.0PD eingenommen." + System.lineSeparator() + "Dafür wurden 1 Krustentiere, 2 Sardellen und 3 Sardinen verkauft." + System.lineSeparator() + "Insgesamt hat der Laden 255.0PD eingenommen.\n" + System.lineSeparator(), outputStreamCaptor.toString().substring(0, (outputStreamCaptor.toString()).length() - 2));

Nach dem '\n' kommt noch ein lineseparator und demnach werden 2 leere Zeilen nach der Ausgabe erwartet, auch wenn in der Aufgabenstellung steht, dass man nur eine haben sollte.

Homework Number: W04H01

In der Spezifikation steht, dass isCentralBin bei Eingaben bis n kleiner gleich 10¹⁷ funktionieren soll. Getestet wird hier aber ein Fall größer als 10¹⁷.

Citation of the homework instruction:
[...]
isCentralBin:
[...]
Eingaben mit n≤10¹⁷ müssen korrekt und effizient behandelt werden.

Testfall
Arguments.of(118264581564861424L, true)

In meinem Fall macht tatsächlich genau das den Unterschied^^

Over here, you refill the hashset:

But shouldn't the elements remain in there?

Homework Number (example: W06H01):

PinguFoodLogisticsTest.java line 47:

wrong assumption about wasted food, statistic shall not be set to zero after clearing orderBook.
Right would be:
"Bisher konnten 15 Tiere mit einem Gesamtgewicht von 2373g nicht verwertet werden.
Claudia und Karl-Heinz ist dadurch ein Profit von 35595PD entgangen."

Homework Number: W08H01

Citation of the homework instruction:

static Path solveMazeFrom(Maze maze, Position position):
[....]
Sollte ein solcher Pfad nicht mehr existieren, soll null zurückgegeben werden.

Additional Information:

Maze
[....]
du kannst aber davon ausgehen, dass der exit immer vom entrance aus erreichbar ist.

Position can be != Entrance, and thus, this case has to be tested aswell.
#33

Thanks @mani 🏅