kinetic_sdk 7.0.0.rc2 → 7.0.0.rc3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d17b093333b568589b62518a2d39f94d34bc0a82833a18df3aa0fae4c9e4d913
-  data.tar.gz: e90ffa58599c343b80e7907102f3a7f858035f1150de4a8fd0a3f5b151dcf66a
+  metadata.gz: 7728373bd9c4a15c643c54aed99acd289e5f776ee0819c8b14f7aea9ec54c106
+  data.tar.gz: d22c5e6907360c11cb886dfb2c58f5c80c5434fcd2b8d669f846b074a57d78ae
 SHA512:
-  metadata.gz: 78a37fe4b1776e46ebe64d64e4c779967cec791292474378155cca09a835a5041e12deea60ba28d7f713621ded57f2dfeb99b50cd14e9d5b802354a885a9bb70
-  data.tar.gz: c3f15b84bbb614618944866ab1f3d5f59eea9875a1ef23a3e4452079eca425e9d5a0eae872345606e063d6be3b35654791b348ba4019a382646afd44afde1637
+  metadata.gz: 0eaa0e7c58dead1010dbd1bd7964f42cf762a712ab7bc888037e6a8e6709b2af1b2562df834ed0d759cc1e677473ffacad035582b10542baaaf45d58401c4979
+  data.tar.gz: f882b337544ec35443094eb971e821859d9480994ab7200519f090607672bc4e35ae284a04bb84aa2ffdc7fb15f1e6755c9712905c9188c518bb3751a06fdde3

data/CHANGELOG.md

@@ -1,6 +1,36 @@
 # Change Log
 
 
+## [7.0.0.rc3](https://github.com/kineticdata/kinetic-sdk-rb/tree/7.0.0.rc3) (2026-09-01)
+
+**Fixed bugs:**
+
+- Fixed a crash in `import_trees` and `import_trees_threaded` that aborted template installs
+  with `NoMethodError: undefined method 'document' for an instance of File`. The tree
+  comparison introduced in 7.x passed a raw `File` handle to `REXML::XPath.first`, which
+  requires a parsed document.
+- Removed the tree comparison entirely, restoring the unconditional import behavior of 5.0.29.
+  The comparison identified trees by a `//definitionId` element that exists only in routine
+  exports, never in the tree exports `import_trees` reads, so it could not succeed. See the
+  specs in `spec/kinetic_sdk/task/trees_spec.rb`.
+- The per-tree and per-routine rescues in the threaded importers no longer discard exceptions
+  silently; failures are logged and propagated.
+- `trees.rb` now requires `rexml/document` explicitly rather than relying on load order.
+
+**Implemented enhancements:**
+
+- Added rspec and the first unit test coverage, run with `bundle exec rspec` or `rake spec`.
+
+
+## [7.0.0.rc2](https://github.com/kineticdata/kinetic-sdk-rb/tree/7.0.0.rc2) (2026-09-01)
+
+**Fixed bugs:**
+
+- Declared `concurrent-ruby` and `rexml` as runtime dependencies. Both were required by the
+  library but missing from the gemspec, so the published gem could fail to load.
+- Pinned `BUNDLED WITH` to a concrete Bundler version; the range value broke `bundle install`.
+
+
 ## [7.0.0-rc1](https://github.com/kineticdata/kinetic-sdk-rb/tree/7.0.0-rc1) (2026-08-31)
 
 **Breaking changes:**

data/Rakefile

@@ -7,7 +7,19 @@ YARD::Rake::YardocTask.new do |t|
   t.stats_options = %w( --list-undoc )
 end
 
+# Run the specs. Guarded so the Rakefile still loads (and `rake build` still
+# works) in an environment where the development dependencies are absent.
+begin
+  require "rspec/core/rake_task"
+  RSpec::Core::RakeTask.new(:spec)
+rescue LoadError
+  desc "Run the specs (rspec is not installed)"
+  task :spec do
+    abort "rspec is not available. Run `bundle install` first."
+  end
+end
+
 # Generate Yard documentation
 task :doc => [:yard]
 task :rdoc => [:yard]
-task :default => [:doc]
+task :default => [:spec, :doc]

data/kinetic_sdk.gemspec

@@ -26,6 +26,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "bundler", ">= 1.17"
   spec.add_development_dependency "rake", ">= 13.0.1"
   spec.add_development_dependency "yard", "~> 0.9.45"
+  spec.add_development_dependency "rspec", "~> 3.13"
 
   spec.metadata["yard.run"] = "yri"
 end

data/lib/kinetic_sdk/task/lib/trees.rb

@@ -1,3 +1,5 @@
+require 'rexml/document'
+
 module KineticSdk
   class Task
     # Delete a tree.
@@ -146,12 +148,7 @@ module KineticSdk
       @logger.info("Importing all Trees from Export Directory")
       Dir["#{@options[:export_directory]}/sources/**/*.xml"].sort.each do |file|
         tree_file = File.new(file, "rb")
-        match = compare_trees(tree_file)
-        if !match
-          import_tree(tree_file, force_overwrite, headers)
-        else
-          #Matching trees, skipping
-        end
+        import_tree(tree_file, force_overwrite, headers)
       end
     end
     def import_trees_threaded(force_overwrite=false, headers=header_basic_auth)
@@ -164,48 +161,18 @@ module KineticSdk
       Dir["#{@options[:export_directory]}/sources/**/*.xml"].sort.each do |file|
         promises << Concurrent::Promise.execute(executor: pool) do
           begin
-
             tree_file = File.new(file, "rb")
-            match = compare_trees(tree_file)
-            if !match
-              import_tree(tree_file, force_overwrite, headers)
-            else
-              #Matching trees, skipping
-            end
-          rescue
+            import_tree(tree_file, force_overwrite, headers)
+          rescue => e
+            @logger.error("Failed to import Tree #{file}: #{e.class}: #{e.message}")
+            raise
           end
-      
         end
       end
       promises.each(&:wait!)
       pool.shutdown
       pool.wait_for_termination        
     end
-    # Import a tree
-    #
-    # If the tree already exists on the server, this will fail unless forced
-    # to overwrite.
-    #
-    # The source named in the tree content must also exist on the server, or
-    # the import will fail.
-    #
-    # @param tree [String] content from tree file
-    # @param force_overwrite [Boolean] whether to overwrite a tree if it exists, default is false
-    # @param headers [Hash] hash of headers to send, default is basic authentication
-    # @return [KineticSdk::Utils::KineticHttpResponse] object, with +code+, +message+, +content_string+, and +content+ properties
-    def compare_trees(tree, headers=header_basic_auth)
-      definitionId = REXML::XPath.first(tree, "//definitionId")&.text
-      @logger.info("Querying for Tree #{definitionId}")
-      response = find_tree(definitionId, { "include" => "details,export" })
-      @logger.info("Comparing new and old tree")
-      prev_tree = response.content
-      if prev_tree == tree
-        return true
-      else
-        return false
-      end
-    end
-
     # Import a routine
     #
     # If the routine already exists on the server, this will fail unless
@@ -248,9 +215,11 @@ module KineticSdk
       Dir["#{@options[:export_directory]}/routines/*.xml"].sort.each do |file|
         promises << Concurrent::Promise.execute(executor: pool) do
           begin
-          routine_file = File.new(file, "rb")
-          import_routine(routine_file, force_overwrite, headers)
-          rescue
+            routine_file = File.new(file, "rb")
+            import_routine(routine_file, force_overwrite, headers)
+          rescue => e
+            @logger.error("Failed to import Routine #{file}: #{e.class}: #{e.message}")
+            raise
           end
         end
       end

data/lib/kinetic_sdk/version.rb

@@ -3,5 +3,5 @@ module KineticSdk
   # Version of Kinetic SDK
   #
   # @return [String] Version of the SDK
-  VERSION = "7.0.0.rc2"
+  VERSION = "7.0.0.rc3"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: kinetic_sdk
 version: !ruby/object:Gem::Version
-  version: 7.0.0.rc2
+  version: 7.0.0.rc3
 platform: ruby
 authors:
 - Kinetic Data
@@ -122,6 +122,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 0.9.45
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.13'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.13'
 description:
 email:
 - support@kineticdata.com