search

Qinfeiii / cs2103aug12-w10-4s

Automatically exported from code.google.com/p/cs2103aug12-w10-4s
0 stars 0 forks source link

UI-to-taskList index mapping causes "/remove" tests to fail #26

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1. Run Delete() test in CommandProcessorTest

What is the expected output? What do you see instead?
The test should pass. It fails.

Please use labels and text to provide additional information.
It seems that the reason is that inside the CommandProcessor.ExecuteRemove 
method, the IndexMap doesn't have a UI to match to.

Original issue reported on code.google.com by pshing...@gmail.com on 30 Oct 2012 at 5:16

GoogleCodeExporter commented 9 years ago 
Just assign a dictionary with direct one-to-one mapping. (DebugMode uses this 
approach.)

Original comment by ready...@gmail.com on 30 Oct 2012 at 11:31

GoogleCodeExporter commented 9 years ago 
Potential fix in unit test in rev 274.
A better solution would be to have the mapping to return a default one-to-one 
mapping if no mappings are supplied (requires changes to Command Processor)

Original comment by ready...@gmail.com on 31 Oct 2012 at 1:57

GoogleCodeExporter commented 9 years ago 
Mapping implemented as of V0.4.

Original comment by ready...@gmail.com on 9 Nov 2012 at 3:52