Skip to content
This repository has been archived by the owner on Jun 11, 2024. It is now read-only.

Convert lisk-cryptography tests to jest - Closes #4677 #4698

Merged
merged 10 commits into from
Jan 15, 2020

Conversation

valamidev
Copy link
Contributor

What was the problem?

Tests were in mocha.

How did I solve it?

Tests converted to jest.

How to manually test it?

npm run test

Review checklist

@valamidev valamidev self-assigned this Jan 14, 2020
@valamidev valamidev requested review from ishantiw and removed request for mitsuaki-u January 14, 2020 16:00
Change back to format if it(should...) from test(should...)
@valamidev valamidev requested a review from shuse2 January 15, 2020 10:01
@valamidev valamidev force-pushed the 4677-cryptography-jest-migration branch from 5dc41b1 to 37d6f0c Compare January 15, 2020 10:39
@shuse2 shuse2 requested review from mitsuaki-u and removed request for ishantiw January 15, 2020 15:42
Copy link
Contributor

@mitsuaki-u mitsuaki-u left a comment

Choose a reason for hiding this comment

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

Can use toBeInstanceOf rather than Object.prototype.toString.call(...

@sonarcloud
Copy link

sonarcloud bot commented Jan 15, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@shuse2 shuse2 merged commit 875bf7a into development Jan 15, 2020
@shuse2 shuse2 deleted the 4677-cryptography-jest-migration branch January 15, 2020 19:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Convert lisk-cryptography mocha test to jest
3 participants