🆕🐥 First Timers Only
This issue is reserved for people who have never contributed to this project or any open source project in general.
We know that creating a pull request (PR) is a major barrier for new contributors.
The goal of this issue and all other issues labeled by 'good first issue' is to help you make your first contribution.
👾 Description of the issue
In the hiero-base module, the FileClientImpl class implements the isDeleted method, which checks if a file has been deleted by interacting with the ProtocolLayerClient. However, this method currently lacks unit and integration tests. Scenarios such as successfully checking if a file is deleted, handling invalid FileId values, and managing exceptions (e.g., HieroException) are untested, leaving the functionality open to potential issues.
Suggested solution
Create a new test class or extend the existing test suite for FileClientImpl to include tests for the isDeleted method. Use JUnit to validate:
Positive scenarios: Successfully checking the deletion status for a valid FileId.
Negative scenarios: Handling null FileId, invalid or non-existent file IDs, and edge cases like files that are not yet deleted.
Error handling: Ensuring that appropriate exceptions (e.g., HieroException) are thrown for failures, such as issues with querying the file info.
📋 Step by step guide to do a contribution
If you have never contributed to an open source project at GitHub, the following step-by-step guide will introduce you to the workflow.
A more detailed general documentation of the GitHub PR workflow can be found here.
🎉 Contribute to Hacktoberfest
Solve this issue as part of the Hacktoberfest event and get a chance to receive cool goodies like a T-Shirt. 🎽
🤔 Additional informantion
If you have any questions, just ask us directly in this issue by adding a comment. A general manual about open-source contributions can be found here.
🆕🐥 First Timers Only
This issue is reserved for people who have never contributed to this project or any open source project in general.
We know that creating a pull request (PR) is a major barrier for new contributors.
The goal of this issue and all other issues labeled by 'good first issue' is to help you make your first contribution.
👾 Description of the issue
In the
hiero-basemodule, theFileClientImplclass implements theisDeletedmethod, which checks if a file has been deleted by interacting with theProtocolLayerClient. However, this method currently lacks unit and integration tests. Scenarios such as successfully checking if a file is deleted, handling invalidFileIdvalues, and managing exceptions (e.g.,HieroException) are untested, leaving the functionality open to potential issues.Suggested solution
Create a new test class or extend the existing test suite for
FileClientImplto include tests for theisDeletedmethod. Use JUnit to validate:Positive scenarios: Successfully checking the deletion status for a valid
FileId.Negative scenarios: Handling null
FileId, invalid or non-existent file IDs, and edge cases like files that are not yet deleted.Error handling: Ensuring that appropriate exceptions (e.g.,
HieroException) are thrown for failures, such as issues with querying the file info.📋 Step by step guide to do a contribution
If you have never contributed to an open source project at GitHub, the following step-by-step guide will introduce you to the workflow.
A more detailed general documentation of the GitHub PR workflow can be found here.
🎉 Contribute to Hacktoberfest
Solve this issue as part of the Hacktoberfest event and get a chance to receive cool goodies like a T-Shirt. 🎽
🤔 Additional informantion
If you have any questions, just ask us directly in this issue by adding a comment. A general manual about open-source contributions can be found here.