Merge branch 'ustinvaskin-upd-range-toString-ternary'

jdeniau · jdeniau · commit d4189ab12e44 · 2025-03-25T19:35:36.000Z
diff --git a/__tests__/Range.ts b/__tests__/Range.ts
@@ -212,4 +212,11 @@ describe('Range', () => {
         .toArray()
     ).toEqual([1, 2]);
   });
+
+  it('toString', () => {
+    expect(Range(0, 0).toString()).toBe('Range []');
+    expect(Range(0, 3).toString()).toBe('Range [ 0...3 ]');
+    expect(Range(0, 10, 2).toString()).toBe('Range [ 0...10 by 2 ]');
+    expect(Range(10, 0, -2).toString()).toBe('Range [ 10...0 by -2 ]');
+  });
 });
diff --git a/src/Range.js b/src/Range.js
@@ -45,17 +45,9 @@ export class Range extends IndexedSeq {
   }
 
   toString() {
-    if (this.size === 0) {
-      return 'Range []';
-    }
-    return (
-      'Range [ ' +
-      this._start +
-      '...' +
-      this._end +
-      (this._step !== 1 ? ' by ' + this._step : '') +
-      ' ]'
-    );
+    return this.size === 0
+      ? 'Range []'
+      : `Range [ ${this._start}...${this._end}${this._step !== 1 ? ' by ' + this._step : ''} ]`;
   }
 
   get(index, notSetValue) {
