Auto merge of #12655 - ChayimFriedman2:debug-bench-filter-bin-target, r=flodiebold

bors · bors · commit ce36446943a3 · 2022-06-30T15:01:42.000Z
Ignore the `bin` artifact for `bench` targets Just like `test`. Fixes #12645. I don't know how to test that.
diff --git a/editors/code/src/toolchain.ts b/editors/code/src/toolchain.ts
@@ -39,7 +39,7 @@ export class Cargo {
         }
 
         const result: ArtifactSpec = { cargoArgs: cargoArgs };
-        if (cargoArgs[0] === "test") {
+        if (cargoArgs[0] === "test" || cargoArgs[0] === "bench") {
             // for instance, `crates\rust-analyzer\tests\heavy_tests\main.rs` tests
             // produce 2 artifacts: {"kind": "bin"} and {"kind": "test"}
             result.filter = (artifacts) => artifacts.filter((it) => it.isTest);

Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,7 @@ export class Cargo {`
`39`	`39`	`}`
`40`	`40`
`41`	`41`	`const result: ArtifactSpec = { cargoArgs: cargoArgs };`
`42`		`- if (cargoArgs[0] === "test") {`
	`42`	`+ if (cargoArgs[0] === "test" \|\| cargoArgs[0] === "bench") {`
`43`	`43`	// for instance, `crates\rust-analyzer\tests\heavy_tests\main.rs` tests
`44`	`44`	`// produce 2 artifacts: {"kind": "bin"} and {"kind": "test"}`
`45`	`45`	`result.filter = (artifacts) => artifacts.filter((it) => it.isTest);`