deltadevsde / deltadevsde/prism

testing(`common::operation`): Improve coverage

Open
#331 0 comments 0 reactions 1 assignee Assigned to @Zombeescott View on GitHub
good first issue testing
Dominant language
Rust
Stars
134
Forks
41
PR merge metrics
No merged PRs in 30d

Description

Quite a few branches are missing from the test coverage for `operation.rs`.

- SignatureBundle::new
- ServiceChallenge::from(signing_key)
- Operation::validate_basic ("id must not empty...") errors, and "service_id must not be.." error
- Operation::validate_basic for AddKey, RevokeKey, SetData, and AddData
- Operation::get_public_key for RevokeKey, AddKey, RegisterService, and Add/SetData. Already called once for CreateAccount

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.