Skip to content
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

SLVS-1490 Refactor test class #5729

Conversation

gabriela-trutan-sonarsource
Copy link
Contributor

@gabriela-trutan-sonarsource gabriela-trutan-sonarsource commented Oct 7, 2024

…log.

This has multiple advantages including better testability of code and the fact that the callers don't have to add references to all the services required to show the ManageBinding dialog,
@hashicorp-vault-sonar-prod hashicorp-vault-sonar-prod bot changed the title SLVS-1490 Refactor test class SLVS-1490 Refactor test class Oct 7, 2024
@gabriela-trutan-sonarsource gabriela-trutan-sonarsource marked this pull request as draft October 7, 2024 10:27
Base automatically changed from gt/validate-fix-suggestion to feature/fix-suggestion October 7, 2024 13:49
@gabriela-trutan-sonarsource gabriela-trutan-sonarsource marked this pull request as ready for review October 7, 2024 13:53
Copy link

sonarqubecloud bot commented Oct 7, 2024

@gabriela-trutan-sonarsource gabriela-trutan-sonarsource merged commit 2fbb374 into feature/fix-suggestion Oct 7, 2024
2 checks passed
@gabriela-trutan-sonarsource gabriela-trutan-sonarsource deleted the gt/binding-suggestion-handler-tests-refactor branch October 7, 2024 14:20
vnaskos-sonar pushed a commit that referenced this pull request Oct 8, 2024
vnaskos-sonar pushed a commit that referenced this pull request Oct 9, 2024
vnaskos-sonar pushed a commit that referenced this pull request Oct 9, 2024
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.

2 participants