Skip to content

Commit fe1be71

Browse files
authored
chore(tests): Deprecate remaining usages of itest in check tests (#26962)
This PR progresses #22907 by deprecating the usage of `itest` in `tests/integration/check_tests.rs`
1 parent 4ab668e commit fe1be71

File tree

7 files changed

+14
-15
lines changed

7 files changed

+14
-15
lines changed

tests/integration/check_tests.rs

-14
Original file line numberDiff line numberDiff line change
@@ -3,23 +3,9 @@
33
use deno_lockfile::NewLockfileOptions;
44
use deno_semver::jsr::JsrDepPackageReq;
55
use test_util as util;
6-
use test_util::itest;
76
use util::TestContext;
87
use util::TestContextBuilder;
98

10-
itest!(check_all {
11-
args: "check --allow-import --quiet --all check/all/check_all.ts",
12-
output: "check/all/check_all.out",
13-
http_server: true,
14-
exit_code: 1,
15-
});
16-
17-
itest!(check_all_local {
18-
args: "check --allow-import --quiet check/all/check_all.ts",
19-
output_str: Some(""),
20-
http_server: true,
21-
});
22-
239
#[test]
2410
fn cache_switching_config_then_no_config() {
2511
let context = TestContext::default();
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
{
2+
"args": "check --allow-import --quiet --all check_all.ts",
3+
"output": "check_all.out",
4+
"exitCode": 1
5+
}
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
{
2+
"args": "check --allow-import --quiet check_all_local.ts",
3+
"output": "",
4+
"exitCode": 0
5+
}
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
import * as a from "http://localhost:4545/subdir/type_error.ts";
2+
3+
console.log(a.a);

tools/lint.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -211,7 +211,7 @@ async function ensureNoNewITests() {
211211
"bench_tests.rs": 0,
212212
"cache_tests.rs": 0,
213213
"cert_tests.rs": 0,
214-
"check_tests.rs": 2,
214+
"check_tests.rs": 0,
215215
"compile_tests.rs": 0,
216216
"coverage_tests.rs": 0,
217217
"eval_tests.rs": 0,

0 commit comments

Comments
 (0)