Updated to retrieve files with long paths with a certain extension using addAssetPath and added tests to cover the case

Author: SC7639

index.js

@@ -85,7 +85,8 @@ module.exports = (http) => {
             let urlParts = url.split('/');
 
             if(url.includes('.')) { // If the url is for an asset
-                let fileType = url.split('.')[2] || url.split('.')[1];
+                let fileType = url.split('.');
+                fileType = fileType[fileType.length - 1];
                 let file = url.replace('/', '');
                 let assetType = urlParts[urlParts.length - 2];
 
@@ -100,7 +101,10 @@ module.exports = (http) => {
                 }
                 else {
                     let path = assetType;
-                    file = this.assetPaths[file].replace(`${path}/`, '');
+
+                    if(this.assetPaths.hasOwnProperty(fileType)) {
+                        file = this.assetPaths[file].replace(`${path}/`, '');
+                    }
 
                     this.renderAsset(path, file, path, res);
                 }
@@ -184,7 +188,7 @@ module.exports = (http) => {
          * @param {Server.Response} res
          */
         renderAsset(path, file, assetType, res) {
-            fs.readFile(`${path}/${file}`, (err, data) => {
+            fs.readFile(`${path}${file}`, (err, data) => {
                 if(err) {
                     res.statusCode = 404;
                     res.end(`File ${path}/${file} doesn't exist`);

package.json

@@ -1,6 +1,6 @@
 {
   "name": "small-router",
-  "version": "1.0.6",
+  "version": "1.0.7",
   "description": "Small router module that creates http server and runs callbacks that have have been added for a route via the api",
   "main": "index.js",
   "scripts": {

tests/index.js

@@ -40,6 +40,7 @@ describe('server', () => {
         });
 
         router.addAssetPath('images', 'tests/images/');
+        router.addAssetPath('js', 'tests/js/');
 
         router.listen(8000);
     });
@@ -170,7 +171,7 @@ describe('server', () => {
         });
     });
 
-    describe('/images/loader.gif', (parameters) => {
+    describe('/images/loader.gif', () => {
         let route = 'images/loader.gif';
 
         it('it should return 200 status code', (done) => {
@@ -197,6 +198,33 @@ describe('server', () => {
         });
     });
 
+    describe('/jspm_packages/github/capaj/systemjs-hot-reloader@0.5.7.js', () => {
+        let route = '/jspm_packages/github/capaj/systemjs-hot-reloader@0.5.7.js';
+
+        it('it should return 200 status code', (done) => {
+            http.get(`${SERVER_URL}/${route}`, (res) => {
+                res.statusCode.should.equal(200);
+                done();
+            });
+        });
+
+        it('it should return contents of systemjs-hot-reloader@0.5.7.js', (done) => {
+            http.get(`${SERVER_URL}/${route}`, (res) => {
+                let data = '';
+                res.on('data', chunk => data += chunk).on('end', () => {
+                    fs.readFile(`tests/js${route}`, (err, data) => {
+                        if(err) {
+                            throw err;
+                        }
+
+                        data.should.equal(data);
+                        done();
+                    });
+                });
+            });
+        });
+    });
+
     describe('/:test([a-z]+)', () => {
         let route = '/test-route';
 

tests/js/jspm_packages/github/capaj/systemjs-hot-reloader@0.5.7.js

@@ -0,0 +1,2 @@
+export * from "github:capaj/systemjs-hot-reloader@0.5.7/hot-reloader.js";
+export {default} from "github:capaj/systemjs-hot-reloader@0.5.7/hot-reloader.js";