Skip to content

Conversation

@Subhrasameerdash
Copy link
Contributor

Description:
Format node unit tests with Black for consistent style.

Run Black on tests/unit/node*.py

Related issue(s):

Fixes #1545

Notes for reviewer:

Checklist

Signed-off-by: SubhraSameerDash <2303105_cseai@gita.edu.in>
Copilot AI review requested due to automatic review settings January 27, 2026 17:51
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 27, 2026

Walkthrough

This pull request applies Black code formatter standardization to four test files in the test suite (node_address_test.py, node_create_transaction_test.py, node_tls_test.py, node_update_transaction_test.py) to improve code consistency. No functional or behavioral changes are introduced.

Changes

Cohort / File(s) Summary
Black formatting applied to test files
tests/unit/node_address_test.py, tests/unit/node_create_transaction_test.py, tests/unit/node_tls_test.py, tests/unit/node_update_transaction_test.py
Formatting standardization: multiline import restructuring, string literal quote normalization (single to double quotes), addition of trailing commas, reflowing of constructor calls and assertions across multiple lines, indentation adjustments, and whitespace consistency. No changes to test logic or assertions.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: formatting node unit tests using Black, which matches the changeset.
Description check ✅ Passed The description is related to the changeset, explaining the purpose of running Black on node test files and referencing the linked issue.
Linked Issues check ✅ Passed The PR implements the core requirement from issue #1545: formatting tests/unit/node*.py files with Black. All four affected test files show only formatting changes with no functional modifications.
Out of Scope Changes check ✅ Passed All changes are within scope—only formatting modifications to node unit test files as specified in issue #1545. No unrelated code changes detected.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
  • 📝 Generate docstrings

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

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

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR applies Black code formatting to node-related unit test files to ensure consistent code style across the test suite.

Changes:

  • Formatted tests/unit/node*.py files with Black formatter
  • Applied line breaking, quote normalization, trailing commas, and whitespace cleanup
  • No logic or functionality changes

Reviewed changes

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

File Description
tests/unit/node_update_transaction_test.py Applied Black formatting: line breaks for long assert statements, trailing commas
tests/unit/node_tls_test.py Applied Black formatting: quote normalization, whitespace cleanup, line breaks, parenthesized context managers
tests/unit/node_create_transaction_test.py Applied Black formatting: line breaks for long assert statements
tests/unit/node_address_test.py Applied Black formatting: quote normalization, line breaks, trailing commas, blank lines

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

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

@manishdait
Copy link
Contributor

Thanks for the PR, @Subhrasameerdash. Please add a corresponding entry to CHANGELOG.md under ## Test.

@Subhrasameerdash Subhrasameerdash deleted the chore/format-node-tests-black branch January 27, 2026 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Good First Issue]: format black tests/unit/node*.py

2 participants