Assert required pipelines test failures with musterr and inline appends

The test-ids acceptance test expects both runs of the split pipeline to
fail, so it uses musterr instead of errcode. The raw requests test
appends inline instead of through a closure that only names the append.

Co-authored-by: Isaac <no-reply@databricks.com>

Author: tharunbhuk-db

acceptance/pipelines/test/test-ids/output.txt

@@ -65,7 +65,7 @@ tests/test_a.py::test_one  PASSED (1.00s)
 === With tests outside root_path, a failing test ID printed in a subfolder runs again from there
 Printed: tests/test_returns.py::test_returned_fails
 
->>> errcode [CLI] pipelines test split tests/test_returns.py::test_returned_fails
+>>> musterr [CLI] pipelines test split tests/test_returns.py::test_returned_fails
 Recommendation: This command runs the last deployed version of the code
 
 If you've made local changes, run 'databricks pipelines deploy' first to ensure they are included.
@@ -82,7 +82,5 @@ tests/test_returns.py::test_returned_fails
 
 1 test: 0 passed, 1 failed, 0 skipped, 0 errors
 
-Exit code: 1
-
 >>> print_requests.py //updates
 {"test_selection":["/Workspace/Users/[USERNAME]/.bundle/test-pipeline-test-ids/default/files/tests/test_returns.py::test_returned_fails"]}

acceptance/pipelines/test/test-ids/script

@@ -17,10 +17,10 @@ trace print_requests.py //updates | jq -c .body.test_options
 
 title "With tests outside root_path, a failing test ID printed in a subfolder runs again from there\n"
 cd src/split
-errcode $CLI pipelines test split > out.txt 2>&1
+musterr $CLI pipelines test split > out.txt 2>&1
 failed_id=$(grep -o '^[^ ]*::test_returned_fails' out.txt | head -1)
 rm out.txt "$OUT_REQUESTS"
 echo "Printed: $failed_id"
-trace errcode $CLI pipelines test split "$failed_id"
+trace musterr $CLI pipelines test split "$failed_id"
 cd ../..
 trace print_requests.py //updates | jq -c .body.test_options

bundle/direct/dresources/pipeline_test.go

@@ -61,16 +61,13 @@ func TestPipelineOverrideChangeDescCascadeOnDestroy(t *testing.T) {
 func TestPipelineRawRequests(t *testing.T) {
 	server := testserver.New(t)
 	var bodies, workspaceIDs []string
-	record := func(req testserver.Request) {
+	server.Handle("POST", "/api/2.0/pipelines", func(req testserver.Request) any {
 		bodies = append(bodies, string(req.Body))
 		workspaceIDs = append(workspaceIDs, req.Headers.Get(auth.WorkspaceIDHeader))
-	}
-	server.Handle("POST", "/api/2.0/pipelines", func(req testserver.Request) any {
-		record(req)
 		return pipelines.CreatePipelineResponse{PipelineId: "p1"}
 	})
 	server.Handle("GET", "/api/2.0/pipelines/{pipeline_id}", func(req testserver.Request) any {
-		record(req)
+		workspaceIDs = append(workspaceIDs, req.Headers.Get(auth.WorkspaceIDHeader))
 		return map[string]any{"pipeline_id": "p1", "spec": map[string]any{"name": "orders"}}
 	})
 	w, err := databricks.NewWorkspaceClient(&databricks.Config{Host: server.URL, Token: "token", WorkspaceID: "123"})
@@ -88,7 +85,7 @@ func TestPipelineRawRequests(t *testing.T) {
 
 	// The raw requests carry the workspace ID, as the SDK's do.
 	assert.Equal(t, []string{"123", "123", "123"}, workspaceIDs)
-	assert.JSONEq(t, `{"name": "orders"}`, bodies[2])
+	assert.JSONEq(t, `{"name": "orders"}`, bodies[1])
 }
 
 func TestPipelineReadReturnsTesting(t *testing.T) {