commit 6872f3969d90ae16f0d7ed22479ee2e7611534ca
author Jonas Finnemann Jensen <jonasfj@google.com> Fri Sep 17 12:25:26 2021 +0200
committer GitHub <noreply@github.com> Fri Sep 17 12:25:26 2021 +0200
tree 7fcd8d70f1c7192c2637cf1ef77886c12c0d88ac
parent c2dc871784b59720e6cf5794dee1a20735df3feb

Improve tests for applicationConfigHome (#67)

test/cli_util_test.dart

diff --git a/test/cli_util_test.dart b/test/cli_util_test.dart
index 95ba0fb..32dcdfc 100644
--- a/test/cli_util_test.dart
+++ b/test/cli_util_test.dart
@@ -7,6 +7,7 @@
 import 'package:cli_util/cli_util.dart';
 import 'package:cli_util/src/utils.dart';
 import 'package:test/test.dart';
+import 'package:path/path.dart' as p;
 
 void main() => defineTests();
 
@@ -38,8 +39,15 @@
   });
 
   group('applicationConfigHome', () {
-    test('returns a string', () {
-      expect(applicationConfigHome('dart'), isA<String>());
+    test('returns a non-empty string', () {
+      expect(applicationConfigHome('dart'), isNotEmpty);
+    });
+
+    test('has an ancestor folder that exists', () {
+      final path = p.split(applicationConfigHome('dart'));
+      // We expect that first two segments of the path exists.. This is really
+      // just a dummy check that some part of the path exists.
+      expect(Directory(p.joinAll(path.take(2))).existsSync(), isTrue);
     });
   });
 }