Skip to content

Commit

Permalink
DeveloperGuide: Change setup to use default test configuration (nus-c…
Browse files Browse the repository at this point in the history
…s2103-AY1718S1#87)

The working directory needs to be changed for the tests to run properly.
Therefore, the developer guide instructs the developer to modify the
generated test configuration.

If the IDE generates another new test configuration, the settings needs
to be changed again, otherwise the tests will fail again.

Let's change the instructions to ask the developers to modify the
default test configuration instead of the generated one.
  • Loading branch information
yamgent authored Sep 10, 2017
1 parent 0c99fd0 commit f714ba6
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions doc/DeveloperGuide.md
Original file line number Diff line number Diff line change
Expand Up @@ -35,9 +35,10 @@
10. Run all the tests (right-click the `test` folder, and click `Run 'All Tests'`)
11. Observe how some tests fail. That is because they try to access the test data from the wrong directory (the working directory is expected to be the root directory, but IntelliJ runs the test with `test\` as the working directory by default). To fix this issue:
1. Go to `Run` -> `Edit Configurations...`
2. On the list at the left, ensure that `All in test` is selected
3. Under `Configuration`, change the `Working directory` to the `addressbook-level3` folder
4. Click `OK`
2. On the list at the left, expand `JUnit`, and remove all the test configurations (e.g. `All in test`) by selecting it and clicking on the '-' icon at the top of the list
3. Expand `Defaults`, and ensure that `JUnit` is selected
4. Under `Configuration`, change the `Working directory` to the `addressbook-level3` folder
5. Click `OK`
12. Run the tests again to ensure they all pass now.

## Design
Expand Down

0 comments on commit f714ba6

Please sign in to comment.