Aligns file structure to top-level package with run command

indigostar-kr · Jun 8, 2020 · 378fbff · 378fbff
1 parent 9d6fe38
commit 378fbff
Show file tree

Hide file tree

Showing 21 changed files with 48 additions and 51 deletions.
diff --git a/yarn/build.go → build.go b/yarn/build.go → build.go
@@ -1,4 +1,4 @@
-package yarn
+package yarninstall
 
 import (
 	"os"

diff --git a/yarn/build_test.go → build_test.go b/yarn/build_test.go → build_test.go
@@ -1,4 +1,4 @@
-package yarn_test
+package yarninstall_test
 
 import (
 	"bytes"
@@ -13,8 +13,8 @@ import (
 	"github.com/paketo-buildpacks/packit/chronos"
 	"github.com/paketo-buildpacks/packit/postal"
 	"github.com/paketo-buildpacks/packit/scribe"
-	"github.com/paketo-buildpacks/yarn-install/yarn"
-	"github.com/paketo-buildpacks/yarn-install/yarn/fakes"
+	yarninstall "github.com/paketo-buildpacks/yarn-install"
+	"github.com/paketo-buildpacks/yarn-install/fakes"
 	"github.com/sclevine/spec"
 
 	. "github.com/onsi/gomega"
@@ -78,7 +78,7 @@ func testBuild(t *testing.T, context spec.G, it spec.S) {
 
 		buffer = bytes.NewBuffer(nil)
 
-		build = yarn.Build(dependencyService, cacheMatcher, installProcess, clock, scribe.NewLogger(buffer))
+		build = yarninstall.Build(dependencyService, cacheMatcher, installProcess, clock, scribe.NewLogger(buffer))
 	})
 
 	it.After(func() {

diff --git a/buildpack.toml b/buildpack.toml
@@ -6,7 +6,7 @@ api = "0.2"
   homepage = "https://github.com/paketo-buildpacks/yarn-install"
 
 [metadata]
-  include_files = ["bin/build", "bin/detect", "buildpack.toml"]
+  include_files = ["bin/build", "bin/detect", "bin/run", "buildpack.toml"]
   pre_package = "./scripts/build.sh"
   [metadata.default_versions]
     yarn = "1.*"

diff --git a/yarn/cache_handler.go → cache_handler.go b/yarn/cache_handler.go → cache_handler.go
@@ -1,4 +1,4 @@
-package yarn
+package yarninstall
 
 type CacheHandler struct{}
 

diff --git a/yarn/cache_handler_test.go → cache_handler_test.go b/yarn/cache_handler_test.go → cache_handler_test.go
@@ -1,9 +1,9 @@
-package yarn_test
+package yarninstall_test
 
 import (
 	"testing"
 
-	"github.com/paketo-buildpacks/yarn-install/yarn"
+	yarninstall "github.com/paketo-buildpacks/yarn-install"
 	"github.com/sclevine/spec"
 
 	. "github.com/onsi/gomega"
@@ -13,11 +13,11 @@ func testCacheHandler(t *testing.T, context spec.G, it spec.S) {
 	var (
 		Expect = NewWithT(t).Expect
 
-		cacheHandler yarn.CacheHandler
+		cacheHandler yarninstall.CacheHandler
 	)
 
 	it.Before(func() {
-		cacheHandler = yarn.NewCacheHandler()
+		cacheHandler = yarninstall.NewCacheHandler()
 	})
 
 	context("Match", func() {

diff --git a/cmd/detect/main.go b/cmd/detect/main.go
diff --git a/yarn/detect.go → detect.go b/yarn/detect.go → detect.go
@@ -1,4 +1,4 @@
-package yarn
+package yarninstall
 
 import (
 	"errors"

diff --git a/yarn/detect_test.go → detect_test.go b/yarn/detect_test.go → detect_test.go
@@ -1,4 +1,4 @@
-package yarn_test
+package yarninstall_test
 
 import (
 	"errors"
@@ -8,8 +8,8 @@ import (
 	"testing"
 
 	"github.com/paketo-buildpacks/packit"
-	"github.com/paketo-buildpacks/yarn-install/yarn"
-	"github.com/paketo-buildpacks/yarn-install/yarn/fakes"
+	yarninstall "github.com/paketo-buildpacks/yarn-install"
+	"github.com/paketo-buildpacks/yarn-install/fakes"
 	"github.com/sclevine/spec"
 
 	. "github.com/onsi/gomega"
@@ -35,7 +35,7 @@ func testDetect(t *testing.T, context spec.G, it spec.S) {
 		versionParser = &fakes.VersionParser{}
 		versionParser.ParseVersionCall.Returns.Version = "some-version"
 
-		detect = yarn.Detect(versionParser)
+		detect = yarninstall.Detect(versionParser)
 	})
 
 	it("returns a plan that provides node_modules", func() {
@@ -50,14 +50,14 @@ func testDetect(t *testing.T, context spec.G, it spec.S) {
 			Requires: []packit.BuildPlanRequirement{
 				{
 					Name: "node_modules",
-					Metadata: yarn.BuildPlanMetadata{
+					Metadata: yarninstall.BuildPlanMetadata{
 						Launch: true,
 					},
 				},
 				{
 					Name:    "node",
 					Version: "some-version",
-					Metadata: yarn.BuildPlanMetadata{
+					Metadata: yarninstall.BuildPlanMetadata{
 						VersionSource: "package.json",
 						Build:         true,
 						Launch:        true,
@@ -86,13 +86,13 @@ func testDetect(t *testing.T, context spec.G, it spec.S) {
 				Requires: []packit.BuildPlanRequirement{
 					{
 						Name: "node_modules",
-						Metadata: yarn.BuildPlanMetadata{
+						Metadata: yarninstall.BuildPlanMetadata{
 							Launch: true,
 						},
 					},
 					{
 						Name: "node",
-						Metadata: yarn.BuildPlanMetadata{
+						Metadata: yarninstall.BuildPlanMetadata{
 							Build:  true,
 							Launch: true,
 						},

diff --git a/yarn/fakes/cache_matcher.go → fakes/cache_matcher.go b/yarn/fakes/cache_matcher.go → fakes/cache_matcher.go
diff --git a/yarn/fakes/dependency_service.go → fakes/dependency_service.go b/yarn/fakes/dependency_service.go → fakes/dependency_service.go
diff --git a/yarn/fakes/executable.go → fakes/executable.go b/yarn/fakes/executable.go → fakes/executable.go
diff --git a/yarn/fakes/install_process.go → fakes/install_process.go b/yarn/fakes/install_process.go → fakes/install_process.go
diff --git a/yarn/fakes/summer.go → fakes/summer.go b/yarn/fakes/summer.go → fakes/summer.go
diff --git a/yarn/fakes/transport.go → fakes/transport.go b/yarn/fakes/transport.go → fakes/transport.go
diff --git a/yarn/fakes/version_parser.go → fakes/version_parser.go b/yarn/fakes/version_parser.go → fakes/version_parser.go
diff --git a/yarn/init_test.go → init_test.go b/yarn/init_test.go → init_test.go
@@ -1,4 +1,4 @@
-package yarn_test
+package yarninstall_test
 
 import (
 	"testing"

diff --git a/yarn/install_process.go → install_process.go b/yarn/install_process.go → install_process.go
@@ -1,4 +1,4 @@
-package yarn
+package yarninstall
 
 import (
 	"bytes"

diff --git a/yarn/install_process_test.go → install_process_test.go b/yarn/install_process_test.go → install_process_test.go
@@ -1,4 +1,4 @@
-package yarn_test
+package yarninstall_test
 
 import (
 	"bytes"
@@ -12,8 +12,8 @@ import (
 
 	"github.com/paketo-buildpacks/packit/pexec"
 	"github.com/paketo-buildpacks/packit/scribe"
-	"github.com/paketo-buildpacks/yarn-install/yarn"
-	"github.com/paketo-buildpacks/yarn-install/yarn/fakes"
+	yarninstall "github.com/paketo-buildpacks/yarn-install"
+	"github.com/paketo-buildpacks/yarn-install/fakes"
 	"github.com/sclevine/spec"
 
 	. "github.com/onsi/gomega"
@@ -26,7 +26,7 @@ func testInstallProcess(t *testing.T, context spec.G, it spec.S) {
 		var (
 			workingDir     string
 			executable     *fakes.Executable
-			installProcess yarn.YarnInstallProcess
+			installProcess yarninstall.YarnInstallProcess
 			summer         *fakes.Summer
 		)
 
@@ -38,7 +38,7 @@ func testInstallProcess(t *testing.T, context spec.G, it spec.S) {
 			executable = &fakes.Executable{}
 			summer = &fakes.Summer{}
 
-			installProcess = yarn.NewYarnInstallProcess(executable, summer, scribe.NewLogger(bytes.NewBuffer(nil)))
+			installProcess = yarninstall.NewYarnInstallProcess(executable, summer, scribe.NewLogger(bytes.NewBuffer(nil)))
 		})
 
 		context("we should run yarn install when", func() {
@@ -127,7 +127,7 @@ func testInstallProcess(t *testing.T, context spec.G, it spec.S) {
 			executable       *fakes.Executable
 			summer           *fakes.Summer
 
-			installProcess yarn.YarnInstallProcess
+			installProcess yarninstall.YarnInstallProcess
 		)
 
 		it.Before(func() {
@@ -159,7 +159,7 @@ func testInstallProcess(t *testing.T, context spec.G, it spec.S) {
 			path = os.Getenv("PATH")
 			os.Setenv("PATH", "/some/bin")
 
-			installProcess = yarn.NewYarnInstallProcess(executable, summer, scribe.NewLogger(buffer))
+			installProcess = yarninstall.NewYarnInstallProcess(executable, summer, scribe.NewLogger(buffer))
 		})
 
 		it.After(func() {

diff --git a/yarn/package_json_parser.go → package_json_parser.go b/yarn/package_json_parser.go → package_json_parser.go
@@ -1,4 +1,4 @@
-package yarn
+package yarninstall
 
 import (
 	"encoding/json"

diff --git a/yarn/package_json_parser_test.go → package_json_parser_test.go b/yarn/package_json_parser_test.go → package_json_parser_test.go
@@ -1,11 +1,11 @@
-package yarn_test
+package yarninstall_test
 
 import (
 	"io/ioutil"
 	"os"
 	"testing"
 
-	"github.com/paketo-buildpacks/yarn-install/yarn"
+	yarninstall "github.com/paketo-buildpacks/yarn-install"
 	"github.com/sclevine/spec"
 
 	. "github.com/onsi/gomega"
@@ -17,7 +17,7 @@ func testPackageJSONParser(t *testing.T, context spec.G, it spec.S) {
 	context("ParseVersion", func() {
 		var (
 			path   string
-			parser yarn.PackageJSONParser
+			parser yarninstall.PackageJSONParser
 		)
 
 		it.Before(func() {
@@ -34,7 +34,7 @@ func testPackageJSONParser(t *testing.T, context spec.G, it spec.S) {
 
 			path = file.Name()
 
-			parser = yarn.NewPackageJSONParser()
+			parser = yarninstall.NewPackageJSONParser()
 		})
 
 		it.After(func() {

diff --git a/cmd/build/main.go → run/main.go b/cmd/build/main.go → run/main.go
@@ -10,18 +10,27 @@ import (
 	"github.com/paketo-buildpacks/packit/pexec"
 	"github.com/paketo-buildpacks/packit/postal"
 	"github.com/paketo-buildpacks/packit/scribe"
-	"github.com/paketo-buildpacks/yarn-install/yarn"
+	yarninstall "github.com/paketo-buildpacks/yarn-install"
 )
 
 func main() {
+	packageJSONParser := yarninstall.NewPackageJSONParser()
 	logger := scribe.NewLogger(os.Stdout)
-
 	transport := cargo.NewTransport()
 	executable := pexec.NewExecutable("yarn")
 	summer := fs.NewChecksumCalculator()
-	installProcess := yarn.NewYarnInstallProcess(executable, summer, logger)
+	installProcess := yarninstall.NewYarnInstallProcess(executable, summer, logger)
 	dependencyService := postal.NewService(transport)
-	cacheHandler := yarn.NewCacheHandler()
+	cacheHandler := yarninstall.NewCacheHandler()
 
-	packit.Build(yarn.Build(dependencyService, cacheHandler, installProcess, chronos.DefaultClock, logger))
+	packit.Run(
+		yarninstall.Detect(packageJSONParser),
+		yarninstall.Build(
+			dependencyService,
+			cacheHandler,
+			installProcess,
+			chronos.DefaultClock,
+			logger,
+		),
+	)
 }