Skip to content

Commit

Permalink
ci: run API tests on PostgreSQL and add test for unicode escaping (#1…
Browse files Browse the repository at this point in the history
…4011)

Signed-off-by: Mason Malone <[email protected]>
  • Loading branch information
MasonM authored Dec 22, 2024
1 parent 1567ed7 commit abdd9da
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 1 deletion.
3 changes: 3 additions & 0 deletions .github/workflows/ci-build.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -238,6 +238,9 @@ jobs:
- test: test-api
profile: mysql
use-api: true
- test: test-api
profile: postgres
use-api: true
- test: test-cli
profile: mysql
use-api: true
Expand Down
7 changes: 6 additions & 1 deletion test/e2e/argo_server_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -1375,7 +1375,8 @@ spec:
templates:
- name: run-archie
container:
image: argoproj/argosay:v2`).
image: argoproj/argosay:v2
args: [echo, "hello \\u0001F44D"]`).
When().
SubmitWorkflow().
WaitForWorkflow(fixtures.ToBeArchived).
Expand Down Expand Up @@ -1527,6 +1528,10 @@ spec:
j.
Path("$.metadata.name").
NotNull()
j.
Path("$.spec.templates[0].container.args[1]").
// make sure unicode escape wasn't mangled
IsEqual("hello \\u0001F44D")
j.
Path(fmt.Sprintf("$.metadata.labels[\"%s\"]", common.LabelKeyWorkflowArchivingStatus)).
IsEqual("Persisted")
Expand Down

0 comments on commit abdd9da

Please sign in to comment.