Skip to content

AB#31366 SQ cleanup AssessmentScoresWidgetViewComponent.cs#1891

Merged
AndreGAot merged 2 commits intodevfrom
feature/AB#31366-SQ-cleanup-AG
Jan 12, 2026
Merged

AB#31366 SQ cleanup AssessmentScoresWidgetViewComponent.cs#1891
AndreGAot merged 2 commits intodevfrom
feature/AB#31366-SQ-cleanup-AG

Conversation

@AndreGAot
Copy link
Copy Markdown
Contributor

No description provided.

@github-actions
Copy link
Copy Markdown

🧪 Unit Test Results (Parallel Execution)

Tests

📊 Summary

Result Count
✅ Passed 270
❌ Failed 0
⚠️ Skipped 0

📄 HTML Reports

  • Merged Tests (HTML): Included in artifacts
    Generated automatically by CI.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request performs SonarQube-driven code cleanup and refactoring of the AssessmentScoresWidgetViewComponent.cs file. The changes modernize the code by adopting C# primary constructor syntax, extracting methods for better maintainability, and using newer C# language features.

Changes:

  • Converted traditional constructor to C# primary constructor pattern, eliminating private field declarations
  • Extracted inline logic into separate helper methods (ResolveAnswers, ResolveAiAnswers, ResolveAiAnswer)
  • Modernized syntax using collection expressions and updated LINQ patterns

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link
Copy Markdown

🧪 Unit Test Results (Parallel Execution)

Tests

📊 Summary

Result Count
✅ Passed 270
❌ Failed 0
⚠️ Skipped 0

📄 HTML Reports

  • Merged Tests (HTML): Included in artifacts
    Generated automatically by CI.

@AndreGAot AndreGAot merged commit 160ebee into dev Jan 12, 2026
22 checks passed
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