-
Notifications
You must be signed in to change notification settings - Fork 35
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Reduce cognitive complexity of AstraUtils#getFullyQualifiedName #88
Comments
Hi .. can you please assign me this issue. |
Hello @kapilgahlot1998, Sure! Thanks for your interest in contributing to Astra. I'll assign this over to you now. |
Hi @RadikalJin .. I have fixed the code also build the service. But I didn't find any test case for this AstraUtils#getFullyQualifiedName. Can you please provide me some test cases details that I can cover in the test cases. Let me know if anything else required from myside |
Hello @kapilgahlot1998, that sounds good! There aren’t specific tests for that function but it’s used very widely across all the tests. If you’re able to run the full set of tests, that should give us the coverage needed to verify the change. If you raise a pull request, we can also run a branch build. |
@RadikalJin Yes, I have run a full build that is success. Let me create a PR. Thanks |
@RadikalJin Created PR for the same. Please review and let me know if any changes required in the code I will happy to do that. |
Reduce cognitive complexity of AstraUtils#getFullyQualifiedName #88
Fixed by @kapilgahlot1998 under #93 |
See https://sonarcloud.io/project/issues?resolved=false&types=CODE_SMELL&id=alfasoftware_astra&open=AXpYXsQlw7HmrLPw-qnT
The text was updated successfully, but these errors were encountered: