chore: format black tests/unit/crypto_utils_test.py

[AkshayKumarSahu]

Description:

Related issue(s):

Fixes #1524

Notes for reviewer:

Checklist

• Documented (Code comments, README, etc.)
• Tested (unit, integration, etc.)

[github-actions]

Hi, this is MergeConflictBot.
Your pull request cannot be merged because it contains merge conflicts.

Please resolve these conflicts locally and push the changes.

Quick Fix for CHANGELOG.md Conflicts

If your conflict is only in CHANGELOG.md, you can resolve it easily using the GitHub web editor:

1. Click on the "Resolve conflicts" button in the PR
2. Accept both changes (keep both changelog entries)
3. Click "Mark as resolved"
4. Commit the merge

For all other merge conflicts, please read:

• Resolving Merge Conflicts
• Rebasing Guide

Thank you for contributing!

[coderabbitai]

Caution

Review failed

An error occurred during the review process. Please try again later.

Walkthrough

The pull request applies black code formatter to standardize the style of the test file tests/unit/crypto_utils_test.py, with no functional or logical changes. A changelog entry was added to document this formatting update.

Changes

Cohort / File(s)	Summary
Changelog Entry CHANGELOG.md	Added release notes entry documenting the black formatting of the crypto_utils_test.py file.
Test File Formatting tests/unit/crypto_utils_test.py	Applied black code formatter: string quotes standardized, assertion statements wrapped across multiple lines for readability. No logic or behavior changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'chore: format black tests/unit/crypto_utils_test.py' directly and clearly summarizes the main change—formatting the crypto_utils_test.py file using Black.
Linked Issues check	✅ Passed	The PR successfully implements the core requirement from issue #1524 by formatting tests/unit/crypto_utils_test.py with Black and includes a changelog entry.
Out of Scope Changes check	✅ Passed	All changes are scoped to formatting tests/unit/crypto_utils_test.py and updating CHANGELOG.md, with no unrelated modifications present.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Description check	✅ Passed	The PR description references issue #1524 which is about formatting tests/unit/crypto_utils_test.py using Black, matching the raw_summary and pr_objectives.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[Copilot]

Pull request overview

Updates formatting and documentation entries related to unit tests, with an additional (unrelated) API typing/docstring tweak in Transaction.to_bytes().

Changes:

• Reformat tests/unit/crypto_utils_test.py using Black (line wrapping, quote style).
• Add return type annotation and tweak docstring wording for Transaction.to_bytes().
• Update CHANGELOG.md with entries for the formatting change and the to_bytes() note.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
tests/unit/crypto_utils_test.py	Black formatting updates (line wrapping, string quoting).
src/hiero_sdk_python/transaction/transaction.py	Adds -> bytes return annotation and adjusts return docstring text for to_bytes().
CHANGELOG.md	Adds/updates changelog bullets for the formatting change and to_bytes() (currently with placement/typo issues).

Comments suppressed due to low confidence (2)

CHANGELOG.md:409

• This entry is being added under the already-released 0.1.8 section. New changes should typically be recorded under [Unreleased] (and only moved into a release section at release time). Please move this bullet to the appropriate [Unreleased] subsection and keep historical release notes immutable.

- Add Google-style docstrings to `AccountInfo` class and its methods in `account_info.py`.
- Added comprehensive Google-style docstrings to the `Logger` class and all utility functions in `src/hiero_sdk_python/logger/logger.py` (#639).
- add AccountRecordsQuery class

CHANGELOG.md:408

• The changelog text says the return of to_bytes() was updated, but the code change here appears to be only a return type annotation/docstring tweak (no behavioral change). Please adjust the entry wording to reflect what actually changed, or drop the entry if it’s not a user-visible change.

- Added comprehensive Google-style docstrings to the `Logger` class and all utility functions in `src/hiero_sdk_python/logger/logger.py` (#639).

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[coderabbitai]

Actionable comments posted: 1

[AkshayKumarSahu]

pls review

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

@@           Coverage Diff           @@
##             main    #1738   +/-   ##
=======================================
  Coverage   93.25%   93.25%           
=======================================
  Files         141      141           
  Lines        9061     9061           
=======================================
  Hits         8450     8450           
  Misses        611      611           

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[exploreriii]

Request step securtiy support please @hiero-ledger/security-maintainers

[AntonioCeppellini]

Request step securtiy support please @hiero-ledger/security-maintainers

I'm from mobile right now :(, I need to study the thing a bit more to resolve it, need a day to come back to my laptop🥲🥲 @exploreriii

[AntonioCeppellini]

Thanks for your contribution @AkshayKumarSahu well done!

[exploreriii]

Thank you @AkshayKumarSahu and congratulations

[github-actions]

🎉 Nice work completing a Good First Issue!

Thank you for your contribution to the Hiero Python SDK! We're excited to have you as part of our community.

Here are some issues labeled Beginner you might be interested in working on next:

1. [Beginner]: Update max assignments reached rejection message with more information

   🐥 Beginner Friendly This issue is intended for contributors who have previously completed a Good First Issue in Hiero and are at a beginner level...

2. [Beginner]: Edit .github/scripts/bot-assignment-check.sh so triage members max assignment is protected from being a mentor

   🐥 Beginner Friendly This issue is intended for contributors who have previously completed a Good First Issue in Hiero and are at a beginner level...

3. [Beginner]: Update .github/scripts/bot-inactivity-unassign.sh to skip if 'discussion' label is added

   🐥 Beginner Friendly This issue is intended for contributors who have previously completed a Good First Issue in Hiero and are at a beginner level...

🌟 Stay connected with the project:

• ⭐ Star this repository to show your support
• 👀 Watch this repository to get notified of new issues and releases

We look forward to seeing more contributions from you! If you have any questions, feel free to ask in our Discord community.

From the Hiero Python SDK Team 🚀