Commit cd582d3c authored by Brian Hall's avatar Brian Hall

Remove unnecessary returns / unset variables from the CoffeeScript -> JS conversion.

parent 7bf6df84
...@@ -11,9 +11,9 @@ ...@@ -11,9 +11,9 @@
beforeEach(function() { beforeEach(function() {
loadFixtures(fixtureName); loadFixtures(fixtureName);
document.querySelector('.js-new-note-form').classList.add('js-main-target-form'); document.querySelector('.js-new-note-form').classList.add('js-main-target-form');
return this.shortcut = new ShortcutsIssuable(); this.shortcut = new ShortcutsIssuable();
}); });
return describe('#replyWithSelectedText', function() { describe('#replyWithSelectedText', function() {
var stubSelection; var stubSelection;
// Stub window.gl.utils.getSelectedFragment to return a node with the provided HTML. // Stub window.gl.utils.getSelectedFragment to return a node with the provided HTML.
stubSelection = function(html) { stubSelection = function(html) {
...@@ -24,64 +24,61 @@ ...@@ -24,64 +24,61 @@
}; };
}; };
beforeEach(function() { beforeEach(function() {
return this.selector = 'form.js-main-target-form textarea#note_note'; this.selector = 'form.js-main-target-form textarea#note_note';
}); });
describe('with empty selection', function() { describe('with empty selection', function() {
it('does not return an error', function() { it('does not return an error', function() {
this.shortcut.replyWithSelectedText(); this.shortcut.replyWithSelectedText();
return expect($(this.selector).val()).toBe(''); expect($(this.selector).val()).toBe('');
}); });
return it('triggers `input`', function() { it('triggers `input`', function() {
var focused; var focused = false;
focused = false;
$(this.selector).on('focus', function() { $(this.selector).on('focus', function() {
return focused = true; focused = true;
}); });
this.shortcut.replyWithSelectedText(); this.shortcut.replyWithSelectedText();
return expect(focused).toBe(true); expect(focused).toBe(true);
}); });
}); });
describe('with any selection', function() { describe('with any selection', function() {
beforeEach(function() { beforeEach(function() {
return stubSelection('<p>Selected text.</p>'); stubSelection('<p>Selected text.</p>');
}); });
it('leaves existing input intact', function() { it('leaves existing input intact', function() {
$(this.selector).val('This text was already here.'); $(this.selector).val('This text was already here.');
expect($(this.selector).val()).toBe('This text was already here.'); expect($(this.selector).val()).toBe('This text was already here.');
this.shortcut.replyWithSelectedText(); this.shortcut.replyWithSelectedText();
return expect($(this.selector).val()).toBe("This text was already here.\n\n> Selected text.\n\n"); expect($(this.selector).val()).toBe("This text was already here.\n\n> Selected text.\n\n");
}); });
it('triggers `input`', function() { it('triggers `input`', function() {
var triggered; var triggered = false;
triggered = false;
$(this.selector).on('input', function() { $(this.selector).on('input', function() {
return triggered = true; triggered = true;
}); });
this.shortcut.replyWithSelectedText(); this.shortcut.replyWithSelectedText();
return expect(triggered).toBe(true); expect(triggered).toBe(true);
}); });
return it('triggers `focus`', function() { it('triggers `focus`', function() {
var focused; var focused = false;
focused = false;
$(this.selector).on('focus', function() { $(this.selector).on('focus', function() {
return focused = true; focused = true;
}); });
this.shortcut.replyWithSelectedText(); this.shortcut.replyWithSelectedText();
return expect(focused).toBe(true); expect(focused).toBe(true);
}); });
}); });
describe('with a one-line selection', function() { describe('with a one-line selection', function() {
return it('quotes the selection', function() { it('quotes the selection', function() {
stubSelection('<p>This text has been selected.</p>'); stubSelection('<p>This text has been selected.</p>');
this.shortcut.replyWithSelectedText(); this.shortcut.replyWithSelectedText();
return expect($(this.selector).val()).toBe("> This text has been selected.\n\n"); expect($(this.selector).val()).toBe("> This text has been selected.\n\n");
}); });
}); });
return describe('with a multi-line selection', function() { describe('with a multi-line selection', function() {
return it('quotes the selected lines as a group', function() { it('quotes the selected lines as a group', function() {
stubSelection("<p>Selected line one.</p>\n\n<p>Selected line two.</p>\n\n<p>Selected line three.</p>"); stubSelection("<p>Selected line one.</p>\n\n<p>Selected line two.</p>\n\n<p>Selected line three.</p>");
this.shortcut.replyWithSelectedText(); this.shortcut.replyWithSelectedText();
return expect($(this.selector).val()).toBe("> Selected line one.\n>\n> Selected line two.\n>\n> Selected line three.\n\n"); expect($(this.selector).val()).toBe("> Selected line one.\n>\n> Selected line two.\n>\n> Selected line three.\n\n");
}); });
}); });
}); });
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment