hyperformula icon indicating copy to clipboard operation
hyperformula copied to clipboard

Add unit tests for function `SUM` and operator `^`

Open sequba opened this issue 1 year ago • 1 comments

Context

Unit tests written when analyzing the behavior of function SUM and operator ^.

How did you test your changes?

unit tests pass

Types of changes

  • [ ] Breaking change (a fix or a feature because of which an existing functionality doesn't work as expected anymore)
  • [ ] New feature or improvement (a non-breaking change that adds functionality)
  • [ ] Bug fix (a non-breaking change that fixes an issue)
  • [ ] Additional language file, or a change to an existing language file (translations)
  • [x] New unit tests

Checklist:

  • [x] I have reviewed the guidelines about Contributing to HyperFormula and I confirm that my code follows the code style of this project.
  • [ ] I have signed the Contributor License Agreement.
  • [x] My change is compliant with the OpenDocument standard.
  • [x] My change is compatible with Microsoft Excel.
  • [x] My change is compatible with Google Sheets.
  • [ ] I described my changes in the CHANGELOG.md file.
  • [ ] My changes require a documentation update.
  • [ ] My changes require a migration guide.

sequba avatar Apr 25 '24 15:04 sequba

Performance comparison of head (72000a46cec85867119930911d11c4d147bdf035) vs base (d44bf8e2e8b78b1bbdfd1967ba2860de3a7bdbc5)

                                     testName |   base |   head | change
------------------------------------------------------------------------
                                      Sheet A | 504.31 | 513.03 | +1.73%
                                      Sheet B | 164.54 | 171.95 | +4.50%
                                      Sheet T | 142.67 | 148.53 | +4.11%
                                Column ranges | 495.79 | 504.11 | +1.68%
Sheet A:  change value, add/remove row/column |  15.49 |  14.58 | -5.87%
 Sheet B: change value, add/remove row/column |  143.4 | 136.39 | -4.89%
                   Column ranges - add column | 141.26 |  147.6 | +4.49%
                Column ranges - without batch |  421.1 | 446.41 | +6.01%
                        Column ranges - batch | 123.52 | 116.39 | -5.77%

github-actions[bot] avatar Apr 25 '24 15:04 github-actions[bot]