Implemented tests and fixed bugs

Author: RangerMauve

index.js

@@ -45,5 +45,5 @@ function addEdge(graph, edge) {
 	var source = edge.source;
 	var target = edge.target;
 	var id = edge.id;
-	graph.setEdge(source, target, edge, label);
+	graph.setEdge(source, target, edge, id);
 }

package.json

@@ -23,8 +23,11 @@
   "homepage": "https://github.com/jsongraph/graphlib-json-graph",
   "dependencies": {
     "graphlib": "^1.0.4",
-    "mocha": "^2.2.5",
     "par": "^0.3.0",
     "prop": "^0.1.1"
+  },
+  "devDependencies": {
+    "chai": "^3.0.0",
+    "mocha": "^2.2.5"
   }
 }

test/index.js

@@ -1,13 +1,117 @@
+var assert = require("chai").assert;
 var toGraph = require("../");
 
 describe("toGraph()", function() {
-	it.skip("should be able to parse a single graph", function() {});
-	it.skip("should be able to parse multiple graphs", function() {});
-	it.skip("should throw an error if there are no graphs", function() {});
-	it.skip("should support graphs without nodes", function() {});
-	it.skip("should support graphs without edges", function() {});
-	it.skip("should respect the directed property of a graph", function() {});
-	it.skip("should make a graph directed if any of the edges are directed", function() {});
-	it.skip("should be a multigraph", function() {});
-	it.skip("should use the id for distinguishing edges between the same nodes", function() {});
+	var defaultGraph = {
+		graph: {
+			nodes: [{
+				id: "1"
+			}, {
+				id: "2"
+			}],
+			edges: [{
+				source: "1",
+				target: "2"
+			}]
+		}
+	};
+
+	it("should be able to parse a single graph", function() {
+		var graph = toGraph(defaultGraph);
+	});
+
+	it("should be able to parse multiple graphs", function() {
+		var graphs = toGraph({
+			graphs: [defaultGraph, defaultGraph, defaultGraph]
+		});
+
+		assert.lengthOf(graphs, 3, "parsed three graphs");
+	});
+
+	it("should throw an error if there are no graphs", function(done) {
+		try {
+			toGraph({});
+		} catch (e) {
+			done();
+		}
+	});
+
+	it("should support graphs without edges", function() {
+		toGraph({
+			graph: {
+				nodes: [{
+					id: 1
+				}]
+			}
+		});
+	});
+
+	it("should support graphs without nodes", function() {
+		toGraph({
+			graph: {}
+		});
+	});
+
+	it("should respect the directed property of a graph", function() {
+		assert.notOk(toGraph({
+			graph: {
+				directed: false
+			}
+		}).isDirected(), "Graph is not directed");
+
+		assert.ok(toGraph({
+			graph: {
+				directed: true
+			}
+		}).isDirected(), "Graph is directed");
+	});
+
+	it("should make a graph directed if any of the edges are directed", function() {
+		assert.ok(toGraph({
+			graph: {
+				nodes: [{
+					id: "1"
+				}, {
+					id: "2"
+				}],
+				edges: [{
+					directed: true,
+					source: 1,
+					target: 1
+				}]
+			}
+		}).isDirected(), "Graph is directed");
+	});
+
+	it("should be a multigraph", function() {
+		assert.ok(toGraph(defaultGraph).isMultigraph(), "Is a multigraph")
+	});
+
+	it("should use the id for distinguishing edges between the same nodes", function() {
+		var graph = toGraph({
+			graph: {
+				nodes: [{
+					id: "1"
+				}, {
+					id: "2"
+				}],
+				edges: [{
+					id: "e1",
+					directed: true,
+					source: "1",
+					target: "2"
+				}, {
+					id: "e2",
+					directed: true,
+					source: "1",
+					target: "2"
+				}]
+			}
+		});
+
+		var e1 = graph.edge("1", "2", "e1");
+		var e2 = graph.edge("1", "2", "e2");
+
+		assert.notEqual(e1, e2, "Edges were different")
+	});
 });