Fix endpoint printout on startup (#32)

Author: callumevans

offline/handler_instance.go

@@ -34,14 +34,13 @@ func (handlerInstance *HandlerInstance) SetExecutionPath(path string) {
 }
 
 func (handlerInstance *HandlerInstance) CompileHandler() (inputFilePaths []string) {
-	dir, err := os.Executable()
+	workingDirectory, err := os.Getwd()
+
 	if err != nil {
 		println(fmt.Errorf("error fetching executable location: %w", err))
 		return
 	}
 
-	workingDirectory := filepath.Dir(dir)
-
 	result := api.Build(api.BuildOptions{
 		EntryPoints: []string{handlerInstance.handlerConfig.Source},
 		Bundle:      true,

offline/offline.go

@@ -28,7 +28,7 @@ func Run(filePath string, moduleName string, port string) error {
 		panic(fmt.Errorf("error parsing .terrable.toml file: %w", err))
 	}
 
-	printConfig(*terrableConfig)
+	printConfig(*terrableConfig, port)
 
 	var wg sync.WaitGroup
 	defer wg.Done()
@@ -51,14 +51,14 @@ func Run(filePath string, moduleName string, port string) error {
 	return nil
 }
 
-func printConfig(config config.TerrableConfig) {
+func printConfig(config config.TerrableConfig, port string) {
 	totalEndpoints := 0
 	printlines := []string{}
 
 	for _, handler := range config.Handlers {
 		for method, path := range handler.Http {
-			totalEndpoints += len(handler.Http)
-			printlines = append(printlines, fmt.Sprintf("   %-*s http://localhost:8080%s\n", 5, method, path))
+			totalEndpoints += 1
+			printlines = append(printlines, fmt.Sprintf("   %-*s http://localhost:%s%s\n", 5, method, port, path))
 		}
 	}
 

offline/offline_test.go

@@ -0,0 +1,70 @@
+package offline
+
+import (
+	"bytes"
+	"fmt"
+	"io"
+	"os"
+	"strings"
+	"testing"
+
+	"github.com/terrable-dev/terrable/config"
+)
+
+func TestPrintConfig(t *testing.T) {
+	testConfig := config.TerrableConfig{
+		Handlers: []config.HandlerMapping{
+			{
+				Name:   "Handler1",
+				Source: "source1",
+				Http: map[string]string{
+					"GET":  "/path1",
+					"POST": "/path1",
+				},
+			},
+			{
+				Name:   "Handler2",
+				Source: "source2",
+				Http: map[string]string{
+					"GET": "/path2",
+				},
+			},
+		},
+	}
+
+	// Capture stdout
+	old := os.Stdout
+	r, w, _ := os.Pipe()
+	os.Stdout = w
+
+	// Call the function
+	printConfig(testConfig, "1234")
+
+	// Restore stdout
+	w.Close()
+	os.Stdout = old
+
+	// Read captured output
+	var buf bytes.Buffer
+	io.Copy(&buf, r)
+	output := buf.String()
+
+	expectedEndpoints := 3
+	expectedLine := fmt.Sprintf("%d Endpoint(s) to prepare...", expectedEndpoints)
+	if !strings.Contains(output, expectedLine) {
+		t.Errorf("Expected output to contain '%s', but it doesn't.\nActual output:\n%s", expectedLine, output)
+	}
+
+	expectedLines := []string{
+		"Starting terrable local server...",
+		"   GET   http://localhost:1234/path1",
+		"   POST  http://localhost:1234/path1",
+		"   GET   http://localhost:1234/path2",
+	}
+
+	for _, line := range expectedLines {
+		if !strings.Contains(output, line) {
+			t.Errorf("Expected output to contain '%s', but it doesn't.\nActual output:\n%s", line, output)
+		}
+	}
+}

terrable_build

@@ -1 +1 @@
-version = 0.1.4
+version = 0.1.5