test: fix flaky test in linux CI/CD environments (#677)

* test: update run test in order to fix flaky result in linux environment * chore: remove unnecessary TODO reminder * chore: add ignoringDependencyMessages as matcher utility Also, apply same fix to workspace test * fix: dart format in matcher utility file --------- Co-authored-by: Jessica Tarra <[email protected]>
invertase · Mar 26, 2024 · 28882a1 · 28882a1
1 parent 2ee575e
commit 28882a1
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 2 deletions.
diff --git a/packages/melos/test/commands/run_test.dart b/packages/melos/test/commands/run_test.dart
@@ -594,7 +594,7 @@ melos run hello_script
 
       expect(
         logger.output.normalizeNewLines(),
-        ignoringAnsii(
+        ignoringDependencyMessages(
           '''
 melos run hello_script
   └> test_script

diff --git a/packages/melos/test/matchers.dart b/packages/melos/test/matchers.dart
@@ -26,6 +26,25 @@ import 'package:path/path.dart' as p;
 import 'package:test/test.dart';
 import 'package:yaml/yaml.dart';
 
+/// Resolve flaky test in linux environments
+Matcher ignoringDependencyMessages(String expected) {
+  return predicate(
+    (actual) {
+      final normalizedActual = actual
+          .toString()
+          .split('\n')
+          .where(
+            (line) =>
+                !line.startsWith('Resolving dependencies...') &&
+                !line.startsWith('Got dependencies!'),
+          )
+          .join('\n');
+      return ignoringAnsii(expected).matches(normalizedActual, {});
+    },
+    'ignores dependency resolution messages',
+  );
+}
+
 Matcher packageNamed(Object? matcher) => _PackageNameMatcher(matcher);
 
 Matcher ignoringAnsii(Object? matcher) {

diff --git a/packages/melos/test/workspace_test.dart b/packages/melos/test/workspace_test.dart
@@ -110,7 +110,7 @@ The packages that caused the problem are:
       );
 
       expect(result.exitCode, 0);
-      expect(result.stdout, 'a\n');
+      expect(result.stdout, ignoringDependencyMessages('a\n'));
     });
 
     test(
-Original file line number
+Diff line change
@@ Expand Up / @@ -110,7 +110,7 @@ The packages that caused the problem are: @@
           );
           expect(result.exitCode, 0);
-          expect(result.stdout, 'a\n');
+          expect(result.stdout, ignoringDependencyMessages('a\n'));
         });
         test(
@@ Expand Down @@