defmastership 1.0.6 → 1.0.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2cd7a521f197f1ab374863af89ab7a24c50bc27d5dfef6161bc47e8fff27825f
-  data.tar.gz: a92ef2ee968e39a48e20fbf2a7d6f3040393c727a422e4f39c935978a29e187f
+  metadata.gz: 0341f43f016456f897b3382a1a8356dd7dc5d35b37059bccfd254f5cd9ccf65d
+  data.tar.gz: 442ebad69c76c21f8bfb972bd53d7ce83e6431795026f7b4abc760a70b450bf7
 SHA512:
-  metadata.gz: ee7d58f8392a43fe5b45e234f9c4965037e2864a37a669c191d33b58016cf7094a99fe29443c3fc60678bab3e591b19188579771d14aa762fba8d4b34b01bd89
-  data.tar.gz: 526ae0002d2f5d66f2ad555d09bc8cf9786b567098e6bc071a8b119b2cbfe6b664e84eaf3fc68e453ca16f41552a4004f23bab6323ed62cba526322f27e88b15
+  metadata.gz: 7106dd56ff07b88ca64e67f3dbd62b46f09a078dff05a955f80a5a1473ca3ff918ff038346a9e598c8355de761e1e5dcc1270db811eadacb875d51de57032b09
+  data.tar.gz: 719a47a9bb86aca473565693a645a85b72ade6036a2ed1ee47cdafc00c85640c52a747e2394d59e8f2ed08803406d49f03914a1b5d74026d20ae01de56b8c474

data/bin/defmastership

@@ -2,7 +2,6 @@
 # Copyright (c) 2020 Jerome Arbez-Gindre
 # frozen_string_literal: true
 
-require('asciidoctor')
 require('csv')
 require('defmastership')
 require('gli')
@@ -52,18 +51,24 @@ module DefMastership
     arg_name 'asciidoctor_files'
     command :export, :exp, :e do |c|
       c.flag(%i[separator sep s], default_value: ',', desc: 'CSV separator')
+      c.switch(%i[no-fail], desc: 'Exit succes even in case of wrong explicit checksum')
+
       c.action do |_global_options, options, args|
-        doc = Asciidoctor.load_file(args[0], safe: :unsafe, parse: false)
-        # FIXME: also escape ifdef, ifndef, ifeval and endif directives
-        # FIXME: do this more carefully by reading line by line; if input
-        # differs by output by leading backslash, restore original line
-        lines = doc.reader.read_lines
         my_doc = DefMastership::Document.new
-        my_doc.parse(lines)
+        my_doc.parse_file_with_preprocessor(args[0])
 
         output_file = args[0].sub(/\.adoc$/, '.csv')
 
         DefMastership::CSVFormatter.new(my_doc, options['separator']).export_to(output_file)
+
+        if my_doc.wrong_explicit_checksum?
+          my_doc.definitions.each do |definition|
+            next if definition.wrong_explicit_checksum.nil?
+
+            warn("WARNING: #{definition.reference} has a wrong explicit checksum (should be #{definition.sha256})")
+          end
+          exit 1 unless options[:"no-fail"]
+        end
       end
     end
 

data/features/checksum.feature

@@ -0,0 +1,244 @@
+Feature: checksum
+  As a Responsible of definitions for a given document
+  In order to detect modifications of defintiions
+  I want defmastership to export and update checksum in definitions
+
+  Scenario: Extract one definition with WRONG explicit checksum to CSV 
+    Given a file named "thedoc.adoc" with:
+    """
+    [define, requirement, TOTO-0001(babe1234)]
+    def one
+    """
+    When I run `defmastership export thedoc.adoc`
+    Then it should fail with:
+    """
+    WARNING: TOTO-0001 has a wrong explicit checksum (should be d27cb5c3)
+    """
+    And the file "thedoc.csv" should contain:
+    """
+    Type,Reference,Value,sha256,Wrong explicit checksum
+    requirement,TOTO-0001,def one,d27cb5c3,babe1234
+    """
+    And the stdout should not contain anything
+
+  Scenario: Extract two definitions with one WRONG explicit checksum to CSV 
+    Given a file named "thedoc.adoc" with:
+    """
+    [define, requirement, TOTO-0001(d27cb5c3)]
+    def one
+
+    [define, requirement, TOTO-0002(babe1234)]
+    def two
+    """
+    When I run `defmastership export thedoc.adoc`
+    Then it should fail with:
+    """
+    WARNING: TOTO-0002 has a wrong explicit checksum (should be b80e1be5)
+    """
+    Then the file "thedoc.csv" should contain:
+    """
+    Type,Reference,Value,sha256,Wrong explicit checksum
+    requirement,TOTO-0001,def one,d27cb5c3,\"\"
+    requirement,TOTO-0002,def two,b80e1be5,babe1234
+    """
+    And the stdout should not contain anything
+
+  Scenario: Export not failing even if WRONG explicit checksum 
+    Given a file named "thedoc.adoc" with:
+    """
+    [define, requirement, TOTO-0001(d27cb5c3)]
+    def one
+
+    [define, requirement, TOTO-0002(babe1234)]
+    def two
+    """
+    When I successfully run `defmastership export --no-fail thedoc.adoc`
+    Then the file "thedoc.csv" should contain:
+    """
+    Type,Reference,Value,sha256,Wrong explicit checksum
+    requirement,TOTO-0001,def one,d27cb5c3,\"\"
+    requirement,TOTO-0002,def two,b80e1be5,babe1234
+    """
+    And the stdout should not contain anything
+    And the stderr should contain:
+    """
+    WARNING: TOTO-0002 has a wrong explicit checksum (should be b80e1be5)
+    """
+
+  Scenario: Extract one definition with explicit checksum to CSV 
+    Given a file named "thedoc.adoc" with:
+    """
+    [define, requirement, TOTO-0001(b86dcbde)]
+    --
+    Exemple of multiline requirement.
+    Second line.
+    --
+    """
+    When I successfully run `defmastership export thedoc.adoc`
+    Then the file "thedoc.csv" should contain:
+    """
+    Type,Reference,Value,sha256
+    requirement,TOTO-0001,"Exemple of multiline requirement.
+    Second line.",b86dcbde
+    """
+    And the stdout should not contain anything
+    And the stderr should not contain anything
+
+  Scenario: Change one modifier
+    Given a file named "modifications.yml" with:
+    """
+    ---
+    :update_requirement_checksum:
+      :type: update_def_checksum
+      :config:
+        :def_type: requirement
+    """
+    And a file named "thedoc.adoc" with:
+    """
+    [define, requirement, TOTO-TEMP-XXX1]
+    --
+    the requirement value.
+    --
+    """
+    When I successfully run `defmastership modify --modifications update_requirement_checksum thedoc.adoc`
+    Then the stdout should not contain anything
+    And the stderr should not contain anything
+    And the file "thedoc.adoc" should contain:
+    """
+    [define, requirement, TOTO-TEMP-XXX1(244eed18)]
+    --
+    the requirement value.
+    --
+    """
+
+  Scenario: Change one modifier with already set checksum
+    Given a file named "modifications.yml" with:
+    """
+    ---
+    :update_requirement_checksum:
+      :type: update_def_checksum
+      :config:
+        :def_type: requirement
+    """
+    And a file named "thedoc.adoc" with:
+    """
+    [define, requirement, TOTO-TEMP-XXX1(abcd1234)]
+    --
+    the requirement value.
+    --
+    """
+    When I successfully run `defmastership modify --modifications update_requirement_checksum thedoc.adoc`
+    Then the stdout should not contain anything
+    And the stderr should not contain anything
+    And the file "thedoc.adoc" should contain:
+    """
+    [define, requirement, TOTO-TEMP-XXX1(244eed18)]
+    --
+    the requirement value.
+    --
+    """
+
+  Scenario: Checksum is calculated on included file for export
+    Given a file named "thedoc.adoc" with:
+    """
+    [define, requirement, TOTO-0001]
+    --
+    include::included.txt[]
+    --
+    """
+    And a file named "included.txt" with:
+    """
+    Exemple of multiline requirement.
+    Second line.
+    """
+    When I successfully run `defmastership export thedoc.adoc`
+    Then the file "thedoc.csv" should contain:
+    """
+    Type,Reference,Value,sha256
+    requirement,TOTO-0001,"Exemple of multiline requirement.
+    Second line.",b86dcbde
+    """
+    And the stdout should not contain anything
+    And the stderr should not contain anything
+
+  Scenario: Checksum is calculated on included file in ref modifcation
+    Given a file named "modifications.yml" with:
+    """
+    ---
+    :update_requirement_checksum:
+      :type: update_def_checksum
+      :config:
+        :def_type: requirement
+    """
+    And a file named "thedoc.adoc" with:
+    """
+    [define, requirement, TOTO-0001]
+    --
+    include::included.txt[]
+    --
+    """
+    And a file named "included.txt" with:
+    """
+    Exemple of multiline requirement.
+    Second line.
+    """
+    When I successfully run `defmastership modify --modifications update_requirement_checksum thedoc.adoc`
+    Then the file "thedoc.adoc" should contain:
+    """
+    [define, requirement, TOTO-0001(b86dcbde)]
+    --
+    include::included.txt[]
+    --
+    """
+    And the stdout should not contain anything
+    And the stderr should not contain anything
+
+  Scenario: Checksum take into account variables in CSV export
+    Given a file named "thedoc.adoc" with:
+    """
+    :variable: multiline
+    [define, requirement, TOTO-0001]
+    --
+    Exemple of {variable} requirement.
+    Second line.
+    --
+    """
+    When I successfully run `defmastership export thedoc.adoc`
+    Then the file "thedoc.csv" should contain:
+    """
+    Type,Reference,Value,sha256
+    requirement,TOTO-0001,"Exemple of multiline requirement.
+    Second line.",b86dcbde
+    """
+    And the stdout should not contain anything
+    And the stderr should not contain anything
+
+  Scenario: Checksum take into account variables in ref modifcation
+    Given a file named "modifications.yml" with:
+    """
+    ---
+    :update_requirement_checksum:
+      :type: update_def_checksum
+      :config:
+        :def_type: requirement
+    """
+    And a file named "thedoc.adoc" with:
+    """
+    :variable: multiline
+    [define, requirement, TOTO-0001]
+    --
+    Exemple of {variable} requirement.
+    Second line.
+    --
+    """
+    When I successfully run `defmastership modify --modifications update_requirement_checksum thedoc.adoc`
+    Then the file "thedoc.adoc" should contain:
+    """
+    [define, requirement, TOTO-0001(b86dcbde)]
+    --
+    Exemple of {variable} requirement.
+    Second line.
+    --
+    """
+    And the stdout should not contain anything
+    And the stderr should not contain anything

data/features/export.feature

@@ -22,59 +22,6 @@ Feature: The extract command
     And the stdout should not contain anything
     And the stderr should not contain anything
 
-  Scenario: Extract one definition with WRONG explicit checksum to CSV 
-    Given a file named "toto.adoc" with:
-    """
-    [define, requirement, TOTO-0001(babe1234)]
-    def one
-    """
-    When I successfully run `defmastership export toto.adoc`
-    Then the file "toto.csv" should contain:
-    """
-    Type,Reference,Value,sha256,Wrong explicit checksum
-    requirement,TOTO-0001,def one,d27cb5c3,babe1234
-    """
-    And the stdout should not contain anything
-    And the stderr should not contain anything
-
-  Scenario: Extract two definitions with one WRONG explicit checksum to CSV 
-    Given a file named "toto.adoc" with:
-    """
-    [define, requirement, TOTO-0001(d27cb5c3)]
-    def one
-
-    [define, requirement, TOTO-0002(babe1234)]
-    def two
-    """
-    When I successfully run `defmastership export toto.adoc`
-    Then the file "toto.csv" should contain:
-    """
-    Type,Reference,Value,sha256,Wrong explicit checksum
-    requirement,TOTO-0001,def one,d27cb5c3,\"\"
-    requirement,TOTO-0002,def two,b80e1be5,babe1234
-    """
-    And the stdout should not contain anything
-    And the stderr should not contain anything
-
-  Scenario: Extract one definition with explicit checksum to CSV 
-    Given a file named "toto.adoc" with:
-    """
-    [define, requirement, TOTO-0001(b86dcbde)]
-    --
-    Exemple of multiline requirement.
-    Second line.
-    --
-    """
-    When I successfully run `defmastership export toto.adoc`
-    Then the file "toto.csv" should contain:
-    """
-    Type,Reference,Value,sha256
-    requirement,TOTO-0001,"Exemple of multiline requirement.
-    Second line.",b86dcbde
-    """
-    And the stdout should not contain anything
-    And the stderr should not contain anything
-
   Scenario: Extract to CSV with alternative CSV separator
     Given a file named "toto.adoc" with:
     """

data/lib/defmastership.rb

@@ -22,3 +22,6 @@ require('defmastership/change_ref_line_modifier')
 
 require('defmastership/rename_included_files_modifier')
 require('defmastership/rename_included_files_line_modifier')
+
+require('defmastership/update_def_checksum_modifier')
+require('defmastership/update_def_checksum_line_modifier')

data/lib/defmastership/change_ref_modifier.rb

@@ -8,8 +8,8 @@ module DefMastership
       %i[replace_refdef replace_irefs]
     end
 
-    def line_modifier_class
-      ChangeRefLineModifier
+    def new_line_modifier(config, _adoc_texts)
+      ChangeRefLineModifier.from_config(config)
     end
   end
 end

data/lib/defmastership/document.rb

@@ -1,6 +1,8 @@
 # Copyright (c) 2020 Jerome Arbez-Gindre
 # frozen_string_literal: true
 
+require('asciidoctor')
+
 module DefMastership
   # Contains the content of a DefMastership document: mainly definitions
 
@@ -20,6 +22,22 @@ module DefMastership
       @definition_parser = DefinitionParser.new(self)
     end
 
+    def parse(lines)
+      lines.reject(&:commented?).each do |line|
+        (@in_literal ? FILTERS : FILTERS_IN_LITERAL).each do |filter|
+          next unless line.match(filter.regexp)
+
+          line = generate_event(filter.event, Regexp.last_match, line)
+
+          break if filter.consumed_line
+        end
+      end
+    end
+
+    def parse_file_with_preprocessor(adoc_file)
+      parse(Asciidoctor.load_file(adoc_file, safe: :unsafe, parse: false).reader.read_lines)
+    end
+
     def wrong_explicit_checksum?
       @definitions.reduce(false) do |res, definition|
         res || !definition.wrong_explicit_checksum.nil?
@@ -38,6 +56,10 @@ module DefMastership
       line
     end
 
+    def ref_to_def(ref)
+      @definitions.find { |definition| definition.reference == ref }
+    end
+
     def add_line(_match, line)
       @definitions.last << line
       line
@@ -96,18 +118,6 @@ module DefMastership
       new_line
     end
 
-    def parse(lines)
-      lines.reject(&:commented?).each do |line|
-        (@in_literal ? FILTERS : FILTERS_IN_LITERAL).each do |filter|
-          next unless line.match(filter.regexp)
-
-          line = generate_event(filter.event, Regexp.last_match, line)
-
-          break if filter.consumed_line
-        end
-      end
-    end
-
     private
 
     def generate_event(event, match, line)

data/lib/defmastership/modifier_base.rb

@@ -12,7 +12,7 @@ module DefMastership
     end
 
     def do_modifications(adoc_texts)
-      line_modifier = line_modifier_class.from_config(@config)
+      line_modifier = new_line_modifier(@config, adoc_texts)
 
       adoc_texts =
         replacements.reduce(adoc_texts) do |texts, method|

data/lib/defmastership/rename_included_files_modifier.rb

@@ -4,27 +4,12 @@
 module DefMastership
   # Change included filenames
   class RenameIncludedFilesModifier < ModifierBase
-    def do_modifications(adoc_texts)
-      line_modifier = line_modifier_class.from_config(@config)
-
-      adoc_texts =
-        replacements.reduce(adoc_texts) do |texts, method|
-          texts.transform_values do |text|
-            text.lines.map { |line| line_modifier.public_send(method, line) }.join
-          end
-        end
-
-      @config = line_modifier.config
-      @changes = line_modifier.changes
-      adoc_texts
-    end
-
     def replacements
       %i[replace]
     end
 
-    def line_modifier_class
-      RenameIncludedFilesLineModifier
+    def new_line_modifier(config, _adoc_texts)
+      RenameIncludedFilesLineModifier.from_config(config)
     end
   end
 end

data/lib/defmastership/update_def_checksum_line_modifier.rb

@@ -0,0 +1,39 @@
+# Copyright (c) 2020 Jerome Arbez-Gindre
+# frozen_string_literal: true
+
+module DefMastership
+  # modify one line after another
+  class UpdateDefChecksumLineModifier < LineModifierBase
+    attr_accessor :document
+
+    def self.from_config(hash)
+      new.from_config(hash)
+    end
+
+    def initialize
+      super
+      @config = {
+        def_type: ''
+      }
+    end
+
+    def replace(line)
+      match = matched?(line)
+
+      return line unless match
+      return line unless match[:type] == def_type
+
+      explicit_checksum_str = match[:explicit_checksum].nil? ? '' : "(#{match[:explicit_checksum]})"
+      line.gsub("#{match[:reference]}#{explicit_checksum_str}") do
+        "#{match[:reference]}(#{@document.ref_to_def(match[:reference]).sha256})"
+      end
+    end
+
+    def matched?(line)
+      return if line.commented?
+      return unless line =~ DMRegexp::DEFINITION
+
+      Regexp.last_match
+    end
+  end
+end

data/lib/defmastership/update_def_checksum_modifier.rb

@@ -0,0 +1,21 @@
+# Copyright (c) 2020 Jerome Arbez-Gindre
+# frozen_string_literal: true
+
+module DefMastership
+  # Update definition checksum wih calculated one
+  class UpdateDefChecksumModifier < ModifierBase
+    def replacements
+      %i[replace]
+    end
+
+    def new_line_modifier(config, adoc_texts)
+      document = Document.new
+      adoc_texts.each do |adoc_file, _|
+        document.parse_file_with_preprocessor(adoc_file)
+      end
+      line_modifier = UpdateDefChecksumLineModifier.from_config(config)
+      line_modifier.document = document
+      line_modifier
+    end
+  end
+end

data/lib/defmastership/version.rb

@@ -2,6 +2,6 @@
 # frozen_string_literal: true
 
 module DefMastership
-  VERSION = '1.0.6'
+  VERSION = '1.0.7'
   public_constant :VERSION
 end

data/spec/unit/defmastership/document_spec.rb

@@ -426,9 +426,38 @@ RSpec.describe(DefMastership::Document) do
     end
   end
 
-  describe '#wrong_explicit_checksum?' do
-    subject(:document) { described_class.new }
+  describe '#parse_file_with_preprocessor' do
+    let(:definition)  { instance_double(DefMastership::Definition, 'definition') }
+    let(:input_lines) { ['[define, requirement, TOTO-0001]']                     }
+    let(:adoc_doc)    { instance_double(Asciidoctor::Document, 'adoc_doc')       }
+    let(:adoc_reader) { instance_double(Asciidoctor::Reader, 'adoc_reader')      }
+
+    before do
+      allow(Asciidoctor).to(
+        receive(:load_file).with('the_file.adoc', { parse: false, safe: :unsafe }).and_return(adoc_doc)
+      )
+      allow(adoc_doc).to(receive(:reader).and_return(adoc_reader))
+      allow(adoc_reader).to(receive(:read_lines).and_return(input_lines))
+      allow(DefMastership::Definition).to(receive(:new).and_return(definition))
+      allow(definition).to(receive(:<<).and_return(definition))
+      allow(definition).to(receive(:labels).and_return(Set.new))
+      document.parse_file_with_preprocessor('the_file.adoc')
+    end
+
+    it { expect(Asciidoctor).to(have_received(:load_file).with('the_file.adoc', { parse: false, safe: :unsafe })) }
+    it { expect(adoc_doc).to(have_received(:reader).with(no_args)) }
+    it { expect(adoc_reader).to(have_received(:read_lines).with(no_args)) }
+
+    it do
+      expect(DefMastership::Definition).to(
+        have_received(:new).with(
+          matchdata_including(type: 'requirement', reference: 'TOTO-0001')
+        )
+      )
+    end
+  end
 
+  describe '#wrong_explicit_checksum?' do
     let(:def1) { instance_double(DefMastership::Definition, 'definition') }
     let(:def2) { instance_double(DefMastership::Definition, 'definition') }
     let(:input_lines) do
@@ -473,4 +502,32 @@ RSpec.describe(DefMastership::Document) do
       it { expect(document.wrong_explicit_checksum?).to(eq(true)) }
     end
   end
+
+  describe '#ref_to_def?' do
+    let(:def1) { instance_double(DefMastership::Definition, 'definition') }
+    let(:def2) { instance_double(DefMastership::Definition, 'definition') }
+    let(:input_lines) do
+      [
+        '[define, requirement, TOTO-0001]',
+        'def one',
+        '',
+        '[define, requirement, TOTO-0002(1234)]',
+        'def two'
+      ]
+    end
+
+    before do
+      allow(DefMastership::Definition).to(receive(:new).twice.and_return(def1, def2))
+      allow(def1).to(receive(:labels)).and_return([])
+      allow(def2).to(receive(:labels)).and_return([])
+      allow(def1).to(receive(:<<).and_return(def1))
+      allow(def2).to(receive(:<<).and_return(def2))
+      allow(def1).to(receive(:reference).and_return('TOTO-0001'))
+      allow(def2).to(receive(:reference).and_return('TOTO-0002'))
+      document.parse(input_lines)
+    end
+
+    it { expect(document.ref_to_def('TOTO-0001')).to(eq(def1)) }
+    it { expect(document.ref_to_def('TOTO-0002')).to(eq(def2)) }
+  end
 end

data/spec/unit/defmastership/update_def_checksum_line_modifier_spec.rb

@@ -0,0 +1,68 @@
+# Copyright (c) 2020 Jerome Arbez-Gindre
+# frozen_string_literal: true
+
+require('defmastership')
+
+RSpec.describe(DefMastership::UpdateDefChecksumLineModifier) do
+  subject(:linemodifier) { described_class.new }
+
+  describe '.new' do
+    it { is_expected.not_to(be(nil)) }
+    it { is_expected.to(have_attributes(def_type: '')) }
+    it { is_expected.to(have_attributes(changes:  [])) }
+    it { expect { linemodifier.user_defined_attribute }.to(raise_error(NoMethodError)) }
+  end
+
+  describe '.from_config' do
+    subject(:linemodifier) do
+      described_class.from_config(
+        def_type: 'requirement'
+      )
+    end
+
+    it { is_expected.not_to(be(nil)) }
+    it { is_expected.to(have_attributes(def_type: 'requirement')) }
+    it { is_expected.to(have_attributes(document: nil)) }
+  end
+
+  describe '#replace' do
+    subject(:linemodifier) do
+      described_class.from_config(
+        def_type: 'requirement'
+      )
+    end
+
+    let(:definition)  { instance_double(DefMastership::Definition, 'definition') }
+    let(:document)    { instance_double(DefMastership::Document, 'document')     }
+    let(:definitions) { { 'REFERENCE' => definition }                            }
+
+    before do
+      linemodifier.document = document
+      allow(File).to(receive(:rename))
+    end
+
+    context 'when definition has not the good type' do
+      it do
+        expect(linemodifier.replace('[define,req,REFERENCE]'))
+          .to(eq('[define,req,REFERENCE]'))
+      end
+    end
+
+    context 'when definition has the good type' do
+      before do
+        allow(document).to(receive(:ref_to_def).with('REFERENCE').and_return(definition))
+        allow(definition).to(receive(:sha256).and_return('abcd1234'))
+      end
+
+      it do
+        expect(linemodifier.replace('[define,requirement,REFERENCE]'))
+          .to(eq('[define,requirement,REFERENCE(abcd1234)]'))
+      end
+
+      it do
+        expect(linemodifier.replace('[define,requirement,REFERENCE(bad)]'))
+          .to(eq('[define,requirement,REFERENCE(abcd1234)]'))
+      end
+    end
+  end
+end

data/spec/unit/defmastership/update_def_checksum_modifier_spec.rb

@@ -0,0 +1,75 @@
+# Copyright (c) 2020 Jerome Arbez-Gindre
+# frozen_string_literal: true
+
+require('defmastership')
+
+RSpec.describe(DefMastership::UpdateDefChecksumModifier) do
+  subject(:modifier) do
+    described_class.new(
+      'fake config'
+    )
+  end
+
+  let(:adoc_texts) do
+    {
+      'file1.adoc' => "file1 line1\nfile1 line2",
+      'file2.adoc' => "file2 line1\nfile2 line2"
+    }
+  end
+
+  describe '.new' do
+    it { is_expected.not_to(be(nil)) }
+    it { is_expected.to(have_attributes(config: 'fake config')) }
+    it { is_expected.to(have_attributes(changes: [])) }
+  end
+
+  describe '#do_modifications' do
+    let(:line_modifier) { instance_double(DefMastership::UpdateDefChecksumLineModifier, 'lineModifier') }
+    let(:document)      { instance_double(DefMastership::Document, 'document')                          }
+
+    before do
+      allow(DefMastership::UpdateDefChecksumLineModifier).to(
+        receive(:from_config).with('fake config').and_return(line_modifier)
+      )
+      allow(DefMastership::Document).to(receive(:new).and_return(document))
+      allow(document).to(receive(:parse_file_with_preprocessor))
+      allow(line_modifier).to(receive(:'document=').with(document))
+      allow(line_modifier).to(receive(:replace).with("file1 line1\n").and_return("new file1 line1\n"))
+      allow(line_modifier).to(receive(:replace).with('file1 line2').and_return('new file1 line2'))
+      allow(line_modifier).to(receive(:replace).with("file2 line1\n").and_return("new file2 line1\n"))
+      allow(line_modifier).to(receive(:replace).with('file2 line2').and_return('new file2 line2'))
+      allow(line_modifier).to(receive(:config).and_return('new fake config'))
+      allow(line_modifier).to(receive(:changes).and_return([%w[from1 to1], %w[from2 to2]]))
+    end
+
+    context 'when detailed expectations' do
+      before { modifier.do_modifications(adoc_texts) }
+
+      it do
+        expect(DefMastership::UpdateDefChecksumLineModifier).to(
+          have_received(:from_config).with('fake config')
+        )
+      end
+
+      it { expect(document).to(have_received(:parse_file_with_preprocessor).with('file1.adoc')) }
+      it { expect(document).to(have_received(:parse_file_with_preprocessor).with('file2.adoc')) }
+      it { expect(line_modifier).to(have_received(:'document=').with(document)) }
+      it { expect(line_modifier).to(have_received(:replace).with("file1 line1\n")) }
+      it { expect(line_modifier).to(have_received(:replace).with('file1 line2')) }
+      it { expect(line_modifier).to(have_received(:replace).with("file2 line1\n")) }
+      it { expect(line_modifier).to(have_received(:replace).with('file2 line2')) }
+      it { expect(line_modifier).to(have_received(:config)) }
+      it { expect(line_modifier).to(have_received(:changes)) }
+      it { is_expected.to(have_attributes(config: 'new fake config')) }
+      it { is_expected.to(have_attributes(changes: [%w[from1 to1], %w[from2 to2]])) }
+    end
+
+    it do
+      expected_adoc = {
+        'file1.adoc' => "new file1 line1\nnew file1 line2",
+        'file2.adoc' => "new file2 line1\nnew file2 line2"
+      }
+      expect(modifier.do_modifications(adoc_texts)).to(eq(expected_adoc))
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: defmastership
 version: !ruby/object:Gem::Version
-  version: 1.0.6
+  version: 1.0.7
 platform: ruby
 authors:
 - Jérôme Arbez-Gindre
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-02-22 00:00:00.000000000 Z
+date: 2021-02-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: aruba
@@ -193,6 +193,7 @@ files:
 - defmastership.gemspec
 - defmastership.rdoc
 - features/changeref.feature
+- features/checksum.feature
 - features/defmastership.feature
 - features/export.feature
 - features/modify.feature
@@ -216,6 +217,8 @@ files:
 - lib/defmastership/modifier_base.rb
 - lib/defmastership/rename_included_files_line_modifier.rb
 - lib/defmastership/rename_included_files_modifier.rb
+- lib/defmastership/update_def_checksum_line_modifier.rb
+- lib/defmastership/update_def_checksum_modifier.rb
 - lib/defmastership/version.rb
 - spec/spec_helper.rb
 - spec/unit/defmastership/batch_modifier_spec.rb
@@ -230,6 +233,8 @@ files:
 - spec/unit/defmastership/document_spec.rb
 - spec/unit/defmastership/rename_included_files_line_modifier_spec.rb
 - spec/unit/defmastership/rename_included_files_modifier_spec.rb
+- spec/unit/defmastership/update_def_checksum_line_modifier_spec.rb
+- spec/unit/defmastership/update_def_checksum_modifier_spec.rb
 - spec/unit/defmastership_spec.rb
 homepage: https://gitlab.com/jjag/defmastership/
 licenses: