User:Victor Grousset - tuxayo/Community work/Reports/2024-05

From Koha Wiki
Jump to navigation Jump to search

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.

Confirmed follow-ups address issues: Bug 35197 - Expose additional_field definitions through REST API

Round 2: Bug 36388 - Mouse operation does not work in draggable fields in authority editor (with Firefox)

Add notes in upcoming (May 15) dev meeting to try to get strait into the contention point (IIUC) Bug 30888 - Add a table for deletedauthorities

Bug 36832 - Adding authority records is broken

Some security bug 1: found issues

Some security bug 2: Partial QA: Code review and try to double check the fix in Perl REPL

Found and fixed issue in follow-up: Bug 36844 - Set library, desk, and cash register menu follow-ups

Round 2: some security bug 1: Partial QA + minor follow-up

All good, just need another pair of eye that there isn't still a blind spot somewhere in that patch.

Opened some security bug 3. When reviewing the above, found a blind spot somewhere else.

It's like late QA for another patch that is already released.

Opened some security bug 4. When reviewing security bug 2, found similar issue

So releasing fix for bug 2 would give a hint of the existence of this vulnerability.

Some security bug 2: Deeper testing on the fixes and to find additional attacks possible

And some code exploration to try to find other vulnerabilities with the same idea. And do additional testing to provide provide more info requested by the Quality Assurance Manager.

Bug 36060 - If issues table includes overdues Renew selected items button is disabled

Found issues: Bug 36694 - Remove HC Sticky library in favor of CSS

Note for the people funding my work and the rest of the community for transparency purposes: QA of an enhancement from ByWater Solutions. For QA of enhancements I'm asked to ideally focus on one monthly.

Found issues and fixed some of them: Bug 35659 - OAI Harvester

Note for the people funding my work and the rest of the community for transparency purposes: QA of an enhancement from BibLibre. For QA of enhancements I'm asked to ideally focus on one monthly.

Some functional analysis and feedback: Bug 36932 - koha-plack: Allow overriding starman environment for enabled dev_install

Round 2: Bug 27769 - Advanced editor shouldn't break copying selected text with Ctrl+C

Opened some security bug 5 when reviewing another security ticket

Signed off and found QA issues: Bug 36932 - koha-plack: Allow overriding starman environment for enabled dev_install

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.

Retest to try confirm someone else having found an issue: Bug 33268 - Overlay rules don't work correctly when source is set to *

(still works for me)

Some security bug 4: thorough testing

Some security bug 3: proposed test plan due to it being missing

Fix seems to work.

Found issue after extensive testing: koha-testing-docker: Enable Plack hot-reload via ENV

https://gitlab.com/koha-community/koha-testing-docker/-/merge_requests/497

Bug 36928 - Patron authentication / password validation will fail with 500 error for local users if external auth (LDAP/CAS/Shibboleth) is enabled

And found a lead to easily add tests necessary for the QA step.

Seems to work, need confirmation on one detail: Bug 34444 - Statistic 1/2 not saving when updating fund after receipt

Bug 36986 - (Bug 26176 follow-up) Fix rename StaffLoginBranchBasedOnIP in BDRev