X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=t%2F85utf8.t;h=af6dedf5fde6d90a03415a2d1a9c3d4099e26d6d;hb=331886ef1b5ab7a732915b8213dd525a3ea63b88;hp=de63f276617b2a123171171ae19af145a7f12fdf;hpb=1415f198da91a23911972ad06b6dafaf6f4c0e8f;p=dbsrgits%2FDBIx-Class.git diff --git a/t/85utf8.t b/t/85utf8.t index de63f27..af6dedf 100644 --- a/t/85utf8.t +++ b/t/85utf8.t @@ -5,6 +5,7 @@ use Test::More; use Test::Warn; use lib qw(t/lib); use DBICTest; +use DBIC::DebugObj; { package A::Comp; @@ -22,6 +23,7 @@ use DBICTest; warnings_are ( sub { + local $ENV{DBIC_UTF8COLUMNS_OK} = 1; package A::Test1; use base 'DBIx::Class::Core'; __PACKAGE__->load_components(qw(Core +A::Comp Ordered UTF8Columns)); @@ -33,6 +35,20 @@ warnings_are ( 'no spurious warnings issued', ); +warnings_like ( + sub { + package A::Test1Loud; + use base 'DBIx::Class::Core'; + __PACKAGE__->load_components(qw(Core +A::Comp Ordered UTF8Columns)); + __PACKAGE__->load_components(qw(Ordered +A::SubComp Row UTF8Columns Core)); + sub store_column { shift->next::method (@_) }; + 1; + }, + [qr/Use of DBIx::Class::UTF8Columns is strongly discouraged/], + 'issued deprecation warning', +); + + my $test1_mro; my $idx = 0; for (@{mro::get_linear_isa ('A::Test1')} ) { @@ -44,7 +60,6 @@ cmp_ok ($test1_mro->{'A::Comp'}, '<', $test1_mro->{'DBIx::Class::UTF8Columns'}, cmp_ok ($test1_mro->{'DBIx::Class::UTF8Columns'}, '<', $test1_mro->{'DBIx::Class::Core'}, 'mro of Test1 correct (UTF8Col before Core)' ); cmp_ok ($test1_mro->{'DBIx::Class::Core'}, '<', $test1_mro->{'DBIx::Class::Row'}, 'mro of Test1 correct (Core before Row)' ); - warnings_like ( sub { package A::Test2; @@ -74,32 +89,94 @@ DBICTest::Schema::CD->load_components('UTF8Columns'); DBICTest::Schema::CD->utf8_columns('title'); Class::C3->reinitialize(); -my $cd = $schema->resultset('CD')->create( { artist => 1, title => "weird\x{466}stuff", year => '2048' } ); +# as per http://search.cpan.org/dist/Test-Simple/lib/Test/More.pm#utf8 +binmode (Test::More->builder->$_, ':utf8') for qw/output failure_output todo_output/; + +my $bytestream_title = my $utf8_title = "weird \x{466} stuff"; +utf8::encode($bytestream_title); +cmp_ok ($bytestream_title, 'ne', $utf8_title, 'unicode/raw differ (sanity check)'); + +my $storage = $schema->storage; +my ($sql, @bind); +my $debugobj = DBIC::DebugObj->new (\$sql, \@bind); +my ($orig_debug, $orig_debugobj) = ($storage->debug, $storage->debugobj); +$storage->debugobj ($debugobj); +$storage->debug (1); -ok( utf8::is_utf8( $cd->title ), 'got title with utf8 flag' ); -ok(! utf8::is_utf8( $cd->{_column_data}{title} ), 'store title without utf8' ); +my $cd = $schema->resultset('CD')->create( { artist => 1, title => $utf8_title, year => '2048' } ); -ok(! utf8::is_utf8( $cd->year ), 'got year without utf8 flag' ); -ok(! utf8::is_utf8( $cd->{_column_data}{year} ), 'store year without utf8' ); +$storage->debugobj ($orig_debugobj); +$storage->debug ($orig_debug); + +# bind values are always alphabetically ordered by column, thus [1] +# the single quotes are an artefact of the debug-system +TODO: { + local $TODO = "This has been broken since rev 1191, Mar 2006"; + is ($bind[1], "'$bytestream_title'", 'INSERT: raw bytes sent to the database'); +} + +# this should be using the cursor directly, no inflation/processing of any sort +my ($raw_db_title) = $schema->resultset('CD') + ->search ($cd->ident_condition) + ->get_column('title') + ->_resultset + ->cursor + ->next; + +is ($raw_db_title, $bytestream_title, 'INSERT: raw bytes retrieved from database'); + +for my $reloaded (0, 1) { + my $test = $reloaded ? 'reloaded' : 'stored'; + $cd->discard_changes if $reloaded; + + ok( utf8::is_utf8( $cd->title ), "got $test title with utf8 flag" ); + ok(! utf8::is_utf8( $cd->{_column_data}{title} ), "in-object $test title without utf8" ); + + ok(! utf8::is_utf8( $cd->year ), "got $test year without utf8 flag" ); + ok(! utf8::is_utf8( $cd->{_column_data}{year} ), "in-object $test year without utf8" ); +} $cd->title('nonunicode'); -ok(! utf8::is_utf8( $cd->title ), 'got title without utf8 flag' ); -ok(! utf8::is_utf8( $cd->{_column_data}{title} ), 'store utf8-less chars' ); +ok(! utf8::is_utf8( $cd->title ), 'update title without utf8 flag' ); +ok(! utf8::is_utf8( $cd->{_column_data}{title} ), 'store utf8-less title' ); + +$cd->update; +$cd->discard_changes; +ok(! utf8::is_utf8( $cd->title ), 'reloaded title without utf8 flag' ); +ok(! utf8::is_utf8( $cd->{_column_data}{title} ), 'reloaded utf8-less title' ); + +$bytestream_title = $utf8_title = "something \x{219} else"; +utf8::encode($bytestream_title); + + +$storage->debugobj ($debugobj); +$storage->debug (1); + +$cd->update ({ title => $utf8_title }); +$storage->debugobj ($orig_debugobj); +$storage->debug ($orig_debug); -my $v_utf8 = "\x{219}"; +is ($bind[0], "'$bytestream_title'", 'UPDATE: raw bytes sent to the database'); +($raw_db_title) = $schema->resultset('CD') + ->search ($cd->ident_condition) + ->get_column('title') + ->_resultset + ->cursor + ->next; +is ($raw_db_title, $bytestream_title, 'UPDATE: raw bytes retrieved from database'); -$cd->update ({ title => $v_utf8 }); -$cd->title($v_utf8); +$cd->discard_changes; +$cd->title($utf8_title); ok( !$cd->is_column_changed('title'), 'column is not dirty after setting the same unicode value' ); -$cd->update ({ title => $v_utf8 }); +$cd->update ({ title => $utf8_title }); $cd->title('something_else'); ok( $cd->is_column_changed('title'), 'column is dirty after setting to something completely different'); TODO: { local $TODO = 'There is currently no way to propagate aliases to inflate_result()'; - $cd = $schema->resultset('CD')->find ({ title => $v_utf8 }, { select => 'title', as => 'name' }); + $cd = $schema->resultset('CD')->find ({ title => $utf8_title }, { select => 'title', as => 'name' }); ok (utf8::is_utf8( $cd->get_column ('name') ), 'utf8 flag propagates via as'); }