Make js.Build fully support modules

Fixes #7816
Fixes #7777
Fixes #7916

1 files changed, 27 insertions, 316 deletions

diff --git a/hugolib/js_test.go b/hugolib/js_test.go
index e34ce3867..6c27219f3 100644
--- a/hugolib/js_test.go
+++ b/hugolib/js_test.go
@@ -14,6 +14,7 @@
 package hugolib
 
 import (
+	"fmt"
 	"os"
 	"os/exec"
 	"path/filepath"
@@ -22,7 +23,6 @@ import (
 
 	"github.com/gohugoio/hugo/htesting"
 
-	"github.com/spf13/afero"
 	"github.com/spf13/viper"
 
 	qt "github.com/frankban/quicktest"
@@ -82,9 +82,7 @@ document.body.textContent = greeter(user);`
   "scripts": {},
 
   "dependencies": {
-		"to-camel-case": "1.0.0",
-		"react": "^16",
-		"react-dom": "^16"
+    "to-camel-case": "1.0.0"
   }
 }
 `
@@ -153,333 +151,46 @@ func TestJSBuild(t *testing.T) {
 
 	c := qt.New(t)
 
-	mainJS := `
-	import "./included";
-	
-	console.log("main");
-
-`
-	includedJS := `
-	console.log("included");
-	
-	`
-
-	workDir, clean, err := htesting.CreateTempDir(hugofs.Os, "hugo-test-js")
-	c.Assert(err, qt.IsNil)
-	defer clean()
-
-	v := viper.New()
-	v.Set("workingDir", workDir)
-	v.Set("disableKinds", []string{"taxonomy", "term", "page"})
-	b := newTestSitesBuilder(t).WithLogger(loggers.NewWarningLogger())
-
-	b.Fs = hugofs.NewDefault(v)
-	b.WithWorkingDir(workDir)
-	b.WithViper(v)
-	b.WithContent("p1.md", "")
-
-	b.WithTemplates("index.html", `
-{{ $js := resources.Get "js/main.js" | js.Build }}
-JS:  {{ template "print" $js }}
-
-
-{{ define "print" }}RelPermalink: {{.RelPermalink}}|MIME: {{ .MediaType }}|Content: {{ .Content | safeJS }}{{ end }}
-
-`)
-
-	jsDir := filepath.Join(workDir, "assets", "js")
-	b.Assert(os.MkdirAll(jsDir, 0777), qt.IsNil)
-	b.Assert(os.Chdir(workDir), qt.IsNil)
-	b.WithSourceFile("assets/js/main.js", mainJS)
-	b.WithSourceFile("assets/js/included.js", includedJS)
-
-	b.Build(BuildCfg{})
-
-	b.AssertFileContent("public/index.html", `
-console.log("included");
-
-`)
-
-}
-
-func TestJSBuildGlobals(t *testing.T) {
-	if !isCI() {
-		t.Skip("skip (relative) long running modules test when running locally")
-	}
-
-	wd, _ := os.Getwd()
-	defer func() {
-		os.Chdir(wd)
-	}()
-
-	c := qt.New(t)
-
-	workDir, clean, err := htesting.CreateTempDir(hugofs.Os, "hugo-test-js")
-	c.Assert(err, qt.IsNil)
-	defer clean()
-
-	v := viper.New()
-	v.Set("workingDir", workDir)
-	v.Set("disableKinds", []string{"taxonomy", "term", "page"})
-	b := newTestSitesBuilder(t).WithLogger(loggers.NewWarningLogger())
-
-	b.Fs = hugofs.NewDefault(v)
-	b.WithWorkingDir(workDir)
-	b.WithViper(v)
-	b.WithContent("p1.md", "")
-
-	jsDir := filepath.Join(workDir, "assets", "js")
-	b.Assert(os.MkdirAll(jsDir, 0777), qt.IsNil)
-	b.Assert(os.Chdir(workDir), qt.IsNil)
-
-	b.WithTemplates("index.html", `
-{{- $js := resources.Get "js/main-project.js" | js.Build -}}
-{{ template "print" (dict "js" $js "name" "root") }}
-
-{{- define "print" -}}
-{{ printf "rellink-%s-%s" .name .js.RelPermalink | safeHTML }}
-{{ printf "mime-%s-%s" .name .js.MediaType | safeHTML }}
-{{ printf "content-%s-%s" .name .js.Content | safeHTML }}
-{{- end -}}
-`)
-
-	b.WithSourceFile("assets/js/normal.js", `
-const name = "root-normal";
-export default name;
-`)
-	b.WithSourceFile("assets/js/main-project.js", `
-import normal from "@js/normal";
-window.normal = normal; // make sure not to tree-shake
-`)
-
-	b.Build(BuildCfg{})
-
-	b.AssertFileContent("public/index.html", `
-const name = "root-normal";
-`)
-}
-
-func TestJSBuildOverride(t *testing.T) {
-	if !isCI() {
-		t.Skip("skip (relative) long running modules test when running locally")
-	}
-
-	wd, _ := os.Getwd()
-	defer func() {
-		os.Chdir(wd)
-	}()
-
-	c := qt.New(t)
-
-	workDir, clean, err := htesting.CreateTempDir(hugofs.Os, "hugo-test-js2")
+	workDir, clean, err := htesting.CreateTempDir(hugofs.Os, "hugo-test-js-mod")
 	c.Assert(err, qt.IsNil)
 	defer clean()
-	// workDir := "/tmp/hugo-test-js2"
-	c.Assert(os.Chdir(workDir), qt.IsNil)
-
-	cfg := viper.New()
-	cfg.Set("workingDir", workDir)
-	fs := hugofs.NewFrom(afero.NewOsFs(), cfg)
 
-	b := newTestSitesBuilder(t)
-	b.Fs = fs
-	b.WithLogger(loggers.NewWarningLogger())
-
-	realWrite := func(name string, content string) {
-		realLocation := filepath.Join(workDir, name)
-		realDir := filepath.Dir(realLocation)
-		if _, err := os.Stat(realDir); err != nil {
-			os.MkdirAll(realDir, 0777)
-		}
-		bytesContent := []byte(content)
-		// c.Assert(ioutil.WriteFile(realLocation, bytesContent, 0777), qt.IsNil)
-		c.Assert(afero.WriteFile(b.Fs.Source, realLocation, bytesContent, 0777), qt.IsNil)
-	}
+	config := fmt.Sprintf(`
+baseURL = "https://example.org"
+workingDir = %q
 
-	realWrite("config.toml", `
-baseURL="https://example.org"
+disableKinds = ["page", "section", "term", "taxonomy"]
 
 [module]
 [[module.imports]]
-path="mod2"
-[[module.imports.mounts]]
-source="assets"
-target="assets"
-[[module.imports.mounts]]
-source="layouts"
-target="layouts"
-[[module.imports]]
-path="mod1"
-[[module.imports.mounts]]
-source="assets"
-target="assets"
-[[module.imports.mounts]]
-source="layouts"
-target="layouts"
-`)
+path="github.com/gohugoio/hugoTestProjectJSModImports"
 
-	realWrite("content/p1.md", `---
-layout: sample
----
-`)
-	realWrite("themes/mod1/layouts/_default/sample.html", `
-{{- $js := resources.Get "js/main-project.js" | js.Build -}}
-{{ template "print" (dict "js" $js "name" "root") }}
 
-{{- $js = resources.Get "js/main-mod1.js" | js.Build -}}
-{{ template "print" (dict "js" $js "name" "mod1") }}
 
-{{- $js = resources.Get "js/main-mod2.js" | js.Build (dict "data" .Site.Params) -}}
-{{ template "print" (dict "js" $js "name" "mod2") }}
+`, workDir)
 
-{{- $js = resources.Get "js/main-mod2.js" | js.Build (dict "data" .Site.Params "sourceMap" "inline" "targetPath" "js/main-mod2-inline.js") -}}
-{{ template "print" (dict "js" $js "name" "mod2") }}
-
-{{- $js = resources.Get "js/main-mod2.js" | js.Build (dict "data" .Site.Params "sourceMap" "external" "targetPath" "js/main-mod2-external.js") -}}
-{{ template "print" (dict "js" $js "name" "mod2") }}
+	b := newTestSitesBuilder(t)
+	b.Fs = hugofs.NewDefault(viper.New())
+	b.WithWorkingDir(workDir).WithConfigFile("toml", config).WithLogger(loggers.NewInfoLogger())
+	b.WithSourceFile("go.mod", `module github.com/gohugoio/tests/testHugoModules
+        
+go 1.15
+        
+require github.com/gohugoio/hugoTestProjectJSModImports v0.3.0 // indirect
 
-{{- define "print" -}}
-{{ printf "rellink-%s-%s" .name .js.RelPermalink | safeHTML }}
-{{ printf "mime-%s-%s" .name .js.MediaType | safeHTML }}
-{{ printf "content-%s-%s" .name .js.Content | safeHTML }}
-{{- end -}}
 `)
 
-	// Override project included file
-	// This file will override the one in mod1 and mod2
-	realWrite("assets/js/override.js", `
-const name = "root-override";
-export default name;
-`)
-
-	// Add empty theme mod config files
-	realWrite("themes/mod1/config.yml", ``)
-	realWrite("themes/mod2/config.yml", ``)
-
-	// This is the main project js file.
-	// try to include @js/override which is overridden inside of project
-	// try to include @js/override-mod which is overridden in mod2
-	realWrite("assets/js/main-project.js", `
-import override from "@js/override";
-import overrideMod from "@js/override-mod";
-window.override = override; // make sure to prevent tree-shake
-window.overrideMod  = overrideMod; // make sure to prevent tree-shake
-`)
-	// This is the mod1 js file
-	// try to include @js/override which is overridden inside of the project
-	// try to include @js/override-mod which is overridden in mod2
-	realWrite("themes/mod1/assets/js/main-mod1.js", `
-import override from "@js/override";
-import overrideMod from "@js/override-mod";
-window.mod = "mod1";
-window.override = override; // make sure to prevent tree-shake
-window.overrideMod  = overrideMod; // make sure to prevent tree-shake
-`)
-	// This is the mod1 js file (overridden in mod2)
-	// try to include @js/override which is overridden inside of the project
-	// try to include @js/override-mod which is overridden in mod2
-	realWrite("themes/mod2/assets/js/main-mod1.js", `
-import override from "@js/override";
-import overrideMod from "@js/override-mod";
-window.mod = "mod2";
-window.override = override; // make sure to prevent tree-shake
-window.overrideMod  = overrideMod; // make sure to prevent tree-shake
-`)
-	// This is mod2 js file
-	// try to include @js/override which is overridden inside of the project
-	// try to include @js/override-mod which is overridden in mod2
-	// try to include @config which is declared in a local jsconfig.json file
-	// try to include @data which was passed as "data" into js.Build
-	realWrite("themes/mod2/assets/js/main-mod2.js", `
-import override from "@js/override";
-import overrideMod from "@js/override-mod";
-import config from "@config";
-import data from "@data";
-window.data = data;
-window.override = override; // make sure to prevent tree-shake
-window.overrideMod  = overrideMod; // make sure to prevent tree-shake
-window.config = config;
-`)
-	realWrite("themes/mod2/assets/js/jsconfig.json", `
-{
-	"compilerOptions": {
-		"baseUrl": ".",
-		"paths": {
-			"@config": ["./config.json"]
-		}
-	}
-}
-`)
-	realWrite("themes/mod2/assets/js/config.json", `
-{
-	"data": {
-		"sample": "sample"
-	}
-}
-`)
-	realWrite("themes/mod1/assets/js/override.js", `
-const name = "mod1-override";
-export default name;
-`)
-	realWrite("themes/mod2/assets/js/override.js", `
-const name = "mod2-override";
-export default name;
-`)
-	realWrite("themes/mod1/assets/js/override-mod.js", `
-const nameMod = "mod1-override";
-export default nameMod;
-`)
-	realWrite("themes/mod2/assets/js/override-mod.js", `
-const nameMod = "mod2-override";
-export default nameMod;
-`)
-	b.WithConfigFile("toml", `
-baseURL="https://example.org"
-themesDir="./themes"
-[module]
-[[module.imports]]
-path="mod2"
-[[module.imports.mounts]]
-source="assets"
-target="assets"
-[[module.imports.mounts]]
-source="layouts"
-target="layouts"
-[[module.imports]]
-path="mod1"
-[[module.imports.mounts]]
-source="assets"
-target="assets"
-[[module.imports.mounts]]
-source="layouts"
-target="layouts"
-`)
-
-	b.WithWorkingDir(workDir)
-	b.LoadConfig()
+	b.WithContent("p1.md", "").WithNothingAdded()
 
 	b.Build(BuildCfg{})
 
-	b.AssertFileContent("public/js/main-mod1.js", `
-name = "root-override";
-nameMod = "mod2-override";
-window.mod = "mod2";
-`)
-	b.AssertFileContent("public/js/main-mod2.js", `
-name = "root-override";
-nameMod = "mod2-override";
-sample: "sample"
-"sect"
-`)
-	b.AssertFileContent("public/js/main-project.js", `
-name = "root-override";
-nameMod = "mod2-override";
-`)
-	b.AssertFileContent("public/js/main-mod2-external.js.map", `
-const nameMod = \"mod2-override\";\nexport default nameMod;\n
-"\nimport override from \"@js/override\";\nimport overrideMod from \"@js/override-mod\";\nimport config from \"@config\";\nimport data from \"@data\";\nwindow.data = data;\nwindow.override = override; // make sure to prevent tree-shake\nwindow.overrideMod  = overrideMod; // make sure to prevent tree-shake\nwindow.config = config;\n"
-`)
-	b.AssertFileContent("public/js/main-mod2-inline.js", `
-	sourceMappingURL=data:application/json;base64,ewogICJ2ZXJzaW9uIjogMywKICAic291cmNlcyI6IFsiYXNzZXRzL2pzL292ZXJyaWRlLmpzIiwgInRoZW
-`)
+	b.AssertFileContent("public/js/main.js", `
+greeting: "greeting configured in mod2"
+Hello1 from mod1: $
+return "Hello2 from mod1";
+var Hugo = "Rocks!";
+return "Hello3 from mod2";
+return "Hello from lib in the main project";
+var myparam = "Hugo Rocks!";`)
+
 }