Foundry376/Mailspring
1
1
Fork 0
mirror of https://github.com/Foundry376/Mailspring.git synced 2024-11-14 21:57:55 +08:00
Code Issues Projects Releases Packages Wiki Activity
Mailspring/internal_packages/composer/spec/contenteditable-quoted-text-spec.cjsx

117 lines
5.6 KiB
Text
Raw Normal View History

tests for contenteditable component
2015-03-06 08:00:56 +08:00
# This tests just quoted text within a contenteditable.
#
# For a test of the basic component itself see
# contenteditable-component-spec.cjsx
#
refactor(utils): switch to regular underscore Summary: Fixes: T1334 remove final InboxApp references move out all underscore-plus methods Mass find and replace of underscore-plus sed -i '' -- 's/underscore-plus/underscore/g' **/*.coffee sed -i '' -- 's/underscore-plus/underscore/g' **/*.cjsx Test Plan: edgehill --test Reviewers: bengotow Reviewed By: bengotow Differential Revision: https://phab.nylas.com/D1534
2015-05-20 07:06:59 +08:00
_ = require "underscore"
tests for contenteditable component
2015-03-06 08:00:56 +08:00
React = require "react/addons"
ReactTestUtils = React.addons.TestUtils
fix(speed): Remove explicit .cjsx extensions from requires
2015-03-21 08:51:49 +08:00
ContenteditableComponent = require "../lib/contenteditable-component",
tests for contenteditable component
2015-03-06 08:00:56 +08:00
remove fdescribe
2015-03-06 08:02:47 +08:00
describe "ContenteditableComponent", ->
tests for contenteditable component
2015-03-06 08:00:56 +08:00
beforeEach ->
@onChange = jasmine.createSpy('onChange')
add a fix, TODO: refactor Summary: fix(service): Remove redundant tail newline when hiding previous in compose view. Fixes T2467. Test Plan: Added a test to verify that all tail newlines are removed except one. Added another to verify that non-tail newlines aren't removed. Tweaked an existing test to make it pass. Reviewers: bengotow Reviewed By: bengotow Maniphest Tasks: T2467 Differential Revision: https://phab.nylas.com/D1864
2015-08-12 02:35:33 +08:00
@html = 'Test <strong>HTML</strong><br>'
tests for contenteditable component
2015-03-06 08:00:56 +08:00
@component = ReactTestUtils.renderIntoDocument(
fix(quoted-text): new system to remove quoted text Summary: We need to remove quoted text completely from bodies in a composer. Unfortunately, that makes it very difficult to determine how to put it back in. For now the scheme is to append the quoted text at the end. However, that means that we need to only pull out quoted text at the end of a message. Unfortunately there are lots of places that quoted text appears inline with regular text. Determining whether or not some content is at the "end" of a message turned out to be non-trivial. We now have a new `DOMUtils` that looks for empty areas at the end. We also have a new quoted HTML parser that finds trailing quotes. Fixes T2335 Test Plan: lots of new quoted text tests Reviewers: bengotow Reviewed By: bengotow Maniphest Tasks: T2335 Differential Revision: https://phab.nylas.com/D1773
2015-07-22 02:34:47 +08:00
<ContenteditableComponent html={@html} onChange={@onChange}/>
tests for contenteditable component
2015-03-06 08:00:56 +08:00
)
feat(draft): escape deletes pristine drafts Summary: quoted text add and remove at filter level fix double space quoted text issue when replying to message Test Plan: edgehill --test Reviewers: bengotow Reviewed By: bengotow Differential Revision: https://review.inboxapp.com/D1341
2015-03-26 02:18:07 +08:00
@htmlWithQuote = 'Test <strong>HTML</strong><br><br><blockquote class="gmail_quote">QUOTE</blockquote>'
tests for contenteditable component
2015-03-06 08:00:56 +08:00
@componentWithQuote = ReactTestUtils.renderIntoDocument(
feat(composer): new composer styles Summary: tooltip styling new styled floating toolbar move buttons to bottom Test Plan: edgehill --test Reviewers: bengotow Reviewed By: bengotow Differential Revision: https://review.inboxapp.com/D1305
2015-03-18 07:19:40 +08:00
<ContenteditableComponent html={@htmlWithQuote}
onChange={@onChange}
mode={showQuotedText: false}/>
tests for contenteditable component
2015-03-06 08:00:56 +08:00
)
describe "quoted-text-control", ->
it "should be rendered", ->
expect(ReactTestUtils.findRenderedDOMComponentWithClass(@component, 'quoted-text-control')).toBeDefined()
it "should be visible if the html contains quoted text", ->
@toggle = ReactTestUtils.findRenderedDOMComponentWithClass(@componentWithQuote, 'quoted-text-control')
expect(@toggle.props.className.indexOf('no-quoted-text') >= 0).toBe(false)
feat(composer): new composer styles Summary: tooltip styling new styled floating toolbar move buttons to bottom Test Plan: edgehill --test Reviewers: bengotow Reviewed By: bengotow Differential Revision: https://review.inboxapp.com/D1305
2015-03-18 07:19:40 +08:00
it "should be have `show-quoted-text` if showQuotedText is true", ->
@componentWithQuote = ReactTestUtils.renderIntoDocument(
<ContenteditableComponent html={@htmlWithQuote} onChange={@onChange} mode={showQuotedText: true}/>
)
tests for contenteditable component
2015-03-06 08:00:56 +08:00
@toggle = ReactTestUtils.findRenderedDOMComponentWithClass(@componentWithQuote, 'quoted-text-control')
expect(@toggle.props.className.indexOf('show-quoted-text') >= 0).toBe(true)
feat(composer): new composer styles Summary: tooltip styling new styled floating toolbar move buttons to bottom Test Plan: edgehill --test Reviewers: bengotow Reviewed By: bengotow Differential Revision: https://review.inboxapp.com/D1305
2015-03-18 07:19:40 +08:00
it "should not have `show-quoted-text` if showQuotedText is false", ->
@componentWithQuote.setState(showQuotedText: false)
tests for contenteditable component
2015-03-06 08:00:56 +08:00
@toggle = ReactTestUtils.findRenderedDOMComponentWithClass(@componentWithQuote, 'quoted-text-control')
expect(@toggle.props.className.indexOf('show-quoted-text') >= 0).toBe(false)
it "should be hidden otherwise", ->
@toggle = ReactTestUtils.findRenderedDOMComponentWithClass(@component, 'quoted-text-control')
expect(@toggle.props.className.indexOf('no-quoted-text') >= 0).toBe(true)
feat(composer): new composer styles Summary: tooltip styling new styled floating toolbar move buttons to bottom Test Plan: edgehill --test Reviewers: bengotow Reviewed By: bengotow Differential Revision: https://review.inboxapp.com/D1305
2015-03-18 07:19:40 +08:00
describe "when showQuotedText is false", ->
fix(quoted-text): new system to remove quoted text Summary: We need to remove quoted text completely from bodies in a composer. Unfortunately, that makes it very difficult to determine how to put it back in. For now the scheme is to append the quoted text at the end. However, that means that we need to only pull out quoted text at the end of a message. Unfortunately there are lots of places that quoted text appears inline with regular text. Determining whether or not some content is at the "end" of a message turned out to be non-trivial. We now have a new `DOMUtils` that looks for empty areas at the end. We also have a new quoted HTML parser that finds trailing quotes. Fixes T2335 Test Plan: lots of new quoted text tests Reviewers: bengotow Reviewed By: bengotow Maniphest Tasks: T2335 Differential Revision: https://phab.nylas.com/D1773
2015-07-22 02:34:47 +08:00
it "should not display quoted text", ->
tests for contenteditable component
2015-03-06 08:00:56 +08:00
@editDiv = ReactTestUtils.findRenderedDOMComponentWithAttr(@componentWithQuote, 'contentEditable')
fix(quoted-text): new system to remove quoted text Summary: We need to remove quoted text completely from bodies in a composer. Unfortunately, that makes it very difficult to determine how to put it back in. For now the scheme is to append the quoted text at the end. However, that means that we need to only pull out quoted text at the end of a message. Unfortunately there are lots of places that quoted text appears inline with regular text. Determining whether or not some content is at the "end" of a message turned out to be non-trivial. We now have a new `DOMUtils` that looks for empty areas at the end. We also have a new quoted HTML parser that finds trailing quotes. Fixes T2335 Test Plan: lots of new quoted text tests Reviewers: bengotow Reviewed By: bengotow Maniphest Tasks: T2335 Differential Revision: https://phab.nylas.com/D1773
2015-07-22 02:34:47 +08:00
expect(React.findDOMNode(@editDiv).innerHTML).toEqual @html
tests for contenteditable component
2015-03-06 08:00:56 +08:00
feat(composer): new composer styles Summary: tooltip styling new styled floating toolbar move buttons to bottom Test Plan: edgehill --test Reviewers: bengotow Reviewed By: bengotow Differential Revision: https://review.inboxapp.com/D1305
2015-03-18 07:19:40 +08:00
describe "when showQuotedText is true", ->
beforeEach ->
@componentWithQuote = ReactTestUtils.renderIntoDocument(
<ContenteditableComponent html={@htmlWithQuote}
onChange={@onChange}
mode={showQuotedText: true}/>
)
tests for contenteditable component
2015-03-06 08:00:56 +08:00
it "should display all the HTML", ->
feat(composer): new composer styles Summary: tooltip styling new styled floating toolbar move buttons to bottom Test Plan: edgehill --test Reviewers: bengotow Reviewed By: bengotow Differential Revision: https://review.inboxapp.com/D1305
2015-03-18 07:19:40 +08:00
@componentWithQuote.setState(showQuotedText: true)
tests for contenteditable component
2015-03-06 08:00:56 +08:00
@editDiv = ReactTestUtils.findRenderedDOMComponentWithAttr(@componentWithQuote, 'contentEditable')
feat(docs): New docs tasks and React 0.13.2 Summary: This diff moves us up to React 0.13.2 and transitions some of the core React components to the new syntax based on plain Javascript objects. `setInitialState` is now just code in the constructor, `getDOMNode(@)` is now `React.findDOMNode(@)`, and `isMounted` is no longer necessary or available. This diff also adds `RegisteredComponent` to match `RegisteredRegion`. In another diff, I think we should change the names of these to be `DynamicComponent` and `DynamicComponentSet`. This diff also includes preliminary API Reference docs for Menu.cjsx and Popover.cjsx. You can build the docs using `grunt docs` from the build folder. It produces a simple html format now, but it's easy to customize. Also we now ignore "Unnecessary fat arrow" Test Plan: Run tests Reviewers: evan Reviewed By: evan Differential Revision: https://review.inboxapp.com/D1437
2015-04-25 02:33:10 +08:00
expect(React.findDOMNode(@editDiv).innerHTML.indexOf('gmail_quote') >= 0).toBe(true)
tests for contenteditable component
2015-03-06 08:00:56 +08:00
feat(composer): new composer styles Summary: tooltip styling new styled floating toolbar move buttons to bottom Test Plan: edgehill --test Reviewers: bengotow Reviewed By: bengotow Differential Revision: https://review.inboxapp.com/D1305
2015-03-18 07:19:40 +08:00
describe "showQuotedText", ->
tests for contenteditable component
2015-03-06 08:00:56 +08:00
it "should default to false", ->
feat(composer): new composer styles Summary: tooltip styling new styled floating toolbar move buttons to bottom Test Plan: edgehill --test Reviewers: bengotow Reviewed By: bengotow Differential Revision: https://review.inboxapp.com/D1305
2015-03-18 07:19:40 +08:00
expect(@component.props.mode?.showQuotedText).toBeUndefined()
tests for contenteditable component
2015-03-06 08:00:56 +08:00
describe "when the html is changed", ->
beforeEach ->
feat(quoted-text): New quoted text engine Summary: Add email reply parser Quoted text detection for HTML More blockquote specs Added 'On … wrote:' Test Plan: edgehill --test Reviewers: bengotow Reviewed By: bengotow Differential Revision: https://phab.nylas.com/D1719
2015-07-09 00:51:33 +08:00
@changedHtmlWithoutQuote = '<head></head><body>Changed <strong>NEW 1 HTML</strong><br><br></body>'
feat(draft): escape deletes pristine drafts Summary: quoted text add and remove at filter level fix double space quoted text issue when replying to message Test Plan: edgehill --test Reviewers: bengotow Reviewed By: bengotow Differential Revision: https://review.inboxapp.com/D1341
2015-03-26 02:18:07 +08:00
@changedHtmlWithQuote = 'Changed <strong>NEW 1 HTML</strong><br><br><blockquote class="gmail_quote">QUOTE</blockquote>'
tests for contenteditable component
2015-03-06 08:00:56 +08:00
@performEdit = (newHTML, component = @componentWithQuote) =>
editDiv = ReactTestUtils.findRenderedDOMComponentWithAttr(component, 'contentEditable')
feat(docs): New docs tasks and React 0.13.2 Summary: This diff moves us up to React 0.13.2 and transitions some of the core React components to the new syntax based on plain Javascript objects. `setInitialState` is now just code in the constructor, `getDOMNode(@)` is now `React.findDOMNode(@)`, and `isMounted` is no longer necessary or available. This diff also adds `RegisteredComponent` to match `RegisteredRegion`. In another diff, I think we should change the names of these to be `DynamicComponent` and `DynamicComponentSet`. This diff also includes preliminary API Reference docs for Menu.cjsx and Popover.cjsx. You can build the docs using `grunt docs` from the build folder. It produces a simple html format now, but it's easy to customize. Also we now ignore "Unnecessary fat arrow" Test Plan: Run tests Reviewers: evan Reviewed By: evan Differential Revision: https://review.inboxapp.com/D1437
2015-04-25 02:33:10 +08:00
React.findDOMNode(editDiv).innerHTML = newHTML
tests for contenteditable component
2015-03-06 08:00:56 +08:00
ReactTestUtils.Simulate.input(editDiv, {target: {value: newHTML}})
feat(composer): new composer styles Summary: tooltip styling new styled floating toolbar move buttons to bottom Test Plan: edgehill --test Reviewers: bengotow Reviewed By: bengotow Differential Revision: https://review.inboxapp.com/D1305
2015-03-18 07:19:40 +08:00
describe "when showQuotedText is true", ->
beforeEach ->
@componentWithQuote = ReactTestUtils.renderIntoDocument(
<ContenteditableComponent html={@htmlWithQuote}
onChange={@onChange}
mode={showQuotedText: true}/>
)
tests for contenteditable component
2015-03-06 08:00:56 +08:00
it "should call `props.onChange` with the entire HTML string", ->
feat(composer): new composer styles Summary: tooltip styling new styled floating toolbar move buttons to bottom Test Plan: edgehill --test Reviewers: bengotow Reviewed By: bengotow Differential Revision: https://review.inboxapp.com/D1305
2015-03-18 07:19:40 +08:00
@componentWithQuote.setState(showQuotedText: true)
tests for contenteditable component
2015-03-06 08:00:56 +08:00
@performEdit(@changedHtmlWithQuote)
ev = @onChange.mostRecentCall.args[0]
expect(ev.target.value).toEqual(@changedHtmlWithQuote)
it "should allow the quoted text to be changed", ->
feat(draft): escape deletes pristine drafts Summary: quoted text add and remove at filter level fix double space quoted text issue when replying to message Test Plan: edgehill --test Reviewers: bengotow Reviewed By: bengotow Differential Revision: https://review.inboxapp.com/D1341
2015-03-26 02:18:07 +08:00
changed = 'Test <strong>NEW 1 HTML</strong><blockquote class="gmail_quote">QUOTE CHANGED!!!</blockquote>'
feat(composer): new composer styles Summary: tooltip styling new styled floating toolbar move buttons to bottom Test Plan: edgehill --test Reviewers: bengotow Reviewed By: bengotow Differential Revision: https://review.inboxapp.com/D1305
2015-03-18 07:19:40 +08:00
@componentWithQuote.setState(showQuotedText: true)
tests for contenteditable component
2015-03-06 08:00:56 +08:00
@performEdit(changed)
ev = @onChange.mostRecentCall.args[0]
expect(ev.target.value).toEqual(changed)
feat(composer): new composer styles Summary: tooltip styling new styled floating toolbar move buttons to bottom Test Plan: edgehill --test Reviewers: bengotow Reviewed By: bengotow Differential Revision: https://review.inboxapp.com/D1305
2015-03-18 07:19:40 +08:00
describe "when showQuotedText is false", ->
fix(quoted-text): new system to remove quoted text Summary: We need to remove quoted text completely from bodies in a composer. Unfortunately, that makes it very difficult to determine how to put it back in. For now the scheme is to append the quoted text at the end. However, that means that we need to only pull out quoted text at the end of a message. Unfortunately there are lots of places that quoted text appears inline with regular text. Determining whether or not some content is at the "end" of a message turned out to be non-trivial. We now have a new `DOMUtils` that looks for empty areas at the end. We also have a new quoted HTML parser that finds trailing quotes. Fixes T2335 Test Plan: lots of new quoted text tests Reviewers: bengotow Reviewed By: bengotow Maniphest Tasks: T2335 Differential Revision: https://phab.nylas.com/D1773
2015-07-22 02:34:47 +08:00
it "should let you change the text, and then append the quoted text part to the end before firing `onChange`", ->
feat(composer): new composer styles Summary: tooltip styling new styled floating toolbar move buttons to bottom Test Plan: edgehill --test Reviewers: bengotow Reviewed By: bengotow Differential Revision: https://review.inboxapp.com/D1305
2015-03-18 07:19:40 +08:00
@componentWithQuote.setState(showQuotedText: false)
tests for contenteditable component
2015-03-06 08:00:56 +08:00
@performEdit(@changedHtmlWithoutQuote)
ev = @onChange.mostRecentCall.args[0]
fix(quoted-text): new system to remove quoted text Summary: We need to remove quoted text completely from bodies in a composer. Unfortunately, that makes it very difficult to determine how to put it back in. For now the scheme is to append the quoted text at the end. However, that means that we need to only pull out quoted text at the end of a message. Unfortunately there are lots of places that quoted text appears inline with regular text. Determining whether or not some content is at the "end" of a message turned out to be non-trivial. We now have a new `DOMUtils` that looks for empty areas at the end. We also have a new quoted HTML parser that finds trailing quotes. Fixes T2335 Test Plan: lots of new quoted text tests Reviewers: bengotow Reviewed By: bengotow Maniphest Tasks: T2335 Differential Revision: https://phab.nylas.com/D1773
2015-07-22 02:34:47 +08:00
withQuote = "<head></head><body>#{@changedHtmlWithQuote}</body>"
expect(ev.target.value).toEqual(withQuote)
tests for contenteditable component
2015-03-06 08:00:56 +08:00
it "should work if the component does not contain quoted text", ->
feat(quoted-text): New quoted text engine Summary: Add email reply parser Quoted text detection for HTML More blockquote specs Added 'On … wrote:' Test Plan: edgehill --test Reviewers: bengotow Reviewed By: bengotow Differential Revision: https://phab.nylas.com/D1719
2015-07-09 00:51:33 +08:00
changed = '<head></head><body>Hallooo! <strong>NEW 1 HTML HTML HTML</strong><br></body>'
feat(composer): new composer styles Summary: tooltip styling new styled floating toolbar move buttons to bottom Test Plan: edgehill --test Reviewers: bengotow Reviewed By: bengotow Differential Revision: https://review.inboxapp.com/D1305
2015-03-18 07:19:40 +08:00
@component.setState(showQuotedText: true)
tests for contenteditable component
2015-03-06 08:00:56 +08:00
@performEdit(changed, @component)
ev = @onChange.mostRecentCall.args[0]
expect(ev.target.value).toEqual(changed)
Reference in a new issue Copy permalink