SqlServerDsc: Add Script Analyzer rule to help detect if resource do not load SMO

[johlju]

Pull Request (PR) description

• SqlServerDsc
  • Added a new script analyzer rule to verify that Import-SQLPSModule or Connect-SQL
    (that implicitly calls Import-SQLPSModule) is present in each Get-, Test-,
    and Set-TargetResource function. If neither command is not needed then the
    analyzer rule should be overridden (issue #1683).
  • Added a new pipeline job that runs Script Analyzer on all PowerShell scripts
    in the source folder. The rules are defined by the Script Analyzer settings
    file .vscode\analyzersettings.psd1 (which also the Visual Studio Code
    PowerShell extension uses).
  • Added unit tests and integration tests for SQL Server 2019
    (issue #1310).
  • Suppressed new custom Script Analyzer rule SqlServerDsc.AnalyzerRules\Measure-CommandsNeededToLoadSMO
    for Get-, Test-, and Set-TargetResource functions in the resources.
  • Minor lint errors throughout the repository.
• SqlSetup
  • Minor refactor due to source code lint errors. The loop what evaluates
    the configuration parameters *FailoverCluster was change to a foreach().

This Pull Request (PR) fixes the following issues

• Fixes SqlServerDsc: Add ScriptAnalyzer rule to validate that resource loads SMO #1683

Task list

• Added an entry to the change log under the Unreleased section of the
  file CHANGELOG.md. Entry should say what was changed and how that
  affects users (if applicable), and reference the issue being resolved
  (if applicable).
• Resource documentation updated in the resource's README.md.
• Resource parameter descriptions updated in schema.mof.
• Comment-based help updated, including parameter descriptions.
• Localization strings updated.
• Examples updated.
• Unit tests updated. See DSC Community Testing Guidelines.
• Integration tests updated (where possible). See DSC Community Testing Guidelines.
• Code changes adheres to DSC Community Style Guidelines.

---

This change is 

[codecov]

Codecov Report

Merging #1684 (40b5af6) into main (511b365) will decrease coverage by 0%.
The diff coverage is 100%.

@@         Coverage Diff          @@
##           main   #1684   +/-   ##
====================================
- Coverage    97%     97%   -1%     
====================================
  Files        38      38           
  Lines      6261    6260    -1     
====================================
- Hits       6104    6103    -1     
  Misses      157     157           

Flag	Coverage Δ
unit	97% <100%> (-1%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
source/DSCResources/DSC_SqlAG/DSC_SqlAG.psm1	100% <ø> (ø)
...Resources/DSC_SqlAGListener/DSC_SqlAGListener.psm1	100% <ø> (ø)
...SCResources/DSC_SqlAGReplica/DSC_SqlAGReplica.psm1	100% <ø> (ø)
...Resources/DSC_SqlAgentAlert/DSC_SqlAgentAlert.psm1	96% <ø> (ø)
...ces/DSC_SqlAgentFailsafe/DSC_SqlAgentFailsafe.psm1	95% <ø> (ø)
...ces/DSC_SqlAgentOperator/DSC_SqlAgentOperator.psm1	82% <ø> (ø)
source/DSCResources/DSC_SqlAlias/DSC_SqlAlias.psm1	98% <ø> (ø)
...DSC_SqlAlwaysOnService/DSC_SqlAlwaysOnService.psm1	94% <ø> (ø)
...ces/DSC_SqlConfiguration/DSC_SqlConfiguration.psm1	94% <ø> (ø)
.../DSCResources/DSC_SqlDatabase/DSC_SqlDatabase.psm1	96% <ø> (ø)
... and 25 more

[johlju]

Reviewed 49 of 49 files at r1.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @johlju)