Bug 24310 - Use Text::Lorem instead of String::Random in Test::Builder
Summary: Use Text::Lorem instead of String::Random in Test::Builder
Status: ASSIGNED
Alias: None
Product: Koha
Classification: Unclassified
Component: Test Suite (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Jonathan Druart
QA Contact: Testopia
URL:
Keywords: dependency
Depends on:
Blocks:
 
Reported: 2019-12-26 17:33 UTC by Jonathan Druart
Modified: 2023-04-27 05:48 UTC (History)
1 user (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 24310: Fix TestBuilder to generate consistent list content (843 bytes, patch)
2019-12-27 12:58 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 24310: Use Text::Lorem instead of String::Random (new dependency) (1.60 KB, patch)
2019-12-27 12:58 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 24310: Fix TestBuilder to generate consistent list content (854 bytes, patch)
2023-04-27 05:48 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 24310: Use Text::Lorem instead of String::Random (new dependency) (1.66 KB, patch)
2023-04-27 05:48 UTC, Jonathan Druart
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Druart 2019-12-26 17:33:52 UTC
That would help to get something more readable. Currently we generate a string of one word that is the size of the column. It produces a lot of overflow when displayed on the screen.
Comment 1 Jonathan Druart 2019-12-27 12:58:01 UTC
Created attachment 96647 [details] [review]
Bug 24310: Fix TestBuilder to generate consistent list content
Comment 2 Jonathan Druart 2019-12-27 12:58:04 UTC
Created attachment 96648 [details] [review]
Bug 24310: Use Text::Lorem instead of String::Random (new dependency)
Comment 3 Katrin Fischer 2019-12-29 14:02:42 UTC
Hi Jonathan, the change sounds good - can you please add a test plan?
Comment 4 Jonathan Druart 2020-02-18 11:15:36 UTC
The context was:
 * https://lists.koha-community.org/pipermail/koha-devel/2019-December/045450.html
 * https://gitlab.com/joubu/koha-plugin-jump

I do not have time at the moment to get back to this, and I have not received much help on the list. So not sure it makes sense to have this into master now.
Comment 5 Jonathan Druart 2023-04-27 05:48:38 UTC
Created attachment 150279 [details] [review]
Bug 24310: Fix TestBuilder to generate consistent list content
Comment 6 Jonathan Druart 2023-04-27 05:48:40 UTC
Created attachment 150280 [details] [review]
Bug 24310: Use Text::Lorem instead of String::Random (new dependency)