Skip to content

Commit 01c09b3

Browse files
pqCommit Queue
authored andcommitted
[CQ] [testing] mock support for test_reflective_loader
Change-Id: I6db62d6adb03576763f65157a45227b884937b17 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/412804 Reviewed-by: Brian Wilkerson <[email protected]> Auto-Submit: Phil Quitslund <[email protected]> Commit-Queue: Paul Berry <[email protected]> Reviewed-by: Paul Berry <[email protected]>
1 parent f04f534 commit 01c09b3

File tree

6 files changed

+28
-14
lines changed

6 files changed

+28
-14
lines changed

pkg/analyzer_utilities/lib/test/mock_packages/mock_packages.dart

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -139,6 +139,11 @@ mixin MockPackagesMixin {
139139
return packageFolder.getChildAssumingFolder('lib');
140140
}
141141

142+
Folder addTestReflectiveLoader() {
143+
var packageFolder = _addFiles('test_reflective_loader');
144+
return packageFolder.getChildAssumingFolder('lib');
145+
}
146+
142147
Folder addUI() {
143148
var packageFolder = _addFiles('ui');
144149
return packageFolder.getChildAssumingFolder('lib');
Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
library test_reflective_loader;
2+
3+
const Object reflectiveTest = _ReflectiveTest();
4+
const Object skippedTest = SkippedTest();
5+
const Object soloTest = _SoloTest();
6+
7+
class SkippedTest {
8+
const SkippedTest({String? issue, String? reason});
9+
}
10+
11+
class _ReflectiveTest {
12+
const _ReflectiveTest();
13+
}
14+
15+
class _SoloTest {
16+
const _SoloTest();
17+
}
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
name: test_reflective_loader

pkg/linter/test/rule_test_support.dart

Lines changed: 3 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -177,7 +177,7 @@ class PubPackageResolutionTest extends _ContextResolutionTest {
177177

178178
bool get addMetaPackageDep => false;
179179

180-
bool get addReflectiveTestLoaderPackageDep => false;
180+
bool get addTestReflectiveLoaderPackageDep => false;
181181

182182
bool get dumpAstOnFailures => true;
183183

@@ -332,17 +332,8 @@ class PubPackageResolutionTest extends _ContextResolutionTest {
332332
configCopy.add(name: 'meta', rootPath: metaPath);
333333
}
334334

335-
if (addReflectiveTestLoaderPackageDep) {
336-
var testReflectiveLoaderPath =
337-
'$workspaceRootPath/test_reflective_loader';
338-
newFile('$testReflectiveLoaderPath/lib/test_reflective_loader.dart', r'''
339-
library test_reflective_loader;
340-
341-
const Object reflectiveTest = _ReflectiveTest();
342-
class _ReflectiveTest {
343-
const _ReflectiveTest();
344-
}
345-
''');
335+
if (addTestReflectiveLoaderPackageDep) {
336+
var testReflectiveLoaderPath = addTestReflectiveLoader().parent.path;
346337
configCopy.add(
347338
name: 'test_reflective_loader',
348339
rootPath: testReflectiveLoaderPath,

pkg/linter/test/rules/strict_top_level_inference_test.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ void main() {
1616
@reflectiveTest
1717
class StrictTopLevelInferenceTest extends LintRuleTest {
1818
@override
19-
bool get addReflectiveTestLoaderPackageDep => true;
19+
bool get addTestReflectiveLoaderPackageDep => true;
2020

2121
@override
2222
List<ErrorCode> get ignoredErrorCodes => [

pkg/linter/test/rules/unreachable_from_main_test.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ class UnreachableFromMainTest extends LintRuleTest {
1818
bool get addMetaPackageDep => true;
1919

2020
@override
21-
bool get addReflectiveTestLoaderPackageDep => true;
21+
bool get addTestReflectiveLoaderPackageDep => true;
2222

2323
@override
2424
String get lintRule => LintNames.unreachable_from_main;

0 commit comments

Comments
 (0)