Merge ~cristiangsp/launchpad:fixing-test_filebug_dupefinderjs-linting into launchpad:master

Proposed by Cristian Gonzalez
Status: Merged
Approved by: Cristian Gonzalez
Approved revision: 371625f8ba31e63b95385e290068929e1a25d3d7
Merge reported by: Otto Co-Pilot
Merged at revision: not available
Proposed branch: ~cristiangsp/launchpad:fixing-test_filebug_dupefinderjs-linting
Merge into: launchpad:master
Diff against target: 29 lines (+6/-6)
1 file modified
lib/lp/bugs/javascript/tests/test_filebug_dupefinder.js (+6/-6)
Reviewer Review Type Date Requested Status
Colin Watson (community) Approve
Review via email: mp+407005@code.launchpad.net

Commit message

Fixing test_filebug_dupefinder.js linting

Description of the change

Fixing test_filebug_dupefinder.js linting

To post a comment you must log in.
Revision history for this message
Colin Watson (cjwatson) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/lib/lp/bugs/javascript/tests/test_filebug_dupefinder.js b/lib/lp/bugs/javascript/tests/test_filebug_dupefinder.js
index eb85aff..e7159f1 100644
--- a/lib/lp/bugs/javascript/tests/test_filebug_dupefinder.js
+++ b/lib/lp/bugs/javascript/tests/test_filebug_dupefinder.js
@@ -24,18 +24,18 @@ YUI.add('lp.bugs.filebug_dupefinder.test', function (Y) {
24 throw new Error("Constructor called as a function");24 throw new Error("Constructor called as a function");
25 }25 }
26 this.calls = [];26 this.calls = [];
27 this.io = function(url, config) {27 this.io = function io_handler(url, config) {
28 this.calls.push(url);28 this.calls.push(url);
29 var response = {responseText: ''};29 var response = {responseText: ''};
30 // We may have been passed text to use in the response.30 // We may have been passed text to use in the response.
31 if (Y.Lang.isValue(arguments.callee.responseText)) {31 if (Y.Lang.isValue(io_handler.responseText)) {
32 response.responseText = arguments.callee.responseText;32 response.responseText = io_handler.responseText;
33 }33 }
34 // We currently only support calling the success handler.34 // We currently only support calling the success handler.
35 config.on.success(undefined, response, arguments.callee.args);35 config.on.success(undefined, response, io_handler.args);
36 // After calling the handler, resume the test.36 // After calling the handler, resume the test.
37 if (Y.Lang.isFunction(arguments.callee.doAfter)) {37 if (Y.Lang.isFunction(io_handler.doAfter)) {
38 test_case.resume(arguments.callee.doAfter);38 test_case.resume(io_handler.doAfter);
39 }39 }
40 };40 };
41 }41 }

Subscribers

People subscribed via source and target branches

to status/vote changes: