Merge pull request nwjs#1388 from kingFighter/tests

rogerwang · rogerwang · commit 0ad59a597b31 · 2013-12-10T23:56:34.000-08:00
[test] temp dir should be removed after app exits
diff --git a/tests/automatic_tests/temp_dir/index.html b/tests/automatic_tests/temp_dir/index.html
@@ -0,0 +1,26 @@
+<!DOCTYPE html>
+<html>
+  <head>
+    <title>Test CASE FOR TMP DIR SHOULD BE REMOVED AFTER APP EXIT</title>
+  </head>
+  <body>
+    <h1>Hello World!</h1>
+    <script>
+      var gui = require('nw.gui');
+      var path = require('path');
+      var temp_dir_path = location.href;
+      var fs = require('fs');
+      var tmp_path = path.join(gui.App.argv[0], 'tmp-nw', 'path.org');
+
+      fs.writeFile(tmp_path, temp_dir_path, function(err) {
+	  if(err) {
+	      console.log(err);
+	  } else {
+	      console.log("The file was saved!");
+	      gui.App.quit();
+	  }
+      }); 
+      
+    </script>
+  </body>
+</html>
diff --git a/tests/automatic_tests/temp_dir/mocha_test.js b/tests/automatic_tests/temp_dir/mocha_test.js
@@ -0,0 +1,67 @@
+var func = require('./' + global.tests_dir +'/start_app/script.js');
+var execPath = func.getExecPath();
+var fs = require('fs-extra');
+var os = require('os');
+var cp = require('child_process');
+var app_test = require('./nw_test_app');
+var current_path = process.cwd();
+var mac_app_path = path.join('tmp-nw', 'node-webkit.app');
+var assert = require('assert');
+var temp_path;
+var execPath;
+
+if (os.platform() == 'win32') {
+    execPath = path.join('tmp-nw', 'app.exe');
+}
+if (os.platform() == 'linux') {
+    execPath = path.join('tmp-nw', 'app');
+}
+if (os.platform() == 'darwin') {
+    execPath = path.join('tmp-nw', 'node-webkit.app', 'Contents', 'MacOS', 'node-webkit');
+}
+
+function make_execuable_file(folder_path, done) {
+    func.copyExecFiles(function() {
+	func.copySourceFiles(folder_path); 
+	func.zipSourceFiles(function() {
+	    func.makeExecuableFile();
+	    if (os.platform() == 'darwin') {
+		var app_path = 'tmp-nw/node-webkit.app/Contents/Resources/app.nw';
+		fs.mkdir(app_path, function(err) {
+		    if(err && err.code !== 'EEXIST') throw err
+		    fs.copy('tmp-nw/index.html', path.join(app_path, 'index.html'));
+		    fs.copy('tmp-nw/package.html', path.join(app_path, 'package.html'));
+		    setTimeout(done, 3000);
+
+		});
+	    } else {
+		setTimeout(function() {
+		    var child = cp.spawn(execPath, [current_path]);
+		    child.on('exit', function() {
+			temp_path = path.dirname(fs.readFileSync(path.join('tmp-nw','path.org'))).substring(8);
+			done();
+		    });
+		}, 3000);
+	    }
+	});
+    });
+}
+
+describe('temp_dir', function() { 
+    this.timeout(0);
+
+    before(function(done) {
+	make_execuable_file('temp_dir', done);
+    });
+
+    after(function() {
+    	fs.remove('tmp-nw', function (er) {
+            if (er) throw er;
+        });
+    });
+
+    it('should be removed after app exit', function() {
+	var exist = fs.existsSync(temp_path);
+	assert.equal(exist, false);
+    });
+});
diff --git a/tests/automatic_tests/temp_dir/package.json b/tests/automatic_tests/temp_dir/package.json
@@ -0,0 +1,4 @@
+{
+  "name": "nw",
+  "main": "index.html"    
+}

-Original file line number
+Diff line change
@@ @@ -0,0 +1,4 @@ @@
 +{
 +  "name": "nw",
 +  "main": "index.html"
 +}