add tests

Author: dcharkes

pkgs/native_assets_builder/lib/src/build_runner/build_runner.dart

@@ -514,20 +514,18 @@ ${e.message}
               await dependenciesHashes.findOutdatedFileSystemEntity();
           if (outdatedFile == null) {
             logger.info(
-              [
-                'Skipping ${hook.name} for ${config.packageName} in $outDir.',
-                'Last build on ${output.timestamp}.',
-              ].join(' '),
+              'Skipping ${hook.name} for ${config.packageName}'
+              ' in ${outDir.toFilePath()}.'
+              ' Last build on ${output.timestamp}.',
             );
             // All build flags go into [outDir]. Therefore we do not have to
             // check here whether the config is equal.
             return output;
           }
           logger.info(
-            [
-              'Running ${hook.name} for ${config.packageName} in $outDir.',
-              '${outdatedFile.toFilePath()} changed.',
-            ].join(' '),
+            'Rerunning ${hook.name} for ${config.packageName}'
+            ' in ${outDir.toFilePath()}.'
+            ' ${outdatedFile.toFilePath()} changed.',
           );
         }
 
@@ -708,7 +706,7 @@ ${e.message}
       if (previousDartExecutable != dartExecutable) {
         mustCompile = true;
         logger.info(
-          'Recompiling $scriptUri, Dart executable changed.',
+          'Recompiling ${scriptUri.toFilePath()}, Dart executable changed.',
         );
       }
 
@@ -717,7 +715,8 @@ ${e.message}
       if (outdatedFile != null) {
         mustCompile = true;
         logger.info(
-          'Recompiling $scriptUri, ${outdatedFile.toFilePath()} changed.',
+          'Recompiling ${scriptUri.toFilePath()}, '
+          '${outdatedFile.toFilePath()} changed.',
         );
       }
     }

pkgs/native_assets_builder/test/build_runner/build_runner_caching_test.dart

@@ -87,10 +87,14 @@ void main() async {
       await copyTestProjects(targetUri: tempUri);
       final packageUri = tempUri.resolve('native_add/');
 
+      final logMessages = <String>[];
+      final logger = createCapturingLogger(logMessages);
+
       await runPubGet(
         workingDirectory: packageUri,
         logger: logger,
       );
+      logMessages.clear();
 
       {
         final result = (await build(
@@ -105,6 +109,7 @@ void main() async {
         await expectSymbols(
             asset: CodeAsset.fromEncoded(result.encodedAssets.single),
             symbols: ['add']);
+        logMessages.clear();
       }
 
       await copyTestProjects(
@@ -122,6 +127,18 @@ void main() async {
           buildValidator: validateCodeAssetBuildOutput,
           applicationAssetValidator: validateCodeAssetInApplication,
         ))!;
+
+        final cUri = packageUri.resolve('src/').resolve('native_add.c');
+        expect(
+          logMessages.join('\n'),
+          stringContainsInOrder(
+            [
+              'Rerunning build for native_add in',
+              '${cUri.toFilePath()} changed.'
+            ],
+          ),
+        );
+
         await expectSymbols(
           asset: CodeAsset.fromEncoded(result.encodedAssets.single),
           symbols: ['add', 'subtract'],
@@ -181,14 +198,13 @@ void main() async {
             buildValidator: validateCodeAssetBuildOutput,
             applicationAssetValidator: validateCodeAssetInApplication,
           ))!;
-          {
-            final compiledHook = logMessages
-                .where((m) =>
-                    m.contains('dart compile kernel') ||
-                    m.contains('dart.exe compile kernel'))
-                .isNotEmpty;
-            expect(compiledHook, isTrue);
-          }
+
+          final hookUri = packageUri.resolve('hook/').resolve('build.dart');
+          expect(
+            logMessages.join('\n'),
+            contains('Recompiling ${hookUri.toFilePath()}'),
+          );
+
           logMessages.clear();
           await expectSymbols(
             asset: CodeAsset.fromEncoded(result.encodedAssets.single),