5105: Really call the report_issue action, instead of fetching "/",

when the "send" button is clicked.

diff --git a/apps/workbench/app/assets/javascripts/report_issue.js b/apps/workbench/app/assets/javascripts/report_issue.js
index f3c323c63ba72bd05dde4e40a87cdcd46e936511..371dacc1afe1eb8bfc8d11144db59cb724c44b15 100644 (file)
--- a/apps/workbench/app/assets/javascripts/report_issue.js
+++ b/apps/workbench/app/assets/javascripts/report_issue.js
@@ -8,7 +8,7 @@ $(document).
     }
     $('div').remove('.modal-footer-status');
 
-    $.ajax('/').
+    $.ajax('/report_issue', {type: 'POST'}).
       success(function(data, status, jqxhr) {
         var $sendButton = $('#report-issue-submit');
         $sendButton.html('Report sent');

diff --git a/apps/workbench/test/integration/errors_test.rb b/apps/workbench/test/integration/errors_test.rb
index 8198be4fb395bc5ff2bca82be45893181c593992..32f16a68dfa9bb7da49f0adbce9881aa0480aeda 100644 (file)
--- a/apps/workbench/test/integration/errors_test.rb
+++ b/apps/workbench/test/integration/errors_test.rb
@@ -103,6 +103,10 @@ class ErrorsTest < ActionDispatch::IntegrationTest
         assert_no_selector 'a,button:not([disabled])', text: 'Send problem report'
         assert_selector 'a,button', text: 'Cancel'
 
+        report = mock
+        report.expects(:deliver).returns true
+        IssueReporter.expects(:send_report).returns report
+
         # enter a report text and click on report
         find_field('report_issue_text').set 'my test report text'
         click_button 'Send problem report'

diff --git a/apps/workbench/test/integration/report_issue_test.rb b/apps/workbench/test/integration/report_issue_test.rb
index 7d4058db4a974126625af8d1648680924b8f9a4f..4a15851636fba6f0502f741d181af42727a921af 100644 (file)
--- a/apps/workbench/test/integration/report_issue_test.rb
+++ b/apps/workbench/test/integration/report_issue_test.rb
@@ -59,6 +59,11 @@ class ReportIssueTest < ActionDispatch::IntegrationTest
       # enter a report text and click on report
       page.find_field('report_issue_text').set 'my test report text'
       assert page.has_button?('Send problem report'), 'Send problem report button not enabled after entering text'
+
+      report = mock
+      report.expects(:deliver).returns true
+      IssueReporter.expects(:send_report).returns report
+
       click_button 'Send problem report'
 
       # ajax success updated button texts and added footer message