kjuulh/dagger
Archived
1
0
Fork 0
Code Issues 1 Pull Requests 1 Projects Releases Wiki Activity

Merge pull request #1770 from talentedmrjones/dagger-do-suggets-update

Browse Source 
errors on missing packages suggests project update
This commit is contained in:
Andrea Luzzardi 2022-03-10 16:58:23 -08:00 committed by GitHub
commit a7d5b366ea
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 18 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
plan/plan.go
View File

@ -49,8 +49,12 @@ func Load(ctx context.Context, cfg Config) (*Plan, error) {
if err != nil {
errstring := err.Error()
if strings.Contains(errstring, "cannot find package") && strings.Contains(errstring, "alpha.dagger.io") {
return nil, ErrIncompatiblePlan
if strings.Contains(errstring, "cannot find package") {
if strings.Contains(errstring, "alpha.dagger.io") {
return nil, ErrIncompatiblePlan
} else if strings.Contains(errstring, pkg.DaggerModule) || strings.Contains(errstring, pkg.UniverseModule) {
return nil, fmt.Errorf("%w: running `dagger project update` may resolve this", err)
}
}
return nil, err

5
tests/plan.bats
View File

@ -46,6 +46,11 @@ setup() {
assert_output --partial "attempting to load a dagger 0.1.0 project."
}
@test "plan/do: missing packages suggests project update" {
run "$DAGGER" "do" -p ./plan/do/missing_dependencies.cue test
assert_output --partial ": running \`dagger project update\` may resolve this"
}
@test "plan/hello" {
# Europa loader handles the cwd differently, therefore we need to CD into the tree at or below the parent of cue.mod
cd "$TESTDIR"

7
tests/plan/do/missing_dependencies.cue Normal file
View File

@ -0,0 +1,7 @@
package main
import (
"dagger.io/daggggger" // intentionally misspelled to force error message
)
dagger.#Plan