test: use expectAsync for rejections

[raphinesse]

Motivation and Context

Make use of expectAsync (available since Jasmine 3.5) to check for expected rejections. This makes for shorter and clearer expectations

This also fixes a test that would otherwise break with apache/cordova-common#121

Description

Uses expectAsync(...).toBeRejectedWithError(...) and expectAsync(...).toBeRejectedWith(...) to replace rejection checking that was done with fail() before. Where it greatly improved readability I switched the tests to async/await.

Testing

npm t

[codecov-io]

Codecov Report

Merging #831 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #831   +/-   ##
=======================================
  Coverage   89.72%   89.72%           
=======================================
  Files          46       46           
  Lines        2132     2132           
=======================================
  Hits         1913     1913           
  Misses        219      219

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 214dca1...5572140. Read the comment docs.