Bug 16177

Summary: Tests for ColumnsSettings are failing
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: Test SuiteAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: normal    
Priority: P5 - low CC: brendan, julian.maurice, m.de.rooy
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 15285    
Bug Blocks:    
Attachments: Bug 16177: Fix tests for ColumnsSsettings.t
Bug 16177: Fix tests for ColumnsSsettings.t
Bug 16177: Fix tests for ColumnsSsettings.t

Description Jonathan Druart 2016-03-31 07:59:23 UTC

    
Comment 1 Jonathan Druart 2016-03-31 08:02:52 UTC Comment hidden (obsolete)
Comment 2 Mark Tompsett 2016-04-01 04:41:09 UTC Comment hidden (obsolete)
Comment 3 Marcel de Rooy 2016-04-01 08:04:32 UTC
Created attachment 49773 [details] [review]
Bug 16177: Fix tests for ColumnsSsettings.t

This failure has been introduced by
  commit 243b797dd11bc0ac1f475195ab765ffbf2220f8b
    Bug 15285: Update common files

This is because the cannot_be_modified key is not always created by
C4::Utils::DataTables::ColumnsSettings subroutines

Test plan:
  prove t/db_dependent/ColumnsSettings.t
should return green
And you can also test a table where the feature is set and a column
cannot be modified from the admin page.

NOTE: Works as described. I was pondering this potential
      solution when I found it in bugzilla.

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 4 Brendan Gallagher 2016-04-01 19:09:56 UTC
Pushed to Master - Should be in the May 2016 release.  Thanks!
Comment 5 Julian Maurice 2016-04-07 08:19:37 UTC
Not needed in stable versions, status changed to RESOLVED