Merge pull request nwjs#1336 from kingFighter/chromium-args-test

rogerwang · rogerwang · commit ad3acfca292a · 2013-11-25T21:11:15.000-08:00
[test] update test-case for chromium-args because 'Intl' is not supporte...
diff --git a/tests/automatic_tests/chromium-args/index.html b/tests/automatic_tests/chromium-args/index.html
@@ -0,0 +1,23 @@
+<!DOCTYPE html>
+<html>
+  <head>
+    <title>test case for chromium-args</title>
+  </head>
+  <body>
+    <h1>Hello World!</h1>
+    <script>
+      var gui = require('nw.gui')
+      var fs = require('fs');
+      fs.openSync('./hi', 'w');
+      var results = false;
+      if (location.href == 'http://www.baidu.com/')
+	  results = true;
+      
+      var client = require('../../nw_test_app').createClient({
+        argv: gui.App.argv,
+        data: results
+      });
+
+    </script>
+  </body>
+</html>
diff --git a/tests/automatic_tests/chromium-args/mocha_test.js b/tests/automatic_tests/chromium-args/mocha_test.js
@@ -1,9 +1,64 @@
 var assert = require('assert');
+var cp = require('child_process');
+var fs = require('fs');
+var fs_extra = require('fs-extra');
+var exec_path = process.execPath;
+var app_path = path.join(global.tests_dir, 'chromium-args');
+var app_test = require('./nw_test_app');
 
 describe('chromium-args', function() {
-  it("window.Intl should be undefined beacuse disabled in package.json",
-    function() {
-      assert.equal(window.Intl, undefined);
-    }
-  )
-})
+    describe('--disable-javascript', function() {
+    	before(function(done) {
+    	    this.timeout(0);
+	    fs_extra.copy(path.join(app_path, 'package1.json'), path.join(app_path, 'package.json'), function (err) {
+		if (err) {
+		    throw err;
+		}
+    		cp.exec(exec_path + " " + app_path, function(err, stdout, stderr) {
+		})
+    	    });
+	    setTimeout(done, 2500);
+    	});
+	
+    	it('javascript should not be executed', function() {
+    	    var result = fs.existsSync(path.join(app_path, 'hi'));
+    	    assert.equal(result, false);
+    	});
+	
+    	after(function() {
+    	    fs.unlink(path.join(app_path, 'hi'), function(err) {if(err && err.code !== 'ENOENT') throw err});
+    	    fs.unlink(path.join(app_path, 'package.json'), function(err) {if(err && err.code !== 'ENOENT') throw err});
+    	});
+    });
+
+    describe('--app=url', function() {
+	var result2 = false;
+    	before(function(done) {
+    	    this.timeout(0);
+	    fs_extra.copy(path.join(app_path, 'package2.json'), path.join(app_path, 'package.json'), function (err) {
+		if (err) {
+		    throw err;
+		}
+    		var child = app_test.createChildProcess({
+		    execPath: exec_path,
+		    appPath: app_path,
+		    end: function(data, app) {
+			result2 = data;
+			app.kill();
+			done()
+		    }
+		});
+    	    });
+    	});
+	
+    	it('website should be the url', function() {
+    	    assert.equal(result2, true);
+    	});
+    
+    	after(function() {
+    	    fs.unlink(path.join(app_path, 'package.json'), function(err) {if(err && err.code !== 'ENOENT') throw err});
+    	    fs.unlink(path.join(app_path, 'hi'), function(err) {if(err && err.code !== 'ENOENT') throw err});
+    	});
+    });
+
+});
diff --git a/tests/automatic_tests/chromium-args/package1.json b/tests/automatic_tests/chromium-args/package1.json
@@ -0,0 +1,9 @@
+{
+  "name": "nw-chromium-args",
+  "main": "index.html",
+  "window": {
+    "show": false
+  },
+  "chromium-args": "--disable-javascript"
+  
+}
diff --git a/tests/automatic_tests/chromium-args/package2.json b/tests/automatic_tests/chromium-args/package2.json
@@ -0,0 +1,9 @@
+{
+  "name": "nw-chromium-args",
+  "main": "index.html",
+  "window": {
+    "show": false
+  },
+  "chromium-args": "--app=http://www.baidu.com"
+  
+}
diff --git a/tests/package.json b/tests/package.json
@@ -22,7 +22,6 @@
     "bignum": "git+https://github.com/owenc4a4/bignum.git",
     "fs-extra": "*"
   },
-  "chromium-args": "--disable-javascript-i18n-api",
   "user-agent": "%name/%nwver/%ver/%webkit_ver/%osinfo",
   "scripts": {
     "install": "python ../tools/build_native_modules.py"
