Added the scaned options for the old and the new files - #145
pratik0316 wants to merge 3 commits into
Conversation
Signed-off-by: Pratikrocks <pratikrocks.dey11@gmail.com>
9abfcb4 to
16ad933
Compare
|
@MaJuRG I have inserted scanned options for new and old file in the deltacode output ,I have not modified the test case so due to this some of the test cases(3 test cases in both windows and osx build ) are not successful as the expected and the actual results in the output are different to some extent(regarding the output in the json file) .Sir if you would kindly review this PR and suggest changes, I would be modifying the test cases too. |
…lta json output scan Signed-off-by: Pratikrocks <pratikrocks.dey11@gmail.com>
ae8e59f to
8d181b2
Compare
steven-esser
left a comment
steven-esser
left a comment
There was a problem hiding this comment.
Is this related to a ticket at all? I am not sure we want this...
|
Thanks sir for the review. |
|
In this Pr my two cases are failing in both Linux and osx build out of 155 test cases |
|
@Pratikrocks You will need to regen those two particular test files since you have added some additional output |
|
okay @MaJuRG I am trying to sort it out |
|
Actually the error is generated due to mismatch in the in the result file |
|
The test files on Travis are taken from the ones in the codebase. |
|
Closing due to stagnation. |
2 participants
In this PR I have added a method to add the scanned options in the new and old file, These options will be available shown in the json file of the deltacode json file output.
The new json file will now contain an additional field of
scanned_optionsin it.So this is my new json file output containing the scanned options present in the new and old file scans
This PR is addressed to the issue of #2