Skip to content

Conversation

@georgii-borovinskikh-sonarsource
Copy link
Contributor

No description provided.

@hashicorp-vault-sonar-prod
Copy link

hashicorp-vault-sonar-prod bot commented Nov 28, 2025

SLVS-2718

browserService = Substitute.For<IBrowserService>();
testSubject = new GeneralOptionsDialogControlViewModel(settings, browserService, openSettingsFileCommand);
focusOnNewCodeService = Substitute.For<IFocusOnNewCodeServiceUpdater>();
focusOnNewCodeService.Current.Returns(new FocusOnNewCodeStatus(false)); // todo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

todo?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

forgot to remove it after fixing the tests

Base automatically changed from gb/report-view-stateful-new-code to feature/persistent-new-code November 28, 2025 13:39
@georgii-borovinskikh-sonarsource georgii-borovinskikh-sonarsource force-pushed the gb/new-code-general-options-checkbox branch from 9234f6b to bef1922 Compare November 28, 2025 13:50
@georgii-borovinskikh-sonarsource georgii-borovinskikh-sonarsource merged commit e07ca64 into feature/persistent-new-code Nov 28, 2025
3 checks passed
@georgii-borovinskikh-sonarsource georgii-borovinskikh-sonarsource deleted the gb/new-code-general-options-checkbox branch November 28, 2025 13:56
@sonarqubecloud
Copy link

SonarQube reviewer guide

Summary: Add "Focus on New Code" toggle to general settings with UI and service integration.

Review Focus: Ensure IFocusOnNewCodeServiceUpdater event subscription is properly disposed to prevent memory leaks. Verify the bidirectional binding between UI and service works correctly when settings change externally.

Start review at: src/Integration.Vsix/Settings/GeneralOptionsDialogControlViewModel.cs. This contains the core logic for the new feature including event handling and state management.

💬 Please send your feedback

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

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