result:path missing filename could use better error report

Bug #999137 reported by Tony Juricic
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
NUnitLite
Fix Released
Medium
Charlie Poole

Bug Description

I understood PATH to be onlu the directory path for result option.
-result:C:\Users\MyUserId\AppData\Local\Temp

Exceptions thrown are hard to interpret as missing file name. Perhaps a default file name can be appended? Like:
if(System.IO.Path.FileName(path)==null)
     System.Io.Path.Combine(path, "NUnitLiteResults.xml");

Related branches

Revision history for this message
Charlie Poole (charlie.poole) wrote :

We should at least intercept the error and give a reasonable message. In addition, make it clearer in the help message.

Changed in nunitlite:
status: New → Triaged
importance: Undecided → Medium
Changed in nunitlite:
assignee: nobody → Charlie Poole (charlie.poole)
milestone: none → 0.8
Revision history for this message
Charlie Poole (charlie.poole) wrote :

After examination, I decided to leave the error message as is but to clarify the meaning of the parameter in the help.

Changed in nunitlite:
status: Triaged → Fix Committed
Changed in nunitlite:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.