Add unit test to make sure issue 1297 will not be introduced again

src/AbsoluteCellRange.ts

@@ -26,21 +26,27 @@
   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
   }
 }
 
 export const simpleCellRange = (start: SimpleCellAddress, end: SimpleCellAddress) => ({start, end})
 
 export class AbsoluteCellRange implements SimpleCellRange {
   public readonly start: SimpleCellAddress
   public readonly end: SimpleCellAddress
 
   constructor(
     start: SimpleCellAddress,
     end: SimpleCellAddress,
   ) {
@@ -51,11 +57,11 @@
     this.end = simpleCellAddress(end.sheet, end.col, end.row)
   }
 
   public get sheet() {
     return this.start.sheet
   }
 
   public static fromSimpleCellAddresses(start: SimpleCellAddress, end: SimpleCellAddress): AbsoluteCellRange {
     if (start.sheet !== end.sheet) {
       throw new SheetsNotEqual(start.sheet, end.sheet)
     }
@@ -74,7 +80,7 @@
     return new AbsoluteColumnRange(start.sheet, start.col, end.col)
   }
 
   public static fromAst(ast: CellRangeAst | ColumnRangeAst | RowRangeAst, baseAddress: SimpleCellAddress): AbsoluteCellRange {
     if (ast.type === AstNodeType.CELL_RANGE) {
       return AbsoluteCellRange.fromCellRange(ast, baseAddress)
     } else if (ast.type === AstNodeType.COLUMN_RANGE) {
@@ -84,7 +90,7 @@
     }
   }
 
   public static fromAstOrUndef(ast: CellRangeAst | ColumnRangeAst | RowRangeAst, baseAddress: SimpleCellAddress): Maybe<AbsoluteCellRange> {
     try {
       return AbsoluteCellRange.fromAst(ast, baseAddress)
     } catch (_e) {
@@ -92,14 +98,14 @@
     }
   }
 
   public static fromCellRange(x: CellRange, baseAddress: SimpleCellAddress): AbsoluteCellRange {
     return new AbsoluteCellRange(
       x.start.toSimpleCellAddress(baseAddress),
       x.end.toSimpleCellAddress(baseAddress),
     )
   }
 
   public static spanFrom(topLeftCorner: SimpleCellAddress, width: number, height: number): AbsoluteCellRange {
     const ret = AbsoluteCellRange.spanFromOrUndef(topLeftCorner, width, height)
     if (ret === undefined) {
       throw new Error(WRONG_RANGE_SIZE)
@@ -107,7 +113,7 @@
     return ret
   }
 
   public static spanFromOrUndef(topLeftCorner: SimpleCellAddress, width: number, height: number): Maybe<AbsoluteCellRange> {
     if (!Number.isFinite(width) && Number.isFinite(height)) {
       if (topLeftCorner.col !== 0) {
         return undefined

test/unit/cruds/change-cell-content.spec.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', () => {