test(): add unit tests for common.go (0% ➝ 100% coverage)

[yuvraj-kolkar17]

Description

Added comprehensive unit tests for the cmd/common.go file to achieve 100% test coverage.
This ensures the core utility functions in the common.go file behave as expected across various edge cases.

Part of #46

How Has This Been Tested?

go test ./cmd -coverprofile=coverage.out
go tool cover -func=coverage.out

Before vs After:

Before	After

Checklist:

• The title of the PR states what changed and the related issues number (used for the release note).
• Does this PR requires documentation updates?
• I've updated documentation as required by this PR.
• I have performed a self-review of my own code.
• I have commented my code, particularly in hard-to-understand areas.
• I have tested it for all user roles.
• I have added all the required unit test cases.

Does this PR introduce a breaking change?

[yuvraj-kolkar17]

Hi @priyank-upadhyay @narmidm @gourishkb @Rahul-D78

Can you please review this PR !

[yuvraj-kolkar17]

Hello @narmidm sir, should I write unit tests, integration tests, or both for the other files in /cmd/? or shift to the /pkg?
Also, PTAL and review this PR and #75 when you have a chance. Thanks a lot!

[Alokzh]

Hello @yuvraj-kolkar17
This PR looks good ! Just two small suggestions:

1. Can you please remove TestGlobalVariables ? It modifies global state which can cause issues & I think global variables don't really need testing since they're just declarations.
2. Also can you please consider adding t.Parallel() for faster test execution.

Thanks !

[yuvraj-kolkar17]

Hello @yuvraj-kolkar17 This PR looks good ! Just two small suggestions:

1. Can you please remove TestGlobalVariables ? It modifies global state which can cause issues & I think global variables don't really need testing since they're just declarations.
2. Also can you please consider adding t.Parallel() for faster test execution.

Thanks !

ok i will do that @Alokzh

[yuvraj-kolkar17]

@Alokzh PTAL
I’ve just updated the changes on the test/add-cmd-common-tests branch.
Are there any other areas where I could help out?

[Alokzh]

Thnx @yuvraj-kolkar17
Can you please sign off the commit