[Koha-bugs] [Bug 33484] Generalise table saveState for all tables

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33484

--- Comment #31 from Emily Lamancusa <emily.lamancusa at montgomerycountymd.gov> ---
I wrote up a test plan to keep myself organized while testing different tables
and states...sharing in case someone else gets to it before me and would like
to use it. (also open to feedback :) )

Considerations:
- test a variety of tables
- must include both tables that use KohaTable (column_settings.inc) and
kohaTable (datatables.js)

For reference: tables that use kohaTable (based on a git grep):
- patron search (patrons_table)
- acqui/parcel.tt (pending_orders)
- admin/background_jobs.tt (table_jobs)
- admin/branches.tt (libraries)
- admin/cities.tt (table_cities)
- admin/identity_provider_domains.tt (identity_provider_domains)
- admin/identity_providers.tt (identity_providers)
- admin/search_filters.tt (search_filters_table)
- admin/smtp_servers.tt (smtp_servers)
- catalogue/detail.tt (table_concerns)
- catalogue/detail.tt (bundle_table)
- cataloguing/concerns.tt (table_concerns)
- pos/register.tt (table_cashups)
- tools/quotes.tt (quotes)

Test plan per table:
1. Edit the config to hide some columns and disable "save state"
2. Test that the "clean" page matches the config settings
3. Make some client-side changes
    a. Hide one or more showing columns
    b. Show one or more hidden columns
    c. Change number of results to display
    d. Add a search filter
4. Navigate away from that page and back
5. Confirm that the state was not saved and the page matches the config
settings
6. Navigate away from that page
7. Edit the config to enable "save state"
8. Return to the page and confirm that it matches the config settings
9. Make some client-side changes
    a. Hide one or more showing columns
    b. Show one or more hidden columns
    c. Change number of results to display
    d. Add a search filter
10. Navigate away from the page and back
11. Confirm that all of the changes were retained

-- 
You are receiving this mail because:
You are watching all bug changes.