Bugzilla – Attachment 27704 Details for
Bug 10212
Columns configuration for tables
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 10212: Columns configuration for tables - Unit tests
Bug-10212-Columns-configuration-for-tables---Unit-.patch (text/plain), 10.83 KB, created by
Jonathan Druart
on 2014-04-28 08:48:07 UTC
(
hide
)
Description:
Bug 10212: Columns configuration for tables - Unit tests
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2014-04-28 08:48:07 UTC
Size:
10.83 KB
patch
obsolete
>From 41c31d99a50d0f7e2285913f18caee8630f74726 Mon Sep 17 00:00:00 2001 >From: Jonathan Druart <jonathan.druart@biblibre.com> >Date: Fri, 10 Jan 2014 18:26:27 +0100 >Subject: [PATCH] Bug 10212: Columns configuration for tables - Unit tests > >Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> >--- > t/db_dependent/ColumnsSettings.t | 289 ++++++++++++++++++++++++++++++++++++++ > 1 file changed, 289 insertions(+) > create mode 100644 t/db_dependent/ColumnsSettings.t > >diff --git a/t/db_dependent/ColumnsSettings.t b/t/db_dependent/ColumnsSettings.t >new file mode 100644 >index 0000000..0201822 >--- /dev/null >+++ b/t/db_dependent/ColumnsSettings.t >@@ -0,0 +1,289 @@ >+#!/usr/bin/perl; >+ >+use Modern::Perl; >+use Test::More tests => 3; >+ >+use C4::Context; >+use C4::Utils::DataTables::ColumnsSettings; >+my $dbh = C4::Context->dbh; >+$dbh->{AutoCommit} = 0; >+$dbh->{RaiseError} = 1; >+ >+$dbh->do(q|DELETE FROM columns_settings|); >+ >+C4::Utils::DataTables::ColumnsSettings::update_columns( >+ { >+ columns => [ >+ { >+ module => 'circ', >+ page => 'circulation', >+ tablename => 'holdst', >+ columnname => 'barcode', >+ is_hidden => 1, >+ cannot_be_toggled => 1 >+ }, >+ { >+ module => 'circ', >+ page => 'circulation', >+ tablename => 'holdst', >+ columnname => 'call_number', >+ is_hidden => 0, >+ cannot_be_toggled => 1 >+ }, >+ { >+ module => 'circ', >+ page => 'circulation', >+ tablename => 'holdst', >+ columnname => 'delete', >+ is_hidden => 1, >+ cannot_be_toggled => 0 >+ }, >+ { >+ module => 'circ', >+ page => 'circulation', >+ tablename => 'holdst', >+ columnname => 'hold_date', >+ is_hidden => 1, >+ cannot_be_toggled => 0 >+ }, >+ { >+ module => 'circ', >+ page => 'circulation', >+ tablename => 'holdst', >+ columnname => 'info', >+ is_hidden => 1, >+ cannot_be_toggled => 0 >+ }, >+ { >+ module => 'circ', >+ page => 'circulation', >+ tablename => 'holdst', >+ columnname => 'priority', >+ is_hidden => 0, >+ cannot_be_toggled => 0 >+ }, >+ { >+ module => 'circ', >+ page => 'circulation', >+ tablename => 'holdst', >+ columnname => 'title', >+ is_hidden => 0, >+ cannot_be_toggled => 1 >+ }, >+ { >+ module => 'circ', >+ page => 'circulation', >+ tablename => 'issuest', >+ columnname => 'call_no', >+ is_hidden => 0, >+ cannot_be_toggled => 0 >+ }, >+ { >+ module => 'circ', >+ page => 'circulation', >+ tablename => 'issuest', >+ columnname => 'charge', >+ is_hidden => 1, >+ cannot_be_toggled => 0 >+ }, >+ { >+ module => 'circ', >+ page => 'circulation', >+ tablename => 'issuest', >+ columnname => 'checked_out_from', >+ is_hidden => 0, >+ cannot_be_toggled => 0 >+ }, >+ { >+ module => 'circ', >+ page => 'circulation', >+ tablename => 'issuest', >+ columnname => 'checked_out_on', >+ is_hidden => 0, >+ cannot_be_toggled => 0 >+ }, >+ { >+ module => 'circ', >+ page => 'circulation', >+ tablename => 'issuest', >+ columnname => 'check_in', >+ is_hidden => 1, >+ cannot_be_toggled => 0 >+ }, >+ { >+ module => 'circ', >+ page => 'circulation', >+ tablename => 'issuest', >+ columnname => 'due_date', >+ is_hidden => 0, >+ cannot_be_toggled => 0 >+ }, >+ { >+ module => 'circ', >+ page => 'circulation', >+ tablename => 'issuest', >+ columnname => 'export', >+ is_hidden => 1, >+ cannot_be_toggled => 0 >+ }, >+ { >+ module => 'circ', >+ page => 'circulation', >+ tablename => 'issuest', >+ columnname => 'item_type', >+ is_hidden => 0, >+ cannot_be_toggled => 0 >+ }, >+ { >+ module => 'circ', >+ page => 'circulation', >+ tablename => 'issuest', >+ columnname => 'price', >+ is_hidden => 1, >+ cannot_be_toggled => 1 >+ }, >+ { >+ module => 'circ', >+ page => 'circulation', >+ tablename => 'issuest', >+ columnname => 'renew', >+ is_hidden => 1, >+ cannot_be_toggled => 0 >+ }, >+ { >+ module => 'circ', >+ page => 'circulation', >+ tablename => 'issuest', >+ columnname => 'title', >+ is_hidden => 0, >+ cannot_be_toggled => 1 >+ }, >+ ] >+ } >+); >+ >+my $modules = C4::Utils::DataTables::ColumnsSettings::get_modules(); >+ >+my $modules_expected = { >+ 'circ' => { >+ 'circulation' => { >+ 'holdst' => [ >+ { >+ 'columnname' => 'hold_date', >+ 'is_hidden' => '1', >+ 'cannot_be_toggled' => '0' >+ }, >+ { >+ 'columnname' => 'title', >+ 'is_hidden' => '0', >+ 'cannot_be_toggled' => '1' >+ }, >+ { >+ 'columnname' => 'call_number', >+ 'is_hidden' => '0', >+ 'cannot_be_toggled' => '1' >+ }, >+ { >+ 'columnname' => 'barcode', >+ 'is_hidden' => '1', >+ 'cannot_be_toggled' => '1' >+ }, >+ { >+ 'columnname' => 'priority', >+ 'is_hidden' => '0', >+ 'cannot_be_toggled' => '0' >+ }, >+ { >+ 'columnname' => 'delete', >+ 'is_hidden' => '1', >+ 'cannot_be_modified' => '1', >+ 'cannot_be_toggled' => '0' >+ }, >+ { >+ 'columnname' => 'info', >+ 'is_hidden' => '1', >+ 'cannot_be_modified' => '1', >+ 'cannot_be_toggled' => '0' >+ } >+ ], >+ 'issuest' => [ >+ { >+ 'columnname' => 'due_date', >+ 'is_hidden' => '0', >+ 'cannot_be_toggled' => '0' >+ }, >+ { >+ 'columnname' => 'title', >+ 'is_hidden' => '0', >+ 'cannot_be_toggled' => '1' >+ }, >+ { >+ 'columnname' => 'item_type', >+ 'is_hidden' => '0', >+ 'cannot_be_toggled' => '0' >+ }, >+ { >+ 'columnname' => 'checked_out_on', >+ 'is_hidden' => '0', >+ 'cannot_be_toggled' => '0' >+ }, >+ { >+ 'columnname' => 'checked_out_from', >+ 'is_hidden' => '0', >+ 'cannot_be_toggled' => '0' >+ }, >+ { >+ 'columnname' => 'call_no', >+ 'is_hidden' => '0', >+ 'cannot_be_toggled' => '0' >+ }, >+ { >+ 'columnname' => 'charge', >+ 'is_hidden' => '1', >+ 'cannot_be_toggled' => '0' >+ }, >+ { >+ 'columnname' => 'price', >+ 'is_hidden' => '1', >+ 'cannot_be_toggled' => '1' >+ }, >+ { >+ 'columnname' => 'renew', >+ 'is_hidden' => '1', >+ 'cannot_be_modified' => '1', >+ 'cannot_be_toggled' => '0' >+ }, >+ { >+ 'columnname' => 'check_in', >+ 'is_hidden' => '1', >+ 'cannot_be_modified' => '1', >+ 'cannot_be_toggled' => '0' >+ }, >+ { >+ 'columnname' => 'export', >+ 'is_hidden' => '1', >+ 'cannot_be_modified' => '1', >+ 'cannot_be_toggled' => '0' >+ } >+ ] >+ } >+ } >+}; >+ >+is_deeply( $modules, $modules_expected, 'get_modules returns all values' ); >+ >+for my $m ( keys %$modules ) { >+ for my $p ( keys %{ $modules->{$m} } ) { >+ for my $t ( keys %{ $modules->{$m}{$p} } ) { >+ my $columns = >+ C4::Utils::DataTables::ColumnsSettings::get_columns( $m, $p, $t ); >+ is_deeply( >+ $columns, >+ $modules->{$m}{$p}{$t}, >+ "columns for $m>$p>$t" >+ ); >+ } >+ } >+} >+ >+$dbh->rollback; >-- >1.7.10.4
<b>You cannot view the attachment while viewing its details because your browser does not support IFRAMEs. <a href="attachment.cgi?id=27704">View the attachment on a separate page</a>.</b>
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 10212
:
17978
|
17979
|
18861
|
20625
|
20640
|
21178
|
21179
|
21180
|
21583
|
21584
|
21585
|
24083
|
24084
|
24085
|
24149
|
24150
|
24151
|
24167
|
24168
|
24577
|
27700
|
27701
|
27702
|
27703
|
27704
|
27705
|
29075
|
29076
|
29077
|
29078
|
29079
|
29080
|
29537
|
29538
|
29539
|
29540
|
29541
|
29701
|
29702
|
29703
|
29704
|
29705
|
29718
|
29830
|
29836
|
29837
|
29838
|
29839
|
29840
|
29841
|
29842
|
29882
|
29883
|
29884
|
29885
|
29886
|
29887
|
29888
|
29947
|
31158
|
31159