nus-oss-test / testrepo4

TEAMMATES system is online at
http://teammatesv4.appspot.com
0 stars 0 forks source link

File path case sensitivity #1620

Closed damithc closed 10 years ago

damithc commented 10 years ago

From quahu...@gmail.com on January 22, 2014 16:31:56

Many file paths used in teammates have wrong casing. On some platforms (E.g. Windows), the win32 namespace just ignores casing while in others, different letter case matters. Operations involved in reading from files with wrong path will be affected, for e.g.: some test cases fail on Linux.

java.lang.RuntimeException: java.io.FileNotFoundException: src/test/resources/pages/studentHomeHTMLEmpty.html (No such file or directory)

Original issue: http://code.google.com/p/teammatespes/issues/detail?id=1539

damithc commented 10 years ago

From quahu...@gmail.com on January 22, 2014 00:34:09

I can fix all the failed test cases that have wrong file path, but does Teammates read from files in other operations other than testing?

damithc commented 10 years ago

From dam...@gmail.com on January 22, 2014 01:26:30

If you fix, I think you need to fix the file name in the code, not the actual file name. Hg does not recognize changes to the case in file name. The file name will change in your computer but it will not propagate to the remote repo.

Status: Accepted

damithc commented 10 years ago

From quahu...@gmail.com on February 12, 2014 03:31:50

https://codereview.appspot.com/62040044

Status: ReadyForReview

damithc commented 10 years ago

From arnold.k...@gmail.com on February 12, 2014 05:20:08

This issue was updated by revision 7ef6a19ac097 .

Status: Delivered

damithc commented 10 years ago

From dam...@gmail.com on February 14, 2014 19:02:37

Status: Deployed
Labels: Milestone-V4.87

damithc commented 10 years ago

From arnold.k...@gmail.com on February 26, 2014 01:58:31

This issue was updated by revision 7ef6a19ac097 .

Status: Delivered

damithc commented 10 years ago

From dam...@gmail.com on February 26, 2014 02:49:10

Status: Deployed