User:Victor Grousset - tuxayo/Community work/Reports/2024-12
QA
Notes:
- An attempt results in feedback provided on the bug. To ask if I correctly followed the test plan due to results that I couldn't interpret. Or if there is indeed an issue. Or questions to clarify something in the code.
- Partial QA is a way to contribute to a ticket that I can't QA alone but for which I can confidently review only part of the code or aren't sure enough about understanding the functional need and change.
Find issues: Bug 37993 - Having a single EDI EAN account produces a bad redirect
Validate follow-ups & find another issue Bug 37993 - Having a single EDI EAN account produces a bad redirect
Try to sum up the issues to confirm that a proper fix is impossible: Bug 32722 - Mandatory subfields from UNIMARC framework shouldn't block the creation of a bib record when field itself is optional
QA feedback and try to find a way around a problem: Bug 38664 - Tidy the whole codebase
Enhancement from Jonathan with the goal of improving QA and backport workflow and general code readability.
QA tooling: Update prettier version in the submitted patches: Bug 38149 - Make ESLint config compatible with version 9 and have ESLint and Prettier installed by default
Quick amendement.
QA tooling: quick follow-up to QA feedback: Bug 38167 - ESLint: migrate config to flat format + cleanup some node dependencies
Find inconsistency with another WIP ticket + try to reproduce an issue Bug 38426 - Node.js v18 EOL around 25.05 release time
To not give away unfixed stuff without blocking too much other fixes.
Send minor follow-up + confirm some remaining issues still there Bug 35635 - Expand Patron attribute type mandatory field
Note for the people funding my work and the rest of the community for transparency purposes: quick QA continuation of an enhancement from ByWater Solutions. For QA of enhancements I'm asked to ideally focus on one monthly.
Minor amendement for commit formatting: Bug 37989 - Add template toolkit to problem_report notice
Confirm issues fixed: Some security bug 4
Did partial QA after signing off last month.
Dig and send relevant old info to the author: Bug 38618 - Remove implicit template translations (in .tt and .inc), use only explicit macro invocations
From old discussion on the topic of performance impact. And sent in chat topic linked in ticket.
Round 2: find issue in follow-up + elaborate on remaining issue: Bug 38411 - When adding multiple items on receive, mandatory fields are not checked
Search to partially validate that this coding guideline is obsolete: Bug 38720 - HTML1 no longer valid? TT tags can be present in HTML without breaking the translator tool
Dig into the very old history of that coding guideline.
WIP investigating possible critical bug found while reviewing Bug 32722 - UNIMARC: Remove mandatory flag from some subfields [...]
Signoffs (testing submitted patches so they can move on to be integrated)
Note: an attempt usually results in feedback provided on the bug. To ask if I correctly followed the test plan due to results that I couldn't interpret. Or if there is indeed an issue.
Success after 2nd attempt: Some security bug 1
Get more troubleshooting info for failure close to what happens in the CI: Bug 38236 - selenium/administration_tasks.t is failing randomly with "no such alert"
It's not QA but it's a problem with tests which are QA tools that are basically automation of a few signoffs to continuously watch for regressions.
It's not a signoff but it's trying to reproduce an issue following a test plan which is the 1st part of most test plans.
So it feels it should be somewhere.
And that was quick anyway. RM asked for help and Jonathan had yet another randomly failing test not reproducing locally :(
Partial QA: Extended testing + follow-up to remove unneeded dependency Bug 38503 - Add a Cypress task to generate objects based on its swagger def spec
Note for the people funding my work: Enhancement from Jonathan, so by transitivity a part of the 3 usual companies. And it's an enhancement that allows extending the test suite so it's ultimately about bugs.
Extended testing + fixes + try find alternative fixes for other issue : Bug 38461 - Table features needs to be covered by e2e tests - Cypress
Note for the people funding my work: Enhancement from Jonathan, so by transitivity a part of the 3 usual companies. And it's an enhancement that allows extending the test suite so it's ultimately about bugs.
Bug 32722 - Mandatory subfields from UNIMARC framework shouldn't block the creation of a bib record when field itself is optional
Bug 22415 - Koha::EDI should not use Log::Log4perl directly
Bug 38423 - EDIFACT invoice files should skip orders that cannot be receipted rather than failing to complete
Some security bug 2: retest and maybe find cause of someone else's testing issues
Some security bug 5
Attempt to test alleged regression that could be fixed by: Bug 30975 - Use event delegation for framework plugins to avoid using private jQuery method _data
The beginning of the ticket suggests it was to fix a regression but the test plan doesn't mention it.
Misc tasks and work on tickets: Analysis or implementation
Research on usability of Bug 38503 to solve Bug 29285 - UI tests: Selenium 4 support or setup Cypress to make them rewritable with it
Follow up on the endless issues like Bug 38236 (selenium/administration_tasks.t is failing randomly) that Jonathan is dealing with. Counted on funded time as it's ultimately about bugs. Both from the usefulness of the test suite and by having it less be a source of bugs.