From 7394726cff6e3297d7d265fd1124515a69721d6c Mon Sep 17 00:00:00 2001
From: Josh Johnson <josh@joshuajohnson.co.uk>
Date: Sat, 2 Nov 2019 12:47:39 +0000
Subject: [PATCH] Add further e2e test to assert on placeholder ordering

---
 cypress/integration/select-one.spec.js | 36 +++++++++++++++++++++++---
 1 file changed, 33 insertions(+), 3 deletions(-)

diff --git a/cypress/integration/select-one.spec.js b/cypress/integration/select-one.spec.js
index 87b8cc2..dd6cdec 100644
--- a/cypress/integration/select-one.spec.js
+++ b/cypress/integration/select-one.spec.js
@@ -479,6 +479,21 @@ describe('Choices - select one', () => {
             .should('not.have.value', 'I am a placeholder');
         });
       });
+
+      describe('when choice list is open', () => {
+        it('displays the placeholder choice first', () => {
+          cy.get('[data-test-hook=placeholder-via-option-value]')
+            .find('.choices__input--cloned')
+            .focus();
+
+          cy.get('[data-test-hook=placeholder-via-option-value]')
+            .find('.choices__list--dropdown .choices__list')
+            .children()
+            .first()
+            .should('have.class', 'choices__placeholder')
+            .should('have.text', 'I am a placeholder');
+        });
+      });
     });
 
     describe('placeholder via option attribute', () => {
@@ -497,21 +512,36 @@ describe('Choices - select one', () => {
 
       describe('when a choice has been selected', () => {
         it('does not display a placeholder', () => {
-          cy.get('[data-test-hook=placeholder-via-option-value]')
+          cy.get('[data-test-hook=placeholder-via-option-attr]')
             .find('.choices__input--cloned')
             .focus();
 
-          cy.get('[data-test-hook=placeholder-via-option-value]')
+          cy.get('[data-test-hook=placeholder-via-option-attr]')
             .find('.choices__list--dropdown .choices__list')
             .children()
             .first()
             .click();
 
-          cy.get('[data-test-hook=placeholder-via-option-value]')
+          cy.get('[data-test-hook=placeholder-via-option-attr]')
             .find('.choices__input--cloned')
             .should('not.have.value', 'I am a placeholder');
         });
       });
+
+      describe('when choice list is open', () => {
+        it('displays the placeholder choice first', () => {
+          cy.get('[data-test-hook=placeholder-via-option-attr]')
+            .find('.choices__input--cloned')
+            .focus();
+
+          cy.get('[data-test-hook=placeholder-via-option-attr]')
+            .find('.choices__list--dropdown .choices__list')
+            .children()
+            .first()
+            .should('have.class', 'choices__placeholder')
+            .should('have.text', 'I am a placeholder');
+        });
+      });
     });
 
     describe('remote data', () => {