Skip to content

Conversation

@claudiamurialdo
Copy link
Collaborator

No description provided.

@claudiamurialdo claudiamurialdo temporarily deployed to external-storage-tests September 28, 2022 18:09 Inactive
@genexusbot
Copy link
Collaborator

Cherry pick to beta success

@claudiamurialdo claudiamurialdo changed the title Submit tests Improve submit tests so when running in parallel with web tests they also call the WaitForAll routine. Sep 28, 2022
@claudiamurialdo claudiamurialdo changed the title Improve submit tests so when running in parallel with web tests they also call the WaitForAll routine. Make submit tests deterministic when running in parallel with web tests Sep 28, 2022
@claudiamurialdo claudiamurialdo merged commit a0bc35d into master Sep 28, 2022
@claudiamurialdo claudiamurialdo deleted the submit-tests branch September 28, 2022 19:04
claudiamurialdo added a commit that referenced this pull request Sep 28, 2022
…ts (#693)

* Test httpcontext on submit unit test.

* Set test of submit to run serially exclusively to ensure GxContext.IsHttpContext is false.

* Fix for running submit test serialized.

* Disable parallelization between MiddlewareTests and SubmitTests (requires running on non-web context)

* Force a ThreadUtil.WaitForEnd(); at the end of SubmitTest to avoid Failed test when running in parallel with MiddlewareTest.

* Remove and sort usings.

* Minor comment fix.

(cherry picked from commit a0bc35d)
claudiamurialdo added a commit that referenced this pull request Sep 29, 2022
…ts (#693)

* Test httpcontext on submit unit test.

* Set test of submit to run serially exclusively to ensure GxContext.IsHttpContext is false.

* Fix for running submit test serialized.

* Disable parallelization between MiddlewareTests and SubmitTests (requires running on non-web context)

* Force a ThreadUtil.WaitForEnd(); at the end of SubmitTest to avoid Failed test when running in parallel with MiddlewareTest.

* Remove and sort usings.

* Minor comment fix.

(cherry picked from commit a0bc35d)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants