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

feat: show front-end e2e test report on PR's comment. #1075

Closed
LiteSun opened this issue Dec 19, 2020 · 2 comments
Closed

feat: show front-end e2e test report on PR's comment. #1075

LiteSun opened this issue Dec 19, 2020 · 2 comments
Assignees
Labels
enhancement New feature or request frontend

Comments

@LiteSun
Copy link
Member

LiteSun commented Dec 19, 2020

Feature request

in order to let developers have clear info about why the front-end e2e test failed, we need show the front-end e2e report in each pr's comment, just like Codecov Report.

refer pr: #991

@juzhiyuan juzhiyuan added the wait for update 1. Solution 2. Due date 3. Assignees if needed label Jan 4, 2021
@juzhiyuan juzhiyuan added this to the 2.4 milestone Jan 8, 2021
@LiteSun
Copy link
Member Author

LiteSun commented Jan 12, 2021

I tried it under my fork branch and it worked as shown here.

image

waiting for admin to install Cypress Github app. https://issues.apache.org/jira/projects/APISIX/issues/APISIX-19?filter=allopenissues

@LiteSun LiteSun self-assigned this Jan 12, 2021
@LiteSun LiteSun removed the wait for update 1. Solution 2. Due date 3. Assignees if needed label Jan 12, 2021
@juzhiyuan
Copy link
Member

This issue is relying on Cypress's Write permission, which needs infra's reply.

@juzhiyuan juzhiyuan removed this from the 2.4 milestone Jan 21, 2021
@juzhiyuan juzhiyuan added enhancement New feature or request and removed testcase labels Jan 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request frontend
Projects
None yet
Development

No branches or pull requests

2 participants