Add Test For Actual Present Case

Signed-off-by: Mahad Zaryab <[email protected]>

cmd/query/app/ui/assets_test.go

@@ -10,6 +10,7 @@ import (
 
 	"github.com/stretchr/testify/require"
 
+	"github.com/jaegertracing/jaeger/cmd/query/app/ui/testdata"
 	"github.com/jaegertracing/jaeger/pkg/testutils"
 )
 
@@ -29,3 +30,20 @@ func TestGetStaticFiles_ReturnsPlaceholderWhenActualNotPresent(t *testing.T) {
 	require.Contains(t, string(bytes), "This is a placeholder for the Jaeger UI home page")
 	require.Contains(t, logBuf.String(), "ui assets not embedded in the binary, using a placeholder")
 }
+
+func TestGetStaticFiles_ReturnsActualWhenPresent(t *testing.T) {
+	// swap out the assets FS for an dummy one and then replace it back when the
+	// test completes
+	currentActualAssets := actualAssetsFS
+	actualAssetsFS = testdata.TestFS
+	t.Cleanup(func() { actualAssetsFS = currentActualAssets })
+
+	logger, logBuf := testutils.NewLogger()
+	fs := GetStaticFiles(logger)
+	file, err := fs.Open("/index.html")
+	require.NoError(t, err)
+	bytes, err := io.ReadAll(file)
+	require.NoError(t, err)
+	require.NotContains(t, string(bytes), "This is a placeholder for the Jaeger UI home page")
+	require.NotContains(t, logBuf.String(), "ui assets not embedded in the binary, using a placeholder")
+}

cmd/query/app/ui/testdata/mock.go

@@ -0,0 +1,6 @@
+package testdata
+
+import "embed"
+
+//go:embed actual/*
+var TestFS embed.FS