fix linter and tests service init

Author: dobrac

packages/envd/internal/services/filesystem/dir_test.go

@@ -34,7 +34,7 @@ func TestListDir(t *testing.T) {
 	require.NoError(t, os.WriteFile(filePath, []byte("Hello, World!"), 0o644))
 
 	// Service instance
-	svc := Service{}
+	svc := mockService()
 
 	// Helper to inject user into context
 	injectUser := func(ctx context.Context, u *user.User) context.Context {
@@ -106,7 +106,7 @@ func TestListDir(t *testing.T) {
 func TestListDirNonExistingPath(t *testing.T) {
 	t.Parallel()
 
-	svc := Service{}
+	svc := mockService()
 	u, err := user.Current()
 	require.NoError(t, err)
 	ctx := authn.SetInfo(t.Context(), u)
@@ -136,7 +136,7 @@ func TestListDirRelativePath(t *testing.T) {
 	require.NoError(t, os.WriteFile(filePath, []byte("Hello, World!"), 0o644))
 
 	// Service instance
-	svc := Service{}
+	svc := mockService()
 	ctx := authn.SetInfo(t.Context(), u)
 
 	req := connect.NewRequest(&filesystem.ListDirRequest{
@@ -188,7 +188,7 @@ func TestListDir_Symlinks(t *testing.T) {
 	require.NoError(t, os.Symlink(realFile, linkToFile))
 	require.NoError(t, os.Symlink(cyclicLink, cyclicLink))
 
-	svc := Service{}
+	svc := mockService()
 
 	t.Run("symlink to directory behaves like directory and the content looks like inside the directory", func(t *testing.T) {
 		t.Parallel()

packages/envd/internal/services/filesystem/move_test.go

@@ -40,7 +40,7 @@ func TestMove(t *testing.T) {
 	destFile := filepath.Join(destDir, "test-file.txt")
 
 	// Service instance
-	svc := Service{}
+	svc := mockService()
 
 	// Call the Move function
 	ctx := authn.SetInfo(t.Context(), u)
@@ -97,7 +97,7 @@ func TestMoveDirectory(t *testing.T) {
 	destDir := filepath.Join(destParent, "test-dir")
 
 	// Service instance
-	svc := Service{}
+	svc := mockService()
 
 	// Call the Move function
 	ctx := authn.SetInfo(t.Context(), u)
@@ -157,7 +157,7 @@ func TestMoveNonExistingFile(t *testing.T) {
 	destFile := filepath.Join(destDir, "moved-file.txt")
 
 	// Service instance
-	svc := Service{}
+	svc := mockService()
 
 	// Call the Move function
 	ctx := authn.SetInfo(t.Context(), u)
@@ -199,7 +199,7 @@ func TestMoveRelativePath(t *testing.T) {
 	destFile := filepath.Join(destFolderPath, "moved-file.txt")
 
 	// Service instance
-	svc := Service{}
+	svc := mockService()
 
 	// Call the Move function with relative paths
 	ctx := authn.SetInfo(t.Context(), u)
@@ -262,7 +262,7 @@ func TestMove_Symlinks(t *testing.T) {
 	require.NoError(t, os.Symlink(realDir, linkToDir))
 	require.NoError(t, os.Symlink(realFile, linkToFile))
 
-	svc := Service{}
+	svc := mockService()
 
 	t.Run("move symlink to directory", func(t *testing.T) {
 		destPath := filepath.Join(destRoot, "moved-link-dir")

packages/envd/internal/services/filesystem/service_test.go

@@ -0,0 +1,14 @@
+package filesystem
+
+import (
+	"github.com/e2b-dev/infra/packages/envd/internal/execcontext"
+	"github.com/e2b-dev/infra/packages/envd/internal/utils"
+)
+
+func mockService() Service {
+	return Service{
+		defaults: &execcontext.Defaults{
+			EnvVars: utils.NewMap[string, string](),
+		},
+	}
+}

packages/envd/internal/services/filesystem/stat_test.go

@@ -44,7 +44,7 @@ func TestStat(t *testing.T) {
 	require.NoError(t, err)
 
 	// Service instance
-	svc := Service{}
+	svc := mockService()
 
 	// Helper to inject user into context
 	injectUser := func(ctx context.Context, u *user.User) context.Context {

packages/orchestrator/internal/template/build/phases/user/builder.go

@@ -60,7 +60,7 @@ func (ub *UserBuilder) Prefix() string {
 	return "base"
 }
 
-func (ub *UserBuilder) String(ctx context.Context) (string, error) {
+func (ub *UserBuilder) String(_ context.Context) (string, error) {
 	return fmt.Sprintf("DEFAULT USER %s", ub.user), nil
 }