rdm 0.4.14.2 → 0.4.17

data/lib/rdm/yml_config/validate_config.rb

@@ -0,0 +1,13 @@
+class Rdm::ValidateConfig
+  def initialize
+    @data = {}
+  end
+
+  def method_missing(method, *args, &block)
+    @data[method] = args.first
+  end
+
+  def to_hash
+    @data
+  end
+end

data/rdm.gemspec

@@ -23,4 +23,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "codecov"
   spec.add_dependency "activesupport"
   spec.add_dependency "commander", "~> 4.4"
+  spec.add_dependency "morf"
+  spec.add_dependency "hcast"
+  spec.add_dependency "attr_validator"
 end

data/spec/fixtures/SampleSource.rb

@@ -1,10 +1,8 @@
 setup do
-  role "example"
-  config_path "config"
+  role                    'example'
+  config_path             'app.yml'
   silence_missing_package true
 end
 
-config :database
-
 package "application/web"
 package "domain/core"

data/spec/fixtures/app.yml

@@ -0,0 +1,17 @@
+core:
+  some_key:
+    - value1
+    - value2
+    - value3
+    
+web:
+  some_key:
+    - value1
+    - value2
+    - value3
+
+factory:
+  some_key:
+    - value1
+    - value2
+    - value3

data/spec/rdm/cli/gen_package_spec.rb

@@ -98,6 +98,4 @@ describe Rdm::CLI::GenPackage do
       expect(stdout.output).to include("Package name was not specified!")
     end
   end
-
-
 end

data/spec/rdm/gen/init_spec.rb

@@ -18,9 +18,6 @@ describe Rdm::Gen::Init do
         ensure_exists("Gemfile")
         ensure_exists("Readme.md")
         ensure_exists("bin/console")
-        ensure_exists("env_files/test.env")
-        ensure_exists("env_files/development.env")
-        ensure_exists("env_files/production.env")
       end
     end
 
@@ -36,15 +33,6 @@ describe Rdm::Gen::Init do
         ensure_exists(".rdm/templates/package/bin/console")
       end
     end
-
-    it "has generated config templates" do
-      subject.generate(current_path: example_project_path, stdout: stdout)
-
-      FileUtils.cd(example_project_path) do
-        ensure_exists(".rdm/templates/configs/<%=config_path%>")
-        ensure_exists(".rdm/templates/configs/<%=role_config_path%>")
-      end
-    end
   end
 
   context "prevents double execution" do

data/spec/rdm/gen/package_spec.rb

@@ -82,6 +82,7 @@ describe Rdm::Gen::Package do
         package_name: "some",
         local_path:   "domain/some",
       )
+      
       expect {
         subject.generate(
           current_path: example_project_path,

data/spec/rdm/package_importer_spec.rb

@@ -8,17 +8,28 @@ describe Rdm::PackageImporter do
     dependencies.each do |dependency|
       package.import(dependency)
     end
+    package.set_environments do
+      array 'some_key', each: :string do
+        length({ min: 3 })
+      end
+    end
+
     package
   end
 
   def build_source(packages:)
     source = Rdm::Source.new(root_path: nil)
-    source.init_with(packages: packages, configs: {})
+    source.init_with(packages: packages)
     source
   end
 
-  before do
+  before(:each) do
     Rdm::PackageImporter.reset!
+    Rdm::ConfigManager.reset!
+
+    Rdm.setup do
+      config_path File.expand_path(File.join(__dir__, '../fixtures/app.yml'))
+    end
   end
 
   describe "#import_package" do
@@ -64,5 +75,26 @@ describe Rdm::PackageImporter do
         expect(imported).to_not include("factory")
       end
     end
+
+    context 'sets config variables' do
+      it 'only for imported packages' do
+        web_pack     = build_package("web", dependencies: ["core"])
+        core_pack    = build_package("core")
+        factory_pack = build_package("factory")
+        
+        source = build_source(packages: {"web" => web_pack, "core" => core_pack, "factory" => factory_pack})
+
+        imported = subject.import_package("web", source: source)
+
+        expect(Rdm::ConfigManager.web.some_key).to match(["value1", "value2", "value3"])
+        expect(Rdm::ConfigManager.core.some_key).to match(["value1", "value2", "value3"])
+
+        expect{
+          Rdm::ConfigManager.factory
+        }.to raise_error(
+          ArgumentError, ":factory configuration was not defined for current package. Add `import 'factory'` to your Package.rb file"
+        )
+      end
+    end
   end
 end

data/spec/rdm/rdm_spec.rb

@@ -29,7 +29,7 @@ describe Rdm do
 
   context "config" do
     it "returns config" do
-      expect(Rdm.config).to be_a(Rdm::ConfigManager)
+      expect(Rdm.config).to eq(Rdm::ConfigManager)
     end
   end
 

data/spec/rdm/source_parser_spec.rb

@@ -32,12 +32,6 @@ describe Rdm::SourceParser do
       expect(paths).to include("application/web")
       expect(paths).to include("domain/core")
     end
-
-    it "parses all config names" do
-      names = @source.config_names
-      expect(names.count).to be(1)
-      expect(names).to include("database")
-    end
   end
 
 
@@ -70,58 +64,5 @@ describe Rdm::SourceParser do
       expect(paths).to include("application/web")
       expect(paths).to include("domain/core")
     end
-
-    it "parses all config names" do
-      names = @source.config_names
-      expect(names.count).to be(2)
-      expect(names).to include("database")
-    end
-  end
-
-  describe "::read_and_init_source" do
-    before { initialize_example_project }
-    after  { reset_example_project }
-
-    subject { described_class }
-    let(:stdout) { SpecLogger.new }
-
-    describe "#init_and_set_env_variables" do
-      context "with defined role" do
-        it "load env_file variables into ENV hash" do
-          subject.read_and_init_source(@rdm_source_file)
-
-          expect(ENV['EXAMPLE_API_KEY']).to eq('example_key_value')
-          expect(ENV['APP_NAME']).to eq('Application')
-        end
-      end
-
-      context "with undefined role" do
-        it "puts warning message" do
-          Rdm::Utils::FileUtils.change_file @rdm_source_file do |line|
-            line.include?("ENV['ENV_FILE'] || \"production\"") ? 'env_file_name "stading"' : line
-          end
-          subject.read_and_init_source(@rdm_source_file, stdout: stdout)
-
-          expect(stdout.output).to include("WARNING! Environment file 'stading' was not found. Please, add /tmp/example/env_files/stading.env file...")
-        end
-      end
-
-      context "when try to overwrite ENV variable" do
-        before do
-          ENV['RUBY_ENV'] = 'test'
-          
-          subject.read_and_init_source(@rdm_source_file, stdout: stdout)
-        end
-
-        it 'puts warning message' do
-          expect(stdout.output).to include("WARNING! Environment file 'production' overwrites ENV['RUBY_ENV'] variable from 'test' to 'production' ...")
-        end
-
-        it 'overwrites ENV variable' do
-          expect(ENV['RUBY_ENV']).to eq('production')
-          expect(ENV['APP_NAME']).to eq('Application')
-        end
-      end
-    end
   end
 end

data/spec/rdm/yml_config/config_caster_spec.rb

@@ -0,0 +1,64 @@
+require 'spec_helper'
+
+describe Rdm::ConfigCaster do
+  subject { described_class.new(env) }
+
+  describe "#to_hcast_string" do
+    context 'for string env' do
+      let(:env) {
+        Rdm::EnvConfig.new(
+          name:        :string_env,
+          type:        :string
+        )
+      }
+
+      it 'generates proper string' do
+        expect(
+          subject.to_hcast_string(env)
+        ).to eq("string :string_env, optional: false")
+      end
+    end
+
+    context 'for array env' do
+      let(:env) {
+        Rdm::EnvConfig.new(
+          name:        :array_env,
+          type:        :array,
+          children: [
+            Rdm::EnvConfig.new(
+              name:        :string_env,
+              type:        :string
+            )
+          ]
+        )
+      }
+
+      it 'generates proper string' do
+        expect(
+          subject.to_hcast_string(env)
+        ).to eq("array :array_env, optional: false, each: :string")
+      end
+    end
+
+    context 'for hash env' do
+      let(:env) {
+        Rdm::EnvConfig.new(
+          name:        :hash_env,
+          type:        :hash,
+          children: [
+            Rdm::EnvConfig.new(
+              name:        :string_env,
+              type:        :string
+            )
+          ]
+        )
+      }
+
+      it 'generates proper string' do
+        expect(
+          subject.to_hcast_string(env)
+        ).to eq("hash :hash_env, optional: false do \n  string :string_env, optional: false \nend")
+      end
+    end
+  end
+end

data/spec/rdm/yml_config/config_manager_spec.rb

@@ -0,0 +1,7 @@
+require 'spec_helper'
+
+describe Rdm::ConfigManager do
+  describe '#load_config' do
+    
+  end
+end

data/spec/rdm/yml_config/config_validator_spec.rb

@@ -0,0 +1,190 @@
+require 'spec_helper'
+
+describe Rdm::ConfigValidator do
+
+  let(:hash_config) {
+    {
+      example_config: {
+        example_array_config:   ['hello', 'world'],
+        example_symbol_config:  :values,
+        example_integer_config: 25,
+        example_email_config:   'info@rdm.com'
+      }
+    }
+  }
+
+  let(:array_config) {
+    Rdm::EnvConfig.new(
+      name:      'example_array_config',
+      type:      :array,
+      validates: nil,
+      children: [
+        Rdm::EnvConfig.new(
+          name: nil,
+          type: :string,
+          validates: {
+            length: {
+              equal_to: 5
+            }
+          }
+        )
+      ]
+    )
+  }
+  let(:symbol_config) {
+    Rdm::EnvConfig.new(
+      name: 'example_symbol_config',
+      type: :symbol,
+      validates: {
+        inclusion: {
+          in: [:example, :symbol, :config, :values]
+        }
+      }
+    )
+  }
+  let(:integer_config) {
+    Rdm::EnvConfig.new(
+      name: 'example_integer_config',
+      type: :integer,
+      validates: {
+        numericality: {
+          greater_than_or_equal_to: 5,
+          less_than: 50
+        }
+      }
+    )
+  }
+  let(:email_config) {
+    Rdm::EnvConfig.new(
+      name:     'example_email_config',
+      type:     :string,
+      optional: true,
+      validates: {
+        email: true,
+        presence: true
+      }
+    )
+  }
+
+  let(:env_config) {
+    Rdm::EnvConfig.new(
+      name:      'example_config',
+      type:      :hash,
+      optional:  true,
+      default:   nil,
+      validates: nil,
+      children: [
+        array_config,
+        symbol_config,
+        integer_config,
+        email_config
+      ]
+    )
+  }
+
+  describe '#validate' do
+    context 'for symbol config' do
+      it 'not raises error for valid params' do
+        expect {
+          described_class.new(symbol_config).validate!({example_symbol_config: :example})
+        }.not_to raise_error
+      end
+
+      it 'raises ArgumentError if invalid params' do
+        expect {
+          described_class.new(symbol_config).validate!({example_symbol_config: :unpermitted_parameter})
+        }.to raise_error Rdm::Errors::InvalidConfig
+      end
+
+      it 'returns OpenStruct' do
+        expect(
+          described_class.new(symbol_config).validate!({example_symbol_config: :example})
+        ).to eq(OpenStruct.new({example_symbol_config: :example}))
+      end
+    end
+
+    context 'for email config' do
+      it 'not raises error for valid params' do
+        expect {
+          described_class.new(email_config).validate!({example_email_config: 'hello@world.ru'})
+        }.not_to raise_error
+      end
+
+      it 'raises ArgumentError if invalid params' do
+        expect {
+          described_class.new(email_config).validate!({example_email_config: 'helloworld'})
+        }.to raise_error Rdm::Errors::InvalidConfig
+      end
+
+      it 'returns OpenStruct' do
+        expect(
+          described_class.new(email_config).validate!({example_email_config: 'hello@world.ru'})
+        ).to eq(OpenStruct.new({example_email_config: 'hello@world.ru'}))
+      end
+    end
+
+    context 'for integer config' do
+      it 'not raises error for valid params' do
+        expect {
+          described_class.new(integer_config).validate!({example_integer_config: 5})
+        }.not_to raise_error
+      end
+
+      it 'not raises error for valid params' do
+        expect {
+          described_class.new(integer_config).validate!({example_integer_config: 4})
+        }.to raise_error Rdm::Errors::InvalidConfig
+      end
+
+      it 'returns OpenStruct' do
+        expect(
+          described_class.new(integer_config).validate!({example_integer_config: 5})
+        ).to eq(OpenStruct.new({example_integer_config: 5}))
+      end
+    end
+
+    context 'for array config' do
+      it 'not raises error for valid params' do
+        expect {
+          described_class.new(symbol_config).validate!({example_array_config: ['hello', 'world']})
+        }.not_to raise_error
+      end
+
+      it 'returns OpenStruct' do
+        expect(
+          described_class.new(integer_config).validate!({example_array_config: ['hello', 'world']})
+        ).to eq(OpenStruct.new({example_array_config: ['hello', 'world']}))
+      end
+    end
+
+    context 'for hash config' do
+      it 'not raises error for valid params' do
+        expect {
+          described_class.new(symbol_config).validate!({example_array_config: ['hello', 'world']})
+        }.not_to raise_error
+      end
+
+      it 'returns OpenStruct' do
+        expect(
+          described_class.new(integer_config).validate!(
+            {
+              example_array_config: ['hello', 'world'],
+              example_integer_config: 5,
+              example_email_config: 'hello@world.pro',
+              example_symbol_config: :example
+            }
+          )
+        ).to match(
+          OpenStruct.new(
+            {
+              example_array_config:   ['hello', 'world'],
+              example_integer_config: 5,
+              example_email_config:   'hello@world.pro',
+              example_symbol_config:  :example
+            }
+          )
+        )
+      end
+    end
+  end
+end