Fix route for "go public" button on the account edit page

diff --git a/app/views/accounts/edit.html.erb b/app/views/accounts/edit.html.erb
index ba809d16ae6348050702a7d265526e3e283a19c6..0dcab17b0d3ba20ffaa9f3febdb21f6e4dd31e96 100644 (file)
--- a/app/views/accounts/edit.html.erb
+++ b/app/views/accounts/edit.html.erb
@@ -61,5 +61,5 @@
 <a name="public"></a>
 <h2><%= t ".public editing note.heading" %></h2>
 <%= t ".public editing note.html" %>
-  <%= button_to t(".make edits public button"), :action => :go_public %>
+  <%= button_to t(".make edits public button"), user_go_public_path %>
 <% end %>

diff --git a/test/controllers/accounts_controller_test.rb b/test/controllers/accounts_controller_test.rb
index 27e05379cd5f4a06734b8f1c37d27ef890f9f4fe..173bcf22c568819bc098a488aa688f0bf5948413 100644 (file)
--- a/test/controllers/accounts_controller_test.rb
+++ b/test/controllers/accounts_controller_test.rb
@@ -128,4 +128,28 @@ class AccountsControllerTest < ActionDispatch::IntegrationTest
     assert_equal user.new_email, email.to.first
     ActionMailer::Base.deliveries.clear
   end
+
+  def test_private_account
+    user = create(:user, :data_public => false)
+
+    # Make sure that you are redirected to the login page when
+    # you are not logged in
+    get edit_account_path
+    assert_response :redirect
+    assert_redirected_to login_path(:referer => "/account/edit")
+
+    # Make sure we get the page when we are logged in as the right user
+    session_for(user)
+    get edit_account_path
+    assert_response :success
+    assert_template :edit
+    assert_select "form#accountForm" do |form|
+      assert_equal "post", form.attr("method").to_s
+      assert_select "input[name='_method']", true
+      assert_equal "/account", form.attr("action").to_s
+    end
+
+    # Make sure we have a button to "go public"
+    assert_select "form.button_to[action='/user/go_public']", true
+  end
 end