chronicle-etl 0.5.0 → 0.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 951fca4c6238d773ec8bc2b9ea474a0cffdabf0c2f5d0c925f78b91b35836224
-  data.tar.gz: 908a7f01fb215cca9936f072b71315c3b62e0d00b3c8f7ffd938682a4cabe42c
+  metadata.gz: 2e9f565004fce3539bdcfec9b98f64e33d8d4f1dde532f573adf54bfb61eadfa
+  data.tar.gz: 81da054f627ae084d45b4fe53480d5ef26fadf9cfd3ad4eb53d908741d8214e4
 SHA512:
-  metadata.gz: 28fc97935e5bd9538877a2057f3201170fdb1eb574385ae6d94901b21abfa5f923618d5fb2caf94395503ec70c0052b607a939b363f27630aaca26df6ca93722
-  data.tar.gz: 0b8e4dedb79e6cbd23487e2c4482d9a8ad9d1653e015593e6b83cac854d94a6cd4702862eebb11ec9f41e63b774eb6f41db929a1ceb12055af6cb08209a6b8eb
+  metadata.gz: ffe35b9c55a442610b9832b7f918f16dff0ccad3245248f76c52afbd88c43d671e8724f72a8d181a13e3cafffae6bc97f5963f65eb5c2a0ddb7080d9b6f1d145
+  data.tar.gz: c063698040b906f99e57cc3f9e829ff8b8ca1d4e5314a3a4179dc965c17ba70da04155d01b0b1fadf573e6a110a69862e9651435bdcdb1160bfe84cb8a55a10a

data/lib/chronicle/etl/cli/jobs.rb

@@ -59,7 +59,7 @@ LONG_DESC
         rescue Chronicle::ETL::JobDefinitionError => e
           message = ""
           job_definition.errors.each_pair do |category, errors|
-            message << "Problem with #{category}:\n  - #{errors.map(&:to_s).join("\n  -")}"
+            message << "Problem with #{category}:\n  - #{errors.map(&:to_s).join("\n  - ")}"
           end
           cli_fail(message: "Error running job.\n#{message}", exception: e)
         end
@@ -112,6 +112,8 @@ LONG_DESC
         private
 
         def run_job(job_definition)
+          # FIXME: have to validate here so next method can work. This is clumsy
+          job_definition.validate!
           # FIXME: clumsy to make CLI responsible for setting secrets here. Think about a better way to do this
           job_definition.apply_default_secrets
 

data/lib/chronicle/etl/loaders/json_loader.rb

@@ -9,10 +9,13 @@ module Chronicle
       setting :output, default: $stdout
 
       def start
-        if @config.output == $stdout
-          @output = @config.output
+        if @config.output.is_a?(IO)
+          # This might seem like a duplication of the default value ($stdout)
+          # but it's because rspec overwrites $stdout (in helper #capture) to
+          # capture output.
+          @output = $stdout.dup
         else
-          @output = File.open(@config.output, "w")
+          @output = File.open(@config.output, "w+")
         end
       end
 
@@ -31,7 +34,7 @@ module Chronicle
       end
 
       def finish
-        @output.close
+        @output.close if @output.is_a?(IO)
       end
 
       private

data/lib/chronicle/etl/runner.rb

@@ -50,7 +50,7 @@ class Chronicle::ETL::Runner
       transformer = @job.instantiate_transformer(extraction)
       record = transformer.transform
 
-      Chronicle::ETL::Logger.info(tty_log_transformation(transformer))
+      Chronicle::ETL::Logger.debug(tty_log_transformation(transformer))
       @job_logger.log_transformation(transformer)
 
       @loader.load(record) unless @job.dry_run?

data/lib/chronicle/etl/version.rb

@@ -1,5 +1,5 @@
 module Chronicle
   module ETL
-    VERSION = "0.5.0"
+    VERSION = "0.5.1"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: chronicle-etl
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.5.1
 platform: ruby
 authors:
 - Andrew Louis
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2022-03-24 00:00:00.000000000 Z
+date: 2022-03-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport