Closed khansaad closed 10 months ago
@khansaad - Have you run the functional test suite?
@khansaad - Have you run the functional test suite?
Yes, attached result of the same:
~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Overall summary of the tests ~~~~~~~~~~~~~~~~~~~~~~~
Total time taken to perform the test 1507 seconds
Total Number of test suites performed 1
Total Number of tests performed 350
Total Number of tests passed 161
Total Number of tests failed 189
[functional_tests.zip](https://github.com/kruize/autotune/files/13559543/functional_tests.zip)
@khansaad - Have you run the functional test suite?
Re-Ran the tests after merging the latest code, now the records are matching with 21 rm image
~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Overall summary of the tests ~~~~~~~~~~~~~~~~~~~~~~~
Total time taken to perform the test 3029 seconds
Total Number of test suites performed 1
Total Number of tests performed 350
Total Number of tests passed 236
Total Number of tests failed 114
@khansaad - Have you run the functional test suite?
Re-Ran the tests after merging the latest code, now the records are matching with 21 rm image
~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Overall summary of the tests ~~~~~~~~~~~~~~~~~~~~~~~ Total time taken to perform the test 3029 seconds Total Number of test suites performed 1 Total Number of tests performed 350 Total Number of tests passed 236 Total Number of tests failed 114
@khansaad - I see the duplicate records fix & tests has been merged into 0.0.19 & master branch #1058 and #1051, I don't see the test changes in mvp_demo. Can you please check with Dino on this. Meanwhile, you can run the test manually with your changes.
This PR fixes #998