Skip to content

[Tests] Remove inline awaits #13041

@stacey-gammon

Description

@stacey-gammon

We should make this part of our style guide as well, but inline awaits are confusing.

e.g. for the following statement is a() being awaited on, or b()?
await a().b()

To avoid this confusion we should just stop inlining awaits. The above usage is prolific in our tests when a() returns a promise, and I suspect causing some test instability.

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions