[YUNIKORN-886] Fix e2e tests after YUNIKORN-794

• Type: Test
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.12.1
• Component/s: shim - kubernetes
• Labels: pull-request-available
• Target Version: 0.12.1
• Assignee: Peter Bacsko
• Reporter: Peter Bacsko
• Created: 21/Oct/21 12:30
• Updated: 21/Jan/22 21:48
• Resolved: 25/Oct/21 11:07

Description

YUNIKORN-794 changed the REST API, which broke the e2e tests:

/home/runner/work/incubator-yunikorn-k8shim/incubator-yunikorn-k8shim/test/e2e/queue_quota_mgmt/queue_quota_mgmt_test.go:34
  Verify_Queue_Quota_Allocation [It]
  /home/runner/work/incubator-yunikorn-k8shim/incubator-yunikorn-k8shim/test/e2e/queue_quota_mgmt/queue_quota_mgmt_test.go:69

  Test Panicked
  interface conversion: interface {} is nil, not string
  /opt/hostedtoolcache/go/1.15.15/x64/src/runtime/iface.go:261

  Full Stack Trace
  github.com/apache/incubator-yunikorn-k8shim/test/e2e/queue_quota_mgmt_test.glob..func3.2()
  	/home/runner/work/incubator-yunikorn-k8shim/incubator-yunikorn-k8shim/test/e2e/queue_quota_mgmt/queue_quota_mgmt_test.go:105 +0x29ff
  github.com/onsi/ginkgo/internal/leafnodes.(*runner).runSync(0xc000038360, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
  	/home/runner/go/pkg/mod/github.com/onsi/ginkgo@v1.16.5/internal/leafnodes/runner.go:113 +0xa3
[...]
  github.com/apache/incubator-yunikorn-k8shim/test/e2e/queue_quota_mgmt_test.TestStateAwareAppScheduling(0xc0000ee780)
  	/home/runner/work/incubator-yunikorn-k8shim/incubator-yunikorn-k8shim/test/e2e/queue_quota_mgmt/queue_quota_mgmt_suite_test.go:83 +0x156
  testing.tRunner(0xc0000ee780, 0x1b75ca8)
  	/opt/hostedtoolcache/go/1.15.15/x64/src/testing/testing.go:1123 +0xef
  created by testing.(*T).Run
  	/opt/hostedtoolcache/go/1.15.15/x64/src/testing/testing.go:1168 +0x2b3
------------------------------

Problem is that we expect strings like "maxcapacity", "usedcapacity", etc. but the JSON response was slightly modified.

Issue Links

• is caused by: YUNIKORN-794 Fix a few typos in the REST API about queue capacity
• links to: GitHub Pull Request #319