Ms4 final: minor fix on running tests (#76)

* Update README.md * Update and rename README to README.md * Update README.md * Update README.md * Update CHANGELOG.md * Update README.md change testing from root to tests/ * Update tests/README.md * Update tests/README.md * Update tests/README.md * Update README.md --------- Co-authored-by: Farhan Bin Faisal <[email protected]>
UBC-MDS · Dec 16, 2024 · 5f48935 · 5f48935
1 parent 9656176
commit 5f48935
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 9 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -36,7 +36,7 @@
 - **Issue 1:** Latest tag was used for docker. This is not ideal because if the user has latest locally, but there is a newer version on the container registry, then Docker will not pull it.<br>
 **Fix 1:**: No Fix. In our readme, we explicitly tell the user to remove the existing local files first. Therefore, we do not run into this issue. <br>
 
-### Milestone 2 Feedback
+### Milestone 3 Feedback
 - N/A
 
 ### Peer Review Feedback
@@ -61,4 +61,4 @@
 [Commit Link](https://github.com/UBC-MDS/DSCI522-2425-22-wine-chromatic-profile/commit/29e1e5b9427cc202c8119da2a91e5e656b47063e#diff-059ae229d14bfbf15abe1e3587fcff94884f9497fffa3b5bdf6c2ff3e94f44d4R113)
 
 - **Issue 6:** The PDF link in Other Formats under the html report is not working.  <br>
-**Fix 6:**: Not Fixed. Not within scope of this project <br>
+**Fix 6:**: Not Fixed. Not within scope of this project <br>
diff --git a/README.md b/README.md
@@ -110,7 +110,8 @@ where you launched the container, and then type `docker compose rm`
 ```
 
 ### Running the test suite
-Use the same docker compose up command as described in the Running the analysis section above to launch Jupyter lab. Tests are run using the pytest command in the root of the project. More details about the test suite can be found in the tests directory.
+Use the same docker compose up command as described in the Running the analysis section above to launch Jupyter lab. Tests are run using the pytest command in the `tests/` directory. More details about the test suite can be found in `tests/`.
+
 
 # License
 This project was created with the [`MIT License`](LICENSE.md)

diff --git a/tests/README → tests/README.md b/tests/README → tests/README.md
@@ -1,20 +1,24 @@
+Use the bash command below to change directory from root to `tests/`  
+`cd tests`
+
+Run each test one by one in the `tests/` directory:
 ```
+pytest test_validate_column_names.py
+
+pytest test_split_data.py
+
 pytest -v test_random_search.py \
     --train_data=../data/proc/wine_train.csv \
     --test_data=../data/proc/wine_test.csv \
     --pipeline_path=../results/models/wine_pipeline.pickle
 
-pytest test_validate_column_names.py 
-
-pytest test_split_data.py
-
 pytest test_evaluation.py
 ```
 
-Test all at once with the command below: 
+Or run all tests at once in the `tests/` directory with the command below: 
 ```
 pytest -v \
     --train_data=../data/proc/wine_train.csv \
     --test_data=../data/proc/wine_test.csv \
     --pipeline_path=../results/models/wine_pipeline.pickle
-```
+```