Skip to content

Commit 466c579

Browse files
refactor: remove test that is already covered
1 parent 9c03dd1 commit 466c579

File tree

1 file changed

+0
-68
lines changed

1 file changed

+0
-68
lines changed

packages/custom_lint_builder/test/analyzer_converter_test.dart

Lines changed: 0 additions & 68 deletions
Original file line numberDiff line numberDiff line change
@@ -83,72 +83,4 @@ void main() {
8383
},
8484
);
8585
});
86-
87-
test('Respects configSeverities when converting errors', () {
88-
final resourceProvider = MemoryResourceProvider();
89-
final source = FileSource(
90-
resourceProvider.newFile(
91-
'/home/user/project/lib/main.dart',
92-
'void main() {}',
93-
),
94-
);
95-
96-
// Create an analysis error with INFO severity
97-
final error = AnalysisError.tmp(
98-
source: source,
99-
offset: 0,
100-
length: 4,
101-
errorCode: const LintCode(
102-
name: 'rule_name_1',
103-
problemMessage: 'This is a lint',
104-
),
105-
);
106-
107-
// Create config severities map that changes rule_name_1 to ERROR
108-
final configSeverities = <String, ErrorSeverity>{
109-
'rule_name_1': ErrorSeverity.ERROR,
110-
'rule_name_2': ErrorSeverity.WARNING,
111-
};
112-
113-
final converter = CustomAnalyzerConverter();
114-
115-
// Convert the error without config severities - should be INFO
116-
final defaultResult = converter.convertAnalysisError(error);
117-
expect(defaultResult.severity.name, 'INFO');
118-
119-
// Convert the error with direct severity override
120-
final withSeverityParam = converter.convertAnalysisError(
121-
error,
122-
severity: ErrorSeverity.ERROR,
123-
);
124-
expect(withSeverityParam.severity.name, 'ERROR');
125-
126-
// Convert the error with config severities through convertAnalysisErrors
127-
final withConfigSeverities = converter.convertAnalysisErrors(
128-
[error],
129-
configSeverities: configSeverities,
130-
).single;
131-
132-
// Config severities should have overridden the default severity
133-
expect(withConfigSeverities.severity.name, 'ERROR');
134-
135-
// Create an error with a rule name that doesn't have a config severity
136-
final errorWithoutConfigSeverity = AnalysisError.tmp(
137-
source: source,
138-
offset: 0,
139-
length: 4,
140-
errorCode: const LintCode(
141-
name: 'no_config_rule',
142-
problemMessage: 'This is another lint',
143-
),
144-
);
145-
146-
final noConfigResult = converter.convertAnalysisErrors(
147-
[errorWithoutConfigSeverity],
148-
configSeverities: configSeverities,
149-
).single;
150-
151-
// Should use default severity when not in config
152-
expect(noConfigResult.severity.name, 'INFO');
153-
});
15486
}

0 commit comments

Comments
 (0)