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

Unit tests for VDR #20

Merged
merged 19 commits into from
Feb 13, 2024
Merged

Unit tests for VDR #20

merged 19 commits into from
Feb 13, 2024

Conversation

DenisRybas
Copy link
Contributor

Added unit tests for VDR:

  • client
  • txn builders, parsers
  • some contract methods

@DenisRybas
Added validation for txn hash len
f0a5fce 
Signed-off-by: DenisRybas <[email protected]>
@DenisRybas
added unit tests for contract
2645203 
Signed-off-by: DenisRybas <[email protected]>
@DenisRybas
added unit tests for client
de1d56c 
Signed-off-by: DenisRybas <[email protected]>
@DenisRybas
added unit tests for role_control
e90fd5e 
Signed-off-by: DenisRybas <[email protected]>
@DenisRybas
uncommented test
a300c51 
Signed-off-by: DenisRybas <[email protected]>
@DenisRybas
added txn tests
bf5c5ea 
Signed-off-by: DenisRybas <[email protected]>
@DenisRybas
linted
4609f40 
Signed-off-by: DenisRybas <[email protected]>
@DenisRybas
made tests work
226d77a 
Signed-off-by: DenisRybas <[email protected]>
@DenisRybas
transaction tests refactoring
e947476 
Signed-off-by: DenisRybas <[email protected]>
@DenisRybas
ignored test
25ebd32 
Signed-off-by: DenisRybas <[email protected]>
@DenisRybas
linted
6ba50af 
Signed-off-by: DenisRybas <[email protected]>
Artemkaaas
Artemkaaas requested changes Feb 8, 2024
View reviewed changes
vdr/src/client/client.rs Outdated Show resolved Hide resolved
vdr/src/client/client.rs Outdated Show resolved Hide resolved
vdr/src/client/client.rs Outdated Show resolved Hide resolved
vdr/src/client/client.rs Outdated Show resolved Hide resolved
vdr/src/client/client.rs Outdated Show resolved Hide resolved
vdr/src/contracts/auth/role_control.rs Outdated Show resolved Hide resolved
vdr/src/contracts/auth/role_control.rs Outdated Show resolved Hide resolved
vdr/src/types/transaction.rs Outdated Show resolved Hide resolved
vdr/src/types/transaction.rs Outdated Show resolved Hide resolved
vdr/src/types/transaction.rs Outdated Show resolved Hide resolved
@DenisRybas
processed comments
d836b83 
Signed-off-by: DenisRybas <[email protected]>
@DenisRybas
Merge branch 'main' of https://github.com/hyperledger/indy-besu into …
0a81701 
…feat/unit-tests

# Conflicts:
#	vdr/Cargo.lock
@DenisRybas
updated dependencies
a69f5ad 
Signed-off-by: DenisRybas <[email protected]>
@DenisRybas
linted
6c88d93 
Signed-off-by: DenisRybas <[email protected]>
Artemkaaas
Artemkaaas reviewed Feb 13, 2024
View reviewed changes
vdr/src/client/client.rs Outdated Show resolved Hide resolved
vdr/src/client/client.rs Show resolved Hide resolved
vdr/src/types/transaction.rs Outdated Show resolved Hide resolved
@DenisRybas
processed comments
1e3a1d7 
Signed-off-by: DenisRybas <[email protected]>
@DenisRybas
processed comments
f7df75a 
Signed-off-by: DenisRybas <[email protected]>
@DenisRybas
Removed warnings
982659c 
Signed-off-by: DenisRybas <[email protected]>
@DenisRybas
fixed tests
2a0f90d 
Signed-off-by: DenisRybas <[email protected]>
@Artemkaaas Artemkaaas merged commit 73a5518 into hyperledger:main Feb 13, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Artemkaaas Artemkaaas Artemkaaas 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.
2 participants
@DenisRybas @Artemkaaas