🩹 [Patch]: Add Static code analysis back to the process (#17)

MariusStorhaug · web-flow · commit 7fb9bf75b66b · 2024-03-27T13:47:05.000+01:00
## Description - Add Static code analysis back to the process ## Type of change  - [ ] 📖 [Docs] - [ ] 🪲 [Fix] - [x] 🩹 [Patch] - [ ] ⚠️ [Security fix] - [ ] 🚀 [Feature] - [ ] 🌟 [Breaking change] ## Checklist  - [x] I have performed a self-review of my own code - [x] I have commented my code, particularly in hard-to-understand areas
diff --git a/.github/workflows/workflow.yml b/.github/workflows/workflow.yml
@@ -55,8 +55,40 @@ permissions:
   statuses: write
 
 jobs:
+  TestSourceCode:
+    name: Test source code
+    strategy:
+      fail-fast: false
+      matrix:
+        shell: [pwsh]
+        os: [ubuntu-latest, macos-latest, windows-latest]
+        include:
+          - shell: powershell
+            os: windows-latest
+    runs-on: ${{ matrix.os }}
+    steps:
+      - name: Checkout Code
+        uses: actions/checkout@v4
+
+      - name: Initialize environment
+        uses: PSModule/Initialize-PSModule@v1
+        with:
+          Version: ${{ inputs.Version }}
+          Prerelease: ${{ inputs.Prerelease }}
+          Shell: ${{ matrix.shell }}
+
+      - name: Test built module
+        uses: PSModule/Test-PSModule@v1
+        if: ${{ inputs.SkipTests != true }}
+        with:
+          Name: ${{ inputs.Name }}
+          Path: ${{ inputs.Path }}
+          Shell: ${{ matrix.shell }}
+          TestType: 'SourceCode'
+
   BuildModule:
     name: Build module
+    needs: TestSourceCode
     runs-on: ubuntu-latest
     steps:
       - name: Checkout Code
@@ -128,6 +160,7 @@ jobs:
           Name: ${{ inputs.Name }}
           Path: ${{ inputs.ModulesOutputPath }}
           Shell: ${{ matrix.shell }}
+          TestType: 'Module'
 
   LintDocs:
     name: Lint documentation
diff --git a/README.md b/README.md
@@ -18,8 +18,11 @@ The workflow is designed to be trigger on pull requests to the repository's defa
 When a pull request is opened, closed, reopened, synchronized (push), or labeled, the workflow will run.
 Depending on the labels in the pull requests, the workflow will result in different outcomes.
 
+- [Test-PSModule](https://github.com/PSModule/Test-PSModule/) - Tests the source code using PSScriptAnalyzer, PSModule source code tests suites. This runs on 4 different environments to check compatibility.
+  - PowerShell 7.x on Windows, Ubuntu and macOS.
+  - Windows PowerShell 5.1 on Windows.
 - [Build-PSModule](https://github.com/PSModule/Build-PSModule/) - Compiles the repository into an efficient PowerShell module.
-- [Test-PSModule](https://github.com/PSModule/Test-PSModule/) - Tests the compiled module using PSScriptAnalyzer, PSModule and module tests suites from the module repository. This runs on 4 different environments to check compatibility.
+- [Test-PSModule](https://github.com/PSModule/Test-PSModule/) - Tests the compiled module using PSScriptAnalyzer, PSModule module tests and custom module tests from the module repository. This runs on 4 different environments to check compatibility.
   - PowerShell 7.x on Windows, Ubuntu and macOS.
   - Windows PowerShell 5.1 on Windows.
 - [Publish-PSModule](https://github.com/PSModule/Publish-PSModule/) - Publishes the module to the PowerShell Gallery, docs to GitHub Pages, and creates a release on the GitHub repository.
diff --git a/tests/src/PSModuleTest/PSModuleTest.psd1 b/tests/src/PSModuleTest/PSModuleTest.psd1
@@ -1,4 +1,5 @@
 @{
-    ModuleVersion = '0.0.0'
-    RootModule    = 'PSModuleTest.psm1'
+    ModuleVersion      = '0.0.0'
+    RootModule         = 'PSModuleTest.psm1'
+    PowerShellHostName = ''
 }

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,5 @@`
`1`	`1`	`@{`
`2`		`- ModuleVersion = '0.0.0'`
`3`		`- RootModule = 'PSModuleTest.psm1'`
	`2`	`+ ModuleVersion = '0.0.0'`
	`3`	`+ RootModule = 'PSModuleTest.psm1'`
	`4`	`+ PowerShellHostName = ''`
`4`	`5`	`}`