View Issue Details

This bug affects 1 person(s).
 4
IDProjectCategoryView StatusLast Update
18596Bug reportsOtherpublic2023-01-24 19:01
Reporterollehar Assigned Toollehar  
PriorityhighSeverityminor 
Status ready for code reviewResolutionopen 
Summary18596: Reintroduce lost unit tests from merge
Description

testGetAvailableSiteSettings
testExportResponses

These two tests overlap during merge, making it impossible to figure out what's what. Should be possible to copy paste. Feel free to add new files instead, to avoid merge conflicts in the future.

TagsNo tags attached.
Bug heat4
Complete LimeSurvey version number (& build)develop
I will donate to the project if issue is resolvedNo
Browser-
Database type & version-
Server OS (if known)-
Webserver software & version (if known)-
PHP Version-

Users monitoring this issue

User List There are no users monitoring this issue.

Activities

gabrieljenik

gabrieljenik

2023-01-16 16:42

manager   ~73475

This is on DEV, right?
Port those test from master to dev, right?

ollehar

ollehar

2023-01-19 10:55

administrator   ~73495

Not even port, just copy-paste the lost code. Or move it to separate test class, yeah.

gabrieljenik

gabrieljenik

2023-01-24 19:01

manager   ~73547

PR: https://github.com/LimeSurvey/LimeSurvey/pull/2867

Issue History

Date Modified Username Field Change
2023-01-16 14:45 ollehar New Issue
2023-01-16 14:45 ollehar Description Updated
2023-01-16 14:46 ollehar Assigned To => gabrieljenik
2023-01-16 14:46 ollehar Priority none => high
2023-01-16 14:46 ollehar Status new => assigned
2023-01-16 16:42 gabrieljenik Note Added: 73475
2023-01-16 16:42 gabrieljenik Bug heat 0 => 2
2023-01-19 10:55 ollehar Note Added: 73495
2023-01-19 10:55 ollehar Bug heat 2 => 4
2023-01-24 19:01 gabrieljenik Assigned To gabrieljenik => ollehar
2023-01-24 19:01 gabrieljenik Status assigned => ready for code review
2023-01-24 19:01 gabrieljenik Note Added: 73547