Ticket #6896 (closed: fixed)
Unit test for ModifyDetectorDotDatFile
Reported by: | Karl Palmen | Owned by: | Karl Palmen |
---|---|---|---|
Priority: | major | Milestone: | Release 2.6 |
Component: | Framework | Keywords: | |
Cc: | anders.markvardsen@… | Blocked By: | |
Blocking: | Tester: | Nick Draper |
Description
Change History
comment:1 Changed 7 years ago by Nick Draper
- Status changed from new to assigned
- Owner set to Karl Palmen
comment:2 Changed 7 years ago by Karl Palmen
There does not seem to be an algorithm that loads a DetectorDotDat file. This would be very useful in the test, so we can verify that the DetectorDotDeat file was modified correctly.
comment:3 Changed 7 years ago by Karl Palmen
Anders has told me that LoadDetectorInfo will load a DetectorDotDat file.
comment:4 Changed 7 years ago by Karl Palmen
LoadDetectorInfo has a unit test, which will be useful for creating this desired unit test.
comment:6 Changed 7 years ago by Karl Palmen
Improve wiki re #6896
Signed-off-by: Karl Palmen <karl.palmen@…>
Changeset: c513d5c7a605427c5cbb14d1b32b28c5a297dabd
comment:10 Changed 7 years ago by Karl Palmen
Add skeleton unit test re #6896
Just tests initialization of the algorithm.
Signed-off-by: Karl Palmen <karl.palmen@…>
Changeset: b49907b2a93cb469894e351969b809b99738ebaf
comment:11 Changed 7 years ago by Karl Palmen
Test header of Output file re #6896
Signed-off-by: Karl Palmen <karl.palmen@…>
Changeset: 3e3ede101e11d18b8b47e14f5e6e9e63badcab93
comment:12 Changed 7 years ago by Karl Palmen
Modify and simplify test file re #6896
Signed-off-by: Karl Palmen <karl.palmen@…>
Changeset: d68e5c208552a7040ea6997ab88e70c5168d1edd
comment:13 Changed 7 years ago by Karl Palmen
Check start of 3rd line of output file re #6896
Signed-off-by: Karl Palmen <karl.palmen@…>
Changeset: 4c68b30fc2aecce1b6d9c68322db37d7f9ced4f0
comment:15 Changed 7 years ago by Karl Palmen
Test some of output file contents re #6896
Signed-off-by: Karl Palmen <karl.palmen@…>
Changeset: 4d4d97b481d24ee8379a9c939e9c46fcffd27afa
comment:16 Changed 7 years ago by Karl Palmen
- Status changed from inprogress to verify
- Resolution set to fixed
To test review the code. Check that there is a unit test ModifyDetectorDotDatFileTest and it tests the essential features of the algorithm as described in the wiki. You may also run the test to check it works on your system.
comment:17 Changed 7 years ago by Nick Draper
- Status changed from verify to verifying
- Tester set to Nick Draper
comment:18 Changed 7 years ago by Nick Draper
Code review looks good. Test is present and takes only 0.12 seconds.
comment:19 Changed 7 years ago by Nick Draper
- Status changed from verifying to closed
Merge remote-tracking branch 'origin/feature/6896_new_unit_test'
comment:20 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 7742