Depreciated Map Constructors

[S010MON]

DirectionFollowAgentTest @Jolijn2020
RayFeedbackTest @MatthewFoster02
WallFollowAgentTest @laurajogi

These tests all use a constructor for the map that is causing significant tech debt. This constructor will be depreciated in phase 3 and these tests will need to be rewritten properly.

Note, some of you are using global variables in multiple tests. This can cause dependency issues due to the asynchronous nature of the Juinit runner. Each test should run in complete isolation, to set up a test environment multiple times, an @BeforeAll and @AfterAll tag can be used to construct and destruct the environment.

[Jolijn2020]

DirectionFollowAgentTest is fixed in branch 323-depreciated-map-constructors

(RayFeedbackTest too)