Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Improved tests structure in files #355

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Improved tests structure in files #355

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
4f7525f
Improved tests structure in element__element__lazy_search_test.py and…
Elena7788 Jun 17, 2021
2b3e068
[#352][#353] TEST: Refactoring of test structure after review
Elena7788 Jun 20, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
32 changes: 16 additions & 16 deletions tests/integration/element__element__lazy_search_test.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,8 @@
def test_search_does_not_start_on_creation_for_both_parent_and_inner(
session_browser,
):
GivenPage(session_browser.driver).opened_empty()
page = GivenPage(session_browser.driver)
page.opened_empty()

non_existent_element = session_browser.element(
'#not-existing-element'
Expand All @@ -41,17 +42,17 @@ def test_search_is_postponed_until_actual_action_like_questioning_displayed(
'.will-exist-inner'
)
page = GivenPage(session_browser.driver)
page.opened_empty()

page.load_body(
page.opened_with_body(
'''
<h1 id="will-be-existing-element">
<span class="will-exist-inner">Hello</span> kitty:*
</h1>'''
</h1>
'''
)
answer = element().is_displayed()

assert answer is True
until_actual_action = element().is_displayed()

assert until_actual_action is True


def test_search_is_updated_on_next_actual_action_like_questioning_displayed(
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please watch this: https://youtu.be/47yhzbiCJSw?t=1147
and refactor the test below, it should have same structure as in video;
assert should be broken into ACT and ASSERT clauses, and all previous page loading including first store of "answer" should be in ARRANGE

also
I would use the update word instead of new for consistency with test name; all this was in video;) watch it!

Expand All @@ -68,19 +69,17 @@ def test_search_is_updated_on_next_actual_action_like_questioning_displayed(
</h1>
'''
)
assert element().is_displayed() is True

page.load_body(
'''
<h1 id="will-be-existing-element">
<span class="will-exist-inner" style="display:none">
Hello
</span> kitty:*
</h1>'''
<span class="will-exist-inner" style="display:none">Hello</span> kitty:*
</h1>
'''
)
new_answer = element().is_displayed()

assert new_answer is False
updated_actual_action = element().is_displayed()

assert updated_actual_action is False


def test_search_finds_exactly_inside_parent(session_browser):
Expand All @@ -92,7 +91,8 @@ def test_search_finds_exactly_inside_parent(session_browser):
<a href="#second">go to Heading 2</a>
<h1 id="first">Heading 1</h1>
<h2 id="second">Heading 2</h2>
/p>'''
</p>
'''
)

session_browser.element('p').element('a').click()
Expand Down
29 changes: 20 additions & 9 deletions tests/integration/element__lazy_search_test.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,24 +37,35 @@ def test_search_is_postponed_until_actual_action_like_questioning_displayed(
):
page = GivenPage(session_browser.driver)
page.opened_empty()

element = session_browser.element('#will-be-existing-element-id')
page.load_body('<h1 id="will-be-existing-element-id">Hello kitty:*</h1>')
page.load_body(
'<h1 id="will-be-existing-element-id">'
'Hello kitty:*'
'</h1>'
)

assert element().is_displayed() is True
until_actual_action = element().is_displayed()

assert until_actual_action is True


def test_search_is_updated_on_next_actual_action_like_questioning_displayed(
session_browser,
):
page = GivenPage(session_browser.driver)
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same logic should be applied as in video mentioned earlier

page.opened_empty()

element = session_browser.element('#will-be-existing-element-id')
page.load_body('<h1 id="will-be-existing-element-id">Hello kitty:*</h1>')
assert element().is_displayed() is True

page.load_body(
'<h1 id="will-be-existing-element-id" style="display:none">Hello kitty:*</h1>'
'<h1 id="will-be-existing-element-id">'
'Hello kitty:*'
'</h1>'
)
page.load_body(
'<h1 id="will-be-existing-element-id" style="display:none">'
'Hello kitty:*'
'</h1>'
)
assert element().is_displayed() is False

updated_actual_action = element().is_displayed()

assert updated_actual_action is False