-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
[Uptime] [Test] Add functional test for monitor with location name but no geo #55915
[Uptime] [Test] Add functional test for monitor with location name but no geo #55915
Conversation
Pinging @elastic/uptime (Team:uptime) |
df76b3c
to
f7bb1f0
Compare
@elasticmachine merge upstream |
6718cfd
to
5716051
Compare
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.
Pretty much LGTM, but I have some questions here before I sign off.
before(async () => { | ||
/** | ||
* This mogrify function will strip the documents of their location | ||
* data (but preserve their locaion name), which is necessary for |
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.
nit s/locaion/location/
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.
Thanks for catching this. We will blame it on awful 2017 MBP keyboard 😆.
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.
Repaired in eb1d96e.
) { | ||
await pageObjects.common.navigateToApp('uptime'); | ||
await pageObjects.timePicker.setAbsoluteRange(datePickerStartValue, datePickerEndValue); | ||
await uptimeService.navigateToMonitorWithId(monitorId); | ||
if ((await uptimeService.getMonitorNameDisplayedOnPageTitle()) !== monitorName) { | ||
if ( | ||
!!monitorName && |
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.
Do we need !!
in javascript? It's useful when you want to return a bool, but isn't it equivalent as just omitting it in a boolean expression?
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.
Deleted in 2056471.
💚 Build SucceededHistory
To update your PR or re-run it, just comment with: |
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.
LGTM
…t no geo (elastic#55915) * Add unit test for missing geo on named location. * WIP writing a functional test. * New named-location with no geo data test is functioning. * Update snaps for functional test attribute. * Rename test. * Remove unnecessary async syntax. * Delete unneeded symbols. * Remove unnecessary boolean conversion. * Fix typo in a comment.
…t no geo (elastic#55915) * Add unit test for missing geo on named location. * WIP writing a functional test. * New named-location with no geo data test is functioning. * Update snaps for functional test attribute. * Rename test. * Remove unnecessary async syntax. * Delete unneeded symbols. * Remove unnecessary boolean conversion. * Fix typo in a comment.
…t no geo (#55915) (#56330) * Add unit test for missing geo on named location. * WIP writing a functional test. * New named-location with no geo data test is functioning. * Update snaps for functional test attribute. * Rename test. * Remove unnecessary async syntax. * Delete unneeded symbols. * Remove unnecessary boolean conversion. * Fix typo in a comment.
…t no geo (#55915) (#56331) * Add unit test for missing geo on named location. * WIP writing a functional test. * New named-location with no geo data test is functioning. * Update snaps for functional test attribute. * Rename test. * Remove unnecessary async syntax. * Delete unneeded symbols. * Remove unnecessary boolean conversion. * Fix typo in a comment.
Summary
Fixes elastic/uptime#141.
Test if the appropriate UI is displayed when monitor has a name but no geo data.
Checklist
Use
strikethroughsto remove checklist items you don't feel are applicable to this PR.For maintainers