Review Tests for Project 8
2 unresolved threads
Closes #2, #3, #4, #5, #6, #9, #10 Related to #7
There were a decent amount of changes (to interfaces and methods) to this one, so I'd appreciate a double check to make sure I didn't break anything or if there are any compatibility issues.
Request to merge
tests
into
master
Last updated by Archie Shahidullah2 2 3 3 import com.google.gson.JsonElement; 4 4 import com.google.gson.JsonParser; 5 import edu.caltech.cs2.interfaces.IDeque; 6 import edu.caltech.cs2.interfaces.IDictionary; 7 import edu.caltech.cs2.interfaces.ISet; 5 import edu.caltech.cs2.interfaces.*; changed this line in version 2 of the diff
- Last updated by Archie Shahidullah
252 290 253 291 IDeque<Location> actualPath = bmg.dijkstra(start, target); 254 292 List<Long> actualPathIDs = new ArrayList<>(); 255 293 if (actualPath != null) { 
This method just tests whether
dijkstraactually produces a path without buildings, rather than checking it for correctness against the trace. It just uses the start and target locations from trace. So I just check for null to only check existing paths. Should I check it for correctness (since it's already done in the stress test)?changed this line in version 3 of the diff
mentioned in commit 183c06a4