Fixed bug + tests ZIP output.

models/submission.rb

@@ -29,9 +29,9 @@ def to_min_secs
   # The filename we'll call it in the ZIP file.
   def filename
     formatted_name = "#{name}".tr(' ', '_')
-    random_string  = SecureRandom.urlsafe_base64[0..4]
+    formatted_time = "#{created_at.hour}:#{created_at.min}:#{created_at.sec}"
 
-    "#{hackathon}/#{formatted_name}_#{random_string}.html"
+    "#{hackathon}/#{formatted_name}_#{formatted_time}.html"
   end
 
   key :name, String

no_light_sinatra.rb

@@ -57,7 +57,7 @@ def show_editor(custom_branding = 'dell')
   end
 
   def create_tempfile
-    @create_tempfile = Tempfile.new(get_hackathon)
+    @create_tempfile ||= Tempfile.new(get_hackathon)
   end
 
   def get_submit_params
@@ -78,7 +78,9 @@ def create_zip_folder(array = [])
       if block_given?
         yield(zip)
       else
-        array.to_a.each { |entry| zip[entry.filename] = entry.html }
+        array.to_a.each do |entry|
+          zip[entry.filename] = entry.html
+        end
       end
     end
   end

spec/admin_can_download_submissions_spec.rb

@@ -45,7 +45,10 @@ def visit_download_and_validate_zip(hackathon)
     page.response_headers['Content-Disposition'].must_equal       'attachment; filename="' + hackathon + '.zip"'
     page.response_headers['Content-Transfer-Encoding'].must_equal 'binary'
 
-    # Need to add tests to check the contents of the ZIP.
-    # Currently `page.body` isn't returning anything. Need to investigate.
+    tempfile = Tempfile.new(hackathon)
+    tempfile.write(page.source)
+    tempfile.close
+
+    Submission.by_hackathon(hackathon).map(&:filename).must_equal Zippy.list(tempfile.path)
   end
 end