Added expected assertions to asynchronous tests.

Ethan-Arrowood / harperdb-connect

Node.JS module for connecting to HarperDB and simplifying HTTP requests.

MIT License

6 stars 10 forks source link

Added expected assertions to asynchronous tests. #9

Closed Trancever closed 6 years ago

Trancever commented 6 years ago

I found out that there is test marked as async but without any await keyword specified inside function body. Shouldn't the async keyword be removed is such case ?

Ethan-Arrowood commented 6 years ago

Thank you contributing 😁

Trancever commented 6 years ago

Thank you also. That was my first merged oss pull request 😁

Ethan-Arrowood commented 6 years ago

Congrats!!