-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Integrate test script and test data into the dev guide #732
Comments
As discussed with Prof, It seems like a good idea to take in the file path as a command line argument, if given, and if it's valid, load the test file else load the default file. This is would be preferred over creating an entire command since this function is not intended for users. What do you think @Zhiyuan-Amos @yamidark @Rinder5 ? |
One limitation of this approach is that it can be used for manual testing only, am I right? |
would you only be able to load in 1 test file? |
Yup! And it can only be done using the jar file.
That should be the case since the use case is such that during the manual testing process, the student would download the other team's jar file along with the test file and then load it into the application using the command line. So it only happens once during the launch of the application. I believe doing it this way would also result in the file being erased from the application once it closes though. |
Sounds good! |
Yep sounds good to me too. |
Currently, we ask students to add a test script and a test data file near the final submission. We can integrate it to the dev guide (e.g. as an appendix), to reduce the number files that need to be submitted. We can also add a command to load data from a file so that there is a legit reason to include a test data file in the code base.
The text was updated successfully, but these errors were encountered: