0
@@ -89,7 +89,7 @@ class ApplicationController < ActionController::Base
0
       authenticate_with_http_basic do |email, api_key|
0
         if email.to_s.any? and api_key.to_s.length == 50
0
           Person.logged_in = @logged_in = Person.find_by_email_and_api_key(email, api_key)
0
-          Person.logged_in = @logged_in = nil unless @logged_in.super_admin?
0
+          Person.logged_in = @logged_in = nil unless @logged_in and @logged_in.super_admin?
0
         end
0
       end
0
       unless @logged_in

0
@@ -92,6 +92,7 @@ tim:
0
   visible_on_printed_directory: true
0
   full_access: true
0
   feed_code: bla
0
+  api_key: dafH2KIiAcnLEr5JxjmX2oveuczq0R6u7Ijd329DtjatgdYcKp
0
 jeremy:
0
   site_id: 1
0
   id: 2
0
@@ -113,6 +114,7 @@ jeremy:
0
   full_access: true
0
   feed_code: bla2
0
   deacon: true
0
+  api_key: dafH2KIiAcnLEr5JxjmX2oveuczq0R6u7Ijd329DtjatgdYcKq
0
 peter:
0
   site_id: 1
0
   id: 3
0
@@ -132,6 +134,7 @@ peter:
0
   visible_on_printed_directory: true
0
   full_access: true
0
   feed_code: bla3
0
+  api_key: abc # invalid key
0
 jennie:
0
   site_id: 1
0
   id: 4

0
@@ -4,6 +4,7 @@ require 'test_help'
0
 require 'notifier'
0
 
0
 require File.dirname(__FILE__) + '/forgeries'
0
+require File.dirname(__FILE__) + '/test_extensions'
0
 
0
 class Test::Unit::TestCase
0
   # Transactional fixtures accelerate your tests by wrapping each test method
0
@@ -74,16 +75,3 @@ class Test::Unit::TestCase
0
 
0
   fixtures :all
0
 end
0
-
0
-module TestExtensions
0
-  def should(name, &block)
0
-    if block_given?
0
-      define_method 'test ' + name, &block
0
-    else
0
-      puts "Unimplemented: " + name
0
-    end
0
-  end
0
-end
0
-
0
-ActionController::TestCase.extend(TestExtensions)
0
-Test::Unit::TestCase.extend(TestExtensions)