defmastership 1.0.4 → 1.0.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 83b29fc13933c5a5030df105c476b86b0db8217d693c56786a1a8c5eec494a97
-  data.tar.gz: 686431e77ee50fc891ca72b1d1a569620cfff7591e30c279044ed49c02926281
+  metadata.gz: 9dd61d32bacab380bbfc110df15fd6617b38540aa5bc8eb14bd5b2736e657b7c
+  data.tar.gz: 1ec7bc921411173863b71ba0c3f4164829ac2efbe0689c321830f632a3333f64
 SHA512:
-  metadata.gz: d87ef7737bbc86a2ad638ac473c2663a710a26d225e17fe1a00cfdea9facb27f01eb810863c03088296bd63d6f665c486d81fc06358cb3f01fcc11da6aef03ba
-  data.tar.gz: 76d543d058079e17bf95c1c495e74d3ccd6f410e07bfbe61f376d058272a08ba52e3c45b4bbc4f73d1838885d657cfe7ecb12ecf7a33eb8caa62e7e4e88afd51
+  metadata.gz: e8dcebcb309fd755a339ab702f7c07c65387e5a961400518661d70e9421911630652855a10ad647ac6155b9686665927fde8315a8efb71e3a2913a8db32fbcd5
+  data.tar.gz: 83e28e54b372de48cc554b8f532e659817453e905f3c6634071b3e280f23fba2431119353af9bdd6b23b97a6d6bf8a2b9a261b39e70591f27be45a3b04dce19b

data/.gitignore

@@ -2,3 +2,5 @@
 /tmp
 /coverage
 defmastership-0.0.1.gem
+/Gemfile.lock
+/pkg/

data/.gitlab-ci.yml

@@ -4,7 +4,6 @@ default:
     - apt-get update
     - ruby -v
     - which ruby
-    - rm Gemfile.lock
     - gem install bundler --no-document
     - bundle install --jobs $(nproc) "${FLAGS[@]}"
 

data/.rubocop.yml

@@ -19,7 +19,7 @@ AllCops:
 
 Style/Copyright:
   Enabled: true
-  Notice: 'Copyright (\(c\) )?2020 Jerome Arbez-Gindre'
+  Notice: 'Copyright (\(c\) )?202[0-9] Jerome Arbez-Gindre'
   AutocorrectNotice: '# Copyright (c) 2020 Jerome Arbez-Gindre'
 
 Lint/ConstantResolution: # Not available ins rubocop 0.81

data/Rakefile

@@ -16,8 +16,7 @@ end
 
 spec = eval(::File.read('defmastership.gemspec'))
 
-Gem::PackageTask.new(spec) do |pkg|
-end
+Gem::PackageTask.new(spec)
 CUKE_RESULTS = 'features_results.html'
 CLEAN << CUKE_RESULTS
 

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')
@@ -50,46 +49,59 @@ module DefMastership
 
     desc 'Export the definition database in CSV'
     arg_name 'asciidoctor_files'
-    command :export do |c|
-      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
+    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|
         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).export_to(output_file)
+        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
 
-    desc 'Change temp references to definitive references'
+    desc 'Apply one or more modifications'
     arg_name 'asciidoctor_files'
-    command :changeref do |c|
-      c.flag(%i[change-summary s], default_value: 'changes.csv', desc: 'generates a change summary in a CSV file')
-
-      c.flag(%i[change-file f], default_value: 'changeref.yaml', desc: 'set the change specification file')
+    command :modify, :mod, :m do |c|
+      c.flag(
+        %i[modifications mod m],
+        must_match: /(?:[\w-]+)(?:,[\w-]+)*/,
+        default_value: 'all',
+        desc: 'comma separated list of modifications to apply'
+      )
+      c.flag(%i[modifications-file mf], default_value: 'modifications.yml', desc: 'modifications description file')
+      c.flag(%i[changes-summary s], default_value: 'changes.csv', desc: 'generates a change summary in a CSV file')
 
       c.action do |_global_options, options, args|
-        ref_changer = ProjectRefChanger.new(
-          YAML.load_file(options['change-file']),
+        changer = BatchModifier.new(
+          YAML.load_file(options[:'modifications-file']),
           args.map { |afile| [afile, File.open(afile).read] }.to_h
         )
 
-        ref_changer.replace_all
+        changer.apply(options[:modifications])
 
-        ref_changer.adoc_texts.each do |adoc_file, adoc_text|
+        changer.adoc_texts.each do |adoc_file, adoc_text|
           File.open(adoc_file, 'w') { |f| f.write(adoc_text) }
         end
-        File.open(options['change-file'], 'w') { |f| f.write(ref_changer.yaml_config) }
 
-        unless options['change-summary'].nil?
-          CSV.open(options['change-summary'], 'wb') do |csv|
-            csv << %w[Was Becomes]
-            ref_changer.changes.each { |row| csv << row }
+        File.open(options[:'modifications-file'], 'w') { |f| f.write(changer.config.to_yaml) }
+
+        unless options['changes-summary'].nil?
+          CSV.open(options['changes-summary'], 'wb') do |csv|
+            csv << %w[Modifier Was Becomes]
+            changer.changes.each { |row| csv << row }
           end
         end
       end

data/cucumber.yml

@@ -0,0 +1,2 @@
+---
+default: --publish --format pretty

data/defmastership.gemspec

@@ -3,12 +3,17 @@
 
 # Ensure we require the local version and not one we might have
 # installed already
-require(File.join([
-                    File.dirname(__FILE__),
-                    'lib',
-                    'defmastership',
-                    'version.rb'
-                  ]))
+require(
+  File.join(
+    [
+      File.dirname(__FILE__),
+      'lib',
+      'defmastership',
+      'version.rb'
+    ]
+  )
+)
+
 Gem::Specification.new do |s|
   s.required_ruby_version = '>= 2.5'
   s.name = 'defmastership'
@@ -29,8 +34,9 @@ Gem::Specification.new do |s|
   s.add_development_dependency('rake', '~> 13')
   s.add_development_dependency('rdoc', '~> 6')
   s.add_development_dependency('rspec', '~> 3')
-  s.add_development_dependency('rubocop', '0.91')
-  s.add_development_dependency('rubocop-rspec', '~> 1')
+  s.add_development_dependency('rubocop', '~> 1.3')
+  s.add_development_dependency('rubocop-rake', '~> 0.5')
+  s.add_development_dependency('rubocop-rspec', '~> 2.0')
   s.add_development_dependency('simplecov', '~> 0')
   s.add_runtime_dependency('aasm', '~> 5')
   s.add_runtime_dependency('asciidoctor', '~> 2')

data/features/changeref.feature

@@ -4,57 +4,32 @@ Feature: The changeref command
   I want defmastership to change reference for asciidoctor documents
 
   Scenario: Change a definition 
-    Given a file named "changeref.yaml" with:
+    Given a file named "modifications.yml" with:
     """
     ---
     :toto:
-      :from_regexp: TOTO-TEMP-[X\d]{4}
-      :to_template: TOTO-%<next_ref>04d
-      :next_ref: 123
+      :type: change_ref
+      :config:
+        :from_regexp: TOTO-TEMP-[X\d]{4}
+        :to_template: TOTO-%<next_ref>04d
+        :next_ref: 123
     """
     And a file named "thedoc.adoc" with:
     """
     [define, requirement, TOTO-TEMP-XXX1]
     """
-    When I run `defmastership changeref thedoc.adoc`
+    When I successfully run `defmastership modify --modifications toto thedoc.adoc`
     Then the stdout should not contain anything
     And the stderr should not contain anything
-    And the file "changeref.yaml" should contain:
+    And the file "modifications.yml" should contain:
     """
     ---
     :toto:
-      :from_regexp: TOTO-TEMP-[X\d]{4}
-      :to_template: TOTO-%<next_ref>04d
-      :next_ref: 124
-    """
-    And the file "thedoc.adoc" should contain:
-    """
-    [define, requirement, TOTO-0123]
-    """
-
-  Scenario: Change a definition with yaml file parameter 
-    Given a file named "pouet.yaml" with:
-    """
-    ---
-    :toto:
-      :from_regexp: TOTO-TEMP-[X\d]{4}
-      :to_template: TOTO-%<next_ref>04d
-      :next_ref: 123
-    """
-    And a file named "thedoc.adoc" with:
-    """
-    [define, requirement, TOTO-TEMP-XXX1]
-    """
-    When I run `defmastership changeref --change-file=pouet.yaml thedoc.adoc`
-    Then the stdout should not contain anything
-    And the stderr should not contain anything
-    And the file "pouet.yaml" should contain:
-    """
-    ---
-    :toto:
-      :from_regexp: TOTO-TEMP-[X\d]{4}
-      :to_template: TOTO-%<next_ref>04d
-      :next_ref: 124
+      :type: change_ref
+      :config:
+        :from_regexp: TOTO-TEMP-[X\d]{4}
+        :to_template: TOTO-%<next_ref>04d
+        :next_ref: 124
     """
     And the file "thedoc.adoc" should contain:
     """
@@ -62,29 +37,33 @@ Feature: The changeref command
     """
 
   Scenario: Change two definitions
-    Given a file named "changeref.yaml" with:
+    Given a file named "modifications.yml" with:
     """
     ---
     :toto:
-      :from_regexp: TOTO-TEMP-[X\d]{4}
-      :to_template: TOTO-%<next_ref>04d
-      :next_ref: 123
+      :type: change_ref
+      :config:
+        :from_regexp: TOTO-TEMP-[X\d]{4}
+        :to_template: TOTO-%<next_ref>04d
+        :next_ref: 123
     """
     And a file named "thedoc.adoc" with:
     """
     [define, requirement, TOTO-TEMP-XXX1]
     [define, requirement, TOTO-TEMP-XXX2]
     """
-    When I run `defmastership changeref thedoc.adoc`
+    When I successfully run `defmastership modify --modifications toto thedoc.adoc`
     Then the stdout should not contain anything
     And the stderr should not contain anything
-    And the file "changeref.yaml" should contain:
+    And the file "modifications.yml" should contain:
     """
     ---
     :toto:
-      :from_regexp: TOTO-TEMP-[X\d]{4}
-      :to_template: TOTO-%<next_ref>04d
-      :next_ref: 125
+      :type: change_ref
+      :config:
+        :from_regexp: TOTO-TEMP-[X\d]{4}
+        :to_template: TOTO-%<next_ref>04d
+        :next_ref: 125
     """
     And the file "thedoc.adoc" should contain:
     """
@@ -92,77 +71,51 @@ Feature: The changeref command
     [define, requirement, TOTO-0124]
     """
 
-  Scenario: Generate a table with changess
-    Given a file named "changeref.yaml" with:
+  Scenario: Change definitions with variable from regexp
+    Given a file named "modifications.yml" with:
     """
     ---
-    :toto:
-      :from_regexp: TOTO-TEMP-[X\d]{4}
-      :to_template: TOTO-%<next_ref>04d
-      :next_ref: 123
-    """
-    And a file named "thedoc.adoc" with:
-    """
-    [define, requirement, TOTO-TEMP-XXX1]
-    [define, requirement, TOTO-TEMP-XXX2]
-    """
-    When I run `defmastership changeref --change-summary=changes.csv thedoc.adoc`
-    Then the stderr should not contain anything
-    And the stdout should not contain anything
-    And the file "changes.csv" should contain:
-    """
-    Was,Becomes
-    TOTO-TEMP-XXX1,TOTO-0123
-    TOTO-TEMP-XXX2,TOTO-0124
-    """
-
-  Scenario: Change two definitions with two schema
-    Given a file named "changeref.yaml" with:
-    """
-    ---
-    :toto:
-      :from_regexp: TOTO-TEMP-[X\d]{4}
-      :to_template: TOTO-%<next_ref>04d
-      :next_ref: 123
     :tata_or_titi:
-      :from_regexp: "(?<tata_or_titi>TATA|TITI)-TEMP-[X\\d]{4}"
-      :to_template: "%<tata_or_titi>s-%<next_ref>07d"
-      :next_ref: 432
+      :type: change_ref
+      :config:
+        :from_regexp: "(?<tata_or_titi>TATA|TITI)-TEMP-[X\\d]{4}"
+        :to_template: "%<tata_or_titi>s-%<next_ref>07d"
+        :next_ref: 432
     """
     And a file named "thedoc.adoc" with:
     """
-    [define, requirement, TOTO-TEMP-XXX1]
     [define, requirement, TITI-TEMP-XXX2]
+    [define, requirement, TATA-TEMP-X2X3]
     """
-    When I run `defmastership changeref thedoc.adoc`
+    When I successfully run `defmastership modify --modifications tata_or_titi thedoc.adoc`
     Then the stdout should not contain anything
     And the stderr should not contain anything
-    And the file "changeref.yaml" should contain:
+    And the file "modifications.yml" should contain:
     """
     ---
-    :toto:
-      :from_regexp: TOTO-TEMP-[X\d]{4}
-      :to_template: TOTO-%<next_ref>04d
-      :next_ref: 124
     :tata_or_titi:
-      :from_regexp: "(?<tata_or_titi>TATA|TITI)-TEMP-[X\\d]{4}"
-      :to_template: "%<tata_or_titi>s-%<next_ref>07d"
-      :next_ref: 433
+      :type: change_ref
+      :config:
+        :from_regexp: "(?<tata_or_titi>TATA|TITI)-TEMP-[X\\d]{4}"
+        :to_template: "%<tata_or_titi>s-%<next_ref>07d"
+        :next_ref: 434
     """
     And the file "thedoc.adoc" should contain:
     """
-    [define, requirement, TOTO-0123]
     [define, requirement, TITI-0000432]
+    [define, requirement, TATA-0000433]
     """
 
   Scenario: Change definitions in two files
-    Given a file named "changeref.yaml" with:
+    Given a file named "modifications.yml" with:
     """
     ---
     :toto:
-      :from_regexp: TOTO-TEMP-[X\d]{4}
-      :to_template: TOTO-%<next_ref>04d
-      :next_ref: 123
+      :type: change_ref
+      :config:
+        :from_regexp: TOTO-TEMP-[X\d]{4}
+        :to_template: TOTO-%<next_ref>04d
+        :next_ref: 123
     """
     And a file named "thedoc.adoc" with:
     """
@@ -172,16 +125,18 @@ Feature: The changeref command
     """
     [define, requirement, TOTO-TEMP-XXX2]
     """
-    When I run `defmastership changeref thedoc.adoc thedoc2.adoc`
+    When I successfully run `defmastership modify --modifications toto thedoc.adoc thedoc2.adoc`
     Then the stdout should not contain anything
     And the stderr should not contain anything
-    And the file "changeref.yaml" should contain:
+    And the file "modifications.yml" should contain:
     """
     ---
     :toto:
-      :from_regexp: TOTO-TEMP-[X\d]{4}
-      :to_template: TOTO-%<next_ref>04d
-      :next_ref: 125
+      :type: change_ref
+      :config:
+        :from_regexp: TOTO-TEMP-[X\d]{4}
+        :to_template: TOTO-%<next_ref>04d
+        :next_ref: 125
     """
     And the file "thedoc.adoc" should contain:
     """
@@ -193,13 +148,15 @@ Feature: The changeref command
     """
 
   Scenario: Change a internal refs
-    Given a file named "changeref.yaml" with:
+    Given a file named "modifications.yml" with:
     """
     ---
     :toto:
-      :from_regexp: TOTO-TEMP-[X\d]{4}
-      :to_template: TOTO-%<next_ref>04d
-      :next_ref: 123
+      :type: change_ref
+      :config:
+        :from_regexp: TOTO-TEMP-[X\d]{4}
+        :to_template: TOTO-%<next_ref>04d
+        :next_ref: 123
     """
     And a file named "thedoc.adoc" with:
     """
@@ -208,7 +165,7 @@ Feature: The changeref command
     [define, requirement, TOTO-TEMP-XXX2]
     is the same as defs:iref[TOTO-TEMP-XXX1]
     """
-    When I run `defmastership changeref thedoc.adoc`
+    When I successfully run `defmastership modify --modifications toto thedoc.adoc`
     Then the stdout should not contain anything
     And the stderr should not contain anything
     And the file "thedoc.adoc" should contain:
@@ -220,13 +177,15 @@ Feature: The changeref command
     """
 
   Scenario: Change definitions and iref in two files
-    Given a file named "changeref.yaml" with:
+    Given a file named "modifications.yml" with:
     """
     ---
     :toto:
-      :from_regexp: TOTO-TEMP-[X\d]{4}
-      :to_template: TOTO-%<next_ref>04d
-      :next_ref: 123
+      :type: change_ref
+      :config:
+        :from_regexp: TOTO-TEMP-[X\d]{4}
+        :to_template: TOTO-%<next_ref>04d
+        :next_ref: 123
     """
     And a file named "thedoc.adoc" with:
     """
@@ -238,17 +197,9 @@ Feature: The changeref command
     [define, requirement, TOTO-TEMP-XXX2]
     defs:iref[TOTO-TEMP-XXX1]
     """
-    When I run `defmastership changeref thedoc.adoc thedoc2.adoc`
+    When I successfully run `defmastership modify --modifications toto thedoc.adoc thedoc2.adoc`
     Then the stdout should not contain anything
     And the stderr should not contain anything
-    And the file "changeref.yaml" should contain:
-    """
-    ---
-    :toto:
-      :from_regexp: TOTO-TEMP-[X\d]{4}
-      :to_template: TOTO-%<next_ref>04d
-      :next_ref: 125
-    """
     And the file "thedoc.adoc" should contain:
     """
     [define, requirement, TOTO-0123]
@@ -261,13 +212,15 @@ Feature: The changeref command
     """
 
   Scenario: Do not change definitions when in literal
-    Given a file named "changeref.yaml" with:
+    Given a file named "modifications.yml" with:
     """
     ---
     :toto:
-      :from_regexp: TOTO-TEMP-[X\d]{4}
-      :to_template: TOTO-%<next_ref>04d
-      :next_ref: 123
+      :type: change_ref
+      :config:
+        :from_regexp: TOTO-TEMP-[X\d]{4}
+        :to_template: TOTO-%<next_ref>04d
+        :next_ref: 123
     """
     And a file named "thedoc.adoc" with:
     """
@@ -275,16 +228,18 @@ Feature: The changeref command
     [define, requirement, TOTO-TEMP-XXX1]
     ....
     """
-    When I run `defmastership changeref thedoc.adoc`
+    When I successfully run `defmastership modify --modifications toto thedoc.adoc`
     Then the stdout should not contain anything
     And the stderr should not contain anything
-    And the file "changeref.yaml" should contain:
+    And the file "modifications.yml" should contain:
     """
     ---
     :toto:
-      :from_regexp: TOTO-TEMP-[X\d]{4}
-      :to_template: TOTO-%<next_ref>04d
-      :next_ref: 123
+      :type: change_ref
+      :config:
+        :from_regexp: TOTO-TEMP-[X\d]{4}
+        :to_template: TOTO-%<next_ref>04d
+        :next_ref: 123
     """
     And the file "thedoc.adoc" should contain:
     """
@@ -292,5 +247,3 @@ Feature: The changeref command
     [define, requirement, TOTO-TEMP-XXX1]
     ....
     """
-
-