Skip to content

testing labels (intermediate)Β #12

Description

@DeathGun44

πŸ§‘β€πŸ’» Intermediate Issue

Welcome! This is an Intermediate Issue touching core SDK architecture.

🏁 When this issue is complete, you will have:

βœ… Researched across multiple files and modules
βœ… Owned implementation decisions
βœ… Written thorough, meaningful tests
βœ… Delivered a clean, review-ready pull request

🐞 Problem Description

πŸ’‘ Expected Solution

πŸ” Background Research

πŸ› οΈ Implementation Notes

πŸ”¬ Technical Domains

  • Intermediate to Advanced Programming (Object oriented design, inheritance, complex type systems)
  • File Specific Knowledge (request β†’ serialization β†’ execution β†’ response mapping)
  • Backward Compatibility (preserving method signatures, defaults, and return types)
  • Protobuf Alignment (reading .proto files, _to_proto() / _from_proto() correctness)
  • Testing (unit, integration, mocking, test coverage for edge cases and failure modes)

🧠 Intermediate Contributors β€” Prerequisites & Expectations

Caution

Intermediate issues are high-risk. We expect more than just a 'working solution' and will recommend beginner issues if the PR does not meet these standards.

🏁 Concrete Prerequisites

  • Advanced Programming Language: Higher level intermediate or advanced programming language.
  • Expertise: Strong understanding of files related to this issue (research before claiming!).
  • Proven History: Successfully completed β‰₯ 5 beginner issues in this repo.

Note

CI/CD Exception: For issues focused on GitHub Actions / Workflows, the repo-specific thresholds above may be waived if the contributor demonstrates intermediate-level proficiency in CI/CD.

If this feels like too big a step, that is completely fine β€” try a Beginner Issue first or a different Intermediate Issue. You can always come back when you are ready.

⚠️ AI Usage Policy

  • Using AI to generate code for Intermediate issues is strictly discouraged
  • Using AI as the main source of research is strictly discouraged, refer to language and library documentation, protobuf definitions and other SDKs.

⏱️ Timeline & Workflow

  • Typical time: ~2 weeks / ~25 hours.
  • πŸ”΄ Completing an intermediate issue in 1–3 days is a red flag.

Tip

Suggested: share your proposed implementation approach as a comment before writing code to get early feedback and avoid wasted effort.

πŸ§ͺ Testing Requirements

How you test depends on the type of change.

Important

At the intermediate level, testing is a major component.
Each method you implement should be verified. Tests should cover happy paths,
edge cases, and error handling.

Source code changes (e.g. in src/):

  • Write unit tests covering happy path, edge cases, and error handling
  • Run tests locally: uv run pytest tests/unit/<specific_file>_test.py -v
  • Verify naming, types, and field ordering match protobuf definitions
  • Check consistency with similar classes already in the SDK β€” use the same patterns
  • Integration tests will run automatically when you push

GitHub Actions / workflow changes (e.g. in .github/): Guide

  • Test by merging to your fork's main and simulating the scenario
  • Create test issues or PRs as evidence and link them in your PR
  • Include screenshots of workflow runs where applicable

Example script changes (e.g. in examples/):

  • Run the example script and confirm output matches expected behavior
  • Compare your example with similar existing examples for consistency
  • You will need a Hedera Portal account for testnet credentials

πŸ›‘οΈ Quality & Review Standards

Intermediate PRs must be "working, maintainable, and aligned with SDK architecture."

  1. Working: The implementation must solve the problem and meet the acceptance criteria.
  2. Maintainable: Code should be clear and concise enough for others to understand and debug without your assistance.
  3. SDK Alignment: The solution should fit well with existing SDK patterns and abstractions.
  4. Backward Compatibility: Public API signatures must be preserved.
  5. Comprehensive Testing: Must include unit and integration tests covering all new logic paths, edge cases, and failure modes. AI generated tests based on AI generated code is grounds for immediate rejection.

⚠️ Breaking changes

  • Before changing any function signature, return type, or public API β€” stop and check
  • If a breaking change is unavoidable: get explicit maintainer approval before implementing
  • All existing tests should pass as-is.

πŸ€– AI notes

  • AI often has outdated real-world knowledge: it will not be able to identify current packages.
  • AI has gaps or incorrect understanding of repo-specific patterns, logic, and protobuf schemes.

βœ… PR Quality Checklist

Before opening your PR, confirm:

  • I have spent the majority of my time researching the problem and solution space extensively, including reviewing relevant code, documentation, and external resources.
  • My implementation works but is also of high quality - including maintainability, readability, and architectural fit.
  • I have checked for breaking changes - no public APIs regress.
  • The system design fits with current Hiero SDK architectural approaches.
  • Every line of code is personally understood and explainable.

Before requesting a review, confirm:

  • I have reviewed the diff line by line.
  • My implementation fully addresses the problem described above.
  • I did not modify files unrelated to this issue
  • Clean git history β€” no rebase artifacts, merge commits, or unrelated files
  • My commits are signed: git commit -S -s -m "chore: description" β€” Signing guide
  • I have verified naming, types, and field ordering against pinned Protobufs.
  • I have applied appropriate linting, code quality, and formatting tools used in this repo.
  • I have included appropriate tests and all CI checks pass.
  • Double and triple check β€” intermediate PRs are time-consuming to review

πŸ“‹ Workflow quick reference

You know the workflow β€” here are the links if you need them:

Step Guide
Claim this issue Comment /assign below
Sync with main Rebasing guide
Open a PR and link this issue PR guide Β· Linking guide
Resolve merge conflicts Merge conflicts guide
Testing Testing guide

πŸ“š Resources & Support

πŸ†˜ Stuck?

Tip

Comment on this issue: and describe what you have tried. A maintainer will respond.

Project references:

Protobuf references:

Python references:

Community:

Metadata

Metadata

Assignees

No one assigned

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions