cafe-grader-web Commit - r275:3720a1af46c5

cafe-grader-web

Commit `r275:3720a1af46c5` public

parent | child

Description:

started cleaning up tests, fixed fixtures loading error, removed error when contest time limit is uninitialized

Commit status:

[Not Reviewed]

References:

default

Diff options:

| | | |

Comments:

0 Commit comments 0 Inline Comments

Unresolved TODOs:

There are no unresolved TODOs

Add another comment

r275:3720a1af46c5 - Tue, 02 Mar 2010 18:33:21 - 19 files changed: 76 inserted, 211 deleted

app/controllers/main_controller.rb ¶ +1 -1

                 def update_user_start_time
                   user = User.find(session[:user_id])
-            -     UserContestStat.update_user_start_time(user)
+            +     user.update_start_time
                 end
                 def reject_announcement_refresh_when_logged_out

app/models/configuration.rb ¶ +28 -22

                 SYSTEM_MODE_CONF_KEY = 'system.mode'
                 TEST_REQUEST_EARLY_TIMEOUT_KEY = 'contest.test_request.early_timeout'
-            -   # set @@cache = true to only reload once.
+            +   cattr_accessor :cache
-            -   @@cache = false
+            +   cattr_accessor :config_cache
+            +   cattr_accessor :task_grading_info
+            +   cattr_accessor :contest_time_str
+            +   cattr_accessor :contest_time
-            -   @@configurations = nil
+            +   # set @@cache = true to only reload once.
-            -   @@task_grading_info = nil
+            +   Configuration.cache = false
+            +
+            +   Configuration.config_cache = nil
+            +   Configuration.task_grading_info = nil
                 def self.get(key)
-            -     if @@cache
+            +     if Configuration.cache
-            -       if @@configurations == nil
+            +       if Configuration.config_cache == nil
                       self.read_config
                     end
-            -       return @@configurations[key]
+            +       return Configuration.config_cache[key]
                   else
                     return Configuration.read_one_key(key)
                   end
                 end
                 def self.clear
-            -     @@configurations = nil
+            +     Configuration.config_cache = nil
                 end
                 def self.cache?
-            -     @@cache
+            +     Configuration.cache
                 end
                 def self.enable_caching
-            -     @@cache = true
+            +     Configuration.cache = true
                 end
                 #
                 end
                 def self.task_grading_info
-            -     if @@task_grading_info==nil
+            +     if Configuration.task_grading_info==nil
                     read_grading_info
                   end
-            -     return @@task_grading_info
+            +     return Configuration.task_grading_info
                 end
                 def self.standard_mode?
                 def self.contest_time_limit
                   contest_time_str = Configuration['contest.time_limit']
-            -     if not defined? @@contest_time_str
+            +     if not defined? Configuration.contest_time_str
-            -       @@contest_time_str = nil
+            +       Configuration.contest_time_str = nil
                   end
-            -     if @@contest_time_str != contest_time_str
+            +     if Configuration.contest_time_str != contest_time_str
-            -       @@contest_time_str = contest_time_str
+            +       Configuration.contest_time_str = contest_time_str
                     if tmatch = /(\d+):(\d+)/.match(contest_time_str)
                       h = tmatch[1].to_i
                       m = tmatch[2].to_i
-            -         @@contest_time = h.hour + m.minute
+            +         Configuration.contest_time = h.hour + m.minute
                     else
-            -         @@contest_time = nil
+            +         Configuration.contest_time = nil
                     end
                   end
-            -     return @@contest_time
+            +     return Configuration.contest_time
                 end
                 protected
                 end
                 def self.read_config
-            -     @@configurations = {}
+            +     Configuration.config_cache = {}
                   Configuration.find(:all).each do |conf|
                     key = conf.key
                     val = conf.value
-            -       @@configurations[key] = Configuration.convert_type(val,conf.value_type)
+            +       Configuration.config_cache[key] = Configuration.convert_type(val,conf.value_type)
                   end
                 end
                 def self.read_grading_info
                   f = File.open(TASK_GRADING_INFO_FILENAME)
-            -     @@task_grading_info = YAML.load(f)
+            +     Configuration.task_grading_info = YAML.load(f)
                   f.close
                 end

app/models/user.rb ¶ +12

                     return site.time_left
                   elsif Configuration.indv_contest_mode?
                     time_limit = Configuration.contest_time_limit
+            +       if time_limit == nil
+            +         return nil
+            +       end
                     if contest_stat==nil
                       return (Time.now.gmtime + time_limit) - Time.now.gmtime
                     else
                   end
                 end
+            +   def update_start_time
+            +     stat = self.contest_stat
+            +     if stat == nil
+            +       stat = UserContestStat.new(:user => self,
+            +                                  :started_at => Time.now.gmtime)
+            +       stat.save
+            +     end
+            +   end
+            +
                 protected
                   def encrypt_new_password
                     return if password.blank?

app/models/user_contest_stat.rb ¶ -9

@@ -2,13 +2,4
2		2
3	belongs_to :user	3	belongs_to :user
4		4
5	- def self.update_user_start_time(user)
6	- stat = user.contest_stat
7	- if stat == nil
8	- stat = UserContestStat.new(:user => user,
9	- :started_at => Time.now.gmtime)
10	- stat.save
11	- end
12	- end
13	-
14	end	5	end

db/migrate/020_add_graders_right_to_admin_role.rb ¶ +1 -1

                 def self.down
                   graders_right = Right.find_by_name('graders_admin')
-            -     graders_right.destroy
+            +     graders_right.destroy if graders_right
                 end
               end

db/migrate/025_add_site_to_user_and_add_default_site.rb ¶ +1 -1

                   remove_column :users, :site_id
                   default_site = Site.find_by_name('default')
-            -     default_site.destroy
+            +     default_site.destroy if default_site
                 end
               end

db/schema.rb ¶ +13 -26

                 end
                 create_table "problems", :force => true do |t|
-            -     t.string   "name",                 :limit => 30
+            +     t.string  "name",                 :limit => 30
-            -     t.string   "full_name"
+            +     t.string  "full_name"
-            -     t.integer  "full_score"
+            +     t.integer "full_score"
-            -     t.date     "date_added"
+            +     t.date    "date_added"
-            -     t.boolean  "available"
+            +     t.boolean "available"
-            -     t.string   "url"
+            +     t.string  "url"
-            -     t.integer  "description_id"
+            +     t.integer "description_id"
-            -     t.boolean  "test_allowed"
+            +     t.boolean "test_allowed"
-            -     t.boolean  "output_only"
+            +     t.boolean "output_only"
-            -     t.integer  "level",                              :default => 0
+            +     t.string  "description_filename"
-            -     t.datetime "updated_at"
-            -     t.string   "description_filename"
                 end
                 create_table "rights", :force => true do |t|
                   t.text     "solution",   :limit => 16777215
                   t.datetime "created_at"
                   t.datetime "updated_at"
-            -     t.integer  "number"
                 end
                 create_table "test_requests", :force => true do |t|
                 end
                 create_table "users", :force => true do |t|
-            -     t.string   "login",               :limit => 50
+            +     t.string   "login",           :limit => 50
                   t.string   "full_name"
                   t.string   "hashed_password"
-            -     t.string   "salt",                :limit => 5
+            +     t.string   "salt",            :limit => 5
                   t.string   "alias"
                   t.string   "email"
                   t.integer  "site_id"
                   t.integer  "country_id"
-            -     t.boolean  "activated",                         :default => false
+            +     t.boolean  "activated",                     :default => false
                   t.datetime "created_at"
                   t.datetime "updated_at"
-            -     t.string   "member1_full_name"
-            -     t.string   "member2_full_name"
-            -     t.string   "member3_full_name"
-            -     t.boolean  "high_school"
-            -     t.string   "member1_school_name"
-            -     t.string   "member2_school_name"
-            -     t.string   "member3_school_name"
-            -     t.string   "school_name"
-            -     t.string   "province"
-            -     t.integer  "year"
                 end
                 add_index "users", ["login"], :name => "index_users_on_login", :unique => true

spec/controllers/main_controller_spec.rb ¶ +1

                 it "should let user sees her own source" do
                   Submission.should_receive(:find).with(@submission.id.to_s).and_return(@submission)
                   User.should_receive(:find).with(1).and_return(@user)
+            +     @user.should_receive(:update_start_time)
                   get 'source', {:id => @submission.id}, {:user_id => 1}
                   response.should be_success
                 end

spec/spec_helper.rb ¶ +1 -1

                 # in your config/boot.rb
                 config.use_transactional_fixtures = true
                 config.use_instantiated_fixtures  = false
-            -   config.fixture_path = RAILS_ROOT + '/spec/fixtures/'
+            +   config.fixture_path = RAILS_ROOT + '/test/fixtures/'
                 # == Fixtures
                 #

test/fixtures/users.yml ¶ +7 -5

               <%
               User.public_class_method :encrypt
-            - SALT = "abc"
+            + salt = "abc"
               %>
               john:
                 login: john
-            -   hashed_password: <%= User.encrypt("hello",SALT) %>
+            +   full_name: john
-            -   salt: <%= SALT %>
+            +   hashed_password: <%= User.encrypt("hello",salt) %>
+            +   salt: <%= salt %>
               mary:
                 login: mary
-            -   hashed_password: <%= User.encrypt("goodbye",SALT) %>
+            +   full_name: mary
-            -   salt: <%= SALT %>
+            +   hashed_password: <%= User.encrypt("goodbye",salt) %>
+            +   salt: <%= salt %>
                 roles: admin

test/functional/announcements_controller_test.rb ¶ -41

@@ -1,45 +1,4
1	require File.dirname(__FILE__) + '/../test_helper'	1	require File.dirname(__FILE__) + '/../test_helper'
2		2
3	class AnnouncementsControllerTest < ActionController::TestCase	3	class AnnouncementsControllerTest < ActionController::TestCase
4	- def test_should_get_index
5	- get :index
6	- assert_response :success
7	- assert_not_nil assigns(:announcements)
8	- end
9	-
10	- def test_should_get_new
11	- get :new
12	- assert_response :success
13	- end
14	-
15	- def test_should_create_announcement
16	- assert_difference('Announcement.count') do
17	- post :create, :announcement => { }
18	- end
19	-
20	- assert_redirected_to announcement_path(assigns(:announcement))
21	- end
22	-
23	- def test_should_show_announcement
24	- get :show, :id => announcements(:one).id
25	- assert_response :success
26	- end
27	-
28	- def test_should_get_edit
29	- get :edit, :id => announcements(:one).id
30	- assert_response :success
31	- end
32	-
33	- def test_should_update_announcement
34	- put :update, :id => announcements(:one).id, :announcement => { }
35	- assert_redirected_to announcement_path(assigns(:announcement))
36	- end
37	-
38	- def test_should_destroy_announcement
39	- assert_difference('Announcement.count', -1) do
40	- delete :destroy, :id => announcements(:one).id
41	- end
42	-
43	- assert_redirected_to announcements_path
44	- end
45	end	4	end

test/functional/contests_controller_test.rb ¶ -41

@@ -1,45 +1,4
1	require 'test_helper'	1	require 'test_helper'
2		2
3	class ContestsControllerTest < ActionController::TestCase	3	class ContestsControllerTest < ActionController::TestCase
4	- test "should get index" do
5	- get :index
6	- assert_response :success
7	- assert_not_nil assigns(:contests)
8	- end
9	-
10	- test "should get new" do
11	- get :new
12	- assert_response :success
13	- end
14	-
15	- test "should create contest" do
16	- assert_difference('Contest.count') do
17	- post :create, :contest => { }
18	- end
19	-
20	- assert_redirected_to contest_path(assigns(:contest))
21	- end
22	-
23	- test "should show contest" do
24	- get :show, :id => contests(:one).to_param
25	- assert_response :success
26	- end
27	-
28	- test "should get edit" do
29	- get :edit, :id => contests(:one).to_param
30	- assert_response :success
31	- end
32	-
33	- test "should update contest" do
34	- put :update, :id => contests(:one).to_param, :contest => { }
35	- assert_redirected_to contest_path(assigns(:contest))
36	- end
37	-
38	- test "should destroy contest" do
39	- assert_difference('Contest.count', -1) do
40	- delete :destroy, :id => contests(:one).to_param
41	- end
42	-
43	- assert_redirected_to contests_path
44	- end
45	end	4	end

test/functional/login_controller_test.rb ¶ +1 -1

               # Re-raise errors caught by the controller.
               class LoginController; def rescue_action(e) raise e end; end
-            - class LoginControllerTest < Test::Unit::TestCase
+            + class LoginControllerTest < ActionController::TestCase
                 fixtures :users

test/functional/main_controller_test.rb ¶ +4 -16

@@ -1,24 +1,12
1	- require File.dirname(__FILE__) + '/../test_helper'	1	+ require File.dirname(__FILE__) + '/../test_helper'
2	- require 'main_controller'
3		2
4	- # Re-raise errors caught by the controller.	3	+ class MainControllerTest < ActionController::TestCase
5	- class MainController; def rescue_action(e) raise e end; end	4	+ fixtures :users
6	-
7	- class MainControllerTest < Test::Unit::TestCase
8	-
9	fixtures :problems	5	fixtures :problems
10	- fixtures :users
11		6
12	- def setup
13	- @controller = MainController.new
14	- @request = ActionController::TestRequest.new
15	- @response = ActionController::TestResponse.new
16	- end
17	-
18	- # Replace this with your real tests.
19	def test_should_redirect_new_user_to_login	7	def test_should_redirect_new_user_to_login
20	get :list	8	get :list
21	- assert_redirected_to :action => 'login'	9	+ assert_redirected_to :controller => 'main', :action => 'login'
22	end	10	end
23		11
24	def test_should_list_available_problems_if_logged_in	12	def test_should_list_available_problems_if_logged_in

test/functional/problems_controller_test.rb ¶ +1 -1

               # Re-raise errors caught by the controller.
               class ProblemsController; def rescue_action(e) raise e end; end
-            - class ProblemsControllerTest < Test::Unit::TestCase
+            + class ProblemsControllerTest < ActionController::TestCase
                 fixtures :problems
                 def setup

test/functional/sites_controller_test.rb ¶ -41

@@ -1,45 +1,4
1	require File.dirname(__FILE__) + '/../test_helper'	1	require File.dirname(__FILE__) + '/../test_helper'
2		2
3	class SitesControllerTest < ActionController::TestCase	3	class SitesControllerTest < ActionController::TestCase
4	- def test_should_get_index
5	- get :index
6	- assert_response :success
7	- assert_not_nil assigns(:sites)
8	- end
9	-
10	- def test_should_get_new
11	- get :new
12	- assert_response :success
13	- end
14	-
15	- def test_should_create_site
16	- assert_difference('Site.count') do
17	- post :create, :site => { }
18	- end
19	-
20	- assert_redirected_to site_path(assigns(:site))
21	- end
22	-
23	- def test_should_show_site
24	- get :show, :id => sites(:one).id
25	- assert_response :success
26	- end
27	-
28	- def test_should_get_edit
29	- get :edit, :id => sites(:one).id
30	- assert_response :success
31	- end
32	-
33	- def test_should_update_site
34	- put :update, :id => sites(:one).id, :site => { }
35	- assert_redirected_to site_path(assigns(:site))
36	- end
37	-
38	- def test_should_destroy_site
39	- assert_difference('Site.count', -1) do
40	- delete :destroy, :id => sites(:one).id
41	- end
42	-
43	- assert_redirected_to sites_path
44	- end
45	end	4	end

test/functional/user_admin_controller_test.rb ¶ +1 -1

               # Re-raise errors caught by the controller.
               class UserAdminController; def rescue_action(e) raise e end; end
-            - class UserAdminControllerTest < Test::Unit::TestCase
+            + class UserAdminControllerTest < ActionController::TestCase
                 fixtures :users
                 fixtures :roles
                 fixtures :rights

test/functional/users_controller_test.rb ¶ +1 -1

               # Re-raise errors caught by the controller.
               class UsersController; def rescue_action(e) raise e end; end
-            - class UsersControllerTest < Test::Unit::TestCase
+            + class UsersControllerTest < ActionController::TestCase
                 def setup
                   @controller = UsersController.new
                   @request    = ActionController::TestRequest.new

test/test_helper.rb ¶ +3 -2

               require File.expand_path(File.dirname(__FILE__) + "/../config/environment")
               require 'test_help'
-            - class Test::Unit::TestCase
+            + class ActiveSupport::TestCase
                 # Transactional fixtures accelerate your tests by wrapping each test method
                 # in a transaction that's rolled back on completion.  This ensures that the
                 # test database remains unchanged so your fixtures don't have to be reloaded
                 # in MySQL.  Turn off transactional fixtures in this case; however, if you
                 # don't care one way or the other, switching from MyISAM to InnoDB tables
                 # is recommended.
-            -   self.use_transactional_fixtures = true
+            +   self.use_transactional_fixtures = false
                 # Instantiated fixtures are slow, but give you @david where otherwise you
                 # would need people(:david).  If you don't want to migrate your existing

Write
Preview

You need to be logged in to leave comments. Login now

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository permissions settings

Sign in to your account

Author

r275:3720a1af46c5 - Tue, 02 Mar 2010 18:33:21 - 19 files changed: 76 inserted, 211 deleted

Sign in to your account

Commit r275:3720a1af46c5 public

Author

r275:3720a1af46c5 - Tue, 02 Mar 2010 18:33:21 - 19 files changed: 76 inserted, 211 deleted

Commit `r275:3720a1af46c5` public