fix: Set default value to output parameters if suspend node timeout. Fixes #12230

[jswxstw]

Fixes #12230

Motivation

Node is marked to Succeeded when suspension is expired, but the value of node outputs parameters was not filled up with ValueFrom.Default if ValueFrom.Supplied != nil.

Modifications

Set default value to output parameters if suspend node timeout or mark the suspend node to error if raw output parameter has not been set and does not have a default value.

Verification

local test and add e2e test

[tczhao]

The test-corefunctional, minimal was succeed before the last commit 2317852,
The last commit only alters the functional test, the test-corefunctional, minimal failure is not an issue

[tooptoop4]

seems all lint on new prs are failing:

level=info msg="Execution took 3m43.470673472s"
			assert.Equal(t, 1, len(status.Outputs.Parameters))
			^
Error: test/e2e/suspend_test.go:124:4: len: use assert.Len (testifylint)
			assert.Equal(t, 1, len(status.Inputs.Parameters))
			^
make: *** [Makefile:466: lint] Error 1
Error: Process completed with exit code 2.

[MasonM]

@tooptoop4 I entered a PR to fix this: #13853

[kush-work]

@terrytangyuan @jswxstw Is there any chance to release a patch with the above fix?

[jswxstw]

@terrytangyuan @jswxstw Is there any chance to release a patch with the above fix?

@kush-work I think this is a reasonable request, but I don't have the permission to do that. @Joibel Could you please take a look at this?