-
Notifications
You must be signed in to change notification settings - Fork 57
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
[ENG-4427] Fix timing issues due to no jQuery integration #1831
[ENG-4427] Fix timing issues due to no jQuery integration #1831
Conversation
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.
I was looking at these test failures when doing the octanify work, and the component tests seem to fail when component is trying to get properties off of relationships (e.g. {{this.draftRegistration.initiator.fullName}}
in template.hbs of draft-reg-card or reviewAction.creator.get('fullName'),
in component.ts of review-action).
The tests pass if you do a await model.belongsTo('relationshipName').load();
before the assertions, so I had to explicitly add this to the draft-reg component test:
await draftRegistration.belongsTo('initiator').load();
await draftRegistration.belongsTo('registrationSchema').load();
await draftRegistration.belongsTo('provider').load();
Not sure if this is the "proper" way of fixing them, or if it's just best to pepper in some await
s. Also may be worth avoiding the use of relationship.get('property)
and the likes in components down the road?
Hmmm. I wonder if, in the components, things are just not being awaited properly. Edit: In at least one case everything seemed fine on the component side. |
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.
Hoping we don't have to pepper in more of these timeouts in the future 🤞
…penScience#1831) ## Purpose Turn jQuery integration off. This causes some subtle timing issues with rendering which are fixable with timeouts. ## Summary of Changes 1. Disable jQuery integration 2. Sprinkle timeouts throughout the tests where necessary to get them to pass again.
…penScience#1831) ## Purpose Turn jQuery integration off. This causes some subtle timing issues with rendering which are fixable with timeouts. ## Summary of Changes 1. Disable jQuery integration 2. Sprinkle timeouts throughout the tests where necessary to get them to pass again.
…penScience#1831) ## Purpose Turn jQuery integration off. This causes some subtle timing issues with rendering which are fixable with timeouts. ## Summary of Changes 1. Disable jQuery integration 2. Sprinkle timeouts throughout the tests where necessary to get them to pass again.
Purpose
Turn jQuery integration off. This causes some subtle timing issues with rendering which are fixable with timeouts.
Summary of Changes
Side Effects
Tests will take a bit longer to run.
QA Notes
No real QA needed for this. Hopefully it won't break selenium with the new timing issues.