From 1f1675817c6ed7965f3e5af173ce9f736d3cced0 Mon Sep 17 00:00:00 2001
From: Masashi Hirano <cherrydog07@gmail.com>
Date: Sun, 8 Jul 2018 08:13:38 +0900
Subject: [PATCH] test: check option start or end is not safe integer

To increase fs readstream coverage, added test to check error when
option.start or end is not safe integer.

PR-URL: https://github.com/nodejs/node/pull/21704
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
Reviewed-By: Claudio Rodriguez <cjrodr@yahoo.com>
Reviewed-By: Weijia Wang <starkwang@126.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
---
 test/parallel/test-fs-read-stream-throw-type-error.js | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/test/parallel/test-fs-read-stream-throw-type-error.js b/test/parallel/test-fs-read-stream-throw-type-error.js
index 9886b3c9a46d97..458df4d347efc4 100644
--- a/test/parallel/test-fs-read-stream-throw-type-error.js
+++ b/test/parallel/test-fs-read-stream-throw-type-error.js
@@ -65,3 +65,12 @@ fs.createReadStream(example, { start: 1, end: 5 });
 
 // Case 6: Should throw RangeError if start is greater than end
 createReadStreamErr(example, { start: 5, end: 1 }, rangeError);
+
+// Case 7: Should throw RangeError if start or end is not safe integer
+const NOT_SAFE_INTEGER = 2 ** 53;
+[
+  { start: NOT_SAFE_INTEGER, end: Infinity },
+  { start: 0, end: NOT_SAFE_INTEGER }
+].forEach((opts) =>
+  createReadStreamErr(example, opts, rangeError)
+);