From 0fc80cffc598aa3848dbe90607b38176652425c9 Mon Sep 17 00:00:00 2001
From: Ujjwal Sharma <usharma1998@gmail.com>
Date: Thu, 8 Nov 2018 11:55:52 +0530
Subject: [PATCH] intl: increase coverage for DateTimeFormat

Increase coverage for the Intl.DateTimeFormat constructor by adding a
test that confirms that ToObject(...) is appropriately called inside the
implementation.
---
 .../constructor-options-toobject.js           | 42 +++++++++++++++++++
 1 file changed, 42 insertions(+)
 create mode 100644 test/intl402/DateTimeFormat/constructor-options-toobject.js

diff --git a/test/intl402/DateTimeFormat/constructor-options-toobject.js b/test/intl402/DateTimeFormat/constructor-options-toobject.js
new file mode 100644
index 0000000000..826a9e8762
--- /dev/null
+++ b/test/intl402/DateTimeFormat/constructor-options-toobject.js
@@ -0,0 +1,42 @@
+// Copyright (C) 2018 Ujjwal Sharma. All rights reserved.
+// This code is governed by the BSD license found in the LICENSE file.
+
+/*---
+esid: sec-initializedatetimeformat
+description: >
+  Tests that Intl.DateTimeFormat contructor converts the options argument
+  to an object using `ToObject` (7.1.13).
+info: |
+  12.1.2 ToDateTimeOptions
+
+  1. If options is undefined, let options be null; otherwise let options be ?
+  ToObject(options).
+---*/
+
+const toObjectResults = [
+  [true, new Boolean(true)],
+  [42, new Number(42)],
+  ['foo', new String('foo')],
+  [{}, {}],
+  [Symbol(), Object(Symbol())]
+];
+
+// Test if ToObject is used to convert primitives to Objects.
+toObjectResults.forEach(pair => {
+  const [value, result] = pair;
+
+  const actual = new Intl.DateTimeFormat(['en-US'], value).resolvedOptions();
+  const expected = new Intl.DateTimeFormat(['en-US'], result).resolvedOptions();
+
+  assert.sameValue(actual.locale, expected.locale);
+  assert.sameValue(actual.calendar, expected.calendar);
+  assert.sameValue(actual.day, expected.day);
+  assert.sameValue(actual.month, expected.month);
+  assert.sameValue(actual.year, expected.year);
+  assert.sameValue(actual.numberingSystem, expected.numberingSystem);
+  assert.sameValue(actual.timeZone, expected.timeZone);
+});
+
+// ToObject throws a TypeError for undefined and null, but it's not called
+// when options is undefined.
+assert.throws(TypeError, () => new Intl.DateTimeFormat(['en-US'], null));