update integration tests

Author: Skycoder42

analysis_options.yaml

@@ -6,8 +6,6 @@ analyzer:
   exclude:
     - "**.freezed.dart"
     - "**.g.dart"
-  errors:
-    included_file_warning: ignore
 
 linter:
   rules:

pubspec.yaml

@@ -36,7 +36,7 @@ dev_dependencies:
   cider: ^0.1.5
   coverage: ^1.6.3
   custom_lint: ^0.3.4
-  dart_test_tools: ^5.0.0
+  dart_test_tools: ^5.0.1
   freezed: ^2.3.3
   json_serializable: ^6.6.2
   mocktail: ^0.3.0

test/integration/integration_test.dart

@@ -85,12 +85,12 @@ void main() {
   }) async {
     final knownTasks = [
       'format',
-      'test-imports',
       'analyze',
+      'custom-lint',
       'flutter-compat',
       'outdated',
       'pull-up-dependencies',
-      'lib-exports',
+      'osv-scanner',
     ];
     final configEditor = YamlEditor('_placeholder: null');
     for (final task in knownTasks) {
@@ -146,9 +146,17 @@ dependencies:
 
 dev_dependencies:
   lint: null
+  custom_lint: null
+  dart_test_tools: '>=5.0.0'
 ''',
     );
 
+    await writeFile('analysis_options.yaml', '''
+analyzer:
+  plugins:
+    - custom_lint
+''');
+
     await writeFile(
       'bin/format.dart',
       '''
@@ -285,40 +293,24 @@ void main() {
     );
   });
 
-  test('test-imports', () async {
+  test('custom-lint', () async {
     final lines = <String>[];
     await git(const ['add', 'test/test.dart']);
     final code = await sut(
-      'test-imports',
-      arguments: const ['--detailed-exit-code', '-ldebug'],
-      failOnError: false,
-      onStdout: lines.add,
-    );
-    expect(code, HookResult.rejected.index);
-    expect(
-      lines,
-      contains(
-        startsWith('  [ERR] Found self import that is not from src: import '),
-      ),
-    );
-  });
-
-  test('lib-exports', () async {
-    final lines = <String>[];
-    await git(const ['add', 'lib']);
-    final code = await sut(
-      'lib-exports',
+      'custom-lint',
       arguments: const ['--detailed-exit-code', '-ldebug'],
       failOnError: false,
       onStdout: lines.add,
     );
     expect(code, HookResult.rejected.index);
     expect(
       lines,
-      contains(
-        allOf(
-          startsWith('  [ERR] '),
-          endsWith('Source file is not exported anywhere'),
+      allOf(
+        contains(
+          '  [INF]   lib/src/analyze.dart:1:6 • The library contains public symbols, but is not exported in any of the package library files. • src_library_not_exported',
+        ),
+        contains(
+          '  [INF]   test/test.dart:1:8 • Libraries in lib/src, test or tool should not import package library files from lib. • no_self_package_imports',
         ),
       ),
     );

test/unit/tasks/provider/default_tasks_loader_test.dart

@@ -1,5 +1,6 @@
 import 'package:dart_pre_commit/src/config/pubspec_config_loader.dart';
 import 'package:dart_pre_commit/src/tasks/analyze_task.dart';
+import 'package:dart_pre_commit/src/tasks/custom_lint_task.dart';
 import 'package:dart_pre_commit/src/tasks/flutter_compat_task.dart';
 import 'package:dart_pre_commit/src/tasks/format_task.dart';
 import 'package:dart_pre_commit/src/tasks/osv_scanner_task.dart';
@@ -57,6 +58,7 @@ void main() {
           mockPubspecConfigLoader.loadPubspecConfig,
           () => mockTaskLoader.registerConfigurableTask(formatTaskProvider),
           () => mockTaskLoader.registerConfigurableTask(analyzeTaskProvider),
+          () => mockTaskLoader.registerTask(customLintTaskProvider),
           () => mockTaskLoader.registerConfigurableTask(outdatedTaskProvider),
           () => mockTaskLoader
               .registerConfigurableTask(pullUpDependenciesTaskProvider),
@@ -78,6 +80,7 @@ void main() {
           mockPubspecConfigLoader.loadPubspecConfig,
           () => mockTaskLoader.registerConfigurableTask(formatTaskProvider),
           () => mockTaskLoader.registerConfigurableTask(analyzeTaskProvider),
+          () => mockTaskLoader.registerTask(customLintTaskProvider),
           () => mockTaskLoader.registerTask(flutterCompatTaskProvider),
           () => mockTaskLoader.registerConfigurableTask(outdatedTaskProvider),
           () => mockTaskLoader