Description:
fix user update
Commit status:
[Not Reviewed]
References:
Diff options:
Comments:
0 Commit comments
0 Inline Comments
Unresolved TODOs:
There are no unresolved TODOs
r639:2826fdf23cce - - 2 files changed: 4 inserted, 4 deleted
@@ -550,6 +550,6 | |||||
|
550 |
|
550 | ||
|
551 | private |
|
551 | private |
|
552 | def user_params |
|
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 | end |
|
554 | end |
|
555 | end |
|
555 | end |
@@ -3,7 +3,7 | |||||
|
3 | class AnnouncementsControllerTest < ActionController::TestCase |
|
3 | class AnnouncementsControllerTest < ActionController::TestCase |
|
4 | setup do |
|
4 | setup do |
|
5 | @announcement = announcements(:one) |
|
5 | @announcement = announcements(:one) |
|
6 |
- @request.session[:user_id] = user |
|
6 | + @request.session[:user_id] = user(:admin).id |
|
7 | end |
|
7 | end |
|
8 |
|
8 | ||
|
9 | test "should get index" do |
|
9 | test "should get index" do |
@@ -19,7 +19,7 | |||||
|
19 |
|
19 | ||
|
20 | test "should create announcement" do |
|
20 | test "should create announcement" do |
|
21 | assert_difference('Announcement.count') do |
|
21 | assert_difference('Announcement.count') do |
|
22 |
- post :create, announcement: { |
|
22 | + post :create, announcement: { } |
|
23 | end |
|
23 | end |
|
24 |
|
24 | ||
|
25 | assert_redirected_to announcement_path(assigns(:announcement)) |
|
25 | assert_redirected_to announcement_path(assigns(:announcement)) |
@@ -36,7 +36,7 | |||||
|
36 | end |
|
36 | end |
|
37 |
|
37 | ||
|
38 | test "should update announcement" do |
|
38 | test "should update announcement" do |
|
39 |
- patch :update, id: @announcement, announcement: { |
|
39 | + patch :update, id: @announcement, announcement: { } |
|
40 | assert_redirected_to announcement_path(assigns(:announcement)) |
|
40 | assert_redirected_to announcement_path(assigns(:announcement)) |
|
41 | end |
|
41 | end |
|
42 |
|
42 |
You need to be logged in to leave comments.
Login now