refactor: validate backup retention days on SQL DB

[v-alexmoraru]

No description provided.

[Copilot]

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds SQLDatabase-specific validation for the properties.backupRetentionDays field when using the fs set command, including a dedicated error message and test coverage.

Changes:

• Added validate_sql_database_property to enforce backupRetentionDays integer range validation.
• Integrated SQLDatabase property validation into fs set execution for SQLDatabase items.
• Added constants and an error message helper, plus unit tests for valid/invalid inputs.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
tests/test_utils/test_fab_cmd_set_utils.py	Adds unit tests covering SQLDatabase backupRetentionDays validation scenarios.
src/fabric_cli/utils/fab_cmd_set_utils.py	Implements SQLDatabase-specific validator for backupRetentionDays.
src/fabric_cli/errors/common.py	Adds a dedicated error message for invalid backupRetentionDays.
src/fabric_cli/core/fab_constant.py	Introduces SQLDatabase validation constants (min/max days and property key).
src/fabric_cli/commands/fs/set/fab_fs_set_item.py	Hooks SQLDatabase property validation into the set command execution path.

[Copilot]

Pull request overview

Copilot reviewed 10 out of 10 changed files in this pull request and generated 7 comments.

[Copilot]

Pull request overview

Copilot reviewed 10 out of 10 changed files in this pull request and generated 4 comments.

[ayeshurun]

@v-alexmoraru please add PR description and link the issue to it (by adding - Fixed #<issue number>

[Copilot]

Pull request overview

Copilot reviewed 13 out of 13 changed files in this pull request and generated 8 comments.