nus-cs2113-AY2223S1 / forum

4 stars 0 forks source link

JUnit test, Assert are not triggered in the tp progress #23

Closed Than-Duc-Huy closed 1 year ago

Than-Duc-Huy commented 1 year ago

We have tried implementing JUnit test, Assert by

But it does not trigger the tp progress check scripts? I wonder if we are doing it correctly?

This information might be relevant as it is mentioned in #22 We removed the .seedu part of the folder structure: src/main/java/tp, src/test/java/tp. Is removing .seedu the reason?

okkhoy commented 1 year ago

Please link your team repo for me to check.

Than-Duc-Huy commented 1 year ago

https://github.com/AY2223S1-CS2113-T18-2/tp For the test, so far we have only make this test https://github.com/AY2223S1-CS2113-T18-2/tp/blob/master/src/test/java/recipeditor/parser/ParserTest.java

Than-Duc-Huy commented 1 year ago

This problem comes from me. I didn't change my user.name to match my github username. Hence the RepoSense noted that I haven't contributed any code at all. I am sorry for the inconvenience!