train-core 3.1.8 → 3.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 49174b0f7d0930b948754aebf097e4d7e86eb81a4c211bcf089a88f03f159bed
-  data.tar.gz: 2d333cba1edba5ffa392bfac19c1719dd0d3f3ef85bf7d37f86f1363c8bc03ee
+  metadata.gz: 753d229b222d0b3db2d8cdaaa8f71232a8eb789c764b26ba57663438ce2b112f
+  data.tar.gz: d11b1c63d3aeb30d3061038948dca02c1a8cc205241865c829d104f53f87ea2c
 SHA512:
-  metadata.gz: fe67e40c52811b843bb2841a1640ef57a5c09180035dce49d2a092e9e13727f91ea9282063e8a002d3b767813fa954beea32ee310a39908bfac123da8d636b52
-  data.tar.gz: 2d42a75181488bc6e076310263b426778d33a88da9dd96c7bf738e5923bbd140dca2ec65726c2bdafd3892141c90a78839f57df304c1a73337cb5410a6e068b5
+  metadata.gz: 8653c7e7c3180d367b60ccc9134d670a74701354cf3009d91a416e6b09dd2fb8112835c010a935f22900e6b27fc974df83d5d7a1c945fc6a6fd7f32f5be13df2
+  data.tar.gz: 8e03272617627c66ee8e27437d3bc052bfaf320c7d6c7eb99ebff240693a70acb086e33d73643d0bf5b0391f490b52a993742e0a04b84c7a76fe2283dd8f1453

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-core
 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