Make sure graph vertex value is converted to string

Author: Pierstoval

src/data-structures/graph/GraphVertex.js

@@ -133,6 +133,6 @@ export default class GraphVertex {
    * @returns {string}
    */
   toString(callback) {
-    return callback ? callback(this.value) : `${this.value}`;
+    return callback ? callback(this.value).toString() : this.value.toString();
   }
 }

src/data-structures/graph/__test__/GraphVertex.test.js

@@ -185,4 +185,20 @@ describe('GraphVertex', () => {
 
     expect(vertexA.getEdges().length).toEqual(3);
   });
+
+  it('should execute callback when passed to toString', () => {
+    const vertex = new GraphVertex('A');
+
+    expect(vertex.toString(() => 'B')).toEqual('B');
+  });
+
+  it('should execute toString on value when calling toString on vertex', () => {
+    const value = {
+      toString() { return 'A'; },
+    };
+
+    const vertex = new GraphVertex(value);
+
+    expect(vertex.toString()).toEqual('A');
+  });
 });