-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[EDR Workflows] Verify flaky osquery tests #169749
Changes from 46 commits
f2d6929
18bc8b0
69b847c
b0e4530
bdeecef
91ca659
5853004
7bd8231
f40ba89
a6fa2ae
4abf5d3
01d1c39
cb282ca
cc11fbc
bffa366
cbf6f8f
969d114
c313e04
835e39a
dcbcd05
dc55d9c
0d9e7d8
e825588
68b9640
2da2515
fc350db
f719820
ef9cc6f
f502a9a
e748b09
008f001
0adfd83
33eb662
70d9943
2fb6446
7a8b0e6
0578de8
b7610b5
dc91194
3d3b2e0
97863fc
00b3d29
b4dbac6
5a647a3
42bc3d0
44c74cb
9105d07
e188c9c
9429418
11479e3
ed51ade
97b8865
76d1e87
212cc0a
03be3ef
89c5342
80c962b
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -18,8 +18,7 @@ import { LIVE_QUERY_EDITOR } from '../../screens/live_query'; | |
import { getAdvancedButton } from '../../screens/integrations'; | ||
import { ServerlessRoleName } from '../../support/roles'; | ||
|
||
// FLAKY: https://github.com/elastic/kibana/issues/169725 | ||
describe.skip('ALL - Live Query', { tags: ['@ess', '@serverless'] }, () => { | ||
describe('ALL - Live Query', { tags: ['@ess', '@serverless'] }, () => { | ||
beforeEach(() => { | ||
cy.login(ServerlessRoleName.SOC_MANAGER); | ||
navigateTo('/app/osquery'); | ||
|
@@ -87,6 +86,6 @@ describe.skip('ALL - Live Query', { tags: ['@ess', '@serverless'] }, () => { | |
|
||
inputQuery('{selectall}{backspace}{selectall}{backspace}'); | ||
// not sure if this is how it used to work when I implemented the functionality, but let's leave it like this for now | ||
cy.get(LIVE_QUERY_EDITOR).invoke('height').should('be.gt', 200).and('be.lt', 380); | ||
cy.get(LIVE_QUERY_EDITOR).invoke('height').should('be.gt', 200).and('be.lt', 400); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. instead of hardcoding values, could we write it in a more dynamic way? checking if the values have changed? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yeah not sure, sometimes these numbers change, but in general it is good to know if it is in some scope, and not changed as from 300px to 3000px. I find this to be a quite easy way to get it tested, now just making the expectations more loose. |
||
}); | ||
}); |
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.
can we remove it?