Add unit test to make sure issue 1297 will not be introduced again (#1535)

* Add unit test to make sure issue 1297 will not be introduced again in the future

* Disable 'Unchanged files with check annotations' section in PR

* Revert last commit

* Revert change made by mistake

* Add jsdoc comment

* Fix type error

Author: sequba

package-lock.json

@@ -26,8 +26,14 @@ export interface SimpleCellRange {
   end: SimpleCellAddress,
 }
 
-export function isSimpleCellRange(obj: any): obj is SimpleCellRange {
-  if (obj && (typeof obj === 'object' || typeof obj === 'function')) {
+/**
+ * Type guard that checks if an object is a valid SimpleCellRange.
+ * @param {unknown} val - Value to check
+ * @returns {boolean} True if and only if the object is a valid SimpleCellRange
+ */
+export function isSimpleCellRange(val: unknown): val is SimpleCellRange {
+  if (val && (typeof val === 'object' || typeof val === 'function')) {
+    const obj = val as Record<string, unknown>
     return 'start' in obj && isSimpleCellAddress(obj.start) && 'end' in obj && isSimpleCellAddress(obj.end)
   } else {
     return false

src/AbsoluteCellRange.ts

@@ -582,6 +582,20 @@ describe('changing cell content', () => {
     expect(engine.getCellValueDetailedType({ sheet: 0, col: 0, row: 1 })).toEqual(CellValueDetailedType.NUMBER_CURRENCY)
     expect(engine.getCellValueDetailedType({ sheet: 0, col: 0, row: 2 })).toEqual(CellValueDetailedType.NUMBER_PERCENT)
   })
+
+  it('should work in scenario from issue https://github.com/handsontable/hyperformula/issues/1297', () => {
+    const engine = HyperFormula.buildFromArray([
+      [1, 2, '=SUM(A1:B1)'],
+      ['=SUM(1:1)', 0, 0],
+      ['=SUM(1:2)', 0, 0],
+    ])
+
+    engine.setCellContents(adr('C1'), [['=SUM(A1:B1)']])
+    engine.setCellContents(adr('A3'), [['=SUM(A1:C2)']])
+    engine.setCellContents(adr('B1'), [[null]])
+
+    expect(engine.getCellValue(adr('A3'))).toEqual(4)
+  })
 })
 
 describe('change multiple cells contents', () => {