Some comments and cleanup.

Author: savetheclocktower

test/unit/server.rb

@@ -2,8 +2,6 @@
 require 'pathname'
 require 'json'
 
-require 'pp'
-
 class UnitTests < Sinatra::Application
 
   PWD = Pathname.new( File.expand_path( File.dirname(__FILE__) ) )
@@ -66,13 +64,17 @@ def self.get_or_post(url, &block)
     redirect to("/test/#{params[:names]}")
   end
 
+  # /test/ will run all tests;
+  # /test/foo,bar will run just "foo" and "bar" tests.
   get '/test/:names?' do
     names = params[:names]
     @suites = names.nil? ? SUITES : names.split(/,/).uniq
     @unique_asset_string = UNIQUE_ASSET_STRING.to_s
     erb :tests, :locals => { :suites => @suites }
   end
 
+  # Will read from disk each time. No server restart necessary when the
+  # distributable is updated.
   get '/prototype.js' do
     content_type 'text/javascript'
     send_file PATH_TO_PROTOTYPE
@@ -83,7 +85,6 @@ def self.get_or_post(url, &block)
   # (a) they should be more prominent in the directory structure;
   # (b) they should never, ever get cached, and we want to enforce that
   #     aggressively.
-
   get '/js/tests/:filename' do
     filename = params[:filename]
     path = PATH_TO_TEST_JS.join(filename)
@@ -110,8 +111,6 @@ def self.get_or_post(url, &block)
       :body    => request.body.read
     }
 
-    pp response[:headers]
-
     content_type 'application/json'
     JSON.dump(response)
   end

test/unit/static/js/test_helpers.js

@@ -1,3 +1,6 @@
+// TODO: Ideally, none of the stuff in this file should use Prototype, so
+// that a broken method inside Prototype does not end up affecting the test
+// reporting.
 
 (function () {
   // Needed because Mocha's HTML test runner assumes the presence of