-
Notifications
You must be signed in to change notification settings - Fork 22
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
Added a search query option for the addwidget dialog #81
Conversation
…hrough code, created search query
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #81 +/- ##
==========================================
+ Coverage 63.41% 63.58% +0.16%
==========================================
Files 75 75
Lines 7867 7892 +25
==========================================
+ Hits 4989 5018 +29
+ Misses 2878 2874 -4 ☔ View full report in Codecov by Sentry. |
For adding unit tests, what would be the most practical to test the new features is to set up a mock NT connection with several sub-topics, and entering different search queries into the text box, and making sure that only certain tables are visible after searching |
--------- Co-authored-by: Gold87 <[email protected]>
Co-authored-by: Gold87 <[email protected]>
Co-authored-by: DanPeled [email protected]
Co-authored-by: DanPeled <[email protected]>
Wasnt sure how to write the test for this, if you (the reviewer) will be able to explain me / do it ill be glad