Modernize tests - Githubissues

dnsimple / dnsimple-node

The DNSimple API client for Node.js.

https://developer.dnsimple.com/

MIT License

48 stars 19 forks source link

Modernize tests #207

Closed ggalmazor closed 4 months ago

ggalmazor commented 4 months ago

In this PR:

Migrate our test suite to Jest and removes dependencies on Mocha, Chai, and Sinon.
Migrate async tests to the recommended async/await style by Jest
Minor tweaks and simplifications on the tests code

Most of the adaptations have been made with jest-codemods following the instructions in the https://jestjs.io/docs/migration-guide. I just had to tweak a couple of tests manually.

ggalmazor commented 4 months ago

@DXTimer, could you please help me debug this failing test? I can't figure out why it's failing while all the others in the same file and using the same fixtures and code structure pass...

ggalmazor commented 4 months ago

Nevermind, @DXTimer :) I figured it out :)