Fix yarn package name parsing (#10)

- Fix yarn package name parsing to account for beginning '@'
- Add tests to test package name parsing function
- Add tests to CI and Deploy workflows

.github/scripts/Tests.ps1

@@ -0,0 +1 @@
+Invoke-Pester -Path (Join-Path $PSScriptRoot "..\..\tests") -Verbose

.github/workflows/CI.yml

@@ -23,3 +23,6 @@ jobs:
       - name: Build module
         run: ./build.ps1
         shell: pwsh
+      - name: Run tests
+        run: ./.github/scripts/Tests.ps1
+        shell: pwsh

.github/workflows/Deploy.yml

@@ -21,6 +21,9 @@ jobs:
       - name: Build Crescendo module
         run: ./build.ps1
         shell: pwsh
+      - name: Run tests
+        run: ./.github/scripts/Tests.ps1
+        shell: pwsh
       - name: Publish PowerShell Module
         run: ./.github/scripts/Deploy.ps1
         shell: pwsh

.gitignore

@@ -1,4 +1,4 @@
 obj/*
 output/*
 yarn.crescendo/*
-sample/node_modules/*
+**/node_modules/*

build.ps1

@@ -57,6 +57,4 @@ $ManifestInfo = @{
     ProjectUri    = 'https://github.com/kelleyma49/yarn.crescendo'
 }
 
-Update-ModuleManifest -Path (Join-Path $Output "yarn.crescendo.psd1") @ManifestInfo -Verbose
-
-# Invoke-Pester -Path "$PSScriptRoot\tests"
+Update-ModuleManifest -Path (Join-Path $Output "yarn.crescendo.psd1") @ManifestInfo -Verbose

src/Helpers/Find-PackageNameWithoutVersion.ps1

@@ -0,0 +1,8 @@
+
+function Find-PackageNameWithoutVersion($name) {
+    $idx = $name.LastIndexOf('@')
+    if ($idx -gt 0) {
+        $name = $name.Substring(0, $idx)
+    }
+    $name
+}

src/Helpers/Get-Packages.ps1

@@ -4,14 +4,15 @@ param ( $CommandName,
     $CommandAst,
     $FakeBoundParameters )
 
+. (Join-Path $PSScriptRoot "Find-PackageNameWithoutVersion.ps1")
+
 if ([string]::IsNullOrEmpty($WordToComplete)) {
     $json = & yarn list --json | ConvertFrom-Json
 }
 else {
     $json = & yarn list --pattern $WordToComplete --json | ConvertFrom-Json
 }
 $names = $json.data.trees | ForEach-Object { 
-    $nameWithoutVersion = $_.name.Split('@')[0] 
-    $nameWithoutVersion
+    Find-PackageNameWithoutVersion $_.name
 }
-return $names 
+return $names 

tests/Helpers.Tests.ps1

@@ -0,0 +1,38 @@
+#Import-Module "$PSScriptRoot\..\yarn-crescendo\yarn.crescendo.psd1" -Force
+BeforeAll {
+    . "$PSScriptRoot\..\src\Helpers\Find-PackageNameWithoutVersion.ps1"
+}
+
+Describe "Find-PackageNameWithoutVersion" {
+    It "should return same string with -" {
+        Find-PackageNameWithoutVersion "some-package" | Should -Be "some-package"
+    }
+
+    It "should return same string with ." {
+        Find-PackageNameWithoutVersion "example.com" | Should -Be "example.com"
+    }
+
+    It "should return same string with _" {
+        Find-PackageNameWithoutVersion "under_score" | Should -Be "under_score"
+    }
+
+    It "should return same string starting with numbers" {
+        Find-PackageNameWithoutVersion "123numeric" | Should -Be "123numeric"
+    }
+
+    It "should return same string starting @" {
+        Find-PackageNameWithoutVersion "@npm/thingy" | Should -Be "@npm/thingy"
+    }
+
+    It "should strip number" {
+        Find-PackageNameWithoutVersion "[email protected]" | Should -Be "thingy"
+    }
+
+    It "should strip number with ^" {
+        Find-PackageNameWithoutVersion "thingy@^1.2.3" | Should -Be "thingy"
+    }
+
+    It "should strip number" {
+        Find-PackageNameWithoutVersion "@npm/[email protected]" | Should -Be "@npm/thingy"
+    }
+}