Bug 70448

Summary: Write more unittests for regression hotspots
Product: LibreOffice Reporter: Björn Michaelsen <bjoern.michaelsen>
Component: LibreOfficeAssignee: Not Assigned <libreoffice-bugs>
Status: NEW ---    
Severity: normal CC: aron.budea, mentoring, robinson.libreoffice
Priority: medium Keywords: difficultyMedium, easyHack, skillCpp, topicQA
Version: unspecified   
Hardware: Other   
OS: All   
See Also: https://bugs.freedesktop.org/show_bug.cgi?id=48024
https://bugs.freedesktop.org/show_bug.cgi?id=39625
https://bugs.freedesktop.org/show_bug.cgi?id=39748
https://bugs.freedesktop.org/show_bug.cgi?id=46723
Whiteboard: target:5.2.0 reviewed:2022
Crash report or crash signature: Regression By:
Bug Depends on: 66750    
Bug Blocks: 143781    

Description Björn Michaelsen 2013-10-14 11:12:47 UTC 
Writing unittests with coverage for all of LibreOffice might appear a daunting and insurmountable task. Given the vastness of the codebase it makes sense to focus on the areas of the codebase that historically have been hit most by regressions and are not already covered well by unittests.
There is a list of those areas at:

 https://wiki.documentfoundation.org/Development/RegressionHotspots

This EasyHack is to write unittests providing good coverage for these hotspots first, which should have the best benefit over cost and is much more limited in scope.
Comment 1 Björn Michaelsen 2015-06-29 08:46:17 UTC 
The regression hotspots can be cross compared with their respective test coverage:

 http://lcov.libreoffice.org/

(backups and historical data at: http://dev-builds.libreoffice.org/lcov_reports/)
Comment 2 Robinson Tryon (qubit) 2015-12-10 11:26:21 UTC Comment hidden (obsolete)
Comment 3 Robinson Tryon (qubit) 2016-02-18 14:51:53 UTC 
JanI is default CC for Easy Hacks (Add Jan; remove LibreOffice Dev List from CC)
[NinjaEdit]
Comment 4 Commit Notification 2016-03-19 15:16:38 UTC 
Jaskaran committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=2fd81edb9a6b89e1aa2d9c83006ad28660f85d71

tdf#70448, Test for SetFormula function

It will be available in 5.2.0.

The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.
Comment 5 jani 2016-04-26 08:52:07 UTC Comment hidden (obsolete)
Comment 6 guillaume.monzerol 2019-09-25 13:11:20 UTC 
Hi,
I will start working on this. I will try to add unit tests for one of the area in the list. If someone could tell me a more specific area that needs unit tests that would be appreciated.
Comment 7 Hossein 2022-08-06 15:27:38 UTC 
Re-evaluating the EasyHack in 2022

This task is still relevant. Finding regression hotspots and writing tests for them is very useful in countering recurring problems.