Merge pull request #1576 from shykes/yarn-cache-optional

yarn.#Build: simpler API, better tests

pkg/universe.dagger.io/yarn/test/yarn-test.cue

@@ -2,9 +2,9 @@ package yarn
 
 import (
 	"dagger.io/dagger"
+	"dagger.io/dagger/engine"
+
 	"universe.dagger.io/yarn"
-	"universe.dagger.io/alpine"
-	"universe.dagger.io/bash"
 )
 
 dagger.#Plan & {
@@ -13,60 +13,48 @@ dagger.#Plan & {
 		testdata2: path: "./testdata2"
 	}
 
-	actions: {
+	actions: tests: {
-		cache: dagger.#CacheDir & {
-			id: "yarn cache"
-		}
 
-		pkg: yarn.#Build & {
+		simple: {
-			source:  inputs.directories.testdata.contents
+			build: yarn.#Build & {
-			"cache": cache
+				source: inputs.directories.testdata.contents
-		}
+			}
-
+
-		_image: alpine.#Build & {
+			verify: #AssertFile & {
-			packages: bash: {}
+				input:    build.output
-		}
+				path:     "test"
-
+				contents: "output\n"
-		test: bash.#Run & {
-			image: _image.output
-			mounts: build: {
-				dest:     "/build"
-				contents: pkg.output
 			}
-			script: contents: #"""
-				test "$(cat /build/test)" = "output"
-				"""#
 		}
 
-		// FIXME: re-enable?
+		// Run yarn.#Build with a custom name
-		// TestSecretsAndFile: {
+		customName: {
-		//  pkg: #Package & {
+			build: yarn.#Build & {
-		//   source:       inputs.directories.testdata2
+				name:   "My Build"
-		//   writeEnvFile: "/.env"
+				source: inputs.directories.testdata.contents
-		//   env: {
+			}
-		//    one: "one"
+			verify: #AssertFile & {
-		//    two: "two"
+				input:    build.output
-		//   }
+				path:     "test"
-		//   secrets: {
+				contents: "output\n"
-		//    secretone: dagger.#Secret @dagger(input)
+			}
-		//    secretwo:  dagger.#Secret @dagger(input)
+		}
-		//   }
-		//  }
-
-		//  test: os.#Container & {
-		//   image: alpine.#Image & {
-		//    package: bash: true
-		//   }
-		//   shell: path: "/bin/bash"
-		//   mount: "/build": from: pkg.build
-		//   command: """
-		//    content="$(cat /build/env)"
-		//    [[ "${content}" = *"SECRETONE="* ]] && \\
-		//    [[ "${content}" = *"SECRETWO="* ]] && \\
-		//    [[ "${content}" = *"ONE=one"* ]] && \\
-		//    [[ "${content}" = *"TWO=two"* ]]
-		//    """
-		//  }
-		// }
 	}
 }
+
+// Make an assertion on the contents of a file
+#AssertFile: {
+	input:    dagger.#FS
+	path:     string
+	contents: string
+
+	_read: engine.#ReadFile & {
+		"input": input
+		"path":  path
+	}
+
+	actual: _read.contents
+
+	// Assertion
+	contents: actual
+}

pkg/universe.dagger.io/yarn/yarn.cue

@@ -13,6 +13,11 @@ import (
 
 // Build a Yarn package
 #Build: {
+	// Custom name for the build.
+	//   When building different apps in the same plan, assign
+	//   different names for optimal caching.
+	name: string | *""
+
 	// Application source code
 	source: dagger.#FS
 
@@ -33,9 +38,6 @@ import (
 	// Fix for shadowing issues
 	let yarnScript = script
 
-	// Cache to use, passed by the caller
-	cache: engine.#CacheDir
-
 	// Optional arguments for the script
 	args: [...string] | *[]
 
@@ -74,7 +76,10 @@ import (
 		mounts: {
 			"yarn cache": {
 				dest:     "/cache/yarn"
-				contents: cache
+				contents: engine.#CacheDir & {
+					// FIXME: are there character limitations in cache ID?
+					id: "universe.dagger.io/yarn.#Build \(name)"
+				}
 			}
 			"package source": {
 				dest:     "/src"