chore: minor code cleanups

Signed-off-by: Abiola Ibrahim <[email protected]>

Author: abiosoft

engine.go

@@ -16,7 +16,7 @@ var (
 	defaultLayout string
 
 	// default filename extenstions for template files
-	defaultExts = []string{"html", "gohtml", "tpl", "tmpl"}
+	defaultExts = []string{".html", ".gohtml", ".tpl", ".tmpl"}
 )
 
 const (
@@ -98,7 +98,7 @@ func walk(fsys fs.FS, exts []string, funcMap template.FuncMap) (set templateSet,
 		}
 
 		ext := filepath.Ext(d.Name())
-		if !validExt(exts, ext) {
+		if !hasExt(exts, ext) {
 			return nil
 		}
 
@@ -260,7 +260,7 @@ func readFile(fsys fs.FS, name string) (string, error) {
 	return string(f), nil
 }
 
-func validExt(exts []string, ext string) bool {
+func hasExt(exts []string, ext string) bool {
 	if ext == "" {
 		return false
 	}
@@ -277,7 +277,7 @@ func validExt(exts []string, ext string) bool {
 
 func validateLayoutFile(exts []string, name string) error {
 	ext := filepath.Ext(name)
-	if !validExt(exts, ext) {
+	if !hasExt(exts, ext) {
 		return fmt.Errorf("unsupported filename extension '%s'", ext)
 	}
 

engine_test.go

@@ -100,7 +100,7 @@ func TestValidExt(t *testing.T) {
 
 	for i, tt := range tests {
 		t.Run(strconv.Itoa(i), func(t *testing.T) {
-			result := validExt(tt.exts, tt.ext)
+			result := hasExt(tt.exts, tt.ext)
 			if result != tt.expected {
 				t.Errorf("validExt(%v, %q) = %v, expected %v", tt.exts, tt.ext, result, tt.expected)
 			}

mold.go

@@ -121,7 +121,7 @@ func WithLayout(layout string) Option {
 // WithExt configures the filename extensions for the templates.
 // Only files with the specified extensions would be parsed.
 //
-//	Default: ["html", "gohtml", "tpl", "tmpl"]
+//	Default: [".html", ".gohtml", ".tpl", ".tmpl"]
 func WithExt(exts ...string) Option {
 	return func(c *Config) { c.exts = newVal(exts) }
 }

mold_test.go

@@ -259,7 +259,7 @@ func TestRender_PartialInvalidPartial(t *testing.T) {
 	)
 
 	if _, err := New(testFS); err == nil {
-		t.Errorf("New() expected error,  got nil")
+		t.Errorf("New() expected error, got nil")
 	}
 }