From: Andy Allan Date: Wed, 21 Dec 2022 12:05:15 +0000 (+0000) Subject: Remove some old debugging comments X-Git-Tag: live~1362^2 X-Git-Url: https://git.openstreetmap.org./rails.git/commitdiff_plain/415ac577f3bbdf8c19e11a2f217170274e7b2515?ds=sidebyside;hp=--cc Remove some old debugging comments --- 415ac577f3bbdf8c19e11a2f217170274e7b2515 diff --git a/test/controllers/api/changesets_controller_test.rb b/test/controllers/api/changesets_controller_test.rb index 9470ec0bb..979b5d8af 100644 --- a/test/controllers/api/changesets_controller_test.rb +++ b/test/controllers/api/changesets_controller_test.rb @@ -1768,7 +1768,7 @@ module Api get changeset_download_path(changeset) assert_response :success - # print @response.body + # FIXME: needs more assert_select tests assert_select "osmChange[version='#{Settings.api_version}'][generator='#{Settings.generator}']" do assert_select "create", :count => 5 diff --git a/test/controllers/diary_entries_controller_test.rb b/test/controllers/diary_entries_controller_test.rb index b1e9b72f2..2003f9a08 100644 --- a/test/controllers/diary_entries_controller_test.rb +++ b/test/controllers/diary_entries_controller_test.rb @@ -325,7 +325,6 @@ class DiaryEntriesControllerTest < ActionDispatch::IntegrationTest assert_select "p", :text => /#{new_body}/, :count => 1 assert_select "abbr[class='geo'][title='#{number_with_precision(new_latitude, :precision => 4)}; #{number_with_precision(new_longitude, :precision => 4)}']", :count => 1 # As we're not logged in, check that you cannot edit - # print @response.body assert_select "a[href='/user/#{ERB::Util.u(entry.user.display_name)}/diary/#{entry.id}/edit']", :text => "Edit this entry", :count => 1 end diff --git a/test/controllers/users_controller_test.rb b/test/controllers/users_controller_test.rb index 634b279a5..9358a5948 100644 --- a/test/controllers/users_controller_test.rb +++ b/test/controllers/users_controller_test.rb @@ -529,7 +529,6 @@ class UsersControllerTest < ActionDispatch::IntegrationTest get user_path(seen_user) assert_response :success - # put @response.body assert_select "div.content-heading" do assert_select "dt", :count => 1, :text => /Contributor terms/ assert_select "dd", /Declined/ diff --git a/test/integration/user_diaries_test.rb b/test/integration/user_diaries_test.rb index 0b3ee930d..57a98823b 100644 --- a/test/integration/user_diaries_test.rb +++ b/test/integration/user_diaries_test.rb @@ -15,7 +15,7 @@ class UserDiariesTest < ActionDispatch::IntegrationTest # We can now login post "/login", :params => { "username" => user.email, "password" => "test", :referer => "/diary/new" } assert_response :redirect - # print @response.body + # Check that there is some payload alerting the user to the redirect # and allowing them to get to the page they are being directed to assert_select "html:root" do @@ -30,8 +30,6 @@ class UserDiariesTest < ActionDispatch::IntegrationTest assert_response :success assert_template "diary_entries/new" - # print @response.body - # print @html_document.to_yaml # We will make sure that the form exists here, full # assert testing of the full form should be done in the diff --git a/test/models/way_test.rb b/test/models/way_test.rb index 420eb42a8..8674b3790 100644 --- a/test/models/way_test.rb +++ b/test/models/way_test.rb @@ -34,7 +34,6 @@ class WayTest < ActiveSupport::TestCase way.add_nd_num(node_b.id) end way.save - # print way.nds.size assert_predicate way, :valid? way.add_nd_num(node_c.id) assert_predicate way, :valid?