From 896994413cad849f470cec7757c4bb7d1b4ffc12 Mon Sep 17 00:00:00 2001
From: Rick Waldron <waldron.rick@gmail.com>
Date: Mon, 31 Aug 2020 15:59:07 -0400
Subject: [PATCH] Missing coverage for Object.defineProperty method. Fixes
 gh-2660 (#2738)

---
 ...th-various-values-and-configurable-true.js | 38 +++++++++++++++++++
 1 file changed, 38 insertions(+)
 create mode 100644 test/built-ins/Object/defineProperty/redefine-length-with-various-values-and-configurable-true.js

diff --git a/test/built-ins/Object/defineProperty/redefine-length-with-various-values-and-configurable-true.js b/test/built-ins/Object/defineProperty/redefine-length-with-various-values-and-configurable-true.js
new file mode 100644
index 0000000000..647887a155
--- /dev/null
+++ b/test/built-ins/Object/defineProperty/redefine-length-with-various-values-and-configurable-true.js
@@ -0,0 +1,38 @@
+// Copyright (c) 2020 Rick Waldron.  All rights reserved.
+// This code is governed by the BSD license found in the LICENSE file.
+
+/*---
+esid: sec-array-exotic-objects-defineownproperty-p-desc
+description: >
+  Redefining "length" to `configurable: true` throws a TypeError exception
+info: |
+  ArraySetLength ( A, Desc )
+
+  ValidateAndApplyPropertyDescriptor ( O, P, extensible, Desc, current )
+---*/
+
+let a = [1];
+
+assert.throws(TypeError, () => {
+  Object.defineProperty(a, "length", {
+    configurable: true
+  });
+});
+assert.throws(TypeError, () => {
+  Object.defineProperty(a, "length", {
+    value: 1,
+    configurable: true
+  });
+});
+assert.throws(TypeError, () => {
+  Object.defineProperty(a, "length", {
+    value: 2,
+    configurable: true
+  });
+});
+assert.throws(TypeError, () => {
+  Object.defineProperty(a, "length", {
+    value: 3,
+    configurable: true
+  });
+});