feat: change createFileSync to getFile and createDirSync to getDirectory

test: change tests accordingly and new tests
kasperpeulen · Dec 12, 2015 · 3edc96b · 3edc96b
1 parent 44f39af
commit 3edc96b
Show file tree

Hide file tree

Showing 2 changed files with 47 additions and 6 deletions.
diff --git a/lib/kp_directory.dart b/lib/kp_directory.dart
@@ -6,12 +6,12 @@ void makeEmptySync(Directory dir) {
   dir.listSync().forEach((f) => f.deleteSync(recursive: true));
 }
 
-File createFileSync(Directory dir, String fileName) {
-  return new File(join(dir.path, fileName))..createSync();
+File getFile(Directory dir, String fileName) {
+  return new File(join(dir.path, fileName));
 }
 
-Directory createDirSync(Directory dir, String directoryName) {
-  return new Directory(join(dir.path, directoryName))..createSync();
+Directory getDirectory(Directory dir, String directoryName) {
+  return new Directory(join(dir.path, directoryName));
 }
 
 void copySync(Directory from, Directory to) {

diff --git a/test/kp_directory_test.dart b/test/kp_directory_test.dart
@@ -1,6 +1,7 @@
 import 'dart:io';
 
 import 'package:kp_directory/kp_directory.dart';
+import 'package:path/path.dart';
 import 'package:test/test.dart';
 
 void main() {
@@ -13,12 +14,52 @@ void main() {
       expect(tempDir.listSync().isEmpty, isTrue);
 
       // populate it
-      createFileSync(tempDir, 'file.txt');
-      createDirSync(tempDir, 'my_dir_name');
+      getFile(tempDir, 'file.txt')..createSync();
+      getDirectory(tempDir, 'my_dir_name')..createSync();
       expect(tempDir.listSync().isEmpty, isFalse);
 
       makeEmptySync(tempDir);
       expect(tempDir.listSync().isEmpty, isTrue);
+      tempDir.deleteSync(recursive: true);
+    });
+  });
+
+  group("createFileSync", () {
+    test("it creates a file", () {
+      final tempDir = Directory.systemTemp.createTempSync();
+
+      final file = getFile(tempDir, 'hello_world.txt')..createSync();
+      expect(file.existsSync(), isTrue);
+      tempDir.deleteSync(recursive: true);
+    });
+  });
+
+  group("createDirSync", () {
+    test("it creates a dir", () {
+      final tempDir = Directory.systemTemp.createTempSync();
+
+      final dir = getDirectory(tempDir, 'hello_world')..createSync();
+      expect(dir.existsSync(), isTrue);
+      tempDir.deleteSync(recursive: true);
+    });
+  });
+
+  group("copySync", () {
+    test("it does copy", () {
+      final tempDir = Directory.systemTemp.createTempSync();
+      final fileInTempDir = getFile(tempDir, 'hello_world.txt')..createSync();
+
+      final newDir = Directory.systemTemp.createTempSync();
+      newDir.deleteSync(recursive: true);
+      expect(newDir.existsSync(), isFalse);
+
+      copySync(tempDir, newDir);
+
+      expect(newDir.existsSync(), isTrue);
+
+      expect(basename(newDir.listSync()[0].path), basename(fileInTempDir.path));
+      newDir.deleteSync(recursive: true);
+      tempDir.deleteSync(recursive: true);
     });
   });
 }