Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix the workspace integration tests that need a username in DB first #9627

Closed
Tracked by #8799
jenting opened this issue Apr 28, 2022 · 0 comments · Fixed by #9628
Closed
Tracked by #8799

Fix the workspace integration tests that need a username in DB first #9627

jenting opened this issue Apr 28, 2022 · 0 comments · Fixed by #9628
Assignees

Comments

@jenting
Copy link
Contributor

jenting commented Apr 28, 2022

Bug description

The following integration tests need a username in DB first, however, the integration test should be able to mock the username into the DB rather than have a real username in DB.

  • TestGitHubContexts
  • TestWorkspaceInstrumentation
  • TestGitActions

Steps to reproduce

$ go test -v ./... \
  -kubeconfig=/home/gitpod/.kube/config \
  -namespace=<namespace>

The above tests are skipped because of missing the username. The error message would be

cannot start server: "cannot access Gitpod server API: \"no suitable user found: make sure there's at least one non-builtin user in the database (e.g. login)\""

Even if we provide the username to it, the integration tests could be failed because the username doesn't in the DB.

$ go test -v ./... \
  -kubeconfig=/home/gitpod/.kube/config \
  -namespace=<namespace>
  -username=integration-test

Ref werft job

Workspace affected

No response

Expected behavior

No response

Example repository

No response

Anything else?

No response

@jenting jenting self-assigned this Apr 28, 2022
@jenting jenting moved this to In Progress in 🌌 Workspace Team Apr 28, 2022
@jenting jenting changed the title Fix broken integration test TestGitActions Fix the integration tests that needs username May 1, 2022
@jenting jenting changed the title Fix the integration tests that needs username Fix the integration tests that needs username in DB first May 1, 2022
@jenting jenting changed the title Fix the integration tests that needs username in DB first Fix the integration tests that need username in DB first May 1, 2022
@jenting jenting changed the title Fix the integration tests that need username in DB first Fix the workspace integration tests that need username in DB first May 1, 2022
@jenting jenting changed the title Fix the workspace integration tests that need username in DB first Fix the workspace integration tests that need a username in DB first May 1, 2022
Repository owner moved this from In Progress to Done in 🌌 Workspace Team May 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant