Skip to content

Adds jest dependency#10845

Merged
tmashuang merged 2 commits intodevelopfrom
add-jest
Apr 8, 2021
Merged

Adds jest dependency#10845
tmashuang merged 2 commits intodevelopfrom
add-jest

Conversation

@tmashuang
Copy link
Contributor

Adds jest dependency as per the discussion with the codefi team to write the swaps test using jest + react-testing-library.

@tmashuang tmashuang requested a review from a team as a code owner April 7, 2021 20:13
@tmashuang tmashuang requested a review from NiranjanaBinoy April 7, 2021 20:13
@github-actions
Copy link
Contributor

github-actions bot commented Apr 7, 2021

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@metamaskbot
Copy link
Collaborator

Builds ready [57d911c]
Page Load Metrics (579 ± 50 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint4610260126
domContentLoaded37772757810651
load37972857910550
domInteractive37772657710651

Gudahtt
Gudahtt previously approved these changes Apr 8, 2021
Copy link
Member

@Gudahtt Gudahtt left a comment

Choose a reason for hiding this comment

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

LGTM!

These lavamoat policy changes aren't necessary here (it's just fsevent and it's dependencies being removed), but we've been allowing these changes for convenience sake until lavamoat better supports optional dependencies.

@metamaskbot
Copy link
Collaborator

Builds ready [fe5e33c]
Page Load Metrics (610 ± 55 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint45716084
domContentLoaded37585460911555
load37685561011555
domInteractive37585460811555

@tmashuang tmashuang merged commit c339f28 into develop Apr 8, 2021
@tmashuang tmashuang deleted the add-jest branch April 8, 2021 17:25
@github-actions github-actions bot locked and limited conversation to collaborators Apr 8, 2021
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.

3 participants