Over the last couple of days I have added all the unit tests and the e2e tests for YUNIKORN-317
The test changes have been committed in three steps: unit tests, e2e tests and a fix for a broken unit test/lint issues. In both test changes there is also a limited number of linter clean up changes included. The e2e tests the refactor was limited to using the newly introduced functions and linter clean up. The test and mock needed for these tests have now all been moved to *_test.go files. This means that they do not leave traces in the production code anymore. Changes in the non test code that were made as part of these changes all were fixes of porting issues or for test failures. All tests pass locally, lint check is far cleaner than it has been in a long time. Travis has some issues with the lint check which is a known issue. Please review the changes, comment and feedback is welcome. There is nothing left to do for this task. I'll check after the weekend what needs to be merged into the branch to allow merging into trunk. Cheers, Wilfred On Tue, 20 Oct 2020 at 15:01, Weiwei Yang <abvclo...@gmail.com> wrote: > Hi Wilfred > > Thanks for the update, I will look at the changes this week. > Thanks for the efforts in putting these together. Thank you! > > Weiwei > > > On Mon, Oct 19, 2020 at 6:37 PM Wilfred Spiegelenburg <wilfr...@apache.org> > wrote: > >> Hi, >> >> Just a status update on the big change of removing the cache from the >> core. >> Wangda started working on this a while ago. I have just finished a move of >> that code and some further refactoring into the dev branch YUNIKORN-317. >> >> In the next day or two I will also add the tests to the branch. >> >> Please feel free to start looking at the changes and comment or ask >> questions if you have any. >> >> Wilfred >> >