Bug 32351: Fix t/db_dependent/Koha/Charges/Sales.t
[koha-ffzg.git] / t / SimpleMARC.t
old mode 100644 (file)
new mode 100755 (executable)
index 880653b..56a92fb
@@ -1,16 +1,21 @@
+#!/usr/bin/perl
+
 use Modern::Perl;
 
-use Test::More tests => 9;
+use Test::More tests => 11;
 
 use_ok("MARC::Field");
 use_ok("MARC::Record");
-use_ok("Koha::SimpleMARC");
+use_ok("Koha::SimpleMARC", qw( field_exists read_field update_field copy_field copy_and_replace_field move_field delete_field field_equals ));
 
 sub new_record {
     my $record = MARC::Record->new;
     $record->leader('03174nam a2200445 a 4500');
     my @fields = (
         MARC::Field->new(
+            '008', '120829t20132012nyu bk 001 0ceng',
+        ),
+        MARC::Field->new(
             100, '1', ' ',
             a => 'Knuth, Donald Ervin',
             d => '1938',
@@ -71,13 +76,17 @@ subtest 'field_exists' => sub {
 subtest 'read_field' => sub {
     plan tests              => 2;
     subtest 'read subfield' => sub {
-        plan tests => 5;
+        plan tests => 6;
         my $record = new_record;
         $record->append_fields(
             MARC::Field->new(
                 650, ' ', '0',
                 a => 'Computer algorithms.',
                 9 => '463',
+            ),
+            MARC::Field->new(
+                600, ' ', '0',
+                0 => '123456',
             )
         );
 
@@ -127,6 +136,20 @@ subtest 'read_field' => sub {
             [],
             'There is no 3 650$a'
         );
+        is_deeply(
+            [
+                read_field(
+                    {
+                        record        => $record,
+                        field         => '600',
+                        subfield      => '0',
+                        field_numbers => [1]
+                    }
+                )
+            ],
+            ['123456'],
+            'first 600$0'
+        );
     };
     subtest 'read field' => sub {
         plan tests => 4;
@@ -188,7 +211,7 @@ subtest 'read_field' => sub {
 subtest 'update_field' => sub {
     plan tests                => 1;
     subtest 'update subfield' => sub {
-        plan tests => 5;
+        plan tests => 6;
         my $record = new_record;
 
         update_field(
@@ -247,6 +270,7 @@ subtest 'update_field' => sub {
                 952, ' ', ' ',
                 p => '3010023917',
                 y => 'BK',
+                0 => '123456',
             ),
         );
         update_field(
@@ -280,6 +304,23 @@ subtest 'update_field' => sub {
             [ '3010023917', '3010023918' ],
             'update all subfields 952$p with the different values'
         );
+
+        update_field(
+            {
+                record   => $record,
+                field    => '952',
+                subfield => '0',
+                values   => [ '654321' ]
+            }
+        );
+        my @fields_9520 =
+          read_field( { record => $record, field => '952', subfield => '0' } );
+        is_deeply(
+            \@fields_9520,
+            [ '654321', '654321' ],
+            'update all subfields 952$0 with the same value'
+        );
+
     };
 };
 
@@ -287,7 +328,7 @@ subtest 'update_field' => sub {
 subtest 'copy_field' => sub {
     plan tests              => 2;
     subtest 'copy subfield' => sub {
-        plan tests => 20;
+        plan tests => 21;
         my $record = new_record;
         $record->append_fields(
             MARC::Field->new(
@@ -638,6 +679,27 @@ subtest 'copy_field' => sub {
             {
                 record        => $record,
                 from_field    => 245,
+                from_subfield => 'a',
+                to_field      => 245,
+                to_subfield   => 'a',
+                regex         => { search => '(art)', replace => 'sm$1 $1' }
+            }
+        );
+        is_deeply(
+            [
+                read_field(
+                    { record => $record, field => '245', subfield => 'a' }
+                )
+            ],
+            ['The art of computer programming', 'The smart art of computer programming'],
+            'Update a subfield: use capture groups'
+        );
+
+        $record = new_record;
+        copy_field(
+            {
+                record        => $record,
+                from_field    => 245,
                 from_subfield => 'c',
                 to_field      => 650,
                 to_subfield   => 'c',
@@ -804,7 +866,7 @@ subtest 'copy_field' => sub {
                 )
             ],
             [ '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(
             [
@@ -857,176 +919,772 @@ subtest 'copy_field' => sub {
     };
 };
 
-# 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 => 7;
+    subtest 'copy and replace subfield' => sub {
+        plan tests => 20;
         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' );
-
-        move_field(
-            {
-                record        => $record,
-                from_field    => '952',
-                from_subfield => 'p',
-                to_field      => '954',
-                to_subfield   => 'p',
-                field_numbers => [1]
-            }
-        );    # Move the first field
-        my @fields_952p =
-          read_field( { record => $record, field => '952', subfield => 'p' } );
-        @fields_954p =
-          read_field( { record => $record, field => '954', subfield => 'p' } );
-        is_deeply( \@fields_952p, ['3010023917'], 'One of 952$p has moved' );
-        is_deeply( \@fields_954p, ['3010023917'], 'Now 954$p 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)'
+        );
 
         $record = new_record;
         $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 => 'p',
-                to_field      => '954',
-                to_subfield   => 'p'
+                from_field    => '650',
+                from_subfield => 'a',
+                to_field      => '651',
+                to_subfield   => 'a'
             }
-        );    # Move all field
-        @fields_952p =
-          read_field( { record => $record, field => '952', subfield => 'p' } );
-        @fields_954p =
-          read_field( { record => $record, field => '954', subfield => 'p' } );
-        is_deeply( \@fields_952p, [], 'All 952$p have moved' );
+        );
+        my @fields_651a =
+          read_field( { record => $record, field => '651', subfield => 'a' } );
         is_deeply(
-            \@fields_954p,
-            [ '3010023917', '3010023917' ],
-            'Now 2 954$p exist'
+            \@fields_651a,
+            [ 'Computer programming.', 'Computer algorithms.' ],
+            'Copy and replace multivalued field (same as copy)'
         );
+        delete_field( { record => $record, field => '651' } );
 
-        move_field(
+        copy_and_replace_field(
             {
                 record        => $record,
-                from_field    => '111',
-                from_subfield => '1',
-                to_field      => '999',
-                to_subfield   => '9'
+                from_field    => '650',
+                from_subfield => 'a',
+                to_field      => '651',
+                to_subfield   => 'a',
+                field_numbers => [1]
             }
         );
-        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 => 9;
-
-        # move_field - fields
-        my $record = new_record;
-        $record->append_fields(
-            MARC::Field->new(
-                952, ' ', ' ',
-                p => '3010023917',
-                y => 'BK',
-            ),
-        );
-
-        #- Move all fields
-        move_field(
-            { record => $record, from_field => '952', to_field => '953' } );
-        is_deeply( [ read_field( { record => $record, field => '952' } ) ],
-            [], "original fields don't exist" );
         is_deeply(
             [
                 read_field(
-                    { record => $record, field => '953', field_numbers => [1] }
+                    { record => $record, field => '651', subfield => 'a' }
                 )
             ],
-            [ '3010023917', 'BK', 'GEN', '2001-06-25' ],
-            "first original fields has been copied"
+            ['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 => '953', field_numbers => [2] }
+                    { record => $record, field => '651', subfield => 'a' }
                 )
             ],
-            [ '3010023917', 'BK' ],
-            "second original fields has been copied"
+            ['Computer algorithms.'],
+            'Copy and replace second field 650$a should erase 651$a'
         );
+        delete_field( { record => $record, field => '651' } );
 
-        #- Move only the first field
-        move_field(
+        copy_and_replace_field(
             {
                 record        => $record,
-                from_field    => '953',
-                to_field      => '954',
-                field_numbers => [1]
+                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(
-            [ read_field( { record => $record, field => '953' } ) ],
-            [ '3010023917', 'BK' ],
-            "only first, the second 953 still exists"
-        );
-        is_deeply(
-            [ read_field( { record => $record, field => '954' } ) ],
-            [ '3010023917', 'BK', 'GEN', '2001-06-25' ],
-            "only first, first 953 has been copied"
+            \@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' } );
 
-        $record = new_record;
-        $record->append_fields(
-            MARC::Field->new(
-                952, ' ', ' ',
-                p => '3010023917',
-                y => 'BK',
-            ),
+        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' } );
 
-        #- Move all fields and modify values using a regex
-        move_field(
+        copy_and_replace_field(
             {
-                record     => $record,
-                from_field => '952',
-                to_field   => '953',
-                regex      => { search => 'BK', replace => 'DVD' }
+                record        => $record,
+                from_field    => '650',
+                from_subfield => 'a',
+                to_field      => '651',
+                to_subfield   => 'a',
+                regex => { search => 'Computer', replace => 'The art of' }
             }
         );
-        is_deeply( [ read_field( { record => $record, field => '952' } ) ],
-            [], "use a regex, original fields don't exist" );
+        @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 => 'a',
+                to_field      => 245,
+                to_subfield   => 'a',
+                regex         => { search => '(art)', replace => 'sm$1 $1' }
+            }
+        );
+        # This is the same as update the subfield
+        is_deeply(
+            [
+                read_field(
+                    { record => $record, field => '245', subfield => 'a' }
+                )
+            ],
+            ['The smart art of computer programming'],
+            'Copy and replace - Update a subfield: use capture groups'
+        );
+
+        $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',
+                field_numbers => [1]
+            }
+        );    # Move the first field
+        my @fields_952p =
+          read_field( { record => $record, field => '952', subfield => 'p' } );
+        @fields_954p =
+          read_field( { record => $record, field => '954', subfield => 'p' } );
+        is_deeply( \@fields_952p, ['3010023917'], 'One of 952$p has moved' );
+        is_deeply( \@fields_954p, ['3010023917'], 'Now 954$p exists' );
+
+        $record = new_record;
+        $record->append_fields(
+            MARC::Field->new(
+                952, ' ', ' ',
+                p => '3010023917',
+                y => 'BK',
+            ),
+        );
+
+        move_field(
+            {
+                record        => $record,
+                from_field    => '952',
+                from_subfield => 'p',
+                to_field      => '954',
+                to_subfield   => 'p'
+            }
+        );    # Move all field
+        @fields_952p =
+          read_field( { record => $record, field => '952', subfield => 'p' } );
+        @fields_954p =
+          read_field( { record => $record, field => '954', subfield => 'p' } );
+        is_deeply( \@fields_952p, [], 'All 952$p have moved' );
+        is_deeply(
+            \@fields_954p,
+            [ '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 => 9;
+
+        # move_field - fields
+        my $record = new_record;
+        $record->append_fields(
+            MARC::Field->new(
+                952, ' ', ' ',
+                p => '3010023917',
+                y => 'BK',
+            ),
+        );
+
+        #- Move all fields
+        move_field(
+            { record => $record, from_field => '952', to_field => '953' } );
+        is_deeply( [ read_field( { record => $record, field => '952' } ) ],
+            [], "original fields don't exist" );
+        is_deeply(
+            [
+                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', field_numbers => [2] }
+                )
+            ],
+            [ '3010023917', 'BK' ],
+            "second original fields has been copied"
+        );
+
+        #- Move only the first field
+        move_field(
+            {
+                record        => $record,
+                from_field    => '953',
+                to_field      => '954',
+                field_numbers => [1]
+            }
+        );
+        is_deeply(
+            [ read_field( { record => $record, field => '953' } ) ],
+            [ '3010023917', 'BK' ],
+            "only first, the second 953 still exists"
+        );
+        is_deeply(
+            [ read_field( { record => $record, field => '954' } ) ],
+            [ '3010023917', 'BK', 'GEN', '2001-06-25' ],
+            "only first, first 953 has been copied"
+        );
+
+        $record = new_record;
+        $record->append_fields(
+            MARC::Field->new(
+                952, ' ', ' ',
+                p => '3010023917',
+                y => 'BK',
+            ),
+        );
+
+        #- Move all fields and modify values using a regex
+        move_field(
+            {
+                record     => $record,
+                from_field => '952',
+                to_field   => '953',
+                regex      => { search => 'BK', replace => 'DVD' }
+            }
+        );
+        is_deeply( [ read_field( { record => $record, field => '952' } ) ],
+            [], "use a regex, original fields don't exist" );
+        is_deeply(
+            [
                 read_field(
                     { record => $record, field => '953', field_numbers => [1] }
                 )
@@ -1063,7 +1721,7 @@ subtest 'move_field' => sub {
 subtest 'delete_field' => sub {
     plan tests                => 2;
     subtest 'delete subfield' => sub {
-        plan tests => 2;
+        plan tests => 3;
         my $record = new_record;
         $record->append_fields(
             MARC::Field->new(
@@ -1097,6 +1755,18 @@ subtest 'delete_field' => sub {
         @fields_952p =
           read_field( { record => $record, field => '952', subfield => 'p' } );
         is_deeply( \@fields_952p, [], 'Delete all 952$p' );
+
+        $record = new_record;
+        $record->append_fields(
+            MARC::Field->new(
+                600, ' ', ' ',
+                a => 'Murakami, Haruki',
+                0 => 'https://id.loc.gov/authorities/names/n81152393.html',
+            ),
+        );
+        delete_field( { record => $record, field => '600', subfield => '0' } );
+        my @fields_600 = read_field( { record => $record, field => '600' } );
+        is_deeply( \@fields_600, ['Murakami, Haruki'], 'Delete all 600$0, only subfield 0 deleted' );
     };
 
     subtest 'delete field' => sub {
@@ -1119,3 +1789,47 @@ subtest 'delete_field' => sub {
         is_deeply( \@fields_952, [], 'Delete all 952, 2 deleted' );
     };
 };
+
+subtest 'field_equals' => sub {
+    plan tests => 2;
+    my $record = new_record;
+    subtest 'standard MARC fields' => sub {
+        plan tests => 2;
+        my $match = Koha::SimpleMARC::field_equals({
+                record => $record,
+                value => 'Donald',
+                field => '100',
+                subfield => 'a',
+            });
+        is_deeply( $match, [], '100$a not equal to "Donald"' );
+
+        $match = Koha::SimpleMARC::field_equals({
+                record => $record,
+                value => 'Donald',
+                field => '100',
+                subfield => 'a',
+                is_regex => 1,
+            });
+        is_deeply( $match, [1], 'first 100$a matches "Donald"');
+    };
+
+    subtest 'control fields' => sub {
+        plan tests => 2;
+        my $match = Koha::SimpleMARC::field_equals({
+                record => $record,
+                value => 'eng',
+                field => '008',
+                subfield => '',
+            });
+        is_deeply( $match, [], '008 control field not equal to "eng"' );
+
+        $match = Koha::SimpleMARC::field_equals({
+                record => $record,
+                value => 'eng',
+                field => '008',
+                subfield => '',
+                is_regex => 1,
+            });
+        is_deeply( $match, [1], 'first 008 control field matches "eng"' );
+    };
+};