Skip to content

Scrypt ut from 3s to 16ms#937

Merged
shargon merged 2 commits intoneo-project:masterfrom
shargon:speed-up-slow-tests
Jul 19, 2019
Merged

Scrypt ut from 3s to 16ms#937
shargon merged 2 commits intoneo-project:masterfrom
shargon:speed-up-slow-tests

Conversation

@shargon
Copy link
Copy Markdown
Member

@shargon shargon commented Jul 19, 2019

No description provided.

@shargon
Copy link
Copy Markdown
Member Author

shargon commented Jul 19, 2019

UT fails because other PR, so please merge (#936) this first

@shargon shargon requested a review from lock9 July 19, 2019 16:24
@codecov-io
Copy link
Copy Markdown

codecov-io commented Jul 19, 2019

Codecov Report

Merging #937 into master will increase coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #937      +/-   ##
==========================================
+ Coverage   45.52%   45.53%   +<.01%     
==========================================
  Files         178      178              
  Lines       12635    12635              
==========================================
+ Hits         5752     5753       +1     
+ Misses       6883     6882       -1
Impacted Files Coverage Δ
neo/Network/P2P/Connection.cs 40% <0%> (-2.23%) ⬇️
neo/Cryptography/SCrypt.cs 82.3% <0%> (+1.32%) ⬆️

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 62766eb...59ea2c0. Read the comment docs.

@shargon shargon merged commit 3e7bee4 into neo-project:master Jul 19, 2019
@shargon shargon deleted the speed-up-slow-tests branch July 19, 2019 16:40
Thacryba pushed a commit to simplitech/neo that referenced this pull request Feb 17, 2020
Tommo-L pushed a commit to Tommo-L/neo that referenced this pull request Jun 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants