Seagate / cortx-ha

CORTX ha (High-Availability) is responsible for ensuring that CORTX Solution is available in case of any hardware component or software service failures. It takes care of failover/ failback control flow for affected services and stabilizes them across CORTX cluster.
https://github.com/Seagate/cortx
GNU Affero General Public License v3.0
4 stars 45 forks source link

CORTX-29708: Add proper validation of args in mock_health_event_publisher.py file #690

Closed gargpalak048 closed 2 years ago

gargpalak048 commented 2 years ago

Signed-off-by: Palak Garg palak.garg@seagate.com

Problem Statement

Design

Coding

Testing

Review Checklist

Review Checklist

Documentation

Checklist for Author

gargpalak048 commented 2 years ago

just give a single space before options in below print(f"Error: Invalid Argument:{option}\n" other chnages looks good to me

Done

Madhura-08 commented 2 years ago

@gargpalak048 Please add Happy Path testing details. ie script execution with supported args.

gargpalak048 commented 2 years ago

@Madhura-08 Added Happy Test path details in test_results.txt file

ArchanaLimaye commented 2 years ago

@gargpalak048 please add link to test file in "Testing" section of the PR

gargpalak048 commented 2 years ago

@gargpalak048 please add link to test file in "Testing" section of the PR

Added