Relationship Graph
View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
16692 | Bug reports | Other | public | 2020-09-23 19:46 | 2020-10-05 08:36 |
Reporter | gabrieljenik | Assigned To | gabrieljenik | ||
Priority | none | Severity | partial_block | ||
Status | closed | Resolution | fixed | ||
Product Version | 3.23.5 | ||||
Target Version | 3.22.13 | ||||
Summary | 16692: Clear process after test script is failing | ||||
Description | This is when it happened: https://github.com/LimeSurvey/LimeSurvey/pull/1586 Survey Imports can't be reused among test scripts Also, smell, something is wrong with the cleanup. | ||||
Steps To Reproduce | Run testChangeQuestionTemplate and EditQuestionTest in the same run. | ||||
Tags | No tags attached. | ||||
Bug heat | 6 | ||||
Complete LimeSurvey version number (& build) | 3.22.13 | ||||
I will donate to the project if issue is resolved | No | ||||
Browser | |||||
Database type & version | Mysql | ||||
Server OS (if known) | |||||
Webserver software & version (if known) | |||||
PHP Version | php | ||||
related to | 16714 | closed | gabrieljenik | Clear process after test script is failing |
Not sure they SHOULD be reused, honestly. If one test needs to change the survey, other unrelated tests might brake, which makes a fragile test suite. |
|
I agree. I just named like that as a way to test if clearing after test is done right. I will rename it though. |
|
Does it worth to rename all current survey files and deduplicate if any is shared among 2 tests? |
|
The issue is caused by 'findByPkCache': Solved by clearing the findByPkCache in TestBaseClass::importSurvey() and clearing cache on Survey::delete() |
|
Besides running travis, deleting a survey should be a step on the manual testing |
|
The cache! It's always the cache. :( |
|
Fix committed to 3.x-LTS branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=30539 |
|
Fixed in Release 3.23.7+201006 |
|
LimeSurvey: 3.x-LTS d99f229f 2020-10-01 15:00:08 Details Diff |
Fixed issue 16692: Clear process after test script is failing Cache was not properly cleared in between tests neither after survey deletion |
Affected Issues 16692 |
|
mod - application/models/Survey.php | Diff File | ||
mod - tests/TestBaseClass.php | Diff File | ||
LimeSurvey: 3.x-LTS 358bf088 2020-10-02 15:16:56 Details Diff |
Fixed issue 16692: Clear process after test script is failing Code fix |
Affected Issues 16692 |
|
mod - application/models/Survey.php | Diff File | ||
LimeSurvey: 3.x-LTS 167b11f4 2020-10-02 18:58:58 Committer: GitHub Details Diff |
Fixed issue 16692: Clear process after test script is failing |
Affected Issues 16692 |
|
mod - application/models/Survey.php | Diff File | ||
mod - tests/TestBaseClass.php | Diff File | ||
mod - tests/controllers/ChangeQuestionTemplateTest.php | Diff File | ||
rm - tests/data/surveys/limesurvey_survey_573387.lss | Diff File |
Date Modified | Username | Field | Change |
---|---|---|---|
2020-09-23 19:46 | gabrieljenik | New Issue | |
2020-09-23 19:46 | gabrieljenik | Status | new => assigned |
2020-09-23 19:46 | gabrieljenik | Assigned To | => gabrieljenik |
2020-09-23 19:48 | gabrieljenik | Steps to Reproduce Updated | |
2020-09-24 11:39 | ollehar | Note Added: 59943 | |
2020-09-24 14:30 | gabrieljenik | Note Added: 59944 | |
2020-09-24 14:31 | gabrieljenik | Summary | Survey Imports can't be reused among test scripts => Clar process after test script is failing |
2020-09-24 14:31 | gabrieljenik | Description Updated | |
2020-09-24 14:33 | gabrieljenik | Note Added: 59945 | |
2020-10-01 14:51 | gabrieljenik | Summary | Clar process after test script is failing => Clear process after test script is failing |
2020-10-01 15:04 | gabrieljenik | Note Added: 60026 | |
2020-10-01 15:10 | gabrieljenik | Note Added: 60027 | |
2020-10-01 15:10 | ollehar | Note Added: 60028 | |
2020-10-01 15:13 | gabrieljenik | Issue cloned: 16714 | |
2020-10-01 15:13 | gabrieljenik | Relationship added | related to 16714 |
2020-10-02 18:59 | ollehar | Changeset attached | => LimeSurvey 3.x-LTS 167b11f4 |
2020-10-02 18:59 | gabrieljenik | Changeset attached | => LimeSurvey 3.x-LTS 358bf088 |
2020-10-02 18:59 | gabrieljenik | Changeset attached | => LimeSurvey 3.x-LTS d99f229f |
2020-10-02 18:59 | gabrieljenik | Note Added: 60056 | |
2020-10-02 18:59 | gabrieljenik | Resolution | open => fixed |
2020-10-05 08:36 | lime_release_bot | Note Added: 60065 | |
2020-10-05 08:36 | lime_release_bot | Status | assigned => closed |