Skip to content

Unit test for Persistence module and part of IO module#912

Closed
eryeer wants to merge 55 commits intoneo-project:masterfrom
eryeer:master
Closed

Unit test for Persistence module and part of IO module#912
eryeer wants to merge 55 commits intoneo-project:masterfrom
eryeer:master

Conversation

@eryeer
Copy link
Copy Markdown
Contributor

@eryeer eryeer commented Jul 11, 2019

Hi,everybody, this is the first submission of the Unit Test by the NGD team. Please take a look and welcome your comments.

We have one suggestion about the code directory structure.

At present, all the test code is placed in the first-level directory. Since there will be more subsequent UT codes to be added, is it possible to establish a one-to-one test code directory structure according to the structure of the Neo code?

For example, if the Neo's Block.cs file is located in Network/P2P/Payloads/Block.cs, the UT_BLOCK.cs file will be placed in the Network/P2P/Payloads/UT_Block.cs location of the UT project.

If this is the case, we can assist with the overall code directory structure adjustment.

@vncoelho vncoelho requested review from igormcoelho and jsolman July 11, 2019 13:19
@shargon
Copy link
Copy Markdown
Member

shargon commented Jul 11, 2019

@lock9 do you have mac, could you fix travis?

@codecov-io
Copy link
Copy Markdown

codecov-io commented Jul 12, 2019

Codecov Report

Merging #912 into master will increase coverage by 5.43%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #912      +/-   ##
==========================================
+ Coverage   45.16%   50.59%   +5.43%     
==========================================
  Files         178      178              
  Lines       12609    12609              
==========================================
+ Hits         5695     6380     +685     
+ Misses       6914     6229     -685
Impacted Files Coverage Δ
neo/Ledger/MemoryPool.cs 77.94% <0%> (ø) ⬆️
neo/Persistence/Helper.cs 8.33% <0%> (+8.33%) ⬆️
neo/SmartContract/Manifest/ContractManifest.cs 91.3% <0%> (+8.69%) ⬆️
neo/IO/Data/LevelDB/Slice.cs 10.52% <0%> (+10.52%) ⬆️
neo/IO/Caching/DataCache.cs 72.83% <0%> (+11.56%) ⬆️
neo/Ledger/TrimmedBlock.cs 16.25% <0%> (+15%) ⬆️
neo/IO/Caching/MetaDataCache.cs 100% <0%> (+15.62%) ⬆️
neo/Ledger/StorageKey.cs 58.33% <0%> (+16.66%) ⬆️
neo/IO/Helper.cs 75.52% <0%> (+20.31%) ⬆️
neo/Persistence/Snapshot.cs 100% <0%> (+21.05%) ⬆️
... and 23 more

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 d662e4a...63aecea. Read the comment docs.

@shargon
Copy link
Copy Markdown
Member

shargon commented Jul 12, 2019

+5% nice work!!!

@eryeer
Copy link
Copy Markdown
Contributor Author

eryeer commented Jul 12, 2019

CI is finally passed, thank you for you all‘s help😄

@shargon
Copy link
Copy Markdown
Member

shargon commented Jul 12, 2019

Please, Take a look to my pr https://github.com/eryeer/neoUT/pull/5

Copy link
Copy Markdown
Member

@vncoelho vncoelho left a comment

Choose a reason for hiding this comment

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

Such a nice job! Congratulations to all involved person.
5% with a different set of tests, covering different parts.

@vncoelho vncoelho mentioned this pull request Jul 12, 2019
@eryeer
Copy link
Copy Markdown
Contributor Author

eryeer commented Jul 15, 2019

Hi everyone. I need to close this PR because we need to remove libleveldb.dll and use Mock to test the code in Persistence module. A new PR will be submitted after we fix these.

@eryeer eryeer closed this Jul 15, 2019
@vncoelho
Copy link
Copy Markdown
Member

Ok, @eryeer.

This is a great job from all of you.
Let's try to merge it as soon as possible.

Thacryba pushed a commit to simplitech/neo that referenced this pull request Feb 17, 2020
* update fees related info

* Update English

* follow up
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.

7 participants