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

Change the E2E class Fixture to allow parallel running #2049

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

xuzhg
Copy link
Member

@xuzhg xuzhg commented Feb 13, 2020

Issues

This pull request fixes issue #xxx.

Description

Briefly describe the changes of this pull request.

Checklist (Uncheck if it is not completed)

  • Test cases added
  • Build and test with one-click build and test script passed

Additional work necessary

If documentation update is needed, please add "Docs Needed" label to the issue and provide details about the required document change in the issue.

@xuzhg xuzhg force-pushed the E2ETestFrameWorkChange branch 6 times, most recently from 2efc1c8 to e5bab38 Compare February 18, 2020 22:06
@xuzhg xuzhg added the Ready for review Use this label if a pull request is ready to be reviewed label Feb 26, 2020
@xuzhg xuzhg added this to the 7.4 Beta milestone Feb 26, 2020
@xuzhg xuzhg modified the milestones: 7.4 Beta, vNext Mar 2, 2020
@Sreejithpin Sreejithpin removed the Ready for review Use this label if a pull request is ready to be reviewed label Jul 21, 2020
@odero
Copy link
Contributor

odero commented Sep 23, 2020

@xuzhg, @Sreejithpin is anyone still working on this?

@xuzhg
Copy link
Member Author

xuzhg commented Sep 23, 2020

@xuzhg, @Sreejithpin is anyone still working on this?

I'd like to keep it open. Some E2E tests case runs conflict because some static settings has conflict when running parallel. I'd like to continue to investigate it until i have the time.

Another part is that we can move to WebApplicationFactory.

@mikepizzo mikepizzo removed the request for review from Sreejithpin January 5, 2021 21:42
@mikepizzo mikepizzo marked this pull request as draft January 5, 2021 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants