Yu-Lin Chen created YUNIKORN-3017:
-------------------------------------

             Summary: [UMBRELLA] Refactor E2E test suites to a consistent code 
structure
                 Key: YUNIKORN-3017
                 URL: https://issues.apache.org/jira/browse/YUNIKORN-3017
             Project: Apache YuniKorn
          Issue Type: Improvement
          Components: test - e2e
            Reporter: Yu-Lin Chen
            Assignee: Yu-Lin Chen


The E2E test suites are separated into two files: XXX_suite_test.go and 
XXX_test.go.

The former contains suite-level variable and suite-level setup/teardown. The 
later is focus on test logic and test-level setup/teardown. 

However, some test suites mixed up the separation . For example:
 # 
BeforeSuite/AfterSuite are set in persistent_volume_test.go instead of 
persistent_volume_suite_test.go
 # BeforeSuite/AfterSuite are set in preemption_test.go instead of 
preemption_suite_test.go
 # BeforeSuite/AfterSuite set in basic_scheduling_test.go instead of 
basic_scheduling_suite_test.go
 # BeforeSuite/AfterSuite set in recovery_and_restart_test.go instead of 
recovery_and_restart_suite_test.go
 # BeforeSuite/AfterSuite set in restart_changed_config_test.go instead of 
restart_changed_config_suite_test.go
 # BeforeSuite/AfterSuite set in simple_preemptor_test.go  instead of 
simple_preemptor_suite_test.go
 # BeforeSuite/AfterSuite set in user_group_limit.go instead of 
user_group_limit_suite_test.go

Below test suite are in good shape:
 # admission_controller
 # bin_packing
 # configmap_suite_test
 # foreign_pod
 # gang_scheduling
 # node_resources
 # predicates
 # priority_scheduling
 # queue_quota_mgmt
 # resource_fairness
 # spark_jobs_scheduling

Above is a rough review so I may missed something, 

The main goal of this Jira is to reshape the test code to a consist struture. 
We can create a subtask for each test suite.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@yunikorn.apache.org
For additional commands, e-mail: dev-h...@yunikorn.apache.org

Reply via email to