defmastership 1.0.19 → 1.1.0

data/spec/unit/{def_mastership → defmastership}/document_spec.rb

@@ -3,7 +3,7 @@
 
 require('defmastership/document')
 
-RSpec.describe(DefMastership::Document) do
+RSpec.describe(Defmastership::Document) do
   subject(:document) { described_class.new }
 
   describe '.new' do
@@ -27,12 +27,12 @@ RSpec.describe(DefMastership::Document) do
     it { expect(document.respond_to?(:not_implemented_random_method)).to(be(false)) }
   end
 
-  describe '#parse' do
+  describe '#do_parse' do
     context 'with valid definitions' do
-      let(:definition) { instance_double(DefMastership::Definition, 'definition') }
+      let(:definition) { instance_double(Defmastership::Definition, 'definition') }
 
       before do
-        allow(DefMastership::Definition).to(receive(:new).and_return(definition))
+        allow(Defmastership::Definition).to(receive(:new).and_return(definition))
         allow(definition).to(
           receive_messages(
             labels: Set.new,
@@ -47,16 +47,16 @@ RSpec.describe(DefMastership::Document) do
         let(:input_lines) { ['[define, requirement, TOTO-0001]'] }
 
         before do
-          allow(DefMastership::Definition).to(
+          allow(Defmastership::Definition).to(
             receive(:new).with(
               matchdata_including(type: 'requirement', reference: 'TOTO-0001')
             ).and_return(definition)
           )
-          document.parse(input_lines)
+          document.__send__(:do_parse, input_lines)
         end
 
         it do
-          expect(DefMastership::Definition).to(
+          expect(Defmastership::Definition).to(
             have_received(:new).with(
               matchdata_including(type: 'requirement', reference: 'TOTO-0001')
             )
@@ -71,16 +71,16 @@ RSpec.describe(DefMastership::Document) do
         let(:input_lines) { ['[define, requirement, TOTO-0001(~ab12)]'] }
 
         before do
-          allow(DefMastership::Definition).to(
+          allow(Defmastership::Definition).to(
             receive(:new).with(
               matchdata_including(type: 'requirement', reference: 'TOTO-0001', explicit_checksum: '~ab12')
             ).and_return(definition)
           )
-          document.parse(input_lines)
+          document.__send__(:do_parse, input_lines)
         end
 
         it do
-          expect(DefMastership::Definition).to(
+          expect(Defmastership::Definition).to(
             have_received(:new).with(
               matchdata_including(type: 'requirement', reference: 'TOTO-0001', explicit_checksum: '~ab12')
             )
@@ -92,16 +92,16 @@ RSpec.describe(DefMastership::Document) do
         let(:input_lines) { ['[define, requirement, TOTO-0001(pouet)]'] }
 
         before do
-          allow(DefMastership::Definition).to(
+          allow(Defmastership::Definition).to(
             receive(:new).with(
               matchdata_including(type: 'requirement', reference: 'TOTO-0001', explicit_version: 'pouet')
             ).and_return(definition)
           )
-          document.parse(input_lines)
+          document.__send__(:do_parse, input_lines)
         end
 
         it do
-          expect(DefMastership::Definition).to(
+          expect(Defmastership::Definition).to(
             have_received(:new).with(
               matchdata_including(type: 'requirement', reference: 'TOTO-0001', explicit_version: 'pouet')
             )
@@ -123,7 +123,7 @@ RSpec.describe(DefMastership::Document) do
 
         before do
           allow(definition).to(receive(:<<).and_return(definition))
-          document.parse(input_lines)
+          document.__send__(:do_parse, input_lines)
         end
 
         it { expect(definition).to(have_received(:<<).with('a')) }
@@ -147,7 +147,7 @@ RSpec.describe(DefMastership::Document) do
 
         before do
           allow(definition).to(receive(:<<).and_return(definition))
-          document.parse(input_lines)
+          document.__send__(:do_parse, input_lines)
         end
 
         it { expect(definition).to(have_received(:<<).twice.with('----')) }
@@ -168,7 +168,7 @@ RSpec.describe(DefMastership::Document) do
 
         before do
           allow(definition).to(receive(:<<).and_return(definition))
-          document.parse(input_lines)
+          document.__send__(:do_parse, input_lines)
         end
 
         it { expect(definition).to(have_received(:<<).with('// comment')) }
@@ -187,7 +187,7 @@ RSpec.describe(DefMastership::Document) do
 
         before do
           allow(definition).to(receive(:<<).and_return(definition))
-          document.parse(input_lines)
+          document.__send__(:do_parse, input_lines)
         end
 
         it { expect(definition).to(have_received(:<<).with('one line')) }
@@ -207,7 +207,7 @@ RSpec.describe(DefMastership::Document) do
           ]
         end
 
-        before { document.parse(input_lines) }
+        before { document.__send__(:do_parse, input_lines) }
 
         it { expect(definition).not_to(have_received(:<<).with('not included')) }
       end
@@ -216,7 +216,7 @@ RSpec.describe(DefMastership::Document) do
         let(:input_lines) { ['--', 'first line', '--'] }
 
         it do
-          document.parse(input_lines)
+          document.__send__(:do_parse, input_lines)
           expect(document.definitions).to(eq([]))
         end
       end
@@ -232,11 +232,11 @@ RSpec.describe(DefMastership::Document) do
 
         before do
           allow(definition).to(receive(:labels).and_return(Set['bla1', 'bla2']))
-          document.parse(input_lines)
+          document.__send__(:do_parse, input_lines)
         end
 
         it do
-          expect(DefMastership::Definition).to(
+          expect(Defmastership::Definition).to(
             have_received(:new).with(
               matchdata_including(type: 'requirement', reference: 'TOTO-0001', labels: 'label1, label2')
             )
@@ -257,11 +257,11 @@ RSpec.describe(DefMastership::Document) do
 
         before do
           allow(definition).to(receive(:labels).and_return(Set['bla1', 'bla2']))
-          document.parse(input_lines)
+          document.__send__(:do_parse, input_lines)
         end
 
         it do
-          expect(DefMastership::Definition).to(
+          expect(Defmastership::Definition).to(
             have_received(:new).with(
               matchdata_including(type: 'requirement', reference: 'TOTO-0001', labels: 'label1,label2')
             )
@@ -281,7 +281,7 @@ RSpec.describe(DefMastership::Document) do
           ]
         end
 
-        before { document.parse(input_lines) }
+        before { document.__send__(:do_parse, input_lines) }
 
         it { expect(document.eref[:implements]).to(eq(prefix: 'Participate to:', url: './other_document.html')) }
       end
@@ -296,7 +296,7 @@ RSpec.describe(DefMastership::Document) do
         end
 
         it do
-          document.parse(input_lines)
+          document.__send__(:do_parse, input_lines)
           expect(document.eref[:implements])
             .to(eq(prefix: 'Participate to:'))
         end
@@ -316,7 +316,7 @@ RSpec.describe(DefMastership::Document) do
             receive(:add_eref).with(:implements, 'SYSTEM-0012, SYSTEM-0014')
               .and_return(definition)
           )
-          document.parse(input_lines)
+          document.__send__(:do_parse, input_lines)
         end
 
         it { expect(definition).to(have_received(:add_eref).with(:implements, 'SYSTEM-0012, SYSTEM-0014')) }
@@ -333,7 +333,7 @@ RSpec.describe(DefMastership::Document) do
 
         before do
           allow(definition).to(receive(:add_iref))
-          document.parse(input_lines)
+          document.__send__(:do_parse, input_lines)
         end
 
         it { expect(definition).to(have_received(:add_iref).with('toto')) }
@@ -350,7 +350,7 @@ RSpec.describe(DefMastership::Document) do
           ]
         end
 
-        before { document.parse(input_lines) }
+        before { document.__send__(:do_parse, input_lines) }
 
         it { expect(document.attributes).to(eq(myattribute: 'My attribute:', myotherone: 'My other attribute:')) }
       end
@@ -365,7 +365,7 @@ RSpec.describe(DefMastership::Document) do
 
         before do
           allow(definition).to(receive(:set_attribute).with(:myattribute, 'My value'))
-          document.parse(input_lines)
+          document.__send__(:do_parse, input_lines)
         end
 
         it { expect(definition).to(have_received(:set_attribute).with(:myattribute, 'My value')) }
@@ -380,7 +380,7 @@ RSpec.describe(DefMastership::Document) do
         end
 
         it do
-          document.parse(input_lines)
+          document.__send__(:do_parse, input_lines)
           expect(document.iref).to(be(false))
         end
       end
@@ -394,16 +394,16 @@ RSpec.describe(DefMastership::Document) do
         end
 
         before do
-          allow(DefMastership::Definition).to(
+          allow(Defmastership::Definition).to(
             receive(:new).with(
               matchdata_including(type: 'requirement', reference: 'TOTO-0001')
             ).and_return(definition)
           )
-          document.parse(input_lines)
+          document.__send__(:do_parse, input_lines)
         end
 
         it do
-          expect(DefMastership::Definition).to(
+          expect(Defmastership::Definition).to(
             have_received(:new).with(
               matchdata_including(type: 'requirement', reference: 'TOTO-0001')
             )
@@ -413,17 +413,17 @@ RSpec.describe(DefMastership::Document) do
     end
 
     context 'when variables replacement' do
-      let(:definition) { instance_double(DefMastership::Definition, 'definition') }
+      let(:definition) { instance_double(Defmastership::Definition, 'definition') }
 
       before do
-        allow(DefMastership::Definition).to(receive(:new).and_return(definition))
+        allow(Defmastership::Definition).to(receive(:new).and_return(definition))
         allow(definition).to(
           receive_messages(
             labels: Set.new,
             '<<': definition
           )
         )
-        document.parse(input_lines)
+        document.__send__(:do_parse, input_lines)
       end
 
       context 'when defined variable' do
@@ -488,13 +488,13 @@ RSpec.describe(DefMastership::Document) do
         end
 
         before do
-          allow(DefMastership::Definition).to(
+          allow(Defmastership::Definition).to(
             receive(:new).and_raise('not a valide definition')
           )
         end
 
         it do
-          document.parse(input_lines)
+          document.__send__(:do_parse, input_lines)
           expect(document).to(have_attributes(definitions: []))
         end
       end
@@ -509,13 +509,13 @@ RSpec.describe(DefMastership::Document) do
         end
 
         before do
-          allow(DefMastership::Definition).to(
+          allow(Defmastership::Definition).to(
             receive(:new).and_raise('not a valide definition')
           )
         end
 
         it do
-          document.parse(input_lines)
+          document.__send__(:do_parse, input_lines)
           expect(document).to(have_attributes(definitions: []))
         end
       end
@@ -523,7 +523,7 @@ RSpec.describe(DefMastership::Document) do
   end
 
   describe '#parse_file_with_preprocessor' do
-    let(:definition)  { instance_double(DefMastership::Definition, 'definition') }
+    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')      }
@@ -534,7 +534,7 @@ RSpec.describe(DefMastership::Document) do
       )
       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(Defmastership::Definition).to(receive(:new).and_return(definition))
       allow(definition).to(
         receive_messages(
           '<<': definition,
@@ -549,7 +549,7 @@ RSpec.describe(DefMastership::Document) do
     it { expect(adoc_reader).to(have_received(:read_lines).with(no_args)) }
 
     it do
-      expect(DefMastership::Definition).to(
+      expect(Defmastership::Definition).to(
         have_received(:new).with(
           matchdata_including(type: 'requirement', reference: 'TOTO-0001')
         )
@@ -560,8 +560,8 @@ RSpec.describe(DefMastership::Document) do
   describe '#wrong_explicit_checksum?' do
     let(:defs) do
       [
-        instance_double(DefMastership::Definition, 'definition'),
-        instance_double(DefMastership::Definition, 'definition')
+        instance_double(Defmastership::Definition, 'definition'),
+        instance_double(Defmastership::Definition, 'definition')
       ]
     end
     let(:input_lines) do
@@ -575,7 +575,7 @@ RSpec.describe(DefMastership::Document) do
     end
 
     before do
-      allow(DefMastership::Definition).to(receive(:new).twice.and_return(defs.first, defs[1]))
+      allow(Defmastership::Definition).to(receive(:new).twice.and_return(defs.first, defs[1]))
       defs.each do |definition|
         allow(definition).to(receive(:labels)).and_return([])
         allow(definition).to(receive(:<<)).and_return(definition)
@@ -585,7 +585,7 @@ RSpec.describe(DefMastership::Document) do
     context 'when no wrong explicit checksum' do
       before do
         defs.each { |definition| allow(definition).to(receive(:wrong_explicit_checksum)).and_return(nil) }
-        document.parse(input_lines)
+        document.__send__(:do_parse, input_lines)
         document.wrong_explicit_checksum?
       end
 
@@ -598,7 +598,7 @@ RSpec.describe(DefMastership::Document) do
       before do
         allow(defs.first).to(receive(:wrong_explicit_checksum)).and_return('toto')
         allow(defs[1]).to(receive(:wrong_explicit_checksum)).and_return(nil)
-        document.parse(input_lines)
+        document.__send__(:do_parse, input_lines)
         document.wrong_explicit_checksum?
       end
 
@@ -609,8 +609,8 @@ RSpec.describe(DefMastership::Document) do
   describe '#explicit_version?' do
     let(:defs) do
       [
-        instance_double(DefMastership::Definition, 'definition'),
-        instance_double(DefMastership::Definition, 'definition')
+        instance_double(Defmastership::Definition, 'definition'),
+        instance_double(Defmastership::Definition, 'definition')
       ]
     end
     let(:input_lines) do
@@ -624,7 +624,7 @@ RSpec.describe(DefMastership::Document) do
     end
 
     before do
-      allow(DefMastership::Definition).to(receive(:new).twice.and_return(defs.first, defs[1]))
+      allow(Defmastership::Definition).to(receive(:new).twice.and_return(defs.first, defs[1]))
       allow(defs.first).to(receive(:labels)).and_return([])
       allow(defs[1]).to(receive(:labels)).and_return([])
       allow(defs.first).to(receive(:<<).and_return(defs.first))
@@ -635,7 +635,7 @@ RSpec.describe(DefMastership::Document) do
       before do
         allow(defs.first).to(receive(:explicit_version)).and_return(nil)
         allow(defs[1]).to(receive(:explicit_version)).and_return(nil)
-        document.parse(input_lines)
+        document.__send__(:do_parse, input_lines)
         document.explicit_version?
       end
 
@@ -648,7 +648,7 @@ RSpec.describe(DefMastership::Document) do
       before do
         allow(defs.first).to(receive(:explicit_version)).and_return('toto')
         allow(defs[1]).to(receive(:explicit_version)).and_return(nil)
-        document.parse(input_lines)
+        document.__send__(:do_parse, input_lines)
         document.explicit_version?
       end
 
@@ -659,8 +659,8 @@ RSpec.describe(DefMastership::Document) do
   describe '#ref_to_def?' do
     let(:definitions) do
       [
-        instance_double(DefMastership::Definition, 'def1'),
-        instance_double(DefMastership::Definition, 'def2')
+        instance_double(Defmastership::Definition, 'def1'),
+        instance_double(Defmastership::Definition, 'def2')
       ]
     end
     let(:input_lines) do
@@ -674,7 +674,7 @@ RSpec.describe(DefMastership::Document) do
     end
 
     before do
-      allow(DefMastership::Definition).to(receive(:new).twice.and_return(definitions.first, definitions[1]))
+      allow(Defmastership::Definition).to(receive(:new).twice.and_return(definitions.first, definitions[1]))
       allow(definitions.first).to(
         receive_messages(
           labels: [],
@@ -689,7 +689,7 @@ RSpec.describe(DefMastership::Document) do
           reference: 'TOTO-0002'
         )
       )
-      document.parse(input_lines)
+      document.__send__(:do_parse, input_lines)
     end
 
     it { expect(document.ref_to_def('TOTO-0001')).to(eq(definitions.first)) }

data/spec/unit/{def_mastership/csv_formatter_body_spec.rb → defmastership/export/body_formatter_spec.rb}

@@ -1,13 +1,13 @@
 # Copyright (c) 2020 Jerome Arbez-Gindre
 # frozen_string_literal: true
 
-require('defmastership/csv_formatter_body')
+require('defmastership/export/body_formatter')
 
-RSpec.describe(DefMastership::CSVFormatterBody) do
+RSpec.describe(Defmastership::Export::BodyFormatter) do
   subject(:formatter) { described_class.new(document, definition) }
 
-  let(:document)   { instance_double(DefMastership::Document, 'document')     }
-  let(:definition) { instance_double(DefMastership::Definition, 'definition') }
+  let(:document)   { instance_double(Defmastership::Document, 'document')     }
+  let(:definition) { instance_double(Defmastership::Definition, 'definition') }
 
   describe '.new' do
     it { is_expected.not_to(be_nil) }

data/spec/unit/{def_mastership/csv_formatter_spec.rb → defmastership/export/csv/formatter_spec.rb}

@@ -2,24 +2,24 @@
 # frozen_string_literal: true
 
 require('csv')
-require('defmastership/csv_formatter')
+require('defmastership/export/csv/formatter')
 require('ostruct')
 
-RSpec.describe(DefMastership::CSVFormatter) do
+RSpec.describe(Defmastership::Export::CSV::Formatter) do
   subject(:formatter) { described_class.new(document, ';') }
 
-  let(:document) { instance_double(DefMastership::Document, 'document') }
+  let(:document) { instance_double(Defmastership::Document, 'document') }
 
   describe '.new' do
     it { is_expected.not_to(be_nil) }
   end
 
   describe '#export_to' do
-    let(:header) { instance_double(DefMastership::CSVFormatterHeader, 'header') }
+    let(:header) { instance_double(Defmastership::Export::HeaderFormatter, 'header') }
     let(:bodies) do
       [
-        instance_double(DefMastership::CSVFormatterBody, 'bodies[0]'),
-        instance_double(DefMastership::CSVFormatterBody, 'bodies[1]')
+        instance_double(Defmastership::Export::BodyFormatter, 'bodies[0]'),
+        instance_double(Defmastership::Export::BodyFormatter, 'bodies[1]')
       ]
     end
     let(:csv) { instance_double(CSV, 'csv') }
@@ -27,9 +27,14 @@ RSpec.describe(DefMastership::CSVFormatter) do
     before do
       allow(CSV).to(receive(:open).with('whatever', 'w:ISO-8859-1', col_sep: ';').and_yield(csv))
       allow(csv).to(receive(:<<))
-      allow(DefMastership::CSVFormatterHeader).to(receive(:new).with(document).and_return(header))
+      allow(Defmastership::Export::HeaderFormatter).to(receive(:new).with(document).and_return(header))
       bodies.each_with_index do |body, index|
-        allow(DefMastership::CSVFormatterBody).to(receive(:new).with(document, :"def#{index}").and_return(body))
+        allow(Defmastership::Export::BodyFormatter).to(
+          receive(:new).with(
+            document,
+            :"def#{index}"
+          ).and_return(body)
+        )
       end
       allow(document).to(receive(:definitions).and_return(%i[def0 def1]))
       allow(document).to(receive(:wrong_explicit_checksum?).with(no_args).and_return(false))

data/spec/unit/{def_mastership/csv_formatter_header_spec.rb → defmastership/export/header_formatter_spec.rb}

@@ -1,12 +1,12 @@
 # Copyright (c) 2020 Jerome Arbez-Gindre
 # frozen_string_literal: true
 
-require('defmastership/csv_formatter_header')
+require('defmastership/export/header_formatter')
 
-RSpec.describe(DefMastership::CSVFormatterHeader) do
+RSpec.describe(Defmastership::Export::HeaderFormatter) do
   subject(:formatter) { described_class.new(document) }
 
-  let(:document) { instance_double(DefMastership::Document, 'document') }
+  let(:document) { instance_double(Defmastership::Document, 'document') }
 
   describe '.new' do
     it { is_expected.not_to(be_nil) }

data/spec/unit/{def_mastership → defmastership}/matching_line_spec.rb

@@ -3,7 +3,7 @@
 
 require('defmastership/matching_line')
 
-RSpec.describe(DefMastership::MatchingLine) do
+RSpec.describe(Defmastership::MatchingLine) do
   describe '.new' do
     subject(:matching_line) do
       described_class.new(:a_match, :a_line)

data/spec/unit/{def_mastership/change_ref_modifier_spec.rb → defmastership/modifier/change_ref_spec.rb}

@@ -1,13 +1,13 @@
 # Copyright (c) 2020 Jerome Arbez-Gindre
 # frozen_string_literal: true
 
-require('defmastership/change_ref_modifier')
+require('defmastership/modifier/change_ref')
 
-RSpec.describe(DefMastership::ChangeRefModifier) do
+RSpec.describe(Defmastership::Modifier::ChangeRef) do
   subject(:refchanger) { described_class.new({}) }
 
   describe '.new' do
-    it { expect(described_class.ancestors).to(include(DefMastership::Modifier)) }
+    it { expect(described_class.ancestors).to(include(Defmastership::Modifier::ModifierCommon)) }
     it { is_expected.not_to(be_nil) }
     it { is_expected.to(have_attributes(from_regexp: '')) }
     it { is_expected.to(have_attributes(to_template: '')) }
@@ -28,7 +28,7 @@ RSpec.describe(DefMastership::ChangeRefModifier) do
     end
   end
 
-  describe '#replace_def' do
+  describe '#replace_refdef' do
     context 'when really simple rule' do
       subject(:refchanger) do
         described_class.new(
@@ -42,6 +42,21 @@ RSpec.describe(DefMastership::ChangeRefModifier) do
           expect(refchanger.replace_refdef('[define, whatever, TEMP]'))
             .to(eq('[define, whatever, TUTU]'))
         end
+
+        it do
+          expect(refchanger.replace_refdef('[define, whatever, TEMP(a~12345678)]'))
+            .to(eq('[define, whatever, TUTU(a~12345678)]'))
+        end
+
+        it do
+          expect(refchanger.replace_refdef('[define, whatever, TEMP(~12345678)]'))
+            .to(eq('[define, whatever, TUTU(~12345678)]'))
+        end
+
+        it do
+          expect(refchanger.replace_refdef('[define, whatever, TEMP(a)]'))
+            .to(eq('[define, whatever, TUTU(a)]'))
+        end
       end
 
       context 'when no valid definition' do
@@ -178,44 +193,4 @@ RSpec.describe(DefMastership::ChangeRefModifier) do
       ).to(eq('defs:iref[TOTO-0132] defs:iref[TOTO-0132] bla'))
     end
   end
-
-  describe '#replace' do
-    subject(:refchanger) do
-      described_class.new(
-        from_regexp: 'TEMP',
-        to_template: 'TUTU'
-      )
-    end
-
-    it do
-      expect(refchanger.replace(:refdef, '[define, whatever, TEMP]'))
-        .to(eq('[define, whatever, TUTU]'))
-    end
-
-    it do
-      expect(refchanger.replace(:refdef, '[define, whatever, TEMP(a~12345678)]'))
-        .to(eq('[define, whatever, TUTU(a~12345678)]'))
-    end
-
-    it do
-      expect(refchanger.replace(:refdef, '[define, whatever, TEMP(~12345678)]'))
-        .to(eq('[define, whatever, TUTU(~12345678)]'))
-    end
-
-    it do
-      expect(refchanger.replace(:refdef, '[define, whatever, TEMP(a)]'))
-        .to(eq('[define, whatever, TUTU(a)]'))
-    end
-
-    it do
-      refchanger.replace(:refdef, '[define, whatever, TEMP]')
-      expect(refchanger.replace(:irefs, 'defs:iref[TEMP] defs:iref[TEMP]'))
-        .to(eq('defs:iref[TUTU] defs:iref[TUTU]'))
-    end
-
-    it do
-      expect { refchanger.replace(:pouet, 'whatever') }
-        .to(raise_error(NoMethodError))
-    end
-  end
 end

data/spec/unit/defmastership/modifier/factory_spec.rb

@@ -0,0 +1,45 @@
+# Copyright (c) 2023 Jerome Arbez-Gindre
+# frozen_string_literal: true
+
+require('defmastership/modifier/factory')
+
+module Defmastership
+  module Modifier
+    # Modifier example
+    class Toto
+      include ModifierCommon
+    end
+
+    # Modifier example
+    class TuTu
+      include ModifierCommon
+    end
+  end
+end
+
+RSpec.describe(Defmastership::Modifier::Factory) do
+  describe('.from_config') do
+    let(:toto)        { instance_double(Defmastership::Modifier::Toto, 'toto') }
+    let(:toto_config) { { type: 'toto', config: { p: 'whatever' } }            }
+    let(:tutu)        { instance_double(Defmastership::Modifier::TuTu, 'tutu') }
+    let(:tutu_config) { { type: 'tu_tu', config: { p: 'whatever' } }           }
+
+    before do
+      allow(Defmastership::Modifier::Toto).to(receive(:new).with(toto_config[:config]).and_return(toto))
+      allow(Defmastership::Modifier::TuTu).to(receive(:new).with(tutu_config[:config]).and_return(tutu))
+    end
+
+    it { expect(described_class.from_config(toto_config)).to(be(toto)) }
+    it { expect(described_class.from_config(tutu_config)).to(be(tutu)) }
+
+    it do
+      described_class.from_config(toto_config)
+      expect(Defmastership::Modifier::Toto).to(have_received(:new).with(toto_config[:config]))
+    end
+
+    it do
+      described_class.from_config(tutu_config)
+      expect(Defmastership::Modifier::TuTu).to(have_received(:new).with(tutu_config[:config]))
+    end
+  end
+end