Skip to content
This repository has been archived by the owner on Apr 6, 2020. It is now read-only.

VM Unit Tests #561

Merged
merged 53 commits into from
Dec 17, 2018
Merged

VM Unit Tests #561

merged 53 commits into from
Dec 17, 2018

Conversation

shargon
Copy link
Member

@shargon shargon commented Dec 11, 2018

  • Fixed GAS cost in InteropService
  • Fixed bug with Array type

- Allow to use InteropTypes
- Fix gas on SYSCALLs
@shargon shargon changed the title VM Unit test for PUSHBYTESXX VM Unit Tests Dec 13, 2018
@shargon
Copy link
Member Author

shargon commented Dec 15, 2018

I included the updated Neo.VM library for merge this until Neo release the last patch of the VM

@codecov
Copy link

codecov bot commented Dec 16, 2018

Codecov Report

❗ No coverage uploaded for pull request base (development@088415c). Click here to learn what that means.
The diff coverage is 42.85%.

Impacted file tree graph

@@              Coverage Diff               @@
##             development     #561   +/-   ##
==============================================
  Coverage               ?   51.71%           
==============================================
  Files                  ?      359           
  Lines                  ?    16246           
  Branches               ?        0           
==============================================
  Hits                   ?     8402           
  Misses                 ?     7844           
  Partials               ?        0
Impacted Files Coverage Δ
src/NeoSharp.VM/Collections/Stack.cs 39.74% <100%> (ø)
src/NeoSharp.VM/Types/InteropService.cs 54.54% <100%> (ø)
test/NeoSharp.VM.TestHelper/VMUTEntry.cs 100% <100%> (ø)
src/NeoSharp.VM.NeoVM/StackItems/ArrayStackItem.cs 35.59% <100%> (ø)
src/NeoSharp.Core/VM/StateMachine.cs 21.92% <31.03%> (ø)
src/NeoSharp.Core/VM/StateReader.cs 31.21% <33.33%> (ø)
src/NeoSharp.Core/Models/StorageKey.cs 56.25% <40%> (ø)
src/NeoSharp.Core/VM/StorageContext.cs 83.33% <75%> (ø)

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 088415c...a02e6df. Read the comment docs.

@osmirnov osmirnov merged commit 7f0d79a into CityOfZion:development Dec 17, 2018
@shargon shargon deleted the pushbytes-ut branch December 17, 2018 17:04
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.

2 participants