cafe-grader-web

Commit r639:2826fdf23cce public

parent | child
Description:
fix user update
Commit status:
[Not Reviewed]
References:
java
Diff options:
Raw Diff | Patch Diff | Download Diff | Ignore whitespace | Increase context
Comments:
0 Commit comments 0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
Side by Side Unified
Expand All Files Collapse All Files Wide Mode Diff
Add another comment
Browse Files

r639:2826fdf23cce - - 2 files changed: 4 inserted, 4 deleted

Show file before | Show file after | Raw diff | Download diff | Ignore whitespace Increase context Show commentsHide comments
@@ -547,9 +547,9
547 547 end
548 548 end
549 549 end
550 550
551 551 private
552 552 def user_params
553 - params.require(:user).permit(:login,:full_name,:hashed_password,:salt,:alias,:email,:site_id,:country_id,:activated,:enabled,:remark,:last_ip,:section)
553 + params.require(:user).permit(:login,:password,:password_confirmation,:email, :alias, :full_name,:remark)
554 554 end
555 555 end
Show file before | Show file after | Raw diff | Download diff | Ignore whitespace Increase context Show commentsHide comments
@@ -1,12 +1,12
1 1 require 'test_helper'
2 2
3 3 class AnnouncementsControllerTest < ActionController::TestCase
4 4 setup do
5 5 @announcement = announcements(:one)
6 - @request.session[:user_id] = users(:admin).id
6 + @request.session[:user_id] = user(:admin).id
7 7 end
8 8
9 9 test "should get index" do
10 10 get :index
11 11 assert_response :success
12 12 assert_not_nil assigns(:announcements)
@@ -16,13 +16,13
16 16 get :new
17 17 assert_response :success
18 18 end
19 19
20 20 test "should create announcement" do
21 21 assert_difference('Announcement.count') do
22 - post :create, announcement: { author: 'test', body: 'haha', published: true}
22 + post :create, announcement: { }
23 23 end
24 24
25 25 assert_redirected_to announcement_path(assigns(:announcement))
26 26 end
27 27
28 28 test "should show announcement" do
@@ -33,13 +33,13
33 33 test "should get edit" do
34 34 get :edit, id: @announcement
35 35 assert_response :success
36 36 end
37 37
38 38 test "should update announcement" do
39 - patch :update, id: @announcement, announcement: { body: 'another body' }
39 + patch :update, id: @announcement, announcement: { }
40 40 assert_redirected_to announcement_path(assigns(:announcement))
41 41 end
42 42
43 43 test "should destroy announcement" do
44 44 assert_difference('Announcement.count', -1) do
45 45 delete :destroy, id: @announcement
You need to be logged in to leave comments. Login now