defmastership 1.0.1 → 1.0.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 86de2f3ea5bbabfae9ed3e773349016d3a122919091bb04d2134dc2de25799a2
-  data.tar.gz: e3d082d8d7f6ee8a22fe178c5eb9691cb6934dae4fff0c6288be0f17e6f94273
+  metadata.gz: 2cd7a521f197f1ab374863af89ab7a24c50bc27d5dfef6161bc47e8fff27825f
+  data.tar.gz: a92ef2ee968e39a48e20fbf2a7d6f3040393c727a422e4f39c935978a29e187f
 SHA512:
-  metadata.gz: 69e865bb1c8e5972d7aa12b5f6ce9203e9d27eadcfef7c3f2becb83b123920769a120e66896957bc26eb89efbe7e9a00389c5262d4d395e2bdb9602b0f3f335c
-  data.tar.gz: cad9f387a71e9668ba1af813865ba161c95daaa4937defa0c42ad41a72b7b3225ad0e3b7ccb5eeefa0553f33b2bd6e3aa6774567fe2bba6d6ea560952190de96
+  metadata.gz: ee7d58f8392a43fe5b45e234f9c4965037e2864a37a669c191d33b58016cf7094a99fe29443c3fc60678bab3e591b19188579771d14aa762fba8d4b34b01bd89
+  data.tar.gz: 526ae0002d2f5d66f2ad555d09bc8cf9786b567098e6bc071a8b119b2cbfe6b664e84eaf3fc68e453ca16f41552a4004f23bab6323ed62cba526322f27e88b15

data/.gitignore

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

data/.gitlab-ci.yml

@@ -0,0 +1,20 @@
+default:
+  image: ruby:2.5
+  before_script:
+    - apt-get update
+    - ruby -v
+    - which ruby
+    - gem install bundler --no-document
+    - bundle install --jobs $(nproc) "${FLAGS[@]}"
+
+cucumber:
+  script:
+    - bundle exec rake features
+
+rspec:
+  script:
+    - bundle exec rake spec
+
+rubocop:
+  script:
+    - bundle exec rake rubocop

data/.rubocop.yml

@@ -13,15 +13,16 @@ require:
   - rubocop-rspec
 
 AllCops:
-  TargetRubyVersion: 2.6
+  TargetRubyVersion: 2.5
   EnabledByDefault: true
   DisplayCopNames: true
 
 Style/Copyright:
-  Enabled: false
-  # AutocorrectNotice: 'Copyright 2020 by Jérôme Arbez-Gindre'
+  Enabled: true
+  Notice: 'Copyright (\(c\) )?202[0-9] Jerome Arbez-Gindre'
+  AutocorrectNotice: '# Copyright (c) 2020 Jerome Arbez-Gindre'
 
-Lint/ConstantResolution:
+Lint/ConstantResolution: # Not available ins rubocop 0.81
   Enabled: false
 
 Style/DocumentationMethod:

data/Gemfile

@@ -1,3 +1,4 @@
+# Copyright (c) 2020 Jerome Arbez-Gindre
 # frozen_string_literal: true
 
 source('https://rubygems.org')

data/LICENSE

@@ -0,0 +1,22 @@
+MIT License
+
+Copyright (c) 2020 Jérôme Arbez-Gindre, and the individual contributors
+to DefMastership.
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in
+all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
+THE SOFTWARE.

data/Rakefile

@@ -1,3 +1,4 @@
+# Copyright (c) 2020 Jerome Arbez-Gindre
 # frozen_string_literal: true
 
 require('rake/clean')
@@ -15,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

@@ -1,4 +1,5 @@
 #!/usr/bin/env ruby
+# Copyright (c) 2020 Jerome Arbez-Gindre
 # frozen_string_literal: true
 
 require('asciidoctor')
@@ -49,8 +50,9 @@ module DefMastership
 
     desc 'Export the definition database in CSV'
     arg_name 'asciidoctor_files'
-    command :export do |c|
-      c.action do |_global_options, _options, args|
+    command :export, :exp, :e do |c|
+      c.flag(%i[separator sep s], default_value: ',', desc: 'CSV separator')
+      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
@@ -61,34 +63,40 @@ module DefMastership
 
         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)
       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

@@ -1,21 +1,27 @@
+# Copyright (c) 2020 Jerome Arbez-Gindre
 # frozen_string_literal: true
 
 # 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.6'
+  s.required_ruby_version = '>= 2.5'
   s.name = 'defmastership'
   s.version = DefMastership::VERSION
   s.author = 'Jérôme Arbez-Gindre'
   s.email = 'jeromearbezgindre@gmail.com'
-  s.licenses = 'Nonstandard'
-  s.homepage = 'http://your.website.com'
+  s.licenses = ['MIT']
+  s.homepage = 'https://gitlab.com/jjag/defmastership/'
   s.platform = Gem::Platform::RUBY
   s.summary = 'Handling of references and definitions with asciidoctor'
   s.files = `git ls-files`.split("\n")
@@ -28,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')
-  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]
     ....
     """
-
-