-
Notifications
You must be signed in to change notification settings - Fork 79
test(commonTests, alert, combobox, dropdown, input-date-picker, modal, popover, tooltip): extracts a general test utility OpenClose
for all components implementing openCloseComponent
util
#7379
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
Merged
Elijbet
merged 56 commits into
main
from
elijbet/4968-add-common-test-util-for-all-OpenCloseComponents
Sep 19, 2023
Merged
Changes from 2 commits
Commits
Show all changes
56 commits
Select commit
Hold shift + click to select a range
04f2697
test(commonTests): extracts a general test utility emitsOpenCloseTran…
Elijbet 57d3038
Merge branch 'main' into elijbet/4968-add-common-test-util-for-all-Op…
Elijbet b25d73e
Merge branch 'main' into elijbet/4968-add-common-test-util-for-all-Op…
Elijbet 16ed200
refactor to simplify var and test names and to eliminate repetitive code
Elijbet d4b1978
add extra checks and move selectors to resources to avoid hard coding
Elijbet dfbc66e
Merge branch 'main' into elijbet/4968-add-common-test-util-for-all-Op…
Elijbet 858848f
Merge branch 'main' into elijbet/4968-add-common-test-util-for-all-Op…
Elijbet 7307dda
Split into 2 test coverages for with and without animation and refact…
Elijbet 1dd336b
get the initial value of toggleProp dynamically and toggle accordingly
Elijbet 00ba742
assume the element will be visible (or not visible) to avoid checking…
Elijbet 01d09d6
wire up this helper in comobobox and alert
Elijbet 136c9dc
rename HTML block
Elijbet c304d33
cleanup
Elijbet 1979adf
add modal test imports of openClose util: regular use case and when i…
Elijbet 4d6d984
Merge branch 'main' into elijbet/4968-add-common-test-util-for-all-Op…
Elijbet bb73ff5
applied to popover, to-do: fix failing tests with disabled animation
Elijbet 4d1ae22
Merge branch 'main' into elijbet/4968-add-common-test-util-for-all-Op…
Elijbet 13a0f1f
fix for popover
Elijbet bb3e086
Merge branch 'main' into elijbet/4968-add-common-test-util-for-all-Op…
Elijbet 62ba3bd
add tooltip tests for checking openClose emits via prop as well as us…
Elijbet 0b4737e
userInputDevice - Optional argument with functions to simulate user i…
Elijbet 3b08700
Merge branch 'main' into elijbet/4968-add-common-test-util-for-all-Op…
Elijbet 4e6caad
Merge branch 'main' into elijbet/4968-add-common-test-util-for-all-Op…
Elijbet 5728d42
modify test for
Elijbet b805d7c
resolve merge
Elijbet 8d28259
cleanups and refactors
Elijbet c8774f5
Merge branch 'main' into elijbet/4968-add-common-test-util-for-all-Op…
Elijbet 59a3ee7
cleanups and refactors
Elijbet 8d99c26
Merge branch 'main' into elijbet/4968-add-common-test-util-for-all-Op…
Elijbet fb3a2ce
Refactor the commonTest util to include newProgrammaticE2EPage setup …
Elijbet 228035a
Merge branch 'main' into elijbet/4968-add-common-test-util-for-all-Op…
Elijbet 49814ec
merge
Elijbet bd76e7c
worked out merge conflicts
Elijbet c162352
Merge branch 'main' into elijbet/4968-add-common-test-util-for-all-Op…
Elijbet d31d215
WIP: cleanup
Elijbet 8f5588a
Merge branch 'main' into elijbet/4968-add-common-test-util-for-all-Op…
Elijbet 90390fb
Merge branch 'main' into elijbet/4968-add-common-test-util-for-all-Op…
Elijbet 7984977
revert unrelated eslint change
Elijbet c061a29
refactors
Elijbet 0524a93
Merge branch 'main' into elijbet/4968-add-common-test-util-for-all-Op…
Elijbet 4048fe5
Merge branch 'main' into elijbet/4968-add-common-test-util-for-all-Op…
Elijbet 3afe7c7
simplify util by extracting a OpenCloseOptions interface
Elijbet 967d2eb
Merge branch 'main' into elijbet/4968-add-common-test-util-for-all-Op…
Elijbet 07c0b55
Merge branch 'main' into elijbet/4968-add-common-test-util-for-all-Op…
Elijbet 2fc3686
merge conflicts
Elijbet 66ed1a9
rename togglePropName to openPropName; since the pattern use open/clo…
Elijbet 39d4518
rename togglePropName to openPropName; since the pattern use open/clo…
Elijbet e82808b
rename userInputDevice to inputOptions
Elijbet f3d01b3
revert
Elijbet c4c3ade
Merge branch 'main' into elijbet/4968-add-common-test-util-for-all-Op…
Elijbet 536e87d
Merge branch 'main' into elijbet/4968-add-common-test-util-for-all-Op…
Elijbet 62aa8a0
refactor openClose call signature in the individual test and add a mo…
Elijbet 1ac7257
for methods, use setUp (verb) vs setup (noun) convention
Elijbet f19a24c
Merge branch 'main' into elijbet/4968-add-common-test-util-for-all-Op…
Elijbet 7d3f147
Shorten comment
Elijbet fe48292
Merge branch 'main' into elijbet/4968-add-common-test-util-for-all-Op…
Elijbet File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.