UI-to-taskList index mapping causes "/remove" tests to fail
Closed this issue · 3 comments
GoogleCodeExporter commented
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
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
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
Mapping implemented as of V0.4.
Original comment by ready...@gmail.com on 9 Nov 2012 at 3:52
- Changed state: Fixed