train 3.1.8 → 3.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e776c228711cbc3b424726dbd8e43655169efe42cd42d3cba18c5d57cb7ff497
-  data.tar.gz: a34d47e64bee5ab52278ef28c1d4d998e7d60ba12519386b9f618812a6db835c
+  metadata.gz: c8dd3de7c1923ee9b164a40fe8b3fb6bfdfe318c3b9cb3b322397551e8a17a5f
+  data.tar.gz: 7b3b63a44c454963d8b47b1c3e7772b17db8f73b20a6519da955a475823d6b3f
 SHA512:
-  metadata.gz: 4acc5d08f23f888fe6411f0914c14e50d53a9f0cfd3e990cfd929c8f8532d87f857f0fb9f199dec9f02eda83a6c7f2f97b786e724d81112321a768b723c06519
-  data.tar.gz: dc8052d53a15ed57eab764bbb355faf994d39145f8210a90911b8198676febbbeda97c79a782d77a12414ddfc554286ecc7f93b9b6873e8601341f64672bb61c
+  metadata.gz: 13928685d3aa1d35942241ce81be9cda15cbdfdfadf2e9c04fdb63da9d0573eba79bfb5431fb907ca57b0732c2a9a6be94abc60dd43261d950297ae764bd2d1e
+  data.tar.gz: 0eb217f2809ff17bc7a0715b9c49566a45d16db343131c208a6dabc9765771095020dc8618af13a3003e5e7c0c72d9804ae649bef56246fd3a77ee690e0e6d15

data/lib/train/file.rb

@@ -29,7 +29,9 @@ module Train
     }.freeze
 
     DATA_FIELDS.each do |m|
-      define_method m.to_sym do
+      next if m == "path"
+
+      define_method m do
         raise NotImplementedError, "File must implement the #{m}() method."
       end
     end

data/lib/train/platforms/platform.rb

@@ -23,10 +23,8 @@ module Train::Platforms
     end
 
     def find_family_hierarchy(platform = self)
-      families = platform.families.each_with_object([]) do |(k, _v), memo|
-        memo << k.name
-        memo << find_family_hierarchy(k) unless k.families.empty?
-      end
+      families = platform.families.map { |k, v| [k.name, find_family_hierarchy(k)] }
+
       @family_hierarchy = families.flatten
     end
 
@@ -42,11 +40,7 @@ module Train::Platforms
 
     def clean_name(force: false)
       @cleaned_name = nil if force
-      @cleaned_name ||= begin
-                          name = (@platform[:name] || @name)
-                          name.downcase!.tr!(" ", "_") if name =~ /[A-Z ]/
-                          name
-                        end
+      @cleaned_name ||= (@platform[:name] || @name).downcase.tr(" ", "_")
     end
 
     def uuid

data/lib/train/transports/mock.rb

@@ -70,11 +70,17 @@ class Train::Transports::Mock
       "mock://"
     end
 
+    DEFAULTS = {
+      name: "mock",
+      family: "mock",
+      release: "unknown",
+      arch: "unknown",
+    }.freeze
+
     def mock_os(value = {})
-      # if a user passes a nil value, set to an empty hash so the merge still succeeds
-      value ||= {}
-      value.each { |k, v| value[k] = "unknown" if v.nil? }
-      value = { name: "mock", family: "mock", release: "unknown", arch: "unknown" }.merge(value)
+      value = DEFAULTS
+        .merge(value || {})
+        .transform_values { |v| v || "unknown" }
 
       platform = Train::Platforms.name(value[:name])
       platform.find_family_hierarchy

data/lib/train/transports/ssh_connection.rb

@@ -285,7 +285,7 @@ class Train::Transports::SSH
       exit_status = nil
       session.open_channel do |channel|
         # wrap commands if that is configured
-        cmd = @cmd_wrapper.run(cmd) unless @cmd_wrapper.nil?
+        cmd = @cmd_wrapper.run(cmd) if @cmd_wrapper
 
         if @transport_options[:pty]
           channel.request_pty do |_ch, success|

data/lib/train/version.rb

@@ -3,5 +3,5 @@
 # Author:: Dominik Richter (<dominik.richter@gmail.com>)
 
 module Train
-  VERSION = "3.1.8".freeze
+  VERSION = "3.2.0".freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: train
 version: !ruby/object:Gem::Version
-  version: 3.1.8
+  version: 3.2.0
 platform: ruby
 authors:
 - Dominik Richter
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-11-13 00:00:00.000000000 Z
+date: 2019-12-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: json
@@ -146,6 +146,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.17'
+- !ruby/object:Gem::Dependency
+  name: azure_mgmt_security
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.18'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.18'
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement