Allow loading external files (urls)

Now you use an url in the config:
files = [
  JASMINE,
  JASMINE_ADAPTER,
  'src/*.js',
  'http://code.jquery.com/jquery-1.7.2.min.js'
];

[changelog]

lib/config.js

@@ -71,7 +71,7 @@ var resolveSinglePattern = function(pattern, done) {
         return null;
       } else {
         // it's a file - if end of pattern, accept the path
-        if (pointer === parts.length) results.push({path: path, mtime: stat.mtime});
+        if (pointer === parts.length) results.push({path: path, mtime: stat.mtime, isUrl: false});
         return finish();
       }
     });
@@ -85,6 +85,8 @@ var resolveSinglePattern = function(pattern, done) {
  * - unique files
  * - absolute paths
  *
+ * Keep absolute url patterns without checking them.
+ *
  * @param {Array.<string>} patterns Array of patterns
  * @param {Array.<string>} exclude Array of patterns to exlclude
  * @param {function(err, files)} done Callback
@@ -99,6 +101,13 @@ var resolve = function(patterns, exclude, done) {
   });
 
   patterns.forEach(function(pattern, index) {
+
+    // By-pass absolute urls
+    if (util.isUrlAbsolute(pattern)) {
+      resultSets[index] = [{path: pattern, isUrl: true, mtime: null}];
+      return;
+    }
+
     waiting++;
 
     resolveSinglePattern(pattern, function(err, files) {
@@ -126,7 +135,7 @@ var resolve = function(patterns, exclude, done) {
                 log.debug('Excluded file ' + file.path);
               } else {
                 uniqueResults.push(file);
-                log.debug('Resolved file ' + file.path);
+                log.debug('Resolved %s %s', file.isUrl ? 'url' : 'file', file.path);
               }
             }
           });
@@ -184,6 +193,7 @@ var parseConfig = function(configFilePath, cliOptions) {
   config.basePath = path.resolve(path.dirname(configFilePath), config.basePath);
 
   var basePathResolve = function(relativePath) {
+    if (util.isUrlAbsolute(relativePath)) return relativePath;
     return path.resolve(config.basePath, relativePath);
   };
 
@@ -215,6 +225,8 @@ var FileGuardian = function(filePatterns, excludePatterns, emitter, autoWatch, a
 
     log.debug('Checking files for modifications...');
     files_.forEach(function(file) {
+      if (file.isUrl) return;
+
       waiting++;
       fs.stat(file.path, function(err, stat) {
         if (file.mtime < stat.mtime) {

lib/util.js

@@ -47,6 +47,11 @@ exports.isString = function(value) {
 };
 
 
+exports.isUrlAbsolute = function(url) {
+  return /^https?:\/\//.test(url);
+};
+
+
 exports.camelToSnake = function(camelCase) {
   return camelCase.replace(/[A-Z]/g, function(match, pos) {
     return (pos > 0 ? '_' : '') + match.toLowerCase();

lib/web-server.js

@@ -65,8 +65,8 @@ var createHandler = function(fileGuardian, STATIC_FOLDER) {
         setNoCacheHeaders(response);
 
         var scriptTags = fileGuardian.getFiles().map(function(file) {
-          // add timestamps only for /context.html, no timestamps for /debug.html
-          return util.format(SCRIPT_TAG, request.url === '/context.html' ?
+          // Add timestamps only for /context.html, no timestamps for /debug.html or urls
+          return util.format(SCRIPT_TAG, request.url === '/context.html' && !file.isUrl ?
               file.path + '?' + file.mtime.getTime() : file.path);
         });
 

test/unit/config.spec.coffee

@@ -57,6 +57,7 @@ describe 'config', ->
       conf:
         'invalid.js': fsMock.file 0, '={function'
         'exclude.js': fsMock.file 0, 'exclude = ["one.js", "sub/two.js"];'
+        'absolute.js': fsMock.file 0, 'files = ["http://some.com", "https://more.org/file.js"];'
 
     # load file under test
     m = loadFile __dirname + '/../../lib/config.js', mocks, {process: mocks.process}
@@ -191,6 +192,17 @@ describe 'config', ->
       waitForFinished()
 
 
+    it 'should keep absolute url patterns', ->
+      m.resolve ['http://one.com', 'https://more.org/some.js', '/home/*.js'], [], (err, files) ->
+        expect(files[0].path).toBe 'http://one.com'
+        expect(files[0].isUrl).toBe true
+        expect(files[1].path).toBe 'https://more.org/some.js'
+        expect(files[1].isUrl).toBe true
+        finished++
+
+      waitForFinished()
+
+
   #============================================================================
   # config.parseConfig()
   # Should parse configuration file and do some basic processing as well
@@ -218,6 +230,11 @@ describe 'config', ->
       expect(config.files).toEqual ['/home/one.js', '/home/sub/two.js']
 
 
+    it 'should keep absolute url file patterns', ->
+      config = e.parseConfig '/conf/absolute.js'
+      expect(config.files).toEqual ['http://some.com', 'https://more.org/file.js']
+
+
     it 'should resolve all exclude patterns', ->
       config = e.parseConfig '/conf/exclude.js'
       expect(config.exclude).toEqual ['/conf/one.js', '/conf/sub/two.js']

test/unit/util.spec.coffee

@@ -143,3 +143,15 @@ describe 'util', ->
 
       expect(destination.a).toBe 1
       expect(result).toEqual {a: 4, b: 2, c: 5}
+
+
+  #==============================================================================
+  # util.isUrlAbsolute()
+  #==============================================================================
+  describe 'isUrlAbsolute', ->
+
+    it 'should check http/https protocol', ->
+      expect(util.isUrlAbsolute 'some/path/http.html').toBe false
+      expect(util.isUrlAbsolute '/some/more.py').toBe false
+      expect(util.isUrlAbsolute 'http://some.com/path').toBe true
+      expect(util.isUrlAbsolute 'https://more.org/some.js').toBe true