Bug 70448 - Write more unittests for regression hotspots
Summary: Write more unittests for regression hotspots
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: LibreOffice (show other bugs)
Version:
(earliest affected)
unspecified
Hardware: Other All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard: target:5.2.0 reviewed:2022
Keywords: difficultyMedium, easyHack, skillCpp, topicQA
Depends on: 66750
Blocks: Dev-related
  Show dependency treegraph
 
Reported: 2013-10-14 11:12 UTC by Björn Michaelsen
Modified: 2022-08-06 15:27 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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.