Get-SqlDatabase accepts Workspace as a pipeline and handles errors correctly

[NowinskiK]

Pull Request

Pull Request (PR) description

Changed

• Get-SqlDatabase accepts Workspace as a pipeline, handles errors correctly and can filter by name (Get-SqlDatabase accepts pipelines & handle errors #117).

Task list

• The PR represents a single logical change. i.e. Cosmetic updates should go in different PRs.
• Added an entry under the Unreleased section of in the CHANGELOG.md as per format.
• Local clean build passes without issue or fail tests (build.ps1 -ResolveDependency -Tasks build, test).
• Comment-based help added/updated.
• Examples appropriately added/updated.
• Unit tests added/updated..
• Integration tests added/updated (where possible).
• Documentation added/updated (where applicable).
• Code follows the contribution guidelines.

[github-actions]

Linux Test Results

4 568 tests   4 567 ✅  42s ⏱️
  405 suites      1 💤
    1 files        0 ❌

Results for commit 768bc5f.

[github-actions]

WinPS51 Test Results

4 773 tests   4 772 ✅  51s ⏱️
  406 suites      1 💤
    1 files        0 ❌

Results for commit 768bc5f.

[github-actions]

WinPS71 Test Results

4 773 tests   4 772 ✅  50s ⏱️
  406 suites      1 💤
    1 files        0 ❌

Results for commit 768bc5f.

[jpomfret]

Hey @NowinskiK

Partly my fault, I didn't press the submit review button on this - but I don't think this PR got an approval before being merged?

[jpomfret]

Should this be Get-FabricSqlDatabase?

[NowinskiK]

Yes! You are absolutely right! I missed it :) Good spot, thanks!!! I will fix it soon.

[NowinskiK]

Hey @NowinskiK

Partly my fault, I didn't press the submit review button on this - but I don't think this PR got an approval before being merged?

No worries at all, Jess. There is no one fault here! I assumed that these changes are small & low risk, hence I decided to merge them. We are at the early stage of the project, so I want to keep the momentum. This happened before and I don't blame anyone as we all know well what we are doing here (I still believe that bigger changes even now should go via full review).