From 2a688db26bdf4e9c3b8b3d3b8985d8719eb5bfb9 Mon Sep 17 00:00:00 2001
From: Antonio Scandurra <as-cii@github.com>
Date: Wed, 19 Apr 2017 11:00:34 +0200
Subject: [PATCH] Add better test coverage for the mini and placeholder-text
 attributes

---
 spec/text-editor-element-spec.js | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/spec/text-editor-element-spec.js b/spec/text-editor-element-spec.js
index 6ea677152..4a6655714 100644
--- a/spec/text-editor-element-spec.js
+++ b/spec/text-editor-element-spec.js
@@ -22,12 +22,24 @@ describe('TextEditorElement', () => {
     jasmineContent.innerHTML = '<atom-text-editor mini>'
     const element = jasmineContent.firstChild
     expect(element.getModel().isMini()).toBe(true)
+
+    element.removeAttribute('mini')
+    expect(element.getModel().isMini()).toBe(false)
+
+    element.setAttribute('mini', '')
+    expect(element.getModel().isMini()).toBe(true)
   })
 
   it("honors the 'placeholder-text' attribute", () => {
     jasmineContent.innerHTML = "<atom-text-editor placeholder-text='testing'>"
     const element = jasmineContent.firstChild
     expect(element.getModel().getPlaceholderText()).toBe('testing')
+
+    element.setAttribute('placeholder-text', 'placeholder')
+    expect(element.getModel().getPlaceholderText()).toBe('placeholder')
+
+    element.removeAttribute('placeholder-text')
+    expect(element.getModel().getPlaceholderText()).toBeNull()
   })
 
   it("only assigns 'placeholder-text' on the model if the attribute is present", () => {