From 0358ee528765157234643d289bce6b8ca5889c72 Mon Sep 17 00:00:00 2001
From: Kevin Brown <kevin@kevin-brown.com>
Date: Thu, 29 Dec 2016 19:15:52 -0500
Subject: [PATCH] Added two test cases (zero and empty string check)

This adds two test cases for 16b4840c0e2df0461998e3b464ee0a546173950d.
---
 tests/data/select-tests.js | 32 ++++++++++++++++++++++++++++++++
 1 file changed, 32 insertions(+)

diff --git a/tests/data/select-tests.js b/tests/data/select-tests.js
index 90d9609089..7d1796686e 100644
--- a/tests/data/select-tests.js
+++ b/tests/data/select-tests.js
@@ -455,3 +455,35 @@ test('data objects use the text of the option', function (assert) {
   assert.equal(item.id, '&');
   assert.equal(item.text, '&');
 });
+
+test('select option construction accepts id=0 (zero) value', function (assert) {
+  var $select = $('#qunit-fixture .single');
+
+  var selectOptions = [{ id: 0, text: 'Zero Value'}];
+  var data = new SelectData($select, selectOptions);
+
+  var optionElem = data.option(selectOptions[0]);
+
+  // If was "Zero Value"", then it ignored id property
+  assert.equal(
+    optionElem[0].value,
+    '0',
+    'Built option value should be "0" (zero as a string).'
+  );
+});
+
+test('select option construction accepts id="" (empty string) value',
+  function (assert) {
+  var $select = $('#qunit-fixture .single');
+
+  var selectOptions = [{ id: '', text: 'Empty String'}];
+  var data = new SelectData($select, selectOptions);
+
+  var optionElem = data.option(selectOptions[0]);
+
+  assert.equal(
+    optionElem[0].value,
+    '',
+    'Built option value should be an empty string.'
+  );
+});