Skip to content

test: add test_navigation.py (part 1) #82

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
merged 1 commit into from
Jul 23, 2020

Conversation

pavelfeldman
Copy link
Member

drive-by replacing asyncio.ensure_future since "create_task() function is the preferred way for creating new Tasks."

@coveralls
Copy link

coveralls commented Jul 23, 2020

Pull Request Test Coverage Report for Build 180079251

  • 18 of 20 (90.0%) changed or added relevant lines in 5 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.2%) to 81.486%

Changes Missing Coverage Covered Lines Changed/Added Lines %
playwright/connection.py 9 10 90.0%
playwright/sync_base.py 0 1 0.0%
Totals Coverage Status
Change from base Build 179350604: 0.2%
Covered Lines: 2117
Relevant Lines: 2598

💛 - Coveralls

if is_chromium:
assert "net::ERR_CONNECTION_REFUSED" in exc_info.value.message
elif is_webkit and is_win:
assert 'Couldn"t connect to server' in exc_info.value.message
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
assert 'Couldn"t connect to server' in exc_info.value.message
assert 'Couldn"t connect to server' in exc_info.value.message
Suggested change
assert 'Couldn"t connect to server' in exc_info.value.message
assert 'Couldn't connect to server' in exc_info.value.message

Copy link
Member

Choose a reason for hiding this comment

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

Probably the reason why the bots are failing.

@pavelfeldman pavelfeldman merged commit 6e1a593 into microsoft:master Jul 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants