Skip to content

Make build.psd1 optional and fix bugs in 1.5.2 #89

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Jaykul merged 17 commits into from
Feb 10, 2020
Merged

Make build.psd1 optional and fix bugs in 1.5.2 #89

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
4e2a651
Update tests so they work better in VSCode
Jaykul Dec 19, 2019
9c08545
Whitespace cleanup commit
Jaykul Jan 11, 2020
84658d2
Fix #87 by switching "ModuleManifest" to "SourcePath"
Jaykul Jan 11, 2020
3dc7800
Update documentation about required changes to module manifests
Jaykul Jan 11, 2020
7d519fa
Finally make the build.psd1 file optional per #44
Jaykul Jan 11, 2020
d28c47d
Add tests to verify prefix works right
Jaykul Jan 11, 2020
cb936ce
Fix #88 by keeping the prefix from build.psd1
Jaykul Jan 11, 2020
e6a894c
Fix build by working around the bug in 1.5.2
Jaykul Jan 11, 2020
4564df7
Fix a few tests that will break if we check module manifests better
Jaykul Jan 14, 2020
0d78ae5
Fix ResolveBuildManifest Tests (it no longer throws) in this branch
Jaykul Jan 14, 2020
59dd957
Add ImportModuleManifest to wrap Get-Module
Jaykul Jan 14, 2020
d873623
Switch to searching for module manifests
Jaykul Jan 14, 2020
55ce9c8
Update to fixed azure-pipeline
Jaykul Jan 14, 2020
12d5fd9
Clean up capitalization
Jaykul Jan 15, 2020
c128593
Fix integration test and test.ps1
Jaykul Jan 15, 2020
2aa4c4f
Clean up debug output
Jaykul Jan 15, 2020
0a8a42f
Fix the normalization / guessing of SourcePath
Jaykul Jan 15, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix ResolveBuildManifest Tests (it no longer throws) in this branch
  • Loading branch information
@Jaykul
Jaykul committed Jan 14, 2020
commit 0d78ae5bdb453819bce2913b07c99ed81d139393
18 changes: 12 additions & 6 deletions Tests/Private/ResolveBuildManifest.Tests.ps1
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,16 +25,22 @@ Describe "ResolveBuildManifest" {
$Expected | Should -Be (Join-Path ${Global:Test Root Path} "build.psd1")
}

It "Should throw when passed a wrong absolute module manifest" {
{InModuleScope ModuleBuilder { ResolveBuildManifest (Join-Path (Join-Path ${Global:Test Root Path} ERROR) ModuleBuilder.psd1) }} | Should -Throw
It "Returns nothing when passed a wrong absolute module manifest" {
InModuleScope ModuleBuilder {
ResolveBuildManifest (Join-Path (Join-Path ${Global:Test Root Path} ERROR) ModuleBuilder.psd1) | Should -BeNullOrEmpty
}
}

It "Should throw when passed the wrong folder path" {
{InModuleScope ModuleBuilder { ResolveBuildManifest (Join-Path ${Global:Test Root Path} "..") }} | Should -Throw
It "Returns nothing when passed the wrong folder path" {
InModuleScope ModuleBuilder {
ResolveBuildManifest (Join-Path ${Global:Test Root Path} "..") | Should -BeNullOrEmpty
}
}

It "Should throw when passed the wrong folder relative path" {
{InModuleScope ModuleBuilder { ResolveBuildManifest (Join-Path . ..) }} | Should -Throw
It "Returns nothing when passed the wrong folder relative path" {
InModuleScope ModuleBuilder {
ResolveBuildManifest (Join-Path . ..) | Should -BeNullOrEmpty
}
}

}