From 9149887fec74b916c074d9d26627c5cfc9cf5ab8 Mon Sep 17 00:00:00 2001
From: Nolan Lawson <nolan@nolanlawson.com>
Date: Sat, 3 Mar 2018 16:12:48 -0800
Subject: [PATCH] add tests for content warnings

---
 routes/_components/compose/ComposeBox.html    | 12 ++--
 routes/_components/compose/ComposeButton.html |  4 +-
 .../compose/ComposeLengthGauge.html           |  6 +-
 .../compose/ComposeLengthIndicator.html       | 10 ++-
 tests/spec/15-compose-content-warnings.js     | 68 +++++++++++++++++++
 tests/utils.js                                |  2 +
 6 files changed, 85 insertions(+), 17 deletions(-)
 create mode 100644 tests/spec/15-compose-content-warnings.js

diff --git a/routes/_components/compose/ComposeBox.html b/routes/_components/compose/ComposeBox.html
index 18a1101..abb0342 100644
--- a/routes/_components/compose/ComposeBox.html
+++ b/routes/_components/compose/ComposeBox.html
@@ -4,11 +4,11 @@
     <ComposeContentWarning :realm :contentWarning />
   {{/if}}
   <ComposeInput :realm :text />
-  <ComposeLengthGauge :textLength :textOverLimit />
+  <ComposeLengthGauge :length :overLimit />
   <ComposeToolbar :realm :postPrivacy :media :contentWarningShown />
-  <ComposeLengthIndicator :textLength :textOverLimit />
+  <ComposeLengthIndicator :length :overLimit />
   <ComposeMedia :realm :media />
-  <ComposeButton :textLength :textOverLimit />
+  <ComposeButton :length :overLimit />
 </div>
 <style>
   .compose-box {
@@ -71,7 +71,11 @@
       defaultPostPrivacyKey: ($currentVerifyCredentials) => $currentVerifyCredentials.source.privacy,
       postPrivacyKey: (composeData, defaultPostPrivacyKey) => composeData.postPrivacy || defaultPostPrivacyKey,
       textLength: (text) => measureText(text),
-      textOverLimit: (textLength) => textLength > CHAR_LIMIT,
+      contentWarningLength: (contentWarning) => measureText(contentWarning),
+      length: (textLength, contentWarningLength, contentWarningShown) => {
+        return textLength + (contentWarningShown ? contentWarningLength : 0)
+      },
+      overLimit: (length) => length > CHAR_LIMIT,
       contentWarningShown: (composeData) => composeData.contentWarningShown,
       contentWarning: (composeData) => composeData.contentWarning || ''
     }
diff --git a/routes/_components/compose/ComposeButton.html b/routes/_components/compose/ComposeButton.html
index 90e4c8c..3eefdda 100644
--- a/routes/_components/compose/ComposeButton.html
+++ b/routes/_components/compose/ComposeButton.html
@@ -16,9 +16,7 @@
   export default {
     store: () => store,
     computed: {
-      disabled: (textOverLimit, textLength) => {
-        return textOverLimit || textLength === 0
-      }
+      disabled: (overLimit, length) => overLimit || length === 0
     }
   }
 </script>
\ No newline at end of file
diff --git a/routes/_components/compose/ComposeLengthGauge.html b/routes/_components/compose/ComposeLengthGauge.html
index af5da70..1ab3607 100644
--- a/routes/_components/compose/ComposeLengthGauge.html
+++ b/routes/_components/compose/ComposeLengthGauge.html
@@ -1,4 +1,4 @@
-<div class="compose-box-length-gauge {{shouldAnimate ? 'should-animate' : ''}}  {{textOverLimit ? 'over-char-limit' : ''}}"
+<div class="compose-box-length-gauge {{shouldAnimate ? 'should-animate' : ''}}  {{overLimit ? 'over-char-limit' : ''}}"
      style="transform: scaleX({{inputLengthAsFractionRoundedAfterRaf || 0}});"
      aria-hidden="true"
 ></div>
@@ -36,9 +36,7 @@
     },
     store: () => store,
     computed: {
-      lengthAsFraction: (textLength) => {
-        return Math.min(CHAR_LIMIT, textLength) / CHAR_LIMIT
-      },
+      lengthAsFraction: (length) => Math.min(CHAR_LIMIT, length) / CHAR_LIMIT,
       lengthAsFractionRounded: (lengthAsFraction) => {
         // We don't need to update the gauge for every decimal point, so round it to the nearest 0.02
         let int = Math.round(lengthAsFraction * 100)
diff --git a/routes/_components/compose/ComposeLengthIndicator.html b/routes/_components/compose/ComposeLengthIndicator.html
index 27ea186..b4ee6e4 100644
--- a/routes/_components/compose/ComposeLengthIndicator.html
+++ b/routes/_components/compose/ComposeLengthIndicator.html
@@ -1,4 +1,4 @@
-<span class="compose-box-length {{textOverLimit ? 'over-char-limit' : ''}}"
+<span class="compose-box-length {{overLimit ? 'over-char-limit' : ''}}"
       aria-label="{{lengthLabel}}">
       {{lengthToDisplayAfterRaf || '0'}}
     </span>
@@ -33,11 +33,9 @@
     },
     store: () => store,
     computed: {
-      lengthToDisplay: (textLength) => {
-        return CHAR_LIMIT - textLength
-      },
-      lengthLabel: (textOverLimit, lengthToDisplay) => {
-        if (textOverLimit) {
+      lengthToDisplay: (length) => CHAR_LIMIT - length,
+      lengthLabel: (overLimit, lengthToDisplay) => {
+        if (overLimit) {
           return `${lengthToDisplay} characters over limit`
         } else {
           return `${lengthToDisplay} characters remaining`
diff --git a/tests/spec/15-compose-content-warnings.js b/tests/spec/15-compose-content-warnings.js
new file mode 100644
index 0000000..15db1a7
--- /dev/null
+++ b/tests/spec/15-compose-content-warnings.js
@@ -0,0 +1,68 @@
+import {
+  composeContentWarning, composeInput, composeLengthIndicator, contentWarningButton, homeNavButton,
+  notificationsNavButton
+} from '../utils'
+import { foobarRole } from '../roles'
+
+fixture`15-compose-content-warnings.js`
+  .page`http://localhost:4002`
+
+test('Changes content warnings', async t => {
+  await t.useRole(foobarRole)
+    .expect(composeContentWarning.exists).notOk()
+    .expect(contentWarningButton.getAttribute('aria-label')).eql('Add content warning')
+    .expect(contentWarningButton.getAttribute('aria-pressed')).eql('false')
+    .click(contentWarningButton)
+    .expect(composeContentWarning.exists).ok()
+    .expect(contentWarningButton.getAttribute('aria-label')).eql('Remove content warning')
+    .expect(contentWarningButton.getAttribute('aria-pressed')).eql('true')
+    .typeText(composeContentWarning, 'hello content warning', {paste: true})
+    .typeText(composeInput, 'secret text', {paste: true})
+    .click(notificationsNavButton)
+    .click(homeNavButton)
+    .expect(contentWarningButton.getAttribute('aria-label')).eql('Remove content warning')
+    .expect(contentWarningButton.getAttribute('aria-pressed')).eql('true')
+    .expect(composeContentWarning.value).eql('hello content warning')
+    .expect(composeInput.value).eql('secret text')
+    .selectText(composeInput)
+    .pressKey('delete')
+    .selectText(composeContentWarning)
+    .pressKey('delete')
+    .expect(composeContentWarning.value).eql('')
+    .expect(composeInput.value).eql('')
+    .click(contentWarningButton)
+    .expect(composeContentWarning.exists).notOk()
+    .expect(contentWarningButton.getAttribute('aria-label')).eql('Add content warning')
+    .expect(contentWarningButton.getAttribute('aria-pressed')).eql('false')
+})
+
+test('Considers content warnings for length limits', async t => {
+  await t.useRole(foobarRole)
+    .expect(composeLengthIndicator.innerText).eql('500')
+    .click(contentWarningButton)
+    .typeText(composeContentWarning, 'my content warning', {paste: true})
+    .expect(composeLengthIndicator.innerText).eql('482')
+    .typeText(composeInput, 'secret text', {paste: true})
+    .expect(composeLengthIndicator.innerText).eql('471')
+    .selectText(composeContentWarning)
+    .pressKey('delete')
+    .expect(composeLengthIndicator.innerText).eql('489')
+    .selectText(composeInput)
+    .pressKey('delete')
+    .expect(composeLengthIndicator.innerText).eql('500')
+})
+
+test('Considers hidden content warnings for length limits', async t => {
+  await t.useRole(foobarRole)
+    .expect(composeLengthIndicator.innerText).eql('500')
+    .click(contentWarningButton)
+    .typeText(composeContentWarning, 'my content warning', {paste: true})
+    .expect(composeLengthIndicator.innerText).eql('482')
+    .click(contentWarningButton)
+    .expect(composeLengthIndicator.innerText).eql('500')
+    .click(contentWarningButton)
+    .expect(composeLengthIndicator.innerText).eql('482')
+    .selectText(composeContentWarning)
+    .pressKey('delete')
+    .expect(composeLengthIndicator.innerText).eql('500')
+})
\ No newline at end of file
diff --git a/tests/utils.js b/tests/utils.js
index 10849b4..1a7797d 100644
--- a/tests/utils.js
+++ b/tests/utils.js
@@ -12,11 +12,13 @@ export const notificationsNavButton = $('nav a[href="/notifications"]')
 export const homeNavButton = $('nav a[href="/"]')
 export const formError = $('.form-error-user-error')
 export const composeInput = $('.compose-box-input')
+export const composeContentWarning = $('.content-warning-input')
 export const composeButton = $('.compose-box-button')
 export const composeLengthIndicator = $('.compose-box-length')
 export const emojiButton = $('.compose-box-toolbar button:first-child')
 export const mediaButton = $('.compose-box-toolbar button:nth-child(2)')
 export const postPrivacyButton = $('.compose-box-toolbar button:nth-child(3)')
+export const contentWarningButton = $('.compose-box-toolbar button:nth-child(4)')
 export const emailInput = $('input#user_email')
 export const passwordInput = $('input#user_password')
 export const authorizeInput = $('button[type=submit]:not(.negative)')