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
OKTA-300462 account for user race condition #387
OKTA-300462 account for user race condition #387
Conversation
8ec087f
to
ed53c21
Compare
ed53c21
to
4515991
Compare
4515991
to
4fe6676
Compare
@@ -12,7 +12,7 @@ | |||
|
|||
namespace Okta.Sdk.IntegrationTests | |||
{ | |||
public class EventHooksClientShould | |||
public class EventHooksClientScenarios |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make sure to update the header as well.
@@ -11,7 +11,7 @@ | |||
|
|||
namespace Okta.Sdk.IntegrationTests | |||
{ | |||
public class InlineHooksClientShould | |||
public class InlineHooksClientScenarios |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make sure to update the header as well.
using System.Threading.Tasks; | ||
using FluentAssertions; | ||
using Xunit; | ||
|
||
namespace Okta.Sdk.IntegrationTests | ||
{ | ||
public class UserTypesClientShould | ||
public class UserTypesClientScenarios |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make sure to update the header as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some minor comments. Other than that, LGTM. 👍
Account for eventual consistency
The primary change introduced here are Thread.Sleep additions on creation of new data. Was seeing intermittent 404 responses when new data entries are made, likely due to GET requests going to read only data replicas. This change waits a few seconds to allow for data replication prior to read attempts.