[ffigen] Support concurrent tests (#2827)

Author: liamappelbe

pkgs/ffigen/dart_test.yaml

@@ -1,5 +1,2 @@
 # This package uses dart:ffi.
 test_on: vm
-
-# https://github.com/dart-lang/native/issues/461
-concurrency: 1

pkgs/ffigen/lib/src/logger.dart

@@ -7,14 +7,16 @@ import 'dart:io';
 import 'package:logging/logging.dart';
 
 /// Creates a default logger that logs to stdout and stderr.
-Logger createDefaultLogger() {
+Logger createDefaultLogger([Level level = Level.INFO]) {
   final logger = Logger.detached('FFIgen');
-  logger.level = Level.INFO;
+  logger.level = level;
   logger.onRecord.listen((record) {
+    final levelStr = '[${record.level.name}]'.padRight(9);
+    final log = '$levelStr: ${record.message}';
     if (record.level >= Level.WARNING) {
-      stderr.writeln(record.message);
+      stderr.writeln(log);
     } else {
-      stdout.writeln(record.message);
+      stdout.writeln(log);
     }
     if (record.error != null) {
       stderr.writeln(record.error);

pkgs/ffigen/test/collision_tests/decl_decl_collision_test.dart

@@ -5,15 +5,11 @@
 import 'package:ffigen/src/code_generator.dart';
 import 'package:ffigen/src/config_provider/config.dart';
 import 'package:ffigen/src/header_parser/parser.dart';
-import 'package:logging/logging.dart';
 import 'package:test/test.dart';
 import '../test_utils.dart';
 
 void main() {
   group('decl_decl_collision_test', () {
-    setUpAll(() {
-      logWarnings(Level.SEVERE);
-    });
     test('declaration conflict', () {
       final context = testContext(
         FfiGenerator(

pkgs/ffigen/test/collision_tests/decl_symbol_address_collision_test.dart

@@ -5,7 +5,6 @@
 import 'package:ffigen/src/code_generator.dart';
 import 'package:ffigen/src/config_provider/config.dart';
 import 'package:ffigen/src/header_parser/parser.dart';
-import 'package:logging/logging.dart';
 import 'package:test/test.dart';
 
 import '../test_utils.dart';
@@ -14,7 +13,6 @@ late Library actual;
 void main() {
   group('decl_symbol_address_collision_test', () {
     setUpAll(() {
-      logWarnings(Level.SEVERE);
       final context = testContext(
         FfiGenerator(
           output: Output(

pkgs/ffigen/test/collision_tests/decl_type_name_collision_test.dart

@@ -5,7 +5,6 @@
 import 'package:ffigen/src/code_generator.dart';
 import 'package:ffigen/src/header_parser.dart' as parser;
 import 'package:ffigen/src/strings.dart' as strings;
-import 'package:logging/logging.dart';
 import 'package:test/test.dart';
 
 import '../test_utils.dart';
@@ -14,7 +13,6 @@ late Library actual;
 void main() {
   group('decl_type_name_collision test', () {
     setUpAll(() {
-      logWarnings(Level.SEVERE);
       actual = parser.parse(
         testContext(
           testConfig('''

pkgs/ffigen/test/collision_tests/reserved_keyword_collision_test.dart

@@ -4,17 +4,13 @@
 
 import 'package:ffigen/src/config_provider.dart';
 import 'package:ffigen/src/header_parser.dart' as parser;
-import 'package:logging/logging.dart';
 import 'package:path/path.dart' as path;
 import 'package:test/test.dart';
 
 import '../test_utils.dart';
 
 void main() {
   group('reserved_keyword_collision_test', () {
-    setUpAll(() {
-      logWarnings(Level.SEVERE);
-    });
     test('reserved keyword collision', () {
       final library = parser.parse(
         testContext(

pkgs/ffigen/test/config_tests/deprecate_assetid_test.dart

@@ -14,8 +14,13 @@ import '../test_utils.dart';
 void main() {
   group('deprecate_assetId_test', () {
     final logArr = <String>[];
-    final logger = logToArray(logArr, Level.WARNING);
-    final config = testConfig('''
+    late FfiGenerator config;
+    setUpAll(() {
+      final logger = createTestLogger(
+        capturedMessages: logArr,
+        level: Level.WARNING,
+      );
+      config = testConfig('''
 ${strings.name}: 'NativeLibrary'
 ${strings.description}: 'Deprecation warning if assetId is used instead of ${strings.ffiNativeAsset}'
 ${strings.output}: 'unused'
@@ -25,9 +30,9 @@ ${strings.headers}:
   ${strings.entryPoints}:
     - '${absPath('test/header_parser_tests/comment_markup.h')}'
 ''', logger: logger);
-    parse(Context(logger, config));
+      parse(Context(logger, config));
+    });
 
-    final logStr = logArr.join('\n');
     test('asset-id is correctly set', () {
       expect(config.output.style is NativeExternalBindings, true);
       expect(
@@ -37,7 +42,7 @@ ${strings.headers}:
     });
 
     test('Deprecation Warning is logged', () {
-      expect(logStr.contains('DEPRECATION WARNING'), true);
+      expect(logArr.join('\n').contains('DEPRECATION WARNING'), true);
     });
   });
 }

pkgs/ffigen/test/config_tests/json_schema_test.dart

@@ -10,7 +10,6 @@ import 'package:ffigen/src/strings.dart' as strings;
 import 'package:file/local.dart';
 import 'package:glob/glob.dart';
 import 'package:json_schema/json_schema.dart';
-import 'package:logging/logging.dart';
 import 'package:path/path.dart' as path;
 import 'package:test/test.dart';
 import 'package:yaml/yaml.dart';
@@ -20,14 +19,14 @@ import '../test_utils.dart';
 void main() {
   group('json_schema_test', () {
     final schema = YamlConfig.getsRootConfigSpec(
-      Logger.root,
+      createTestLogger(),
     ).generateJsonSchema(strings.ffigenJsonSchemaId);
 
     test('Schema Changes', () {
       final actualJsonSchema =
           const JsonEncoder.withIndent(strings.ffigenJsonSchemaIndent).convert(
             YamlConfig.getsRootConfigSpec(
-              Logger.root,
+              createTestLogger(),
             ).generateJsonSchema(strings.ffigenJsonSchemaId),
           );
       final expectedJsonSchema = File(

pkgs/ffigen/test/config_tests/no_cursor_definition_warn_test.dart

@@ -15,7 +15,10 @@ void main() {
   group('no_cursor_definition_warn_test', () {
     setUpAll(() {
       final logArr = <String>[];
-      final logger = logToArray(logArr, Level.WARNING);
+      final logger = createTestLogger(
+        capturedMessages: logArr,
+        level: Level.WARNING,
+      );
       final config = testConfig('''
 ${strings.name}: 'NativeLibrary'
 ${strings.description}: 'Warn for no cursor definition.'

pkgs/ffigen/test/config_tests/unknown_keys_warn_test.dart

@@ -13,7 +13,10 @@ void main() {
   group('unknown_keys_warn_test', () {
     setUpAll(() {
       final logArr = <String>[];
-      final logger = logToArray(logArr, Level.WARNING);
+      final logger = createTestLogger(
+        capturedMessages: logArr,
+        level: Level.WARNING,
+      );
       testConfig('''
 ${strings.name}: 'NativeLibrary'
 ${strings.description}: 'Warn for unknown keys.'