Bug 8022

Summary: Permissions test doesn't check all languages
Product: Koha Reporter: Jared Camins-Esakov <jcamins>
Component: Test SuiteAssignee: Jared Camins-Esakov <jcamins>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P5 - low CC: chris, koha.sekjal, magnus, paul.poulain
Version: 3.6   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 8022: Permissions test doesn't check all languages
Bug 8022: Permissions test doesn't check all languages

Description Jared Camins-Esakov 2012-04-28 14:03:11 UTC
The permissions test does not check the permissions files for nb-NO and es-ES. It should.
Comment 1 Jared Camins-Esakov 2012-04-28 14:05:33 UTC Comment hidden (obsolete)
Comment 2 Magnus Enger 2012-05-07 10:59:05 UTC
Created attachment 9426 [details] [review]
Bug 8022: Permissions test doesn't check all languages

Add the permissions files for es-ES and nb-NO to the permissions test.

Signed-off-by: Magnus Enger <magnus@enger.priv.no>
Running xt/permissions.t before the patch leaves out nb-NO and es-ES.
After the patch they are picked up and the test fails correctly
because of the missing permissions described in Bug 7984.
As far as I can tall, all permission files are now picked up by
this test.
Comment 3 Ian Walls 2012-05-10 15:56:07 UTC
Adds necessary languages to test.  Marking Passed QA.
Comment 4 Chris Cormack 2012-05-14 20:55:15 UTC
Will be in 3.8.1
Comment 5 Jared Camins-Esakov 2012-06-14 11:52:39 UTC
This patch will be in 3.6.6.