novox/mesh-catalog
18
0
Fork 0
Code Issues Pull Requests 3 Actions Packages Projects Releases Wiki Activity

Update the build modules' manifest tests for the toolchain provisions (issue 1573)
mesh/merge-gate head 36259f3406f7 merged into trunk f0c8958dd217 — pass: builds dev-tools, lab, mesh-build-go, mesh-build-node → ace, g14, novox, shanks; n…
Details
mesh/repo-check head 36259f3406f7 merged into trunk f0c8958dd217 — pass: THE CHANGE ALTERS ITS OWN CHECK (merge-check.sh): main's version judged it; the ch…
Details
mesh/delivery covered: a later merge that contains it was delivered: novox/mesh-catalog@e023f69989ab (merged as b5dbfbb7 into main, walk plan-17925769516…
Details

Browse Source
This commit is contained in:
mesh-tools
2026-10-21 09:53:48 +00:00
parent d8d153295a
commit 36259f3406
3 changed files with 7 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
modules/mesh-build-go/cmd/mesh-build-go/forge_test.go
+2 -2
View File
@@ -143,8 +143,8 @@ func TestForgeFilesRenderFromTheBoundValuesInTheModulesOwnHome(t *testing.T) {
// template in the files/ archive (novox/hq ADR 0349 §2). // template in the files/ archive (novox/hq ADR 0349 §2).
func TestManifestRequiresForgeAccountKeepsKeysAndTakesFilesFromTemplates(t *testing.T) { func TestManifestRequiresForgeAccountKeepsKeysAndTakesFilesFromTemplates(t *testing.T) {
m := readForgeManifest(t) m := readForgeManifest(t)
if len(m.Requires) != 1 || m.Requires[0] != "forge-account" { if len(m.Requires) != 2 || m.Requires[0] != "forge-account" || m.Requires[1] != "go-toolchain" {
t.Errorf("requires = %v, want [forge-account]", m.Requires) t.Errorf("requires = %v, want [forge-account go-toolchain]", m.Requires)
} }
keeps := false keeps := false
for _, s := range m.State { for _, s := range m.State {
modules/mesh-build-go/cmd/mesh-build-go/manifest_test.go
+1 -1
View File
@@ -64,7 +64,7 @@ func TestManifestIsValidAndDeclaresTheAccountTheCapabilityAndThePackages(t *test
} }
} }
slices.Sort(packages) slices.Sort(packages)
if want := []string{"git", "go", "openssh", "postgresql"}; !slices.Equal(packages, want) { if want := []string{"git", "openssh", "postgresql"}; !slices.Equal(packages, want) {
t.Errorf("packages = %v, want %v", packages, want) t.Errorf("packages = %v, want %v", packages, want)
} }
} }
modules/mesh-build-node/cmd/mesh-build-node/manifest_test.go
+4 -1
View File
@@ -66,7 +66,7 @@ func TestManifestIsValidAndDeclaresTheAccountTheCapabilityAndThePackages(t *test
} }
} }
slices.Sort(packages) slices.Sort(packages)
if want := []string{"git", "nodejs", "npm", "openssh"}; !slices.Equal(packages, want) { if want := []string{"git", "openssh"}; !slices.Equal(packages, want) {
t.Errorf("packages = %v, want %v", packages, want) t.Errorf("packages = %v, want %v", packages, want)
} }
} }
@@ -134,6 +134,9 @@ func TestTheModuleCarriesItsToolchainLockAndTheManifestPlacesIt(t *testing.T) {
t.Errorf("the module carries no toolchain/%s: %v", name, err) t.Errorf("the module carries no toolchain/%s: %v", name, err)
} }
} }
if !slices.Contains(m.Requires, "node-toolchain") {
t.Errorf("requires = %v lack node-toolchain, which provides the nodejs and npm packages (issue 1573)", m.Requires)
}
if !slices.Contains(m.Requires, "npm-package-registry") { if !slices.Contains(m.Requires, "npm-package-registry") {
t.Errorf("requires = %v lack npm-package-registry, which the install resolves @novox from", m.Requires) t.Errorf("requires = %v lack npm-package-registry, which the install resolves @novox from", m.Requires)
} }