Add new tests, cleanup

.gitignore

@@ -0,0 +1 @@
+node_modules

Makefile

@@ -6,3 +6,6 @@ xlsx.js: $(DEPS)
 .PHONY: clean
 clean:
 	rm xlsx.js
+
+test:
+	npm test

package.json

@@ -21,7 +21,8 @@
 		"jasmine-node": "x"
 	},
 	"scripts": {
-		"test": "jasmine-node tests/"
+		"pretest": "git submodule init && git submodule update",
+		"test": "jasmine-node --verbose tests/"
 	},
 	"engines": { "node": ">=0.6" }
 }

tests/Common.js

@@ -0,0 +1,25 @@
+var XLSX = require('../');
+
+var tests = {
+	'should be able to open workbook': function (file) {
+		var xlsx = XLSX.readFile('tests/files/' + file);
+		expect(xlsx).toBeTruthy();
+		expect(xlsx).toEqual(jasmine.any(Object));
+	},
+	'should define all api properties correctly': function (file) {
+		var xlsx = XLSX.readFile('tests/files/' + file);
+		expect(xlsx.Workbook).toEqual(jasmine.any(Object));
+		expect(xlsx.Props).toBeDefined();
+		expect(xlsx.Deps).toBeDefined();
+		expect(xlsx.Sheets).toEqual(jasmine.any(Object));
+		expect(xlsx.SheetNames).toEqual(jasmine.any(Array));
+		expect(xlsx.Strings).toBeDefined();
+		expect(xlsx.Styles).toBeDefined();
+	}
+};
+
+module.exports = function (file) {
+	for (var key in tests) {
+		it(key, tests[key].bind(undefined, file));
+	}
+};

tests/EncodedSpec.js

@@ -0,0 +1,8 @@
+var XLSX = require('../');
+var testCommon = require('./Common.js');
+
+var file = 'חישוב_נקודות_זיכוי.xlsx';
+
+describe(file, function () {
+	testCommon(file);
+});

tests/FormulaSpec.js

@@ -0,0 +1,9 @@
+var XLSX = require('../');
+var testCommon = require('./Common.js');
+
+var file = 'formula_stress_test.xlsx';
+
+describe(file, function () {
+	// Opening the file currently crashes node
+	//testCommon(file);
+});

tests/InterviewSpec.js

@@ -0,0 +1,8 @@
+var XLSX = require('../');
+var testCommon = require('./Common.js');
+
+var file = 'interview.xlsx';
+
+describe(file, function () {
+	testCommon(file);
+});

tests/IssueSpec.js

@@ -0,0 +1,8 @@
+var XLSX = require('../');
+var testCommon = require('./Common.js');
+
+var file = 'issue.xlsx';
+
+describe(file, function () {
+	testCommon(file);
+});

tests/MixedSpec.js

@@ -0,0 +1,8 @@
+var XLSX = require('../');
+var testCommon = require('./Common.js');
+
+var file = 'mixed_sheets.xlsx';
+
+describe(file, function () {
+	testCommon(file);
+});

tests/NamedRangesSpec.js

@@ -0,0 +1,8 @@
+var XLSX = require('../');
+var testCommon = require('./Common.js');
+
+var file = 'named_ranges_2011.xlsx';
+
+describe(file, function () {
+	testCommon(file);
+});