assembly-image 1.7.8 → 2.0.0

data/spec/image_spec.rb

@@ -1,45 +1,25 @@
 # frozen_string_literal: true
 
 require 'spec_helper'
+require 'fileutils'
 
 RSpec.describe Assembly::Image do
-  let(:ai) { described_class.new(input_path) }
+  let(:assembly_image) { described_class.new(input_path) }
   let(:input_path) { TEST_TIF_INPUT_FILE }
+  let(:jp2_output_file) { File.join(TEST_OUTPUT_DIR, File.basename(input_path).gsub('.tif', '.jp2')) }
 
-  after do
-    # after each test, empty out the input and output test directories
-    remove_files(TEST_INPUT_DIR)
-    remove_files(TEST_OUTPUT_DIR)
-  end
+  before { cleanup }
 
   describe '#jp2_filename' do
     it 'indicates the default jp2 filename' do
-      expect(ai.jp2_filename).to eq TEST_TIF_INPUT_FILE.gsub('.tif', '.jp2')
+      expect(assembly_image.jp2_filename).to eq input_path.gsub('.tif', '.jp2')
     end
 
     context 'with a file with no extension' do
       let(:input_path) { '/path/to/a/file_with_no_extension' }
 
       it 'indicates the default jp2 filename' do
-        expect(ai.jp2_filename).to eq '/path/to/a/file_with_no_extension.jp2'
-      end
-    end
-  end
-
-  describe '#dpg_jp2_filename' do
-    context 'with a dpg tiff file' do
-      let(:input_path) { TEST_DPG_TIF_INPUT_FILE }
-
-      it 'indicates the default DPG jp2 filename' do
-        expect(ai.dpg_jp2_filename).to eq TEST_DPG_TIF_INPUT_FILE.gsub('.tif', '.jp2').gsub('_00_', '_05_')
-      end
-    end
-
-    context 'with a file with no extension' do
-      let(:input_path) { '/path/to/a/file_with_no_00_extension' }
-
-      it 'indicates the default jp2 filename' do
-        expect(ai.dpg_jp2_filename).to eq '/path/to/a/file_with_no_05_extension.jp2'
+        expect(assembly_image.jp2_filename).to eq '/path/to/a/file_with_no_extension.jp2'
       end
     end
   end
@@ -49,178 +29,243 @@ RSpec.describe Assembly::Image do
       let(:input_path) { '' }
 
       it 'does not run if no input file is passed in' do
-        expect { ai.create_jp2 }.to raise_error
+        expect { assembly_image.create_jp2 }.to raise_error(RuntimeError)
       end
     end
 
     context 'when given an uncompressed compressed RGB tif with more than 4GB of image data', skip: 'This test will create a 4GB test image and a 4GB temporary image, so skipping by default.' do
+      let(:input_path) { File.join(TEST_INPUT_DIR, 'rgb.tif') }
+
       before do
-        generate_test_image(TEST_TIF_INPUT_FILE, compress: 'none', width: '37838', height: '37838')
+        generate_test_image(input_path, compress: 'none', width: '37838', height: '37838')
       end
 
       it 'creates the jp2 with a temp file' do
-        expect(File).to exist TEST_TIF_INPUT_FILE
-        expect(File).not_to exist TEST_JP2_OUTPUT_FILE
-        result = ai.create_jp2(output: TEST_JP2_OUTPUT_FILE)
-        expect(ai.tmp_path).not_to be_nil
+        expect(File).to exist input_path
+        expect(File).not_to exist jp2_output_file
+        result = assembly_image.create_jp2(output: jp2_output_file)
+        expect(assembly_image.tmp_path).not_to be_nil
         expect(result).to be_a_kind_of described_class
-        expect(result.path).to eq TEST_JP2_OUTPUT_FILE
-        expect(TEST_JP2_OUTPUT_FILE).to be_a_jp2
+        expect(result.path).to eq jp2_output_file
+        expect(jp2_output_file).to be_a_jp2
         expect(result.exif.colorspace).to eq 'sRGB'
-        jp2 = described_class.new(TEST_JP2_OUTPUT_FILE)
-        expect(jp2.height).to eq 37_838
-        expect(jp2.width).to eq 37_838
+        expect(result.height).to eq 37_838
+        expect(result.width).to eq 37_838
       end
     end
 
     context 'when given an LZW compressed RGB tif with more than 4GB of image data', skip: 'This test will create a 4GB temporary image, so skipping by default.' do
+      let(:input_path) { File.join(TEST_INPUT_DIR, 'lzw.tif') }
+
       before do
-        generate_test_image(TEST_TIF_INPUT_FILE, compress: 'lzw', width: '37838', height: '37838')
+        generate_test_image(input_path, compress: 'lzw', width: '37838', height: '37838')
       end
 
       it 'creates the jp2 with a temp file' do
-        expect(File).to exist TEST_TIF_INPUT_FILE
-        expect(File).not_to exist TEST_JP2_OUTPUT_FILE
-        result = ai.create_jp2(output: TEST_JP2_OUTPUT_FILE)
-        expect(ai.tmp_path).not_to be_nil
+        expect(File).to exist input_path
+        expect(File).not_to exist jp2_output_file
+        expect(assembly_image.exif.samplesperpixel).to be 3
+        expect(assembly_image.exif.bitspersample).to eql '8 8 8'
+        expect(assembly_image).to be_a_valid_image
+        expect(assembly_image).to be_jp2abl
+        result = assembly_image.create_jp2(output: jp2_output_file)
+        expect(assembly_image.tmp_path).not_to be_nil
         expect(result).to be_a_kind_of described_class
-        expect(result.path).to eq TEST_JP2_OUTPUT_FILE
-        expect(TEST_JP2_OUTPUT_FILE).to be_a_jp2
+        expect(result.path).to eq jp2_output_file
+        expect(jp2_output_file).to be_a_jp2
         expect(result.exif.colorspace).to eq 'sRGB'
-        jp2 = described_class.new(TEST_JP2_OUTPUT_FILE)
-        expect(jp2.height).to eq 37_838
-        expect(jp2.width).to eq 37_838
+        expect(result.height).to eq 37_838
+        expect(result.width).to eq 37_838
       end
     end
 
     context 'when given a bitonal tif' do
+      let(:input_path) { File.join(TEST_INPUT_DIR, 'bitonal.tif') }
+
       before do
-        # Need to force group4 compression to get ImageMagick to create bitonal tiff
-        generate_test_image(TEST_TIF_INPUT_FILE, image_type: 'Bilevel', compress: 'group4')
+        generate_test_image(input_path, color: 'bin', bands: 1, depth: 1)
       end
 
-      it 'creates grayscale jp2' do
-        expect(File).to exist TEST_TIF_INPUT_FILE
-        expect(File).not_to exist TEST_JP2_OUTPUT_FILE
-        result = ai.create_jp2(output: TEST_JP2_OUTPUT_FILE)
-        expect(TEST_JP2_OUTPUT_FILE).to be_a_jp2
+      it 'creates valid jp2' do
+        expect(File).to exist input_path
+        expect(File).not_to exist jp2_output_file
+        expect(assembly_image.exif.samplesperpixel).to be 1
+        expect(assembly_image.exif.bitspersample).to be 1
+        expect(assembly_image).not_to have_color_profile
+        result = assembly_image.create_jp2(output: jp2_output_file)
+        expect(result).to be_a_kind_of described_class
+        expect(result.path).to eq jp2_output_file
+        expect(jp2_output_file).to be_a_jp2
         expect(result.exif.colorspace).to eq 'Grayscale'
       end
     end
 
     context 'when given a color tif but bitonal image data (1 channels and 1 bits per pixel)' do
+      let(:input_path) { File.join(TEST_INPUT_DIR, 'color.tif') }
+
       before do
-        generate_test_image(TEST_TIF_INPUT_FILE, color: 'Bilevel', image_type: 'TrueColor', profile: '')
+        generate_test_image(input_path, color: 'bin', bands: 3)
       end
 
       it 'creates color jp2' do
-        expect(File).to exist TEST_TIF_INPUT_FILE
-        expect(File).not_to exist TEST_JP2_OUTPUT_FILE
-        expect(ai).not_to have_color_profile
-        result = ai.create_jp2(output: TEST_JP2_OUTPUT_FILE)
-        expect(TEST_JP2_OUTPUT_FILE).to be_a_jp2
+        expect(File).to exist input_path
+        expect(File).not_to exist jp2_output_file
+        expect(assembly_image).not_to have_color_profile
+        expect(assembly_image.exif.samplesperpixel).to be 3
+        expect(assembly_image.exif.bitspersample).to eql '8 8 8'
+        expect(assembly_image).to be_a_valid_image
+        expect(assembly_image).to be_jp2able
+        result = assembly_image.create_jp2(output: jp2_output_file)
+        expect(result).to be_a_kind_of described_class
+        expect(result.path).to eq jp2_output_file
+        expect(jp2_output_file).to be_a_jp2
         expect(result.exif.colorspace).to eq 'sRGB'
       end
     end
 
     context 'when given a graycale tif but with bitonal image data (1 channel and 1 bits per pixel)' do
+      let(:input_path) { File.join(TEST_INPUT_DIR, 'gray.tif') }
+
       before do
-        generate_test_image(TEST_TIF_INPUT_FILE, color: 'Bilevel', image_type: 'Grayscale', profile: '')
+        generate_test_image(input_path, color: 'grey', bands: 1)
       end
 
       it 'creates grayscale jp2' do
-        expect(File).to exist TEST_TIF_INPUT_FILE
-        expect(File).not_to exist TEST_JP2_OUTPUT_FILE
-        expect(ai).not_to have_color_profile
-        result = ai.create_jp2(output: TEST_JP2_OUTPUT_FILE)
-        expect(TEST_JP2_OUTPUT_FILE).to be_a_jp2
+        expect(File).to exist input_path
+        expect(File).not_to exist jp2_output_file
+        expect(assembly_image).not_to have_color_profile
+        expect(assembly_image.exif.samplesperpixel).to be 1
+        expect(assembly_image.exif.bitspersample).to be 8
+        expect(assembly_image).to be_a_valid_image
+        expect(assembly_image).to be_jp2able
+        result = assembly_image.create_jp2(output: jp2_output_file)
+        expect(jp2_output_file).to be_a_jp2
+        expect(result).to be_a_kind_of described_class
+        expect(result.path).to eq jp2_output_file
         expect(result.exif.colorspace).to eq 'Grayscale'
       end
     end
 
     context 'when given a color tif but with greyscale image data (1 channel and 8 bits per pixel)' do
+      let(:input_path) { File.join(TEST_INPUT_DIR, 'color_gray.tif') }
+
       before do
-        generate_test_image(TEST_TIF_INPUT_FILE, color: 'Grayscale', image_type: 'TrueColor', profile: '')
+        generate_test_image(input_path, color: 'grey')
       end
 
       it 'creates color jp2' do
-        expect(File).to exist TEST_TIF_INPUT_FILE
-        expect(File).not_to exist TEST_JP2_OUTPUT_FILE
-        expect(ai).not_to have_color_profile
-        result = ai.create_jp2(output: TEST_JP2_OUTPUT_FILE)
-        expect(TEST_JP2_OUTPUT_FILE).to be_a_jp2
+        expect(File).to exist input_path
+        expect(File).not_to exist jp2_output_file
+        expect(assembly_image.exif.samplesperpixel).to be 3
+        expect(assembly_image.exif.bitspersample).to eql '8 8 8'
+        expect(assembly_image).to be_a_valid_image
+        expect(assembly_image).to be_jp2able
+        expect(assembly_image).not_to have_color_profile
+        result = assembly_image.create_jp2(output: jp2_output_file)
+        expect(result).to be_a_kind_of described_class
+        expect(result.path).to eq jp2_output_file
+        expect(jp2_output_file).to be_a_jp2
         expect(result.exif.colorspace).to eq 'sRGB'
       end
     end
 
+    context 'when given a cmyk tif' do
+      let(:input_path) { File.join(TEST_INPUT_DIR, 'cmky.tif') }
+
+      before do
+        generate_test_image(input_path, color: 'cmyk', cg_type: 'cmyk', profile: 'cmyk', bands: 4)
+      end
+
+      it 'creates an srgb jp2', skip: 'Need to verify the color space is correct in jp2' do
+        expect(File).to exist input_path
+        expect(File).not_to exist jp2_output_file
+        expect(assembly_image.exif.samplesperpixel).to be 4
+        expect(assembly_image.exif.bitspersample).to eql '8 8 8 8'
+        expect(assembly_image).to be_a_valid_image
+        expect(assembly_image).to be_jp2able
+        expect(assembly_image).to have_color_profile
+        result = assembly_image.create_jp2(output: jp2_output_file)
+        expect(result).to be_a_kind_of described_class
+        expect(result.path).to eq jp2_output_file
+        expect(jp2_output_file).to be_a_jp2
+        # note, we verify the CMYK has been converted to an SRGB JP2 correctly by using ruby-vips instead of exif, since exif does not correctly
+        #  identify the color space...note: this line current does not work in circleci, potentially due to libvips version differences
+        expect(Vips::Image.new_from_file(jp2_output_file).get_value('interpretation')).to eq :srgb
+      end
+    end
+
     context 'when the source image has no profile' do
+      let(:input_path) { File.join(TEST_INPUT_DIR, 'no_profile.tif') }
+
       before do
-        generate_test_image(TEST_TIF_INPUT_FILE, profile: '') # generate a test input with no profile
+        generate_test_image(input_path)
       end
 
       it 'creates a jp2' do
-        expect(File).to exist TEST_TIF_INPUT_FILE
-        expect(File).not_to exist TEST_JP2_OUTPUT_FILE
-        expect(ai).not_to have_color_profile
-        expect(ai).to be_a_valid_image
-        expect(ai).to be_jp2able
-        ai.create_jp2(output: TEST_JP2_OUTPUT_FILE)
-        expect(TEST_JP2_OUTPUT_FILE).to be_a_jp2
+        expect(File).to exist input_path
+        expect(File).not_to exist jp2_output_file
+        expect(assembly_image.exif.samplesperpixel).to be 3
+        expect(assembly_image.exif.bitspersample).to eql '8 8 8'
+        expect(assembly_image).not_to have_color_profile
+        expect(assembly_image).to be_a_valid_image
+        expect(assembly_image).to be_jp2able
+        assembly_image.create_jp2(output: jp2_output_file)
+        expect(jp2_output_file).to be_a_jp2
       end
     end
 
     context "when the output file exists and you don't allow overwriting" do
       before do
-        generate_test_image(TEST_TIF_INPUT_FILE)
-        generate_test_image(TEST_JP2_OUTPUT_FILE)
+        generate_test_image(input_path)
+        FileUtils.touch(jp2_output_file) # just need a file with this name, don't care what
       end
 
       it 'does not run' do
-        expect(File).to exist TEST_TIF_INPUT_FILE
-        expect(File).to exist TEST_JP2_OUTPUT_FILE
-        expect { ai.create_jp2(output: TEST_JP2_OUTPUT_FILE) }.to raise_error(SecurityError)
+        expect(File).to exist input_path
+        expect(File).to exist jp2_output_file
+        expect { assembly_image.create_jp2(output: jp2_output_file) }.to raise_error(SecurityError)
       end
     end
 
     context 'when given a test tiff' do
       before do
-        generate_test_image(TEST_TIF_INPUT_FILE)
+        generate_test_image(input_path)
       end
 
       it 'gets the correct image height and width' do
-        expect(ai.height).to eq 100
-        expect(ai.width).to eq 100
+        expect(assembly_image.height).to eq 36
+        expect(assembly_image.width).to eq 43
       end
     end
 
     context 'when the input file is a jp2' do
       before do
-        generate_test_image(TEST_TIF_INPUT_FILE, profile: '') # generate a test input with no profile
+        generate_test_image(input_path)
       end
 
       it 'does not run' do
-        expect(File).to exist TEST_TIF_INPUT_FILE
-        expect(File).not_to exist TEST_JP2_OUTPUT_FILE
-        expect(ai).not_to have_color_profile
-        expect(ai).to be_a_valid_image
-        expect(ai).to be_jp2able
-        ai.create_jp2(output: TEST_JP2_OUTPUT_FILE)
-        expect(TEST_JP2_OUTPUT_FILE).to be_a_jp2
-        jp2_file = described_class.new(TEST_JP2_OUTPUT_FILE)
+        expect(File).to exist input_path
+        expect(File).not_to exist jp2_output_file
+        expect(assembly_image).not_to have_color_profile
+        expect(assembly_image).to be_a_valid_image
+        expect(assembly_image).to be_jp2able
+        assembly_image.create_jp2(output: jp2_output_file)
+        expect(jp2_output_file).to be_a_jp2
+        jp2_file = described_class.new(jp2_output_file)
         expect(jp2_file).to be_valid_image
         expect(jp2_file).not_to be_jp2able
-        expect { jp2_file.create_jp2 }.to raise_error
+        expect { jp2_file.create_jp2 }.to raise_error(RuntimeError)
       end
     end
 
     context 'when you specify a bogus output profile' do
       before do
-        generate_test_image(TEST_TIF_INPUT_FILE)
+        generate_test_image(input_path)
       end
 
       it 'runs, because this is not currently an option' do
-        expect(File).to exist TEST_TIF_INPUT_FILE
-        result = ai.create_jp2(output_profile: 'bogusness')
+        expect(File).to exist input_path
+        result = assembly_image.create_jp2(output_profile: 'bogusness')
         expect(result).to be_a_kind_of described_class
         expect(result.path).to eq TEST_JP2_INPUT_FILE
         expect(TEST_JP2_INPUT_FILE).to be_a_jp2
@@ -239,19 +284,19 @@ RSpec.describe Assembly::Image do
         bogus_folder = '/crapsticks'
         expect(File).to exist TEST_JPEG_INPUT_FILE
         expect(File).not_to exist bogus_folder
-        expect { ai.create_jp2(tmp_folder: bogus_folder) }.to raise_error
+        expect { assembly_image.create_jp2(tmp_folder: bogus_folder) }.to raise_error(RuntimeError)
       end
     end
 
     context 'when no output file is specified' do
       before do
-        generate_test_image(TEST_TIF_INPUT_FILE)
+        generate_test_image(input_path)
       end
 
       it 'creates a jp2 of the same filename and in the same location as the input and cleans up the tmp file' do
-        expect(File).to exist TEST_TIF_INPUT_FILE
+        expect(File).to exist input_path
         expect(File.exist?(TEST_JP2_INPUT_FILE)).to be false
-        result = ai.create_jp2
+        result = assembly_image.create_jp2
         expect(result).to be_a_kind_of described_class
         expect(result.path).to eq TEST_JP2_INPUT_FILE
         expect(TEST_JP2_INPUT_FILE).to be_a_jp2
@@ -261,17 +306,17 @@ RSpec.describe Assembly::Image do
 
     context 'when the output file exists and you allow overwriting' do
       before do
-        generate_test_image(TEST_TIF_INPUT_FILE)
-        generate_test_image(TEST_JP2_OUTPUT_FILE)
+        generate_test_image(input_path)
+        FileUtils.touch(jp2_output_file) # just need a file with this name, don't care what
       end
 
       it 'recreates jp2' do
-        expect(File).to exist TEST_TIF_INPUT_FILE
-        expect(File).to exist TEST_JP2_OUTPUT_FILE
-        result = ai.create_jp2(output: TEST_JP2_OUTPUT_FILE, overwrite: true)
+        expect(File).to exist input_path
+        expect(File).to exist jp2_output_file
+        result = assembly_image.create_jp2(output: jp2_output_file, overwrite: true)
         expect(result).to be_a_kind_of described_class
-        expect(result.path).to eq TEST_JP2_OUTPUT_FILE
-        expect(TEST_JP2_OUTPUT_FILE).to be_a_jp2
+        expect(result.path).to eq jp2_output_file
+        expect(jp2_output_file).to be_a_jp2
         expect(result.exif.colorspace).to eq 'sRGB'
       end
     end

data/spec/images_spec.rb

@@ -3,45 +3,45 @@
 require 'spec_helper'
 
 RSpec.describe Assembly::Images do
-  after do
-    # after each test, empty out the input and output test directories
-    remove_files(TEST_INPUT_DIR)
-    remove_files(TEST_OUTPUT_DIR)
+  before { cleanup }
+
+  describe '#batch_generate_jp2' do
+    it 'does not run if no input folder is passed in' do
+      expect{ described_class.batch_generate_jp2('') }.to raise_error(RuntimeError)
+    end
+
+    it 'does not run if a non-existent input folder is passed in' do
+      expect{ described_class.batch_generate_jp2('/junk/path') }.to raise_error(RuntimeError)
+    end
+
+    it 'runs and batch produces jp2s from input tiffs' do
+      ['test1', 'test2', 'test3'].each { |image| generate_test_image(File.join(TEST_INPUT_DIR, "#{image}.tif"), profile: 'AdobeRGB1998') }
+      described_class.batch_generate_jp2(TEST_INPUT_DIR, output: TEST_OUTPUT_DIR)
+      expect(File.directory?(TEST_OUTPUT_DIR)).to be true
+      ['test1', 'test2', 'test3'].each { |image| expect(File.join(TEST_OUTPUT_DIR, "#{image}.jp2")).to be_a_jp2 }
+    end
   end
 
-  it 'does not run if no input folder is passed in' do
-    expect{ described_class.batch_generate_jp2('') }.to raise_error
-  end
-
-  it 'does not run if a non-existent input folder is passed in' do
-    expect{ described_class.batch_generate_jp2('/junk/path') }.to raise_error
-  end
-
-  it 'runs and batch produces jp2s from input tiffs' do
-    ['test1', 'test2', 'test3'].each { |image| generate_test_image(File.join(TEST_INPUT_DIR, "#{image}.tif")) }
-    described_class.batch_generate_jp2(TEST_INPUT_DIR, output: TEST_OUTPUT_DIR)
-    expect(File.directory?(TEST_OUTPUT_DIR)).to be true
-    ['test1', 'test2', 'test3'].each { |image| expect(File.join(TEST_OUTPUT_DIR, "#{image}.jp2")).to be_a_jp2 }
-  end
-
-  it 'runs and batch add color profile descriptions input tiffs with no color profile descriptions' do
-    ['test1', 'test2', 'test3'].each { |image| generate_test_image(File.join(TEST_INPUT_DIR, "#{image}.tif"), profile: '') }
-    ['test1', 'test2', 'test3'].each { |image| expect(Assembly::Image.new(File.join(TEST_INPUT_DIR, "#{image}.tif")).exif.profiledescription).to be_nil }
-    described_class.batch_add_exif_profile_descr(TEST_INPUT_DIR, 'Adobe RGB 1998')
-    ['test1', 'test2', 'test3'].each { |image| expect(Assembly::Image.new(File.join(TEST_INPUT_DIR, "#{image}.tif")).exif.profiledescription).to eq 'Adobe RGB (1998)' }
-  end
-
-  it 'runs and batch add color profile descriptions input tiffs, forcing over existing color profile descriptions' do
-    ['test1', 'test2', 'test3'].each { |image| generate_test_image(File.join(TEST_INPUT_DIR, "#{image}.tif")) }
-    ['test1', 'test2', 'test3'].each { |image| expect(Assembly::Image.new(File.join(TEST_INPUT_DIR, "#{image}.tif")).exif.profiledescription).to eq 'sRGB IEC61966-2.1' }
-    described_class.batch_add_exif_profile_descr(TEST_INPUT_DIR, 'Adobe RGB 1998', force: true) # force overwrite
-    ['test1', 'test2', 'test3'].each { |image| expect(Assembly::Image.new(File.join(TEST_INPUT_DIR, "#{image}.tif")).exif.profiledescription).to eq 'Adobe RGB (1998)' }
-  end
-
-  it 'runs and batch add color profile descriptions input tiffs, not overwriting existing color profile descriptions' do
-    ['test1', 'test2', 'test3'].each { |image| generate_test_image(File.join(TEST_INPUT_DIR, "#{image}.tif")) }
-    ['test1', 'test2', 'test3'].each { |image| expect(Assembly::Image.new(File.join(TEST_INPUT_DIR, "#{image}.tif")).exif.profiledescription).to eq 'sRGB IEC61966-2.1' }
-    described_class.batch_add_exif_profile_descr(TEST_INPUT_DIR, 'Adobe RGB 1998') # do not force overwrite
-    ['test1', 'test2', 'test3'].each { |image| expect(Assembly::Image.new(File.join(TEST_INPUT_DIR, "#{image}.tif")).exif.profiledescription).to eq 'sRGB IEC61966-2.1' }
+  describe '#batch_add_exif_profile_descr' do
+    it 'runs and batch adds color profile descriptions to input tiffs that had no color profile descriptions' do
+      ['test1', 'test2', 'test3'].each { |image| generate_test_image(File.join(TEST_INPUT_DIR, "#{image}.tif")) }
+      ['test1', 'test2', 'test3'].each { |image| expect(Assembly::Image.new(File.join(TEST_INPUT_DIR, "#{image}.tif")).exif.profiledescription).to be_nil }
+      described_class.batch_add_exif_profile_descr(TEST_INPUT_DIR, 'Adobe RGB 1998')
+      ['test1', 'test2', 'test3'].each { |image| expect(Assembly::Image.new(File.join(TEST_INPUT_DIR, "#{image}.tif")).exif.profiledescription).to eq 'Adobe RGB (1998)' }
+    end
+
+    it 'runs and batch adds color profile descriptions to input tiffs, forcing over existing color profile descriptions' do
+      ['test1', 'test2', 'test3'].each { |image| generate_test_image(File.join(TEST_INPUT_DIR, "#{image}.tif"), profile: 'sRGBIEC6196621') }
+      ['test1', 'test2', 'test3'].each { |image| expect(Assembly::Image.new(File.join(TEST_INPUT_DIR, "#{image}.tif")).exif.profiledescription).to eq 'sRGB IEC61966-2.1' }
+      described_class.batch_add_exif_profile_descr(TEST_INPUT_DIR, 'Adobe RGB 1998', force: true) # force overwrite
+      ['test1', 'test2', 'test3'].each { |image| expect(Assembly::Image.new(File.join(TEST_INPUT_DIR, "#{image}.tif")).exif.profiledescription).to eq 'Adobe RGB (1998)' }
+    end
+
+    it 'runs and batch adds color profile descriptions to input tiffs, not overwriting existing color profile descriptions' do
+      ['test1', 'test2', 'test3'].each { |image| generate_test_image(File.join(TEST_INPUT_DIR, "#{image}.tif"), profile: 'sRGBIEC6196621') }
+      ['test1', 'test2', 'test3'].each { |image| expect(Assembly::Image.new(File.join(TEST_INPUT_DIR, "#{image}.tif")).exif.profiledescription).to eq 'sRGB IEC61966-2.1' }
+      described_class.batch_add_exif_profile_descr(TEST_INPUT_DIR, 'Adobe RGB 1998') # do not force overwrite
+      ['test1', 'test2', 'test3'].each { |image| expect(Assembly::Image.new(File.join(TEST_INPUT_DIR, "#{image}.tif")).exif.profiledescription).to eq 'sRGB IEC61966-2.1' }
+    end
   end
 end