Skip to content

[SC] Additional storage tests#524

Merged
rowandh merged 2 commits intostratisproject:masterfrom
rowandh:null-storage-test
Apr 20, 2021
Merged

[SC] Additional storage tests#524
rowandh merged 2 commits intostratisproject:masterfrom
rowandh:null-storage-test

Conversation

@rowandh
Copy link
Contributor

@rowandh rowandh commented Apr 19, 2021

Add tests to demonstrate the behaviour when storing null and byte[0] in the repository

@rowandh rowandh requested review from YakupIpek and quantumagi April 19, 2021 13:10
repository.Commit();

// We have pushed null to the kv store. Should come back as null
StateRepositoryRoot repository2 = new StateRepositoryRoot(stateDB, repository.Root);
Copy link
Contributor

Choose a reason for hiding this comment

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

You use var and actual type together as style. Which one we should follow ?

@rowandh rowandh merged commit eb184b9 into stratisproject:master Apr 20, 2021
zeptin pushed a commit to zeptin/StratisFullNode that referenced this pull request Jun 15, 2021
* Add repository null tests

* Add test demonstrating null and empty behaviour
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