Description:
- more test on user admin and authorization
- fix more routing
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
r754:4cbf294c126c - - 8 files changed: 140 inserted, 79 deleted
@@ -51,13 +51,17 | |||
|
51 | 51 | flash[:notice] = 'Wrong site' |
|
52 | 52 | redirect_to :controller => 'main', :action => 'login' and return |
|
53 | 53 | end |
|
54 | 54 | if (site.password) and (site.password == params[:login][:password]) |
|
55 | 55 | session[:site_id] = site.id |
|
56 | 56 | redirect_to :controller => 'site', :action => 'index' |
|
57 | 57 | else |
|
58 | 58 | flash[:notice] = 'Wrong site password' |
|
59 | 59 | redirect_to :controller => 'site', :action => 'login' |
|
60 | 60 | end |
|
61 | 61 | end |
|
62 | 62 | |
|
63 | + def logout | |
|
64 | + redirect_to root_path | |
|
63 | 65 | end |
|
66 | + | |
|
67 | + end |
@@ -12,43 +12,49 | |||
|
12 | 12 | # system only in some tab |
|
13 | 13 | prepend_before_action :reject_announcement_refresh_when_logged_out, |
|
14 | 14 | :only => [:announcements] |
|
15 | 15 | |
|
16 | 16 | before_action :authenticate_by_ip_address, :only => [:list] |
|
17 | 17 | |
|
18 | 18 | # NOTE: This method is not actually needed, 'config/routes.rb' has |
|
19 | 19 | # assigned action login as a default action. |
|
20 | 20 | def index |
|
21 | 21 | redirect_to :action => 'login' |
|
22 | 22 | end |
|
23 | 23 | |
|
24 | + #reset login, clear session | |
|
24 | 25 | def login |
|
25 | 26 | saved_notice = flash[:notice] |
|
26 | 27 | reset_session |
|
27 | 28 | flash.now[:notice] = saved_notice |
|
28 | 29 | |
|
29 | 30 | # EXPERIMENT: |
|
30 | 31 | # Hide login if in single user mode and the url does not |
|
31 | 32 | # explicitly specify /login |
|
32 | 33 | # |
|
33 | 34 | # logger.info "PATH: #{request.path}" |
|
34 | 35 | # if GraderConfiguration['system.single_user_mode'] and |
|
35 | 36 | # request.path!='/main/login' |
|
36 | 37 | # @hidelogin = true |
|
37 | 38 | # end |
|
38 | 39 | |
|
39 | 40 | @announcements = Announcement.frontpage |
|
40 | 41 | render :action => 'login', :layout => 'empty' |
|
41 | 42 | end |
|
42 | 43 | |
|
44 | + def logout | |
|
45 | + reset_session | |
|
46 | + redirect_to root_path | |
|
47 | + end | |
|
48 | + | |
|
43 | 49 | def list |
|
44 | 50 | prepare_list_information |
|
45 | 51 | end |
|
46 | 52 | |
|
47 | 53 | def help |
|
48 | 54 | @user = User.find(session[:user_id]) |
|
49 | 55 | end |
|
50 | 56 | |
|
51 | 57 | def submit |
|
52 | 58 | user = User.find(session[:user_id]) |
|
53 | 59 | |
|
54 | 60 | @submission = Submission.new |
@@ -12,18 +12,18 | |||
|
12 | 12 | - if GraderConfiguration['ui.show_score'] |
|
13 | 13 | %strong=t 'main.score' |
|
14 | 14 | = "#{(submission.points*100/submission.problem.full_score).to_i} " |
|
15 | 15 | = " [" |
|
16 | 16 | %tt.grader-comment |
|
17 | 17 | = submission.grader_comment |
|
18 | 18 | = "]" |
|
19 | 19 | %br |
|
20 | 20 | %strong View: |
|
21 | 21 | - if GraderConfiguration.show_grading_result |
|
22 | 22 | = link_to '[detailed result]', :action => 'result', :id => submission.id |
|
23 | 23 | /= link_to "#{t 'main.cmp_msg'}", {:action => 'compiler_msg', :id => submission.id}, {popup: true,class: 'btn btn-xs btn-info'} |
|
24 |
- = link_to "#{t 'main.cmp_msg'}", compiler_msg_submission_path(submission |
|
|
25 |
- = link_to "#{t 'main.src_link'}", |
|
|
24 | + = link_to "#{t 'main.cmp_msg'}", compiler_msg_submission_path(submission), {popup: true,remote: true,class: 'btn btn-xs btn-info'} | |
|
25 | + = link_to "#{t 'main.src_link'}",download_submission_path(submission), class: 'btn btn-xs btn-info' | |
|
26 | 26 | = link_to "#{t 'main.submissions_link'}", problem_submissions_path(problem_id), class: 'btn btn-xs btn-info' |
|
27 | 27 | - if GraderConfiguration.show_testcase |
|
28 | 28 | = link_to "testcases", show_problem_testcases_path(problem_id), class: 'btn btn-xs btn-info' |
|
29 | 29 |
@@ -8,18 +8,18 | |||
|
8 | 8 | %th |
|
9 | 9 | - @admins.each_with_index do |user, i| |
|
10 | 10 | %tr |
|
11 | 11 | %td= i+1 |
|
12 | 12 | %td= user.login |
|
13 | 13 | %td= user.full_name |
|
14 | 14 | %td |
|
15 | 15 | - if user.login!='root' |
|
16 | 16 | = link_to '[revoke]', :action => 'revoke_admin', :id => user.id |
|
17 | 17 | %hr |
|
18 | 18 | |
|
19 | 19 | = form_tag :action => 'grant_admin' do |
|
20 | - Grant admin permission to: | |
|
21 | - = text_field_tag 'login' | |
|
22 | - = submit_tag 'Grant' | |
|
20 | + = label_tag :login, 'Grant admin permission to:' | |
|
21 | + = text_field_tag 'login',nil, class: 'input-field' | |
|
22 | + = submit_tag 'Grant', class: 'btn btn-primary' | |
|
23 | 23 | |
|
24 | 24 | %hr/ |
|
25 | 25 | = link_to '[go back to index]', :action => 'index' |
@@ -1,26 +1,23 | |||
|
1 | 1 | Rails.application.routes.draw do |
|
2 | 2 | resources :tags |
|
3 | 3 | get "sources/direct_edit" |
|
4 | 4 | |
|
5 | 5 | root :to => 'main#login' |
|
6 | 6 | |
|
7 | 7 | #logins |
|
8 | 8 | match 'login/login', to: 'login#login', via: [:get,:post] |
|
9 | 9 | |
|
10 | - | |
|
11 | 10 | resources :contests |
|
12 | - | |
|
13 | 11 | resources :sites |
|
14 | - | |
|
15 | 12 | resources :test |
|
16 | 13 | |
|
17 | 14 | resources :messages do |
|
18 | 15 | collection do |
|
19 | 16 | get 'console' |
|
20 | 17 | end |
|
21 | 18 | end |
|
22 | 19 | |
|
23 | 20 | resources :announcements do |
|
24 | 21 | member do |
|
25 | 22 | get 'toggle','toggle_front' |
|
26 | 23 | end |
@@ -94,24 +91,25 | |||
|
94 | 91 | |
|
95 | 92 | #user admin |
|
96 | 93 | resources :user_admin do |
|
97 | 94 | collection do |
|
98 | 95 | match 'bulk_manage', via: [:get, :post] |
|
99 | 96 | get 'bulk_mail' |
|
100 | 97 | get 'user_stat' |
|
101 | 98 | get 'import' |
|
102 | 99 | get 'new_list' |
|
103 | 100 | get 'admin' |
|
104 | 101 | get 'active' |
|
105 | 102 | get 'mass_mailing' |
|
103 | + get 'revoke_admin' | |
|
106 | 104 | post 'grant_admin' |
|
107 | 105 | match 'create_from_list', via: [:get, :post] |
|
108 | 106 | match 'random_all_passwords', via: [:get, :post] |
|
109 | 107 | end |
|
110 | 108 | member do |
|
111 | 109 | get 'clear_last_ip' |
|
112 | 110 | end |
|
113 | 111 | end |
|
114 | 112 | |
|
115 | 113 | resources :contest_management, only: [:index] do |
|
116 | 114 | collection do |
|
117 | 115 | get 'user_stat' |
@@ -135,29 +133,31 | |||
|
135 | 133 | get 'problem_hof/:id', action: 'problem_hof' |
|
136 | 134 | get 'current_score' |
|
137 | 135 | get 'max_score' |
|
138 | 136 | post 'show_max_score' |
|
139 | 137 | end |
|
140 | 138 | #get 'report/current_score', to: 'report#current_score', as: 'report_current_score' |
|
141 | 139 | #get 'report/problem_hof(/:id)', to: 'report#problem_hof', as: 'report_problem_hof' |
|
142 | 140 | #get "report/login" |
|
143 | 141 | #get 'report/max_score', to: 'report#max_score', as: 'report_max_score' |
|
144 | 142 | #post 'report/show_max_score', to: 'report#show_max_score', as: 'report_show_max_score' |
|
145 | 143 | |
|
146 | 144 | resource :main, only: [], controller: 'main' do |
|
145 | + get 'login' | |
|
146 | + get 'logout' | |
|
147 | 147 | get 'list' |
|
148 | 148 | get 'submission(/:id)', action: 'submission', as: 'main_submission' |
|
149 | - post 'submit' | |
|
150 | 149 | get 'announcements' |
|
151 | 150 | get 'help' |
|
151 | + post 'submit' | |
|
152 | 152 | end |
|
153 | 153 | #main |
|
154 | 154 | #get "main/list" |
|
155 | 155 | #get 'main/submission(/:id)', to: 'main#submission', as: 'main_submission' |
|
156 | 156 | #post 'main/submit', to: 'main#submit' |
|
157 | 157 | #get 'main/announcements', to: 'main#announcements' |
|
158 | 158 | |
|
159 | 159 | |
|
160 | 160 | # |
|
161 | 161 | get 'tasks/view/:file.:ext' => 'tasks#view' |
|
162 | 162 | get 'tasks/download/:id/:file.:ext' => 'tasks#download' |
|
163 | 163 | get 'heartbeat/:id/edit' => 'heartbeat#edit' |
@@ -1,5 +1,5 | |||
|
1 | - class ChangeSubmissionSourceSize < ActiveRecord::Migration | |
|
1 | + class ChangeSubmissionSourceSize < ActiveRecord::Migration[4.2] | |
|
2 | 2 | def change |
|
3 | 3 | change_column :submissions, :source, :text, :limit => 1.megabyte |
|
4 | 4 | end |
|
5 | 5 | end |
@@ -1,307 +1,306 | |||
|
1 | 1 | # This file is auto-generated from the current state of the database. Instead |
|
2 | 2 | # of editing this file, please use the migrations feature of Active Record to |
|
3 | 3 | # incrementally modify your database, and then regenerate this schema definition. |
|
4 | 4 | # |
|
5 | 5 | # Note that this schema.rb definition is the authoritative source for your |
|
6 | 6 | # database schema. If you need to create the application database on another |
|
7 | 7 | # system, you should be using db:schema:load, not running all the migrations |
|
8 | 8 | # from scratch. The latter is a flawed and unsustainable approach (the more migrations |
|
9 | 9 | # you'll amass, the slower it'll run and the greater likelihood for issues). |
|
10 | 10 | # |
|
11 | 11 | # It's strongly recommended that you check this file into your version control system. |
|
12 | 12 | |
|
13 | - ActiveRecord::Schema.define(version: 20180612102327) do | |
|
13 | + ActiveRecord::Schema.define(version: 2018_06_12_102327) do | |
|
14 | 14 | |
|
15 |
- create_table "announcements", |
|
|
15 | + create_table "announcements", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t| | |
|
16 | 16 |
t.string |
|
17 | - t.text "body", limit: 65535 | |
|
17 | + t.text "body" | |
|
18 | 18 |
t.boolean |
|
19 | 19 | t.datetime "created_at" |
|
20 | 20 | t.datetime "updated_at" |
|
21 | 21 |
t.boolean |
|
22 | 22 |
t.boolean |
|
23 | 23 |
t.string |
|
24 | 24 |
t.string |
|
25 | 25 | end |
|
26 | 26 | |
|
27 |
- create_table "contests", |
|
|
27 | + create_table "contests", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t| | |
|
28 | 28 |
t.string |
|
29 | 29 |
t.boolean |
|
30 | 30 | t.datetime "created_at" |
|
31 | 31 | t.datetime "updated_at" |
|
32 | 32 |
t.string |
|
33 | 33 | end |
|
34 | 34 | |
|
35 |
- create_table "contests_problems", id: false, force: :cascade |
|
|
35 | + create_table "contests_problems", id: false, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t| | |
|
36 | 36 | t.integer "contest_id" |
|
37 | 37 | t.integer "problem_id" |
|
38 | 38 | end |
|
39 | 39 | |
|
40 |
- create_table "contests_users", id: false, force: :cascade |
|
|
40 | + create_table "contests_users", id: false, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t| | |
|
41 | 41 | t.integer "contest_id" |
|
42 | 42 | t.integer "user_id" |
|
43 | 43 | end |
|
44 | 44 | |
|
45 |
- create_table "countries", |
|
|
45 | + create_table "countries", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t| | |
|
46 | 46 |
t.string |
|
47 | 47 | t.datetime "created_at" |
|
48 | 48 | t.datetime "updated_at" |
|
49 | 49 | end |
|
50 | 50 | |
|
51 |
- create_table "descriptions", |
|
|
52 | - t.text "body", limit: 65535 | |
|
51 | + create_table "descriptions", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t| | |
|
52 | + t.text "body" | |
|
53 | 53 |
t.boolean |
|
54 | 54 | t.datetime "created_at" |
|
55 | 55 | t.datetime "updated_at" |
|
56 | 56 | end |
|
57 | 57 | |
|
58 |
- create_table "grader_configurations", |
|
|
58 | + create_table "grader_configurations", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t| | |
|
59 | 59 |
t.string |
|
60 | 60 |
t.string |
|
61 | 61 |
t.string |
|
62 | 62 | t.datetime "created_at" |
|
63 | 63 | t.datetime "updated_at" |
|
64 |
- t.text |
|
|
64 | + t.text "description" | |
|
65 | 65 | end |
|
66 | 66 | |
|
67 |
- create_table "grader_processes", |
|
|
67 | + create_table "grader_processes", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t| | |
|
68 | 68 |
t.string |
|
69 | 69 |
t.integer |
|
70 | 70 |
t.string |
|
71 | 71 |
t.boolean |
|
72 | 72 | t.datetime "created_at" |
|
73 | 73 | t.datetime "updated_at" |
|
74 | 74 |
t.integer |
|
75 | 75 |
t.string |
|
76 | 76 |
t.boolean |
|
77 |
- t.index ["host", "pid"], name: "index_grader_processes_on_ |
|
|
77 | + t.index ["host", "pid"], name: "index_grader_processes_on_ip_and_pid" | |
|
78 | 78 | end |
|
79 | 79 | |
|
80 |
- create_table "groups", |
|
|
80 | + create_table "groups", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci", force: :cascade do |t| | |
|
81 | 81 | t.string "name" |
|
82 | 82 | t.string "description" |
|
83 | 83 | end |
|
84 | 84 | |
|
85 |
- create_table "groups_problems", id: false, force: :cascade |
|
|
85 | + create_table "groups_problems", id: false, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci", force: :cascade do |t| | |
|
86 | 86 | t.integer "problem_id", null: false |
|
87 | 87 |
t.integer "group_id", |
|
88 |
- t.index ["group_id", "problem_id"], name: "index_groups_problems_on_group_id_and_problem_id" |
|
|
88 | + t.index ["group_id", "problem_id"], name: "index_groups_problems_on_group_id_and_problem_id" | |
|
89 | 89 | end |
|
90 | 90 | |
|
91 |
- create_table "groups_users", id: false, force: :cascade |
|
|
91 | + create_table "groups_users", id: false, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci", force: :cascade do |t| | |
|
92 | 92 | t.integer "group_id", null: false |
|
93 | 93 |
t.integer "user_id", |
|
94 |
- t.index ["user_id", "group_id"], name: "index_groups_users_on_user_id_and_group_id" |
|
|
94 | + t.index ["user_id", "group_id"], name: "index_groups_users_on_user_id_and_group_id" | |
|
95 | 95 | end |
|
96 | 96 | |
|
97 |
- create_table "heart_beats", |
|
|
97 | + create_table "heart_beats", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=latin1", force: :cascade do |t| | |
|
98 | 98 |
t.integer |
|
99 | 99 |
t.string |
|
100 | - t.datetime "created_at" | |
|
101 | - t.datetime "updated_at" | |
|
100 | + t.datetime "created_at", null: false | |
|
101 | + t.datetime "updated_at", null: false | |
|
102 | 102 |
t.string |
|
103 |
- t.index ["updated_at"], name: "index_heart_beats_on_updated_at" |
|
|
103 | + t.index ["updated_at"], name: "index_heart_beats_on_updated_at" | |
|
104 | 104 | end |
|
105 | 105 | |
|
106 |
- create_table "languages", |
|
|
106 | + create_table "languages", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t| | |
|
107 | 107 |
t.string "name", |
|
108 | 108 | t.string "pretty_name" |
|
109 | 109 |
t.string "ext", |
|
110 | 110 | t.string "common_ext" |
|
111 | 111 | end |
|
112 | 112 | |
|
113 |
- create_table "logins", |
|
|
113 | + create_table "logins", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=latin1", force: :cascade do |t| | |
|
114 | 114 |
t.integer |
|
115 | 115 |
t.string |
|
116 | - t.datetime "created_at" | |
|
117 | - t.datetime "updated_at" | |
|
116 | + t.datetime "created_at", null: false | |
|
117 | + t.datetime "updated_at", null: false | |
|
118 | 118 | end |
|
119 | 119 | |
|
120 |
- create_table "messages", |
|
|
120 | + create_table "messages", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t| | |
|
121 | 121 |
t.integer |
|
122 | 122 |
t.integer |
|
123 | 123 |
t.integer |
|
124 | - t.text "body", limit: 65535 | |
|
124 | + t.text "body" | |
|
125 | 125 |
t.boolean |
|
126 | 126 | t.datetime "created_at" |
|
127 | 127 | t.datetime "updated_at" |
|
128 | 128 | end |
|
129 | 129 | |
|
130 |
- create_table "problems", |
|
|
130 | + create_table "problems", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t| | |
|
131 | 131 |
t.string |
|
132 | 132 |
t.string |
|
133 | 133 | t.integer "full_score" |
|
134 | 134 |
t.date |
|
135 | 135 | t.boolean "available" |
|
136 | 136 |
t.string |
|
137 | 137 | t.integer "description_id" |
|
138 | 138 | t.boolean "test_allowed" |
|
139 | 139 | t.boolean "output_only" |
|
140 | 140 |
t.string |
|
141 | 141 | t.boolean "view_testcase" |
|
142 | 142 | end |
|
143 | 143 | |
|
144 |
- create_table "problems_tags", |
|
|
144 | + create_table "problems_tags", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci", force: :cascade do |t| | |
|
145 | 145 | t.integer "problem_id" |
|
146 | 146 | t.integer "tag_id" |
|
147 |
- t.index ["problem_id", "tag_id"], name: "index_problems_tags_on_problem_id_and_tag_id", unique: true |
|
|
148 |
- t.index ["problem_id"], name: "index_problems_tags_on_problem_id" |
|
|
149 |
- t.index ["tag_id"], name: "index_problems_tags_on_tag_id" |
|
|
147 | + t.index ["problem_id", "tag_id"], name: "index_problems_tags_on_problem_id_and_tag_id", unique: true | |
|
148 | + t.index ["problem_id"], name: "index_problems_tags_on_problem_id" | |
|
149 | + t.index ["tag_id"], name: "index_problems_tags_on_tag_id" | |
|
150 | 150 | end |
|
151 | 151 | |
|
152 |
- create_table "rights", |
|
|
152 | + create_table "rights", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t| | |
|
153 | 153 | t.string "name" |
|
154 | 154 | t.string "controller" |
|
155 | 155 | t.string "action" |
|
156 | 156 | end |
|
157 | 157 | |
|
158 |
- create_table "rights_roles", id: false, force: :cascade |
|
|
158 | + create_table "rights_roles", id: false, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t| | |
|
159 | 159 | t.integer "right_id" |
|
160 | 160 | t.integer "role_id" |
|
161 |
- t.index ["role_id"], name: "index_rights_roles_on_role_id" |
|
|
161 | + t.index ["role_id"], name: "index_rights_roles_on_role_id" | |
|
162 | 162 | end |
|
163 | 163 | |
|
164 |
- create_table "roles", |
|
|
164 | + create_table "roles", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t| | |
|
165 | 165 | t.string "name" |
|
166 | 166 | end |
|
167 | 167 | |
|
168 |
- create_table "roles_users", id: false, force: :cascade |
|
|
168 | + create_table "roles_users", id: false, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t| | |
|
169 | 169 | t.integer "role_id" |
|
170 | 170 | t.integer "user_id" |
|
171 |
- t.index ["user_id"], name: "index_roles_users_on_user_id" |
|
|
171 | + t.index ["user_id"], name: "index_roles_users_on_user_id" | |
|
172 | 172 | end |
|
173 | 173 | |
|
174 |
- create_table "sessions", |
|
|
174 | + create_table "sessions", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t| | |
|
175 | 175 |
t.string |
|
176 | - t.text "data", limit: 65535 | |
|
176 | + t.text "data" | |
|
177 | 177 | t.datetime "updated_at" |
|
178 |
- t.index ["session_id"], name: "index_sessions_on_session_id" |
|
|
179 |
- t.index ["updated_at"], name: "index_sessions_on_updated_at" |
|
|
178 | + t.index ["session_id"], name: "index_sessions_on_session_id" | |
|
179 | + t.index ["updated_at"], name: "index_sessions_on_updated_at" | |
|
180 | 180 | end |
|
181 | 181 | |
|
182 |
- create_table "sites", |
|
|
182 | + create_table "sites", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t| | |
|
183 | 183 |
t.string |
|
184 | 184 |
t.boolean |
|
185 | 185 | t.datetime "start_time" |
|
186 | 186 | t.datetime "created_at" |
|
187 | 187 | t.datetime "updated_at" |
|
188 | 188 |
t.integer |
|
189 | 189 |
t.string |
|
190 | 190 | end |
|
191 | 191 | |
|
192 |
- create_table "submission_view_logs", |
|
|
192 | + create_table "submission_view_logs", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=latin1", force: :cascade do |t| | |
|
193 | 193 |
t.integer |
|
194 | 194 |
t.integer |
|
195 | - t.datetime "created_at" | |
|
196 | - t.datetime "updated_at" | |
|
195 | + t.datetime "created_at", null: false | |
|
196 | + t.datetime "updated_at", null: false | |
|
197 | 197 | end |
|
198 | 198 | |
|
199 |
- create_table "submissions", |
|
|
199 | + create_table "submissions", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t| | |
|
200 | 200 |
t.integer |
|
201 | 201 |
t.integer |
|
202 | 202 |
t.integer |
|
203 | 203 |
t.text |
|
204 | - t.binary "binary", limit: 65535 | |
|
204 | + t.binary "binary" | |
|
205 | 205 | t.datetime "submitted_at" |
|
206 | 206 | t.datetime "compiled_at" |
|
207 |
- t.text |
|
|
207 | + t.text "compiler_message" | |
|
208 | 208 | t.datetime "graded_at" |
|
209 | 209 |
t.integer |
|
210 |
- t.text |
|
|
210 | + t.text "grader_comment" | |
|
211 | 211 |
t.integer |
|
212 | 212 |
t.string |
|
213 |
- t.float |
|
|
213 | + t.float "max_runtime" | |
|
214 | 214 |
t.integer |
|
215 | 215 |
t.integer |
|
216 | 216 |
t.string |
|
217 |
- t.index ["user_id", "problem_id", "number"], name: "index_submissions_on_user_id_and_problem_id_and_number", unique: true |
|
|
218 |
- t.index ["user_id", "problem_id"], name: "index_submissions_on_user_id_and_problem_id" |
|
|
217 | + t.index ["user_id", "problem_id", "number"], name: "index_submissions_on_user_id_and_problem_id_and_number", unique: true | |
|
218 | + t.index ["user_id", "problem_id"], name: "index_submissions_on_user_id_and_problem_id" | |
|
219 | 219 | end |
|
220 | 220 | |
|
221 |
- create_table "tags", |
|
|
221 | + create_table "tags", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci", force: :cascade do |t| | |
|
222 | 222 |
t.string |
|
223 |
- t.text |
|
|
223 | + t.text "description" | |
|
224 | 224 |
t.boolean |
|
225 | 225 |
t.datetime "created_at", |
|
226 | 226 |
t.datetime "updated_at", |
|
227 | 227 | end |
|
228 | 228 | |
|
229 |
- create_table "tasks", |
|
|
229 | + create_table "tasks", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t| | |
|
230 | 230 |
t.integer |
|
231 | 231 | t.datetime "created_at" |
|
232 | 232 |
t.integer |
|
233 | 233 | t.datetime "updated_at" |
|
234 |
- t.index ["submission_id"], name: "index_tasks_on_submission_id" |
|
|
234 | + t.index ["submission_id"], name: "index_tasks_on_submission_id" | |
|
235 | 235 | end |
|
236 | 236 | |
|
237 |
- create_table "test_pairs", |
|
|
237 | + create_table "test_pairs", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t| | |
|
238 | 238 |
t.integer |
|
239 | 239 |
t.text |
|
240 | 240 |
t.text |
|
241 | 241 | t.datetime "created_at" |
|
242 | 242 | t.datetime "updated_at" |
|
243 | 243 | end |
|
244 | 244 | |
|
245 |
- create_table "test_requests", |
|
|
245 | + create_table "test_requests", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t| | |
|
246 | 246 |
t.integer |
|
247 | 247 |
t.integer |
|
248 | 248 |
t.integer |
|
249 | 249 |
t.string |
|
250 | 250 |
t.string |
|
251 | 251 |
t.string |
|
252 | 252 |
t.integer |
|
253 | 253 | t.datetime "updated_at" |
|
254 | 254 | t.datetime "submitted_at" |
|
255 | 255 | t.datetime "compiled_at" |
|
256 |
- t.text |
|
|
256 | + t.text "compiler_message" | |
|
257 | 257 | t.datetime "graded_at" |
|
258 | 258 |
t.string |
|
259 | 259 | t.datetime "created_at" |
|
260 |
- t.float |
|
|
260 | + t.float "running_time" | |
|
261 | 261 |
t.string |
|
262 | 262 |
t.integer |
|
263 |
- t.index ["user_id", "problem_id"], name: "index_test_requests_on_user_id_and_problem_id" |
|
|
263 | + t.index ["user_id", "problem_id"], name: "index_test_requests_on_user_id_and_problem_id" | |
|
264 | 264 | end |
|
265 | 265 | |
|
266 |
- create_table "testcases", |
|
|
266 | + create_table "testcases", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci", force: :cascade do |t| | |
|
267 | 267 |
t.integer |
|
268 | 268 |
t.integer |
|
269 | 269 |
t.integer |
|
270 | 270 |
t.integer |
|
271 | 271 |
t.text |
|
272 | 272 |
t.text |
|
273 | 273 | t.datetime "created_at" |
|
274 | 274 | t.datetime "updated_at" |
|
275 |
- t.index ["problem_id"], name: "index_testcases_on_problem_id" |
|
|
275 | + t.index ["problem_id"], name: "index_testcases_on_problem_id" | |
|
276 | 276 | end |
|
277 | 277 | |
|
278 |
- create_table "user_contest_stats", |
|
|
278 | + create_table "user_contest_stats", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t| | |
|
279 | 279 |
t.integer |
|
280 | 280 | t.datetime "started_at" |
|
281 | 281 | t.datetime "created_at" |
|
282 | 282 | t.datetime "updated_at" |
|
283 | - t.boolean "forced_logout" | |
|
284 | 283 | end |
|
285 | 284 | |
|
286 |
- create_table "users", |
|
|
285 | + create_table "users", id: :integer, options: "ENGINE=InnoDB DEFAULT CHARSET=utf8", force: :cascade do |t| | |
|
287 | 286 |
t.string |
|
288 | 287 |
t.string |
|
289 | 288 |
t.string |
|
290 | 289 |
t.string |
|
291 | 290 |
t.string |
|
292 | 291 |
t.string |
|
293 | 292 |
t.integer |
|
294 | 293 |
t.integer |
|
295 | 294 |
t.boolean |
|
296 | 295 | t.datetime "created_at" |
|
297 | 296 | t.datetime "updated_at" |
|
298 | 297 |
t.string |
|
299 | 298 |
t.boolean |
|
300 | 299 |
t.string |
|
301 | 300 |
t.string |
|
302 |
- t.index ["login"], name: "index_users_on_login", unique: true |
|
|
301 | + t.index ["login"], name: "index_users_on_login", unique: true | |
|
303 | 302 | end |
|
304 | 303 | |
|
305 | 304 | add_foreign_key "problems_tags", "problems" |
|
306 | 305 | add_foreign_key "problems_tags", "tags" |
|
307 | 306 | end |
@@ -32,19 +32,71 | |||
|
32 | 32 | |
|
33 | 33 | within('tr', text: 'McTestface') do |
|
34 | 34 | click_on 'Edit' |
|
35 | 35 | end |
|
36 | 36 | |
|
37 | 37 | fill_in 'Alias', with: 'hahaha' |
|
38 | 38 | fill_in 'Remark', with: 'section 2' |
|
39 | 39 | click_on 'Update User' |
|
40 | 40 | |
|
41 | 41 | assert_text 'section 2' |
|
42 | 42 | end |
|
43 | 43 | |
|
44 | + test "add multiple users" do | |
|
45 | + login 'admin', 'admin' | |
|
46 | + within 'header' do | |
|
47 | + click_on 'Manage' | |
|
48 | + click_on 'Users', match: :first | |
|
49 | + end | |
|
50 | + | |
|
51 | + click_on 'New list of users', match: :first | |
|
52 | + find(:css, 'textarea').fill_in with:"abc1,Boaty McBoatface,abcdef,alias1,remark1,\nabc2,Boaty2 McSecond,acbdef123,aias2,remark2" | |
|
53 | + click_on 'create users' | |
|
54 | + | |
|
55 | + assert_text('remark1') | |
|
56 | + assert_text('remark2') | |
|
57 | + end | |
|
58 | + | |
|
59 | + test "grant admin right" do | |
|
60 | + login 'admin', 'admin' | |
|
61 | + within 'header' do | |
|
62 | + click_on 'Manage' | |
|
63 | + click_on 'Users', match: :first | |
|
64 | + end | |
|
65 | + | |
|
66 | + click_on "View administrator" | |
|
67 | + fill_in 'login', with: 'john' | |
|
68 | + click_on "Grant" | |
|
69 | + | |
|
70 | + visit logout_main_path | |
|
71 | + login 'john','hello' | |
|
72 | + within 'header' do | |
|
73 | + click_on 'Manage' | |
|
74 | + click_on 'Problem', match: :first | |
|
75 | + end | |
|
76 | + assert_text "Turn off all problems" | |
|
77 | + end | |
|
78 | + | |
|
79 | + test "try using admin from normal user" do | |
|
80 | + login 'admin','admin' | |
|
81 | + visit bulk_manage_user_admin_index_path | |
|
82 | + assert_current_path bulk_manage_user_admin_index_path | |
|
83 | + visit logout_main_path | |
|
84 | + | |
|
85 | + login 'jack','morning' | |
|
86 | + visit bulk_manage_user_admin_index_path | |
|
87 | + assert_current_path root_path | |
|
88 | + assert_text 'You are not authorized' | |
|
89 | + | |
|
90 | + login 'james','morning' | |
|
91 | + visit new_list_user_admin_index_path | |
|
92 | + assert_current_path root_path | |
|
93 | + assert_text 'You are not authorized' | |
|
94 | + end | |
|
95 | + | |
|
44 | 96 | def login(username,password) |
|
45 | 97 | visit root_path |
|
46 | 98 | fill_in "Login", with: username |
|
47 | 99 | fill_in "Password", with: password |
|
48 | 100 | click_on "Login" |
|
49 | 101 | end |
|
50 | 102 | end |
You need to be logged in to leave comments.
Login now