From 50f3fca7a0eac6b6e8e5e9aee7af3c2a05831261 Mon Sep 17 00:00:00 2001
From: Richard Gibson <richard.gibson@gmail.com>
Date: Fri, 3 Sep 2021 15:59:25 -0400
Subject: [PATCH] Test Math.hypot argument coercion (#3177)

* Test Math.hypot argument coercion

* Fix constructor
---
 .../Math/hypot/Math.hypot_ToNumberErr.js      | 38 +++++++++++++++++++
 .../built-ins/Math/hypot/Math.hypot_Zero_2.js |  6 ++-
 2 files changed, 43 insertions(+), 1 deletion(-)
 create mode 100644 test/built-ins/Math/hypot/Math.hypot_ToNumberErr.js

diff --git a/test/built-ins/Math/hypot/Math.hypot_ToNumberErr.js b/test/built-ins/Math/hypot/Math.hypot_ToNumberErr.js
new file mode 100644
index 0000000000..818fe74666
--- /dev/null
+++ b/test/built-ins/Math/hypot/Math.hypot_ToNumberErr.js
@@ -0,0 +1,38 @@
+// Copyright (c) 2021 Richard Gibson. All rights reserved.
+// This code is governed by the BSD license found in the LICENSE file.
+
+/*---
+description: >
+  Math.hypot should coerce all arguments before inspecting them.
+esid: sec-math.hypot
+info: |
+  1. Let _coerced_ be a new empty List.
+  2. For each element _arg_ of _args_, do
+    a. Let _n_ be ? ToNumber(_arg_).
+    b. Append _n_ to _coerced_.
+  3. For each element _number_ of _coerced_, do
+---*/
+
+let uniqueErrorCount = 0;
+class UniqueError extends Error {
+  constructor() { super(); uniqueErrorCount++; }
+}
+
+assert.throws(
+  UniqueError,
+  function() {
+    Math.hypot(
+      Infinity,
+      -Infinity,
+      NaN,
+      0,
+      -0,
+      {valueOf(){ throw new UniqueError(); }},
+      {valueOf(){ throw new UniqueError(); }},
+    );
+  },
+  'Math.hypot propagates an abrupt completion from coercing an argument to Number'
+);
+
+assert.sameValue(uniqueErrorCount, 1,
+    'Math.hypot aborts argument processing at the first abrupt completion');
diff --git a/test/built-ins/Math/hypot/Math.hypot_Zero_2.js b/test/built-ins/Math/hypot/Math.hypot_Zero_2.js
index 159cf95375..7352851e6b 100644
--- a/test/built-ins/Math/hypot/Math.hypot_Zero_2.js
+++ b/test/built-ins/Math/hypot/Math.hypot_Zero_2.js
@@ -2,12 +2,16 @@
 // This code is governed by the BSD license found in the LICENSE file.
 
 /*---
+esid: sec-math.hypot
 es6id: 20.2.2.18
 author: Ryan Lewis
-description: Return 0 if all arguments being are 0 or -0.
+description: Math.hypot should return 0 if all arguments are 0 or -0.
 ---*/
 
+assert.sameValue(Math.hypot(0), 0, 'Math.hypot(0)');
+assert.sameValue(Math.hypot(-0), 0, 'Math.hypot(-0)');
 assert.sameValue(Math.hypot(0, 0), 0, 'Math.hypot(0, 0)');
 assert.sameValue(Math.hypot(0, -0), 0, 'Math.hypot(0, -0)');
 assert.sameValue(Math.hypot(-0, 0), 0, 'Math.hypot(-0, 0)');
 assert.sameValue(Math.hypot(-0, -0), 0, 'Math.hypot(-0, -0)');
+assert.sameValue(Math.hypot(0, -0, -0), 0, 'Math.hypot(0, -0, -0)');