Generator tests: prevent parallel running tests from sending events

Author: greenrobot-team

generator/lib/src/analysis/analysis.dart

@@ -18,14 +18,20 @@ import 'build_properties.dart';
 /// Requires [tokenFilePath] to exist, otherwise does nothing. See the
 /// associated test (analysis_test.dart) on how to create this file.
 class ObjectBoxAnalysis {
-  static const _debug = false;
-
   /// Path is relative to lib folder.
-  static const tokenFilePath = "assets/analysis-token.txt";
+  static const defaultTokenFilePath = "assets/analysis-token.txt";
 
   static const _url = "api.mixpanel.com";
   static const _path = "track";
 
+  final bool _debug;
+  final String _tokenFilePath;
+
+  ObjectBoxAnalysis(
+      {bool debug = false, String tokenFilePath = defaultTokenFilePath})
+      : _debug = debug,
+        _tokenFilePath = tokenFilePath;
+
   /// Builds a Build event and sends it with [sendEvent]. May not send if it
   /// fails to store a unique identifier and last time sent, or if no valid API
   /// token is found.
@@ -126,7 +132,7 @@ class ObjectBoxAnalysis {
   }
 
   Future<String?> _getToken() async {
-    final uri = Uri.parse("package:objectbox_generator/$tokenFilePath");
+    final uri = Uri.parse("package:objectbox_generator/$_tokenFilePath");
     final resolvedUri = await Isolate.resolvePackageUri(uri);
     if (resolvedUri != null) {
       final file = File.fromUri(resolvedUri);

generator/test/analysis_test.dart

@@ -24,39 +24,41 @@ void main() {
     var obfuscatedToken = _obfuscateToken(token);
     final keyBase64 = obfuscatedToken.keyBase64;
     final dataBase64 = obfuscatedToken.dataBase64;
-    print("Store this in generator/lib/${ObjectBoxAnalysis.tokenFilePath}:");
+    print(
+        "Store this in generator/lib/${ObjectBoxAnalysis.defaultTokenFilePath}:");
     print("$keyBase64\n$dataBase64");
 
-    final decryptedToken =
-        ObjectBoxAnalysis().decryptAndVerifyToken(keyBase64, dataBase64);
+    final decryptedToken = ObjectBoxAnalysis(debug: true)
+        .decryptAndVerifyToken(keyBase64, dataBase64);
     expect(decryptedToken, equals(token));
   }, skip: true);
 
   test("send test event", () async {
-    // Create a token file just for this test (delete right after to avoid
-    // CI sending events).
+    // Create a token file just for this test
     final token = Platform.environment["DART_ANALYSIS_TOKEN"];
     if (token == null) {
       markTestSkipped("DART_ANALYSIS_TOKEN not set");
       return;
     }
     var obfuscatedToken = _obfuscateToken(token);
-    final tokenFile = File("lib/${ObjectBoxAnalysis.tokenFilePath}");
+    // Use a token file different from the default to prevent parallel running
+    // tests from sending events.
+    final tokenFilePath = 'assets/test-analysis-token.txt';
+    final tokenFile = File("lib/$tokenFilePath");
     await tokenFile.writeAsString(
         "${obfuscatedToken.keyBase64}\n${obfuscatedToken.dataBase64}");
+    addTearDown(() async => tokenFile.delete());
 
     final testPubspec = Pubspec("test", dependencies: {
       "flutter": SdkDependency("flutter"),
       "objectbox": HostedDependency(version: VersionConstraint.parse("^1.2.3"))
     });
 
-    final analysis = ObjectBoxAnalysis();
+    final analysis =
+        ObjectBoxAnalysis(tokenFilePath: tokenFilePath, debug: true);
     final event = analysis.buildEvent("Test Event", "test-uid", testPubspec);
     final response = await analysis.sendEvent(event);
 
-    // Delete token before test may fail to ensure CI does not send events.
-    await tokenFile.delete();
-
     expect(response!.statusCode, 200);
     expect(response.body, "1");
   });