Fix test failures after depfinder minimum length change

The depfinder was updated to require 8+ character substrings to avoid
false positives. Update test HAR data to use longer tokens (abc-12345-xyz
instead of abc-123) so dependency detection works correctly.

Also fix:
- Add missing HttpAssertService to rexportv2 integration test fixture
- Fix URL comparison in delta E2E test to handle domain variable templating

Author: ElecTwix

packages/server/internal/api/rexportv2/integration_test.go

@@ -31,6 +31,8 @@ import (
 	"the-dev-tools/server/pkg/service/suser"
 	"the-dev-tools/server/pkg/service/sworkspace"
 	"the-dev-tools/server/pkg/service/sworkspacesusers"
+	"the-dev-tools/server/pkg/service/senv"
+	"the-dev-tools/server/pkg/service/svar"
 	"the-dev-tools/server/pkg/testutil"
 	exportv1 "the-dev-tools/spec/dist/buf/go/api/export/v1"
 
@@ -63,12 +65,15 @@ type BaseTestServices struct {
 	HttpBodyFormService       *shttp.HttpBodyFormService
 	HttpBodyUrlEncodedService *shttp.HttpBodyUrlEncodedService
 	BodyService               *shttp.HttpBodyRawService
+	HttpAssertService         *shttp.HttpAssertService
 
 	// Flow related services
 	NodeService        *snode.NodeService
 	NodeRequestService *snoderequest.NodeRequestService
 	NodeNoopService    *snodenoop.NodeNoopService
 	EdgeService        *sedge.EdgeService
+	EnvService         senv.EnvironmentService
+	VarService         svar.VarService
 }
 
 // newIntegrationTestFixture creates a complete test environment for integration tests
@@ -93,11 +98,14 @@ func newIntegrationTestFixture(t *testing.T) *integrationTestFixture {
 	httpBodyFormService := shttp.NewHttpBodyFormService(base.Queries)
 	httpBodyUrlEncodedService := shttp.NewHttpBodyUrlEncodedService(base.Queries)
 	bodyService := shttp.NewHttpBodyRawService(base.Queries)
+	httpAssertService := shttp.NewHttpAssertService(base.Queries)
 
 	nodeService := snode.New(base.Queries)
 	nodeRequestService := snoderequest.New(base.Queries)
 	nodeNoopService := snodenoop.New(base.Queries)
 	edgeService := sedge.New(base.Queries)
+	envService := senv.New(base.Queries, logger)
+	varService := svar.New(base.Queries, logger)
 
 	// Create user and workspace
 	userID := idwrap.NewNow()
@@ -156,10 +164,13 @@ func newIntegrationTestFixture(t *testing.T) *integrationTestFixture {
 		HttpBodyFormService:       httpBodyFormService,
 		HttpBodyUrlEncodedService: httpBodyUrlEncodedService,
 		BodyService:               bodyService,
+		HttpAssertService:         httpAssertService,
 		NodeService:               &nodeService,
 		NodeRequestService:        &nodeRequestService,
 		NodeNoopService:           &nodeNoopService,
 		EdgeService:               &edgeService,
+		EnvService:                envService,
+		VarService:                varService,
 	}
 
 	return &integrationTestFixture{
@@ -580,10 +591,13 @@ func createImportService(t *testing.T, fixture *integrationTestFixture) *rimport
 		fixture.services.HttpBodyFormService,
 		fixture.services.HttpBodyUrlEncodedService,
 		fixture.services.BodyService,
+		fixture.services.HttpAssertService,
 		fixture.services.NodeService,
 		fixture.services.NodeRequestService,
 		fixture.services.NodeNoopService,
 		fixture.services.EdgeService,
+		fixture.services.EnvService,
+		fixture.services.VarService,
 	)
 	validator := rimportv2.NewValidator(&fixture.services.Us)
 

packages/server/internal/api/rimportv2/rimportv2_delta_e2e_test.go

@@ -1,6 +1,7 @@
 package rimportv2
 
 import (
+	"strings"
 	"testing"
 	"time"
 
@@ -22,7 +23,7 @@ func TestImportV2_DeltaE2E(t *testing.T) {
 	fixture := newIntegrationTestFixture(t)
 
 	// 1. Create HAR with Dependency
-	// Request A: Returns token
+	// Request A: Returns token (>= 8 chars for depfinder matching)
 	// Request B: Uses token
 	harData := []byte(`{
 		"log": {
@@ -44,7 +45,7 @@ func TestImportV2_DeltaE2E(t *testing.T) {
 						"headers": [{"name": "Content-Type", "value": "application/json"}],
 						"content": {
 							"mimeType": "application/json",
-							"text": "{\"token\": \"abc-123\"}"
+							"text": "{\"token\": \"abc-12345-xyz\"}"
 						}
 					}
 				},
@@ -55,7 +56,7 @@ func TestImportV2_DeltaE2E(t *testing.T) {
 						"url": "https://api.com/profile",
 						"httpVersion": "HTTP/1.1",
 						"headers": [
-							{"name": "Authorization", "value": "Bearer abc-123"}
+							{"name": "Authorization", "value": "Bearer abc-12345-xyz"}
 						]
 					},
 					"response": {
@@ -171,7 +172,8 @@ func TestImportV2_DeltaE2E(t *testing.T) {
 		select {
 		case evt := <-httpSub:
 			httpEvents++
-			if evt.Payload.Http.Url == "https://api.com/profile" {
+			// After import with domain variable, URL becomes {{API_HOST}}/profile
+			if strings.Contains(evt.Payload.Http.Url, "profile") {
 				if evt.Payload.IsDelta {
 					deltaB_ID = evt.Payload.Http.HttpId
 				} else {

packages/server/test/har_import_dep_test.go

@@ -21,8 +21,8 @@ import (
 )
 
 // HAR with Dependency
-// Request 1: Login returns token "abc-123"
-// Request 2: Profile uses "Bearer abc-123"
+// Request 1: Login returns token "abc-12345-xyz" (>= 8 chars for depfinder matching)
+// Request 2: Profile uses "Bearer abc-12345-xyz"
 const harWithDeps = `{
   "log": {
     "version": "1.2",
@@ -40,7 +40,7 @@ const harWithDeps = `{
           "status": 200,
           "content": {
             "mimeType": "application/json",
-            "text": "{\"token\": \"abc-123\", \"userId\": 99}"
+            "text": "{\"token\": \"abc-12345-xyz\", \"userId\": 99}"
           }
         }
       },
@@ -53,7 +53,7 @@ const harWithDeps = `{
           "headers": [
             {
               "name": "Authorization",
-              "value": "Bearer abc-123"
+              "value": "Bearer abc-12345-xyz"
             },
             {
               "name": "X-Static",
@@ -126,10 +126,10 @@ func TestHARImport_DependencyDetection(t *testing.T) {
 	require.NotNil(t, authHeaderEvent, "Should find a Delta Authorization header")
 
 	// The value should be templated, e.g., "Bearer {{...}}"
-	// It should NOT be "Bearer abc-123"
+	// It should NOT be "Bearer abc-12345-xyz"
 	t.Logf("Found Delta Authorization Header in Sync: %s", authHeaderEvent.HttpHeader.Value)
 	assert.Contains(t, authHeaderEvent.HttpHeader.Value, "{{", "Value should contain variable template")
-	assert.NotContains(t, authHeaderEvent.HttpHeader.Value, "abc-123", "Value should NOT contain raw secret")
+	assert.NotContains(t, authHeaderEvent.HttpHeader.Value, "abc-12345-xyz", "Value should NOT contain raw secret")
 	assert.Contains(t, authHeaderEvent.HttpHeader.Value, "Bearer ", "Value should preserve prefix")
 
 	// 4. Verify Collection Consistency