-
Notifications
You must be signed in to change notification settings - Fork 917
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
Add missing data-test-subj tags for data plugin #9051
Conversation
Signed-off-by: Suchit Sahoo <[email protected]>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #9051 +/- ##
==========================================
- Coverage 60.87% 60.84% -0.04%
==========================================
Files 3808 3808
Lines 91209 91209
Branches 14410 14410
==========================================
- Hits 55526 55492 -34
- Misses 32142 32222 +80
+ Partials 3541 3495 -46
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Signed-off-by: Suchit Sahoo <[email protected]>
src/plugins/data/public/query/query_string/language_service/lib/query_result.tsx
Outdated
Show resolved
Hide resolved
src/plugins/data/public/ui/query_editor/query_editor_top_row.tsx
Outdated
Show resolved
Hide resolved
5bb43a9
to
0f42fcf
Compare
Signed-off-by: Suchit Sahoo <[email protected]>
0f42fcf
to
528982a
Compare
@@ -150,7 +153,7 @@ export const SingleLineInput: React.FC<SingleLineInputProps> = ({ | |||
triggerSuggestOnFocus={true} | |||
/> | |||
{editorIsFocused && ( | |||
<div className="queryEditor__footer"> | |||
<div className="queryEditor__footer" data-test-subj="queryEditorFooter"> |
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.
If I remember correctly, I added this for the single line editor's footer. Could be a little more descriptive for this to avoid confusion with the other editors.
* Add missing data-test-subj tags for data plugin * Update Snapshots * Fix as per the comments --------- Signed-off-by: Suchit Sahoo <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> (cherry picked from commit 16115ba) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Description
Add missing data-test-subj tags for data plugin
Issues Resolved
Screenshot
Testing the changes
Changelog
Check List
yarn test:jest
yarn test:jest_integration