fix: remove specific tests, remove planBuild from test

executor/linux/build.go

@@ -367,7 +367,7 @@ func (c *client) AssembleBuild(ctx context.Context) error {
 			continue
 		}
 
-		// ignore secret origins coming from plugins
+		// skip over non-plugin secrets origins
 		if container.Empty() {
 			continue
 		}

executor/linux/build_test.go

@@ -591,34 +591,6 @@ func TestLinux_AssembleBuild_EnforceTrustedRepos(t *testing.T) {
 		},
 	}
 
-	// focus these tests
-	tests = []struct {
-		name                string
-		failure             bool
-		build               *library.Build
-		repo                *library.Repo
-		pipeline            string
-		privilegedImages    []string
-		enforceTrustedRepos bool
-	}{{
-		name:                "enforce trusted repos enabled: privileged steps pipeline with untrusted repo and init step name",
-		failure:             true,
-		build:               _build,
-		repo:                _untrustedRepo,
-		pipeline:            "testdata/build/steps/name_init.yml",
-		privilegedImages:    _privilegedImagesStepsPipeline, // this matches the image from test.pipeline
-		enforceTrustedRepos: true,
-	},
-		{
-			name:                "enforce trusted repos enabled: non-privileged steps pipeline with untrusted repo and init step name",
-			failure:             true,
-			build:               _build,
-			repo:                _untrustedRepo,
-			pipeline:            "testdata/build/steps/name_init.yml",
-			privilegedImages:    _privilegedImagesStepsPipeline, // this matches the image from test.pipeline
-			enforceTrustedRepos: true,
-		}}
-
 	// run test
 	for _, test := range tests {
 		t.Run(test.name, func(t *testing.T) {
@@ -652,11 +624,6 @@ func TestLinux_AssembleBuild_EnforceTrustedRepos(t *testing.T) {
 				t.Errorf("CreateBuild returned err: %v", err)
 			}
 
-			err = _engine.PlanBuild(context.Background())
-			if err != nil {
-				t.Errorf("PlanBuild returned err: %v", err)
-			}
-
 			err = _engine.AssembleBuild(context.Background())
 
 			if test.failure {