Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Feature: smartcontract and state unit tests #289

Merged
merged 8 commits into from
Jun 2, 2021
Merged

Feature: smartcontract and state unit tests #289

merged 8 commits into from
Jun 2, 2021

Conversation

MurashovVen
Copy link
Contributor

  1. Added smartcontract unit tests.
  2. Added state unit tests.
  3. Mocks:
  • Added new mocks.
  • Existing mocks are moved in a single package.

@MurashovVen
Mocks:
2e911f6 
- adding new mocks
- moving existing mocks in single package
- adding generate_mocks.sh
@MurashovVen
Add state tests
e3cfe5e
@MurashovVen
Add smartcontract tests
f947bc0
@MurashovVen
gofmt
05f1989
@Sriep Sriep mentioned this pull request May 18, 2021
Closed
Sriep
Sriep suggested changes May 18, 2021
View reviewed changes
code/go/0chain.net/chaincore/smartcontract/bccontext_test.go Outdated Show resolved Hide resolved
code/go/0chain.net/chaincore/smartcontract/handler_test.go Show resolved Hide resolved
code/go/0chain.net/chaincore/smartcontract/handler_test.go Outdated Show resolved Hide resolved
code/go/0chain.net/chaincore/smartcontract/handler_test.go Outdated Show resolved Hide resolved
code/go/0chain.net/chaincore/state/state_debug_test.go Outdated Show resolved Hide resolved
code/go/0chain.net/chaincore/state/state_debug_test.go Outdated Show resolved Hide resolved
code/go/0chain.net/chaincore/state/state_debug_test.go Outdated Show resolved Hide resolved
code/go/0chain.net/chaincore/state/state_debug_test.go Outdated Show resolved Hide resolved
code/go/0chain.net/chaincore/state/state_debug_test.go Outdated Show resolved Hide resolved
code/go/0chain.net/chaincore/state/state_debug_test.go Outdated Show resolved Hide resolved
@MurashovVen
Fixes:
a635cf7 
- code refactoring
- fixing race issues
@MurashovVen
Merge remote-tracking branch 'origin/master' into feature/smartcontra…
81f115f 
…ct_and_state_unit_tests

# Conflicts:
#	generate_mocks.sh
Sriep
Sriep suggested changes May 31, 2021
View reviewed changes
Copy link
Contributor

@Sriep Sriep left a comment

Choose a reason for hiding this comment

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

Need to merge in changes to TestGetSmartContract.

Sriep added 2 commits June 2, 2021 11:43
@Sriep
Merge remote-tracking branch 'origin/master' into feature/smartcontra…
9696573 
…ct_and_state_unit_tests

# Conflicts:
#	code/go/0chain.net/chaincore/smartcontract/handler_test.go
@Sriep
merge with master
3638714
@Sriep Sriep merged commit 3e23cc5 into master Jun 2, 2021
@Sriep Sriep deleted the feature/smartcontract_and_state_unit_tests branch June 2, 2021 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Sriep Sriep Sriep approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@MurashovVen @Sriep