Skip to content
This repository has been archived by the owner on Jun 8, 2024. It is now read-only.

Enable skipped tests #8

Closed
springmeyer opened this issue Aug 8, 2013 · 1 comment
Closed

Enable skipped tests #8

springmeyer opened this issue Aug 8, 2013 · 1 comment
Assignees

Comments

@springmeyer
Copy link

Need to enable the two skipped/pending (blue tests) once upstream issues are fixed:

@DennisOSRM - can you let me know if those issues are ones you'd like to jump on or would prefer me to take a look at?

Once solved these tests can be enabled by doing:

diff --git a/test/osrm.test.js b/test/osrm.test.js
index e03ba19..1664415 100644
--- a/test/osrm.test.js
+++ b/test/osrm.test.js
@@ -16,7 +16,7 @@ describe('osrm', function() {
     });

     // @TODO not safe yet to test: https://github.com/DennisOSRM/Project-OSRM/issues/692
-    it.skip('should throw if ini file is blank', function(done) {
+    it('should throw if ini file is blank', function(done) {
         assert.throws(function() { new osrm.Engine("./test/data/bogus.ini"); },
         /@TODO/);
         done();
@@ -38,7 +38,7 @@ describe('osrm', function() {


     // @TODO not safe yet to test: https://github.com/DennisOSRM/Project-OSRM/issues/693
-    it.skip('should throw if ini references corrupt files', function(done) {
+    it('should throw if ini references corrupt files', function(done) {
         assert.throws(function() { new osrm.Engine("./test/data/references-corrupt-files.ini"); },
         /hsgr not found/);
         done();
@DennisOSRM
Copy link
Contributor

I'll jump on this.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants