Fix make test

A panic would occur if a field in the config file (e.g. youtube-dl.youtube-dl-path) was not set

Makefile

@@ -12,6 +12,9 @@ debug:
 	$(GO) build -o $(EXE) -gcflags=all="-N -l" cmd/$(EXE)/*.go
 	dlv exec ./dischord
 
+fmt:
+	find . -type f -name '*.go' -exec gofmt -w '{}' ';'
+
 install: all
 	mkdir -p $(DESTDIR)$(PREFIX)/bin
 	cp -f $(EXE) $(DESTDIR)$(PREFIX)/bin
@@ -48,9 +51,9 @@ uninstall:
 	rm -f $(DESTDIR)$(CFGPREFIX)/systemd/system/$(EXE).service
 
 test:
-	$(GO) test -count=1 -v $(EXE)/extractor
+	$(GO) test -count=1 -v git.nobrain.org/r4/dischord/extractor/
 
-.PHONY: all debug install uninstall clean
+.PHONY: all debug fmt install uninstall clean
 
 clean:
 	rm -f $(EXE)

cmd/dischord/dischord.go

@@ -10,6 +10,7 @@ import (
 	"git.nobrain.org/r4/dischord/player"
 	"git.nobrain.org/r4/dischord/util"
 
+	_ "embed"
 	"errors"
 	"flag"
 	"fmt"
@@ -21,7 +22,6 @@ import (
 	"strings"
 	"sync"
 	"syscall"
-	_ "embed"
 )
 
 var copyright bool

config/config.go

@@ -97,14 +97,20 @@ func macosEnableExecutable(filename string) error {
 // configuration file does not exist or is invalid.
 func Load(filename string) (*Config, error) {
 	cfg := &Config{}
-	_, err := toml.DecodeFile(filename, cfg)
+	meta, err := toml.DecodeFile(filename, cfg)
 	if err != nil {
+		if pe, ok := err.(toml.ParseError); ok {
+			fmt.Println(pe.ErrorWithUsage())
+		}
 		return nil, err
 	}
+	if undec := meta.Undecoded(); len(undec) > 0 {
+		return nil, fmt.Errorf("%v: field '%v' could not be decoded", filename, undec[0])
+	}
 	if cfg.Token == defaultToken || cfg.Token == "" {
 		return nil, ErrTokenNotSet
 	}
-	if err := cfg.Extractors.CheckTypes(); err != nil {
+	if err := cfg.Extractors.CheckValidity(); err != nil {
 		return nil, err
 	}
 	if _, err := exec.LookPath(cfg.Extractors["youtube-dl"]["youtube-dl-path"].(string)); err != nil {

extractor/extractor.go

@@ -8,7 +8,7 @@ import (
 )
 
 var (
-	ErrNoSearchResults      = errors.New("no search provider available")
+	ErrNoSearchResults      = errors.New("no search results")
 	ErrNoSearchProvider     = errors.New("no search provider available")
 	ErrNoSuggestionProvider = errors.New("no search suggestion provider available")
 )
@@ -22,7 +22,7 @@ var (
 )
 
 func Extract(cfg Config, input string) ([]Data, error) {
-	if err := cfg.CheckTypes(); err != nil {
+	if err := cfg.CheckValidity(); err != nil {
 		return nil, err
 	}
 	for _, e := range extractors {
@@ -45,7 +45,7 @@ func Extract(cfg Config, input string) ([]Data, error) {
 }
 
 func Search(cfg Config, input string) ([]Data, error) {
-	if err := cfg.CheckTypes(); err != nil {
+	if err := cfg.CheckValidity(); err != nil {
 		return nil, err
 	}
 	for _, s := range searchers {
@@ -59,7 +59,7 @@ func Search(cfg Config, input string) ([]Data, error) {
 }
 
 func Suggest(cfg Config, input string) ([]string, error) {
-	if err := cfg.CheckTypes(); err != nil {
+	if err := cfg.CheckValidity(); err != nil {
 		return nil, err
 	}
 	for _, s := range suggestors {
@@ -115,16 +115,16 @@ func DefaultConfig() Config {
 	}
 }
 
-func (cfg Config) CheckTypes() error {
+func (cfg Config) CheckValidity() error {
-	for provider, pCfg := range cfg {
+	for chkProvider, chkCfg := range DefaultConfig() {
-		if pCfg == nil {
+		if _, ok := cfg[chkProvider]; !ok {
-			return fmt.Errorf("extractor config for %v is nil", provider)
+			return fmt.Errorf("extractor config for %v is nil", chkProvider)
 		}
-		for k, v := range pCfg {
+		for k, v := range chkCfg {
-			got, expected := reflect.TypeOf(v), reflect.TypeOf(DefaultConfig()[provider][k])
+			expected, got := reflect.TypeOf(v), reflect.TypeOf(cfg[chkProvider][k])
 			if got != expected {
 				return &ConfigTypeError{
-					Provider: provider,
+					Provider: chkProvider,
 					Key:      k,
 					Expected: expected,
 					Got:      got,
@@ -151,7 +151,7 @@ func (e *ConfigTypeError) Error() string {
 	if e.Got != nil {
 		gotName = e.Got.Name()
 	}
-	return "extractor config type error: "+e.Provider+"."+e.Key+": expected "+expectedName+" but got "+gotName
+	return "invalid extractor configuration: " + e.Provider + "." + e.Key + ": expected " + expectedName + " but got " + gotName
 }
 
 type ProviderConfig map[string]any

extractor/youtube/youtube_decrypt.go

@@ -140,7 +140,7 @@ func getDecryptFunction(baseJs string) (string, error) {
 	}
 	fn := baseJs[start+len(startMatch):]
 	end := strings.Index(fn, endMatch)
-	if start == -1 {
+	if end == -1 {
 		return "", ErrDecryptGettingFunction
 	}
 	return fn[:end], nil
@@ -170,7 +170,7 @@ func getDecryptOps(baseJs string) ([]decryptorOp, error) {
 		}
 		ops = baseJs[start+len(startMatch):]
 		end := strings.Index(ops, endMatch)
-		if start == -1 {
+		if end == -1 {
 			return nil, ErrDecryptGettingOpTable
 		}
 		ops = ops[:end]