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.

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

[docs] Describe the allowUnawaitedPromise assertion option #2355

Merged

Conversation

VasilyStrelyaev
Copy link
Collaborator

\cc @DevExpress/testcafe-docs @AndreyBelym

@testcafe-build-bot
Copy link
Collaborator

❌ Tests for the commit cf6aa86 have failed. See details.

and [client functions](../obtaining-data-from-the-client/README.md) can be passed as the assertion's actual value.
They trigger the [Smart Assertion Query Mechanism](#smart-assertion-query-mechanism).
If you pass a regular unawaited promise, this will tell TestCafe to compare the promise itself with the expected value.
Copy link
Collaborator

Choose a reason for hiding this comment

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

AFAIK, It should throws a warning/error as well. /cc @AndreyBelym

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes, it throws

@testcafe-build-bot
Copy link
Collaborator

✅ Tests for the commit 593ab35 have passed. See details.

@testcafe-build-bot
Copy link
Collaborator

✅ Tests for the commit 1ecc79f have passed. See details.

@VasilyStrelyaev
Copy link
Collaborator Author

@dirk-pieterse , please check the changes (or the whole doc if you wish)

@testcafe-build-bot
Copy link
Collaborator

✅ Tests for the commit ebed8eb have passed. See details.

@MargaritaLoseva MargaritaLoseva merged commit ef9b89e into DevExpress:master May 7, 2018
kirovboris pushed a commit to kirovboris/testcafe-phoenix that referenced this pull request Dec 18, 2019
…s#2355)

* Describe the allowUnawaitedPromise assertion option

* Fix broken links

* Address remarks

* Update README.md

* Update README.md
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.

7 participants