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

Adding useful information to backend E2E test cases #1136

Closed
2 tasks done
juzhiyuan opened this issue Dec 28, 2020 · 7 comments · Fixed by #1319
Closed
2 tasks done

Adding useful information to backend E2E test cases #1136

juzhiyuan opened this issue Dec 28, 2020 · 7 comments · Fixed by #1319
Assignees
Milestone

Comments

@juzhiyuan
Copy link
Member

juzhiyuan commented Dec 28, 2020

Feature request

Please describe your feature

  • Which test case is failed (would better support error highlighting);
  • Which file is this test case in?

cc @idbeta @nic-chen

@juzhiyuan juzhiyuan added this to the 2.4 milestone Dec 28, 2020
@membphis
Copy link
Member

@idbeta @nic-chen what is your opinion?

@membphis
Copy link
Member

related issue? #1140

if yes, then we can close one of them

@nic-chen
Copy link
Member

yes, it's duplicate.
we can use the ginkgo framework to improve the readability of e2e test information.

@nic-chen
Copy link
Member

since bug of e2e failed has fixed, we could put it to M2.4

@idbeta
Copy link
Contributor

idbeta commented Dec 30, 2020

@idbeta @nic-chen what is your opinion?

I agree with @juzhiyuan .

@nic-chen
Copy link
Member

nic-chen commented Feb 2, 2021

finish framework and route part in M2.4

@nic-chen nic-chen reopened this Feb 5, 2021
@nic-chen nic-chen modified the milestones: 2.4, 2.5 Feb 5, 2021
@nic-chen
Copy link
Member

We have another issue #1500 to track. And we have used ginkgo to rewrite many e2e test cases in milestone 2.5.
This issue can be closed now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants