-
Notifications
You must be signed in to change notification settings - Fork 8
Description
Overview
In order for Line Coverage to be calculated accurately, we need to tell the autograder where to look for the tests and classes of those tests. Originally, the Phase 5 Unit Tests are stored in a package called client, and the autograder will look for any packages called client, but that feels weird to put a client package in the client module (or maybe I'm wrong).
In addition, there is no starter code for the ServerFacade, so students must create the package and file for where it should go, and if done incorrectly, students can get upset about where they needed to put their unit tests and Server Facade class.
To mitigate these issues, I propose putting the starter code into a "facade" package for both the tests and the ServerFacade. This will be clear that it is a package in client that holds anything Facade related, such as the ServerFacade. Name can be altered if desired. In addition, by adding the facade package with the bare minimum of a ServerFacade class, then students will know exactly where their code should go and they shouldn't run into the issue of having their unit tests fail.
This issue is connected to softwareconstruction240/autograder#398
To Do
- Add a facade package into the chess template code, specifically in the client module.
- This facade package will also contain the bare minimum of a ServerFacade class (not much more than public class ServerFacade{} honestly).