Bug 21832: add unit test
[koha.git] / t / SimpleMARC.t
index fff1096..5958508 100644 (file)
@@ -1,6 +1,6 @@
 use Modern::Perl;
 
-use Test::More tests => 9;
+use Test::More tests => 10;
 
 use_ok("MARC::Field");
 use_ok("MARC::Record");
@@ -41,14 +41,14 @@ sub new_record {
 subtest 'field_exists' => sub {
     plan tests => 3;
     my $record = new_record;
-    is(
+    is_deeply(
         field_exists( { record => $record, field => '650', subfield => 'a' } ),
-        'Computer programming.',
+        [1],
         '650$a exists'
     );
-    is(
+    is_deeply(
         field_exists( { record => $record, field => '650', subfield => 'b' } ),
-        undef,
+        [],
         '650$b does not exist'
     );
 
@@ -60,10 +60,10 @@ subtest 'field_exists' => sub {
         )
     );
 
-    is(
+    is_deeply(
         field_exists( { record => $record, field => '650', subfield => 'a' } ),
-        'Computer programming.',
-        '650$a exists, field_exists returns the first one'
+        [ 1, 2 ],
+        '650$a exists, field_exists returns the 2 field numbers'
     );
 };
 
@@ -83,42 +83,48 @@ subtest 'read_field' => sub {
 
         my @fields_650a =
           read_field( { record => $record, field => '650', subfield => 'a' } );
-        is( $fields_650a[0], 'Computer programming.', 'first 650$a' );
-        is( $fields_650a[1], 'Computer algorithms.',  'second 650$a' );
-        is(
-            read_field(
-                {
-                    record   => $record,
-                    field    => '650',
-                    subfield => 'a',
-                    n        => 1
-                }
-            ),
-            'Computer programming.',
+        is_deeply( $fields_650a[0], 'Computer programming.', 'first 650$a' );
+        is_deeply( $fields_650a[1], 'Computer algorithms.',  'second 650$a' );
+        is_deeply(
+            [
+                read_field(
+                    {
+                        record        => $record,
+                        field         => '650',
+                        subfield      => 'a',
+                        field_numbers => [1]
+                    }
+                )
+            ],
+            ['Computer programming.'],
             'first 650$a bis'
         );
-        is(
-            read_field(
-                {
-                    record   => $record,
-                    field    => '650',
-                    subfield => 'a',
-                    n        => 2
-                }
-            ),
-            'Computer algorithms.',
+        is_deeply(
+            [
+                read_field(
+                    {
+                        record        => $record,
+                        field         => '650',
+                        subfield      => 'a',
+                        field_numbers => [2]
+                    }
+                )
+            ],
+            ['Computer algorithms.'],
             'second 650$a bis'
         );
-        is(
-            read_field(
-                {
-                    record   => $record,
-                    field    => '650',
-                    subfield => 'a',
-                    n        => 3
-                }
-            ),
-            undef,
+        is_deeply(
+            [
+                read_field(
+                    {
+                        record        => $record,
+                        field         => '650',
+                        subfield      => 'a',
+                        field_numbers => [3]
+                    }
+                )
+            ],
+            [],
             'There is no 3 650$a'
         );
     };
@@ -148,9 +154,9 @@ subtest 'read_field' => sub {
             [
                 read_field(
                     {
-                        record => $record,
-                        field  => '650',
-                        n      => 1
+                        record        => $record,
+                        field         => '650',
+                        field_numbers => [1]
                     }
                 )
             ],
@@ -158,12 +164,20 @@ subtest 'read_field' => sub {
             'Get the all subfield values for the first field 650'
         );
         is_deeply(
-            [ read_field( { record => $record, field => '650', n => 2 } ) ],
+            [
+                read_field(
+                    { record => $record, field => '650', field_numbers => [2] }
+                )
+            ],
             [ 'Computer algorithms.', '463' ],
             'Get the all subfield values for the second field 650'
         );
         is_deeply(
-            [ read_field( { record => $record, field => '650', n => 3 } ) ],
+            [
+                read_field(
+                    { record => $record, field => '650', field_numbers => [3] }
+                )
+            ],
             [],
 'Get the all subfield values for the third field 650 which does not exist'
         );
@@ -186,10 +200,12 @@ subtest 'update_field' => sub {
             }
         );
         is_deeply(
-            read_field(
-                { record => $record, field => '952', subfield => 'p' }
-            ),
-            '3010023918',
+            [
+                read_field(
+                    { record => $record, field => '952', subfield => 'p' }
+                )
+            ],
+            ['3010023918'],
             'update existing subfield 952$p'
         );
         delete_field( { record => $record, field => '952' } );
@@ -210,10 +226,12 @@ subtest 'update_field' => sub {
             }
         );
         is_deeply(
-            read_field(
-                { record => $record, field => '952', subfield => 'p' }
-            ),
-            '3010023918',
+            [
+                read_field(
+                    { record => $record, field => '952', subfield => 'p' }
+                )
+            ],
+            ['3010023918'],
             'create subfield 952$p'
         );
         is_deeply(
@@ -269,7 +287,7 @@ subtest 'update_field' => sub {
 subtest 'copy_field' => sub {
     plan tests              => 2;
     subtest 'copy subfield' => sub {
-        plan tests => 15;
+        plan tests => 20;
         my $record = new_record;
         $record->append_fields(
             MARC::Field->new(
@@ -288,25 +306,29 @@ subtest 'copy_field' => sub {
             }
         );
         is_deeply(
-            read_field(
-                { record => $record, field => '245', subfield => 'a' }
-            ),
-            'The art of computer programming',
+            [
+                read_field(
+                    { record => $record, field => '245', subfield => 'a' }
+                )
+            ],
+            ['The art of computer programming'],
             'After copy 245$a still exists'
         );
         is_deeply(
-            read_field(
-                { record => $record, field => '246', subfield => 'a' }
-            ),
-            'The art of computer programming',
+            [
+                read_field(
+                    { record => $record, field => '246', subfield => 'a' }
+                )
+            ],
+            ['The art of computer programming'],
             '246$a is a new field'
         );
         delete_field( { record => $record, field => '246' } );
-        is(
+        is_deeply(
             field_exists(
                 { record => $record, field => '246', subfield => 'a' }
             ),
-            undef,
+            [],
             '246$a does not exist anymore'
         );
 
@@ -335,14 +357,16 @@ subtest 'copy_field' => sub {
                 from_subfield => 'a',
                 to_field      => '651',
                 to_subfield   => 'a',
-                n             => 1
+                field_numbers => [1]
             }
         );
         is_deeply(
-            read_field(
-                { record => $record, field => '651', subfield => 'a' }
-            ),
-            'Computer programming.',
+            [
+                read_field(
+                    { record => $record, field => '651', subfield => 'a' }
+                )
+            ],
+            ['Computer programming.'],
             'Copy first field 650$a'
         );
         delete_field( { record => $record, field => '652' } );
@@ -354,14 +378,16 @@ subtest 'copy_field' => sub {
                 from_subfield => 'a',
                 to_field      => '651',
                 to_subfield   => 'a',
-                n             => 2
+                field_numbers => [2]
             }
         );
         is_deeply(
-            read_field(
-                { record => $record, field => '651', subfield => 'a' }
-            ),
-            'Computer algorithms.',
+            [
+                read_field(
+                    { record => $record, field => '651', subfield => 'a' }
+                )
+            ],
+            ['Computer programming.', 'Computer algorithms.'],
             'Copy second field 650$a'
         );
         delete_field( { record => $record, field => '651' } );
@@ -383,6 +409,7 @@ subtest 'copy_field' => sub {
             [ 'The art of programming.', 'The art of algorithms.' ],
             'Copy field using regex'
         );
+        delete_field( { record => $record, field => '651' } );
 
         copy_field(
             {
@@ -430,7 +457,7 @@ subtest 'copy_field' => sub {
                 to_field      => '651',
                 to_subfield   => 'a',
                 regex => { search => 'Computer', replace => 'The art of' },
-                n     => 1
+                field_numbers => [1]
             }
         );
         @fields_651a =
@@ -523,6 +550,7 @@ subtest 'copy_field' => sub {
             'Copy field using regex'
         );
 
+        $record = new_record;
         $record->append_fields(
             MARC::Field->new(
                 952, ' ', ' ',
@@ -542,15 +570,114 @@ subtest 'copy_field' => sub {
         );
         my @fields_952d =
           read_field( { record => $record, field => '952', subfield => 'd' } );
+        # FIXME We need a new action "duplicate" if we don't want to modify the original field
         is_deeply(
             \@fields_952d,
-            [ '2001-06-25', '2001-06-25' ],
+            [ '2001-06-25', '2001-06-25', '2001-06-25' ],
             'copy 952$d into others 952 field'
         );
+
+        copy_field(
+            {
+                record        => $record,
+                from_field    => '111',
+                from_subfield => '1',
+                to_field      => '999',
+                to_subfield   => '9'
+            }
+        );
+        my @fields_9999 =
+          read_field( { record => $record, field => '999', subfield => '9' } );
+        is_deeply( \@fields_9999, [],
+            'copy a nonexistent subfield does not create a new one' );
+
+        $record = new_record;
+        copy_field(
+            {
+                record        => $record,
+                from_field    => 245,
+                from_subfield => 'a',
+                to_field      => 245,
+                to_subfield   => 'a',
+                regex         => { search => '^', replace => 'BEGIN ' }
+            }
+        );
+        is_deeply(
+            [
+                read_field(
+                    { record => $record, field => '245', subfield => 'a' }
+                )
+            ],
+            ['The art of computer programming', 'BEGIN The art of computer programming'],
+            'Update a subfield: add a string at the beginning'
+        );
+
+        $record = new_record;
+        copy_field(
+            {
+                record        => $record,
+                from_field    => 245,
+                from_subfield => 'a',
+                to_field      => 245,
+                to_subfield   => 'a',
+                regex         => { search => '$', replace => ' END' }
+            }
+        );
+        is_deeply(
+            [
+                read_field(
+                    { record => $record, field => '245', subfield => 'a' }
+                )
+            ],
+            ['The art of computer programming', 'The art of computer programming END'],
+            'Update a subfield: add a string at the end'
+        );
+
+        $record = new_record;
+        copy_field(
+            {
+                record        => $record,
+                from_field    => 245,
+                from_subfield => 'c',
+                to_field      => 650,
+                to_subfield   => 'c',
+            }
+        );
+
+        is_deeply(
+            [
+                read_field(
+                    { record => $record, field => '650' }
+                )
+            ],
+            [ 'Computer programming.', '462', 'Donald E. Knuth.' ],
+            'Copy a subfield to an existent field but inexistent subfield'
+        );
+
+        $record = new_record;
+        copy_field(
+            {
+                record        => $record,
+                from_field    => 245,
+                from_subfield => 'c',
+                to_field      => 650,
+                to_subfield   => '9',
+            }
+        );
+
+        is_deeply(
+            [
+                read_field(
+                    { record => $record, field => '650' }
+                )
+            ],
+            [ 'Computer programming.', '462', 'Donald E. Knuth.' ],
+            'Copy a subfield to an existent field / subfield'
+        );
     };
 
     subtest 'copy field' => sub {
-        plan tests => 11;
+        plan tests => 14;
         my $record = new_record;
         $record->append_fields(
             MARC::Field->new(
@@ -565,22 +692,38 @@ subtest 'copy_field' => sub {
             { record => $record, from_field => '952', to_field => '953' } );
         my @fields_952 = read_field( { record => $record, field => '952' } );
         is_deeply(
-            [ read_field( { record => $record, field => '952', n => 1 } ) ],
+            [
+                read_field(
+                    { record => $record, field => '952', field_numbers => [1] }
+                )
+            ],
             [ '3010023917', 'BK', 'GEN', '2001-06-25' ],
             "copy all: original first field still exists"
         );
         is_deeply(
-            [ read_field( { record => $record, field => '952', n => 2 } ) ],
+            [
+                read_field(
+                    { record => $record, field => '952', field_numbers => [2] }
+                )
+            ],
             [ '3010023918', 'CD' ],
             "copy all: original second field still exists"
         );
         is_deeply(
-            [ read_field( { record => $record, field => '953', n => 1 } ) ],
+            [
+                read_field(
+                    { record => $record, field => '953', field_numbers => [1] }
+                )
+            ],
             [ '3010023917', 'BK', 'GEN', '2001-06-25' ],
             "copy all: first original fields has been copied"
         );
         is_deeply(
-            [ read_field( { record => $record, field => '953', n => 2 } ) ],
+            [
+                read_field(
+                    { record => $record, field => '953', field_numbers => [2] }
+                )
+            ],
             [ '3010023918', 'CD' ],
             "copy all: second original fields has been copied"
         );
@@ -588,19 +731,27 @@ subtest 'copy_field' => sub {
         #- copy only the first field
         copy_field(
             {
-                record     => $record,
-                from_field => '953',
-                to_field   => '954',
-                n          => 1
+                record        => $record,
+                from_field    => '953',
+                to_field      => '954',
+                field_numbers => [1]
             }
         );
         is_deeply(
-            [ read_field( { record => $record, field => '953', n => 1 } ) ],
+            [
+                read_field(
+                    { record => $record, field => '953', field_numbers => [1] }
+                )
+            ],
             [ '3010023917', 'BK', 'GEN', '2001-06-25' ],
             "copy first: first original fields has been copied"
         );
         is_deeply(
-            [ read_field( { record => $record, field => '953', n => 2 } ) ],
+            [
+                read_field(
+                    { record => $record, field => '953', field_numbers => [2] }
+                )
+            ],
             [ '3010023918', 'CD' ],
             "copy first: second original fields has been copied"
         );
@@ -629,66 +780,695 @@ subtest 'copy_field' => sub {
             }
         );
         is_deeply(
-            [ read_field( { record => $record, field => '952', n => 1 } ) ],
+            [
+                read_field(
+                    { record => $record, field => '952', field_numbers => [1] }
+                )
+            ],
             [ '3010023917', 'BK', 'GEN', '2001-06-25' ],
             "copy all with regex: original first field still exists"
         );
         is_deeply(
-            [ read_field( { record => $record, field => '952', n => 2 } ) ],
+            [
+                read_field(
+                    { record => $record, field => '952', field_numbers => [2] }
+                )
+            ],
             [ '3010023918', 'CD' ],
             "copy all with regex: original second field still exists"
         );
         is_deeply(
-            [ read_field( { record => $record, field => '953', n => 1 } ) ],
+            [
+                read_field(
+                    { record => $record, field => '953', field_numbers => [1] }
+                )
+            ],
             [ '4242423917', 'BK', 'GEN', '2001-06-25' ],
-            "copy all wirh regex: first original fields has been copied"
+            "copy all with regex: first original fields has been copied"
         );
         is_deeply(
-            [ read_field( { record => $record, field => '953', n => 2 } ) ],
+            [
+                read_field(
+                    { record => $record, field => '953', field_numbers => [2] }
+                )
+            ],
             [ '4242423918', 'CD' ],
             "copy all with regex: second original fields has been copied"
         );
+        copy_field(
+            {
+                record     => $record,
+                from_field => '111',
+                to_field   => '999',
+            }
+        );
+        my @fields_9999 =
+          read_field( { record => $record, field => '999', subfield => '9' } );
+        is_deeply( \@fields_9999, [],
+            'copy a nonexistent field does not create a new one' );
+
+        $record = new_record;
+        copy_field(
+            {
+                record        => $record,
+                from_field    => 245,
+                to_field      => 650,
+            }
+        );
+
+        is_deeply(
+            [
+                read_field(
+                    { record => $record, field => '650', field_numbers => [2] }
+                )
+            ],
+            [ 'The art of computer programming', 'Donald E. Knuth.' ],
+            'Copy a field to existent fields should create a new field'
+        );
+        is_deeply(
+            [
+                read_field(
+                    { record => $record, field => '650', field_numbers => [1] }
+                )
+            ],
+            [ 'Computer programming.', '462' ],
+            'Copy a field to existent fields should create a new field, the original one should not have been updated'
+        );
     };
 };
 
-# move_field - subfields
-subtest 'move_field' => sub {
+# copy_and_replace_field - subfield
+subtest 'copy_and_replace_field' => sub {
     plan tests              => 2;
-    subtest 'move subfield' => sub {
-        plan tests => 6;
+    subtest 'copy and replace subfield' => sub {
+        plan tests => 19;
         my $record = new_record;
-        my ( @fields_952d, @fields_952c, @fields_954c, @fields_954p );
         $record->append_fields(
             MARC::Field->new(
-                952, ' ', ' ',
-                p => '3010023917',
-                y => 'BK',
-            ),
+                650, ' ', '0',
+                a => 'Computer algorithms.',
+                9 => '463',
+            )
         );
-        move_field(
+        copy_and_replace_field(
             {
                 record        => $record,
-                from_field    => '952',
-                from_subfield => 'c',
-                to_field      => '954',
-                to_subfield   => 'c'
+                from_field    => '245',
+                from_subfield => 'a',
+                to_field      => '246',
+                to_subfield   => 'a'
             }
         );
-        @fields_952c =
-          read_field( { record => $record, field => '952', subfield => 'c' } );
-        @fields_954c =
-          read_field( { record => $record, field => '954', subfield => 'c' } );
-        is_deeply( \@fields_952c, [],      'The 952$c has moved' );
-        is_deeply( \@fields_954c, ['GEN'], 'Now 954$c exists' );
+        is_deeply(
+            [
+                read_field(
+                    { record => $record, field => '245', subfield => 'a' }
+                )
+            ],
+            ['The art of computer programming'],
+            'Copy and replace should not have modify original subfield 245$a (same as copy)'
+        );
+        is_deeply(
+            [
+                read_field(
+                    { record => $record, field => '246', subfield => 'a' }
+                )
+            ],
+            ['The art of computer programming'],
+            'Copy and replace should create a new 246$a (same as copy)'
+        );
 
-        move_field(
-            {
-                record        => $record,
+        $record = new_record;
+        $record->append_fields(
+            MARC::Field->new(
+                650, ' ', '0',
+                a => 'Computer algorithms.',
+                9 => '463',
+            )
+        );
+        copy_and_replace_field(
+            {
+                record        => $record,
+                from_field    => '650',
+                from_subfield => 'a',
+                to_field      => '651',
+                to_subfield   => 'a'
+            }
+        );
+        my @fields_651a =
+          read_field( { record => $record, field => '651', subfield => 'a' } );
+        is_deeply(
+            \@fields_651a,
+            [ 'Computer programming.', 'Computer algorithms.' ],
+            'Copy and replace multivalued field (same as copy)'
+        );
+        delete_field( { record => $record, field => '651' } );
+
+        copy_and_replace_field(
+            {
+                record        => $record,
+                from_field    => '650',
+                from_subfield => 'a',
+                to_field      => '651',
+                to_subfield   => 'a',
+                field_numbers => [1]
+            }
+        );
+        is_deeply(
+            [
+                read_field(
+                    { record => $record, field => '651', subfield => 'a' }
+                )
+            ],
+            ['Computer programming.'],
+            'Copy and replace first field 650$a should only copy the 1st (same as copy)'
+        );
+
+        copy_and_replace_field(
+            {
+                record        => $record,
+                from_field    => '650',
+                from_subfield => 'a',
+                to_field      => '651',
+                to_subfield   => 'a',
+                field_numbers => [2]
+            }
+        );
+        is_deeply(
+            [
+                read_field(
+                    { record => $record, field => '651', subfield => 'a' }
+                )
+            ],
+            ['Computer algorithms.'],
+            'Copy and replace second field 650$a should erase 651$a'
+        );
+        delete_field( { record => $record, field => '651' } );
+
+        copy_and_replace_field(
+            {
+                record        => $record,
+                from_field    => '650',
+                from_subfield => 'a',
+                to_field      => '651',
+                to_subfield   => 'a',
+                regex => { search => 'Computer', replace => 'The art of' }
+            }
+        );
+        @fields_651a =
+          read_field( { record => $record, field => '651', subfield => 'a' } );
+        is_deeply(
+            \@fields_651a,
+            [ 'The art of programming.', 'The art of algorithms.' ],
+            'Copy and replace field using regex (same as copy)'
+        );
+        delete_field( { record => $record, field => '651' } );
+
+        copy_and_replace_field(
+            {
+                record        => $record,
+                from_field    => '650',
+                from_subfield => 'a',
+                to_field      => '651',
+                to_subfield   => 'a',
+                regex => { search => 'Computer', replace => 'The mistake of' }
+            }
+        );
+        @fields_651a =
+          read_field( { record => $record, field => '651', subfield => 'a' } );
+        is_deeply(
+            \@fields_651a,
+            [ 'The mistake of programming.', 'The mistake of algorithms.' ],
+            'Copy and replace fields using regex on existing fields (same as copy)'
+        );
+        delete_field( { record => $record, field => '651' } );
+
+        copy_and_replace_field(
+            {
+                record        => $record,
+                from_field    => '650',
+                from_subfield => 'a',
+                to_field      => '651',
+                to_subfield   => 'a',
+                regex => { search => 'Computer', replace => 'The art of' }
+            }
+        );
+        @fields_651a =
+          read_field( { record => $record, field => '651', subfield => 'a' } );
+        is_deeply(
+            \@fields_651a,
+            [ 'The art of programming.', 'The art of algorithms.', ],
+            'Copy and replace all fields using regex (same as copy)'
+        );
+        delete_field( { record => $record, field => '651' } );
+
+        copy_and_replace_field(
+            {
+                record        => $record,
+                from_field    => '650',
+                from_subfield => 'a',
+                to_field      => '651',
+                to_subfield   => 'a',
+                regex => { search => 'Computer', replace => 'The art of' },
+                field_numbers => [1]
+            }
+        );
+        @fields_651a =
+          read_field( { record => $record, field => '651', subfield => 'a' } );
+        is_deeply(
+            \@fields_651a,
+            [ 'The art of programming.', ],
+            'Copy and replace first field using regex (same as copy)'
+        );
+        delete_field( { record => $record, field => '651' } );
+
+        # Copy and replace with regex modifiers
+        $record = new_record;
+        $record->append_fields(
+            MARC::Field->new(
+                650, ' ', '0',
+                a => 'Computer algorithms.',
+                9 => '463',
+            )
+        );
+        copy_and_replace_field(
+            {
+                record        => $record,
+                from_field    => '650',
+                from_subfield => 'a',
+                to_field      => '652',
+                to_subfield   => 'a',
+                regex         => { search => 'o', replace => 'foo' }
+            }
+        );
+        my @fields_652a =
+          read_field( { record => $record, field => '652', subfield => 'a' } );
+        is_deeply(
+            \@fields_652a,
+            [ 'Cfoomputer programming.', 'Cfoomputer algorithms.' ],
+            'Copy and replace field using regex (same as copy)'
+        );
+
+        copy_and_replace_field(
+            {
+                record        => $record,
+                from_field    => '650',
+                from_subfield => 'a',
+                to_field      => '653',
+                to_subfield   => 'a',
+                regex => { search => 'o', replace => 'foo', modifiers => 'g' }
+            }
+        );
+        my @fields_653a =
+          read_field( { record => $record, field => '653', subfield => 'a' } );
+        is_deeply(
+            \@fields_653a,
+            [ 'Cfoomputer prfoogramming.', 'Cfoomputer algfoorithms.' ],
+            'Copy and replace field using regex (same as copy)'
+        );
+
+        copy_and_replace_field(
+            {
+                record        => $record,
+                from_field    => '650',
+                from_subfield => 'a',
+                to_field      => '654',
+                to_subfield   => 'a',
+                regex => { search => 'O', replace => 'foo', modifiers => 'i' }
+            }
+        );
+        my @fields_654a =
+          read_field( { record => $record, field => '654', subfield => 'a' } );
+        is_deeply(
+            \@fields_654a,
+            [ 'Cfoomputer programming.', 'Cfoomputer algorithms.' ],
+            'Copy and replace field using regex (same as copy)'
+        );
+
+        copy_and_replace_field(
+            {
+                record        => $record,
+                from_field    => '650',
+                from_subfield => 'a',
+                to_field      => '655',
+                to_subfield   => 'a',
+                regex => { search => 'O', replace => 'foo', modifiers => 'gi' }
+            }
+        );
+        my @fields_655a =
+          read_field( { record => $record, field => '655', subfield => 'a' } );
+        is_deeply(
+            \@fields_655a,
+            [ 'Cfoomputer prfoogramming.', 'Cfoomputer algfoorithms.' ],
+            'Copy and replace field using regex (same as copy)'
+        );
+
+        $record = new_record;
+        $record->append_fields(
+            MARC::Field->new(
+                952, ' ', ' ',
+                p => '3010023917',
+                y => 'BK',
+            ),
+        );
+
+        copy_and_replace_field(
+            {
+                record        => $record,
+                from_field    => '952',
+                from_subfield => 'd',
+                to_field      => '952',
+                to_subfield   => 'd'
+            }
+        );
+        my @fields_952d =
+          read_field( { record => $record, field => '952', subfield => 'd' } );
+        is_deeply(
+            \@fields_952d,
+            [ '2001-06-25', '2001-06-25' ],
+            'copy and replace 952$d into others 952 field'
+        );
+
+        copy_and_replace_field(
+            {
+                record        => $record,
+                from_field    => '111',
+                from_subfield => '1',
+                to_field      => '999',
+                to_subfield   => '9'
+            }
+        );
+        my @fields_9999 =
+          read_field( { record => $record, field => '999', subfield => '9' } );
+        is_deeply( \@fields_9999, [],
+            'copy and replace a nonexistent subfield does not create a new one (same as copy)' );
+
+        $record = new_record;
+        copy_and_replace_field(
+            {
+                record        => $record,
+                from_field    => 245,
+                from_subfield => 'a',
+                to_field      => 245,
+                to_subfield   => 'a',
+                regex         => { search => '^', replace => 'BEGIN ' }
+            }
+        );
+        # This is the same as update the subfield
+        is_deeply(
+            [
+                read_field(
+                    { record => $record, field => '245', subfield => 'a' }
+                )
+            ],
+            ['BEGIN The art of computer programming'],
+            'Copy and replace - Update a subfield: add a string at the beginning'
+        );
+
+        $record = new_record;
+        copy_and_replace_field(
+            {
+                record        => $record,
+                from_field    => 245,
+                from_subfield => 'a',
+                to_field      => 245,
+                to_subfield   => 'a',
+                regex         => { search => '$', replace => ' END' }
+            }
+        );
+        # This is the same as update the subfield
+        is_deeply(
+            [
+                read_field(
+                    { record => $record, field => '245', subfield => 'a' }
+                )
+            ],
+            ['The art of computer programming END'],
+            'Copy and replace - Update a subfield: add a string at the end'
+        );
+
+        $record = new_record;
+        copy_and_replace_field(
+            {
+                record        => $record,
+                from_field    => 245,
+                from_subfield => 'c',
+                to_field      => 650,
+                to_subfield   => 'c',
+            }
+        );
+
+        is_deeply(
+            [
+                read_field(
+                    { record => $record, field => '650' }
+                )
+            ],
+            [ 'Computer programming.', '462', 'Donald E. Knuth.' ],
+            'Copy and replace a subfield to an existent field but inexistent subfield (same as copy)'
+        );
+
+        $record = new_record;
+        copy_and_replace_field(
+            {
+                record        => $record,
+                from_field    => 245,
+                from_subfield => 'c',
+                to_field      => 650,
+                to_subfield   => '9',
+            }
+        );
+
+        is_deeply(
+            [
+                read_field(
+                    { record => $record, field => '650' }
+                )
+            ],
+            [ 'Computer programming.', 'Donald E. Knuth.' ],
+            'Copy and replace a subfield to an existent field / subfield, the origin subfield is replaced'
+        );
+    };
+
+    subtest 'copy and replace field' => sub {
+        plan tests => 14;
+        my $record = new_record;
+        $record->append_fields(
+            MARC::Field->new(
+                952, ' ', ' ',
+                p => '3010023918',
+                y => 'CD',
+            ),
+        );
+
+        #- copy all fields
+        copy_and_replace_field(
+            { record => $record, from_field => '952', to_field => '953' } );
+        my @fields_952 = read_field( { record => $record, field => '952' } );
+        is_deeply(
+            [
+                read_field(
+                    { record => $record, field => '952', field_numbers => [1] }
+                )
+            ],
+            [ '3010023917', 'BK', 'GEN', '2001-06-25' ],
+            "copy all: original first field still exists (same as copy)"
+        );
+        is_deeply(
+            [
+                read_field(
+                    { record => $record, field => '952', field_numbers => [2] }
+                )
+            ],
+            [ '3010023918', 'CD' ],
+            "copy all: original second field still exists (same as copy)"
+        );
+        is_deeply(
+            [
+                read_field(
+                    { record => $record, field => '953', field_numbers => [1] }
+                )
+            ],
+            [ '3010023917', 'BK', 'GEN', '2001-06-25' ],
+            "copy all: first original fields has been copied (same as copy)"
+        );
+        is_deeply(
+            [
+                read_field(
+                    { record => $record, field => '953', field_numbers => [2] }
+                )
+            ],
+            [ '3010023918', 'CD' ],
+            "copy all: second original fields has been copied (same as copy)"
+        );
+
+        #- copy only the first field
+        copy_and_replace_field(
+            {
+                record        => $record,
+                from_field    => '953',
+                to_field      => '954',
+                field_numbers => [1]
+            }
+        );
+        is_deeply(
+            [
+                read_field(
+                    { record => $record, field => '953', field_numbers => [1] }
+                )
+            ],
+            [ '3010023917', 'BK', 'GEN', '2001-06-25' ],
+            "copy and replace first: first original fields has been copied (same as copy)"
+        );
+        is_deeply(
+            [
+                read_field(
+                    { record => $record, field => '953', field_numbers => [2] }
+                )
+            ],
+            [ '3010023918', 'CD' ],
+            "copy and replace first: second original fields has been copied (same as copy)"
+        );
+        is_deeply(
+            [ read_field( { record => $record, field => '954' } ) ],
+            [ '3010023917', 'BK', 'GEN', '2001-06-25' ],
+            "copy and replace first: only first, first 953 has been copied (same as copy)"
+        );
+
+        $record = new_record;
+        $record->append_fields(
+            MARC::Field->new(
+                952, ' ', ' ',
+                p => '3010023918',
+                y => 'CD',
+            ),
+        );
+
+        #- copy and replace all fields and modify values using a regex
+        copy_and_replace_field(
+            {
+                record     => $record,
+                from_field => '952',
+                to_field   => '953',
+                regex      => { search => '30100', replace => '42424' }
+            }
+        );
+        is_deeply(
+            [
+                read_field(
+                    { record => $record, field => '952', field_numbers => [1] }
+                )
+            ],
+            [ '3010023917', 'BK', 'GEN', '2001-06-25' ],
+            "copy and replace all with regex: original first field still exists (same as copy)"
+        );
+        is_deeply(
+            [
+                read_field(
+                    { record => $record, field => '952', field_numbers => [2] }
+                )
+            ],
+            [ '3010023918', 'CD' ],
+            "copy and replace all with regex: original second field still exists (same as copy)"
+        );
+        is_deeply(
+            [
+                read_field(
+                    { record => $record, field => '953', field_numbers => [1] }
+                )
+            ],
+            [ '4242423917', 'BK', 'GEN', '2001-06-25' ],
+            "copy and replace all with regex: first original fields has been copied (same as copy)"
+        );
+        is_deeply(
+            [
+                read_field(
+                    { record => $record, field => '953', field_numbers => [2] }
+                )
+            ],
+            [ '4242423918', 'CD' ],
+            "copy and replace all with regex: second original fields has been copied (same as copy)"
+        );
+        copy_and_replace_field(
+            {
+                record     => $record,
+                from_field => '111',
+                to_field   => '999',
+            }
+        );
+        my @fields_9999 =
+          read_field( { record => $record, field => '999', subfield => '9' } );
+        is_deeply( \@fields_9999, [],
+            'copy and replace a nonexistent field does not create a new one (same as copy)' );
+
+        $record = new_record;
+        copy_and_replace_field(
+            {
+                record        => $record,
+                from_field    => 245,
+                to_field      => 650,
+            }
+        );
+
+        is_deeply(
+            [
+                read_field(
+                    { record => $record, field => '650', field_numbers => [1] }
+                )
+            ],
+            [ 'The art of computer programming', 'Donald E. Knuth.' ],
+            'Copy and replace to an existent field should erase the original field'
+        );
+        is_deeply(
+            [
+                read_field(
+                    { record => $record, field => '650', field_numbers => [2] }
+                )
+            ],
+            [],
+            'Copy and replace to an existent field should not create a new field'
+        );
+    };
+};
+
+# move_field - subfields
+subtest 'move_field' => sub {
+    plan tests              => 2;
+    subtest 'move subfield' => sub {
+        plan tests => 7;
+        my $record = new_record;
+        my ( @fields_952d, @fields_952c, @fields_954c, @fields_954p );
+        $record->append_fields(
+            MARC::Field->new(
+                952, ' ', ' ',
+                p => '3010023917',
+                y => 'BK',
+            ),
+        );
+        move_field(
+            {
+                record        => $record,
+                from_field    => '952',
+                from_subfield => 'c',
+                to_field      => '954',
+                to_subfield   => 'c'
+            }
+        );
+        @fields_952c =
+          read_field( { record => $record, field => '952', subfield => 'c' } );
+        @fields_954c =
+          read_field( { record => $record, field => '954', subfield => 'c' } );
+        is_deeply( \@fields_952c, [],      'The 952$c has moved' );
+        is_deeply( \@fields_954c, ['GEN'], 'Now 954$c exists' );
+
+        move_field(
+            {
+                record        => $record,
                 from_field    => '952',
                 from_subfield => 'p',
                 to_field      => '954',
                 to_subfield   => 'p',
-                n             => 1
+                field_numbers => [1]
             }
         );    # Move the first field
         my @fields_952p =
@@ -726,10 +1506,24 @@ subtest 'move_field' => sub {
             [ '3010023917', '3010023917' ],
             'Now 2 954$p exist'
         );
+
+        move_field(
+            {
+                record        => $record,
+                from_field    => '111',
+                from_subfield => '1',
+                to_field      => '999',
+                to_subfield   => '9'
+            }
+        );
+        my @fields_9999 =
+          read_field( { record => $record, field => '999', subfield => '9' } );
+        is_deeply( \@fields_9999, [],
+            'move a nonexistent subfield does not create a new one' );
     };
 
     subtest 'move field' => sub {
-        plan tests => 8;
+        plan tests => 9;
 
         # move_field - fields
         my $record = new_record;
@@ -747,12 +1541,20 @@ subtest 'move_field' => sub {
         is_deeply( [ read_field( { record => $record, field => '952' } ) ],
             [], "original fields don't exist" );
         is_deeply(
-            [ read_field( { record => $record, field => '953', n => 1 } ) ],
+            [
+                read_field(
+                    { record => $record, field => '953', field_numbers => [1] }
+                )
+            ],
             [ '3010023917', 'BK', 'GEN', '2001-06-25' ],
             "first original fields has been copied"
         );
         is_deeply(
-            [ read_field( { record => $record, field => '953', n => 2 } ) ],
+            [
+                read_field(
+                    { record => $record, field => '953', field_numbers => [2] }
+                )
+            ],
             [ '3010023917', 'BK' ],
             "second original fields has been copied"
         );
@@ -760,10 +1562,10 @@ subtest 'move_field' => sub {
         #- Move only the first field
         move_field(
             {
-                record     => $record,
-                from_field => '953',
-                to_field   => '954',
-                n          => 1
+                record        => $record,
+                from_field    => '953',
+                to_field      => '954',
+                field_numbers => [1]
             }
         );
         is_deeply(
@@ -798,15 +1600,36 @@ subtest 'move_field' => sub {
         is_deeply( [ read_field( { record => $record, field => '952' } ) ],
             [], "use a regex, original fields don't exist" );
         is_deeply(
-            [ read_field( { record => $record, field => '953', n => 1 } ) ],
+            [
+                read_field(
+                    { record => $record, field => '953', field_numbers => [1] }
+                )
+            ],
             [ '3010023917', 'DVD', 'GEN', '2001-06-25' ],
             "use a regex, first original fields has been copied"
         );
         is_deeply(
-            [ read_field( { record => $record, field => '953', n => 2 } ) ],
+            [
+                read_field(
+                    { record => $record, field => '953', field_numbers => [2] }
+                )
+            ],
             [ '3010023917', 'DVD' ],
             "use a regex, second original fields has been copied"
         );
+
+        move_field(
+            {
+                record     => $record,
+                from_field => '111',
+                to_field   => '999',
+            }
+        );
+        my @fields_9999 =
+          read_field( { record => $record, field => '999', subfield => '9' } );
+        is_deeply( \@fields_9999, [],
+            'move a nonexistent field does not create a new one' );
+
     };
 };
 
@@ -825,7 +1648,13 @@ subtest 'delete_field' => sub {
         );
 
         delete_field(
-            { record => $record, field => '952', subfield => 'p', n => 1 } );
+            {
+                record        => $record,
+                field         => '952',
+                subfield      => 'p',
+                field_numbers => [1]
+            }
+        );
         my @fields_952p =
           read_field( { record => $record, field => '952', subfield => 'p' } );
         is_deeply( \@fields_952p, ['3010023917'], 'Delete first 952$p' );