Info: Do not just add a e2e test

[hzxuzhonghu]

For small features or bug fixes, if it can be tested by a UT, do not add e2e again.

As one e2e case can cause 20+ seconds, it will make the CI takes more time.

[TommyLike]

@hzxuzhonghu would you help on setting up the UT framework and moving some of the them?

[k82cn]

/kind feature
/priority high

[hzxuzhonghu]

@hzxuzhonghu would you help on setting up the UT framework and moving some of the them?

sure, but i have been working on other stuff these days. May have bandwidth in the middle of next week.

[k82cn]

/assign @asifdxtreme , please help on that.

[asifdxtreme]

/cc @thandayuthapani start writing a basic and generic framework which can be used through out the project , you can start with job package for now

[thandayuthapani]

Have written UT cases for two files in Job package, ref #201 & #213
Files in Job package

• job_controller_plugins
• job_controller_handler
• job_controller_actions

[thandayuthapani]

UT for other packages are to be added, listing out other packages to which UT cases to be added.

• pkg/cli
• pkg/controllers/queue
• pkg/controllers/garbagecollector
• pkg/controllers/apis
• pkg/controllers/cache
• pkg/controllers/job/state

[k82cn]

@thandayuthapani , please also help to provide code coverage of each package :)

[asifdxtreme]

UT has been added for all the packages and the current coverage for UT is around 61%, In my opinion we can go ahead and close this issue

[hzxuzhonghu]

Can we remove some e2e cases, which is already covered by e2e?

[asifdxtreme]

Can we remove some e2e cases, which is already covered by e2e?

We went through all E2E test case, In our opinion all are required as of now, the test scenarios for UT are restricted to a particular functions/methods whereas E2E is tesing the complete flow for each scenario, so the both have different scope while testing. Going ahead we need to add more E2E's to cover all scenario, I don't think so we can reduce the E2E.