Skip to content
This repository has been archived by the owner on Dec 8, 2022. It is now read-only.

Fix wait service teardown race condition #1739

Merged
merged 30 commits into from
Jun 29, 2018

Conversation

blackbaud-conorwright
Copy link
Contributor

Resolves: #1276

Made the skywait service's pageWait teardown yield using a timeout so that the creation of the pagewait can complete before being torn down.

Needs to yield so that wait creation can finish
before it is dismissed in the event of a race.
*/
setTimeout(() => {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Possible to reference setTimeout from SkyWindowRefService, just for consistency?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done :)

@codecov-io
Copy link

codecov-io commented Jun 22, 2018

Codecov Report

Merging #1739 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1739      +/-   ##
==========================================
+ Coverage   99.98%   99.98%   +<.01%     
==========================================
  Files         410      410              
  Lines        8527     8530       +3     
  Branches     1251     1252       +1     
==========================================
+ Hits         8526     8529       +3     
  Misses          1        1
Impacted Files Coverage Δ
src/modules/wait/wait.service.ts 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7ef56d7...cc292c0. Read the comment docs.

@Blackbaud-SteveBrush
Copy link
Member

@blackbaud-conorwright The unit tests are failing for this one.

Also: Is it possible to create a unit test to confirm that the race condition is no longer happening?

@blackbaud-conorwright
Copy link
Contributor Author

test added

@Blackbaud-SteveBrush Blackbaud-SteveBrush merged commit 1d7aae7 into master Jun 29, 2018
@Blackbaud-SteveBrush Blackbaud-SteveBrush deleted the fix-wait-service-teardown-race-condition branch June 29, 2018 16:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants