roku_builder 4.23.0 → 4.24.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 373790650070231c745746cd44c6ad316f34746e27fd73a4849b2d02279df624
-  data.tar.gz: de065657de7cf806f4db60a01230eef93d1f92aba581c5456752367e01554900
+  metadata.gz: 5c8cbcadc9f204bd809c6d0b42ef03c0fe462209c493381d654db1b6c4f51b17
+  data.tar.gz: 51b96420ec3a859d6af35a16989bc5560bd6987c84e8ee3c8bbea4b3159aacbf
 SHA512:
-  metadata.gz: 16a02701b5eaed8f0f6dbee5edd0c42d3f26481139384aac9ecc545c96695259d0c4509274b3543cf5fd137c55f9ae19e27dda9f21a0813a725f6d1e0ae870ce
-  data.tar.gz: 01fbc2cd1d4bf08f4ee2f8a8fed846300d0a6629b54cb6a45484af7b4ff0388dde3ebfc84ac7d940bed5697700c5be690bc7491c10fcc366a374c59a8caaf75f
+  metadata.gz: 2247b8355b001e0151255e0ffb45032b92ff1f5adb6e5ec6c98be8013c142d859d689cd056b1e757cda82445bf77cae19b9ab0775419d2d95041a105cc7643ca
+  data.tar.gz: 3459696d4a13f4faf6000c3b2a17a82e392e3935adcee2a77bb312e1ed06fcdbf1ac6e57b4486198128e26235846126fc3508bd61427d52f3a42ffddb536a990

data/README.md

@@ -1,9 +1,9 @@
 # RokuBuilder
 
 [![Gem Version](https://badge.fury.io/rb/roku_builder.svg)](https://badge.fury.io/rb/roku_builder)
-[![Build Status](https://travis-ci.org/ViacomInc/roku_builder.svg?branch=master)](https://travis-ci.org/ViacomInc/roku_builder)
 [![Coverage Status](https://coveralls.io/repos/github/ViacomInc/roku_builder/badge.svg?branch=master)](https://coveralls.io/github/ViacomInc/roku_builder?branch=master)
 [![Code Climate](https://codeclimate.com/github/ViacomInc/roku_builder/badges/gpa.svg)](https://codeclimate.com/github/ViacomInc/roku_builder)
+<!-- [![Build Status](https://travis-ci.org/ViacomInc/roku_builder.svg?branch=master)](https://travis-ci.org/ViacomInc/roku_builder) -->
 
 A tool to help with Roku Development. Assists with many development/deployment
 tasks. More information can be found in the [wiki](https://github.com/ViacomInc/roku_builder/wiki).

data/lib/roku_builder/config_parser.rb

@@ -169,14 +169,17 @@ module RokuBuilder
     end
 
     def setup_key_config
-      if @parsed[:stage]
-        @parsed[:key] = @parsed[:stage][:key]
-        get_global_key_config if @parsed[:key].class == String
-        test_key_file
+      if @options.keyed_command?
+        if @parsed[:stage]
+          @parsed[:key] = @parsed[:stage][:key]
+          get_global_key_config if @parsed[:key].class == String
+          test_key_file
+        end
       end
     end
 
     def get_global_key_config
+      raise ParseError, "No Keys Configured" unless @config[:keys]
       raise ParseError, "Unknown Key: #{@parsed[:key]}" unless @config[:keys][@parsed[:key].to_sym]
       @parsed[:key] = @config[:keys][@parsed[:key].to_sym].dup
       if @config[:keys][:key_dir]  and !@parsed[:key][:keyed_pkg].start_with?("./")

data/lib/roku_builder/config_validator.rb

@@ -26,7 +26,7 @@ module RokuBuilder
     KEY_MISSING_PATH          = 19
     KEY_MISSING_PASSWORD      = 20
     INVALID_MAPPING_INFO      = 21
-    MISSING_KEY               = 22
+    #                         = 22
     MISSING_STAGE_METHOD      = 23
 
     def initialize(config:)
@@ -153,7 +153,6 @@ module RokuBuilder
       errors= [
         [STAGE_MISSING_BRANCH, (!stage[:branch] and project[:stage_method] == :git)],
         [STAGE_MISSING_SCRIPT, (!stage[:script] and project[:stage_method] == :script)],
-        [MISSING_KEY, (!!stage[:key] and stage[:key].class == String and (!@config[:keys] or !@config[:keys][stage[:key].to_sym]))]
       ]
       process_errors(errors: errors)
     end

data/lib/roku_builder/options.rb

@@ -31,6 +31,10 @@ module RokuBuilder
       device_commands.include?(command)
     end
 
+    def keyed_command?
+      keyed_commands.include?(command)
+    end
+
     def has_source?
       !(keys & sources).empty?
     end
@@ -41,7 +45,7 @@ module RokuBuilder
       RokuBuilder.plugins.each do |plugin|
         plugin.commands.each do |command, attributes|
           commands << command
-          [:device, :source, :exclude].each do |type|
+          [:device, :source, :exclude, :keyed].each do |type|
             if attributes[type]
               send("#{type}_commands".to_sym) << command
             end
@@ -148,5 +152,11 @@ module RokuBuilder
     def device_commands
       @device_commands ||= []
     end
+
+    # List of commands that require a key
+    # @return [Array<Symbol>] List of commands that require a key
+    def keyed_commands
+      @keyed_commands ||= []
+    end
   end
 end

data/lib/roku_builder/plugins/core.rb

@@ -8,7 +8,7 @@ module RokuBuilder
     def self.commands
       {
         configure: {},
-        validate: {},
+        validate: {keyed: true},
         increment: {source: true, stage: true},
         dostage: {source: true},
         dounstage: {source: true}

data/lib/roku_builder/plugins/indentation_inspector.rb

@@ -14,7 +14,7 @@ module RokuBuilder
     end
 
     def check_line(line:, number:, comment:)
-      #byebug if number == 10 and @path.ends_with?(".brs")
+      #byebug if number == 190 and @path.ends_with?("ScreenManager.brs")
       set_indentation(line: line)
       regexp = /^#{@character}{#{@ind}}[^#{@character}]/
       unless line =~ regexp or line == "\n" or line =~ /\'indent-ignore/ or comment
@@ -47,10 +47,12 @@ module RokuBuilder
               @ind += @count
             elsif @prev_line =~ /:\s*\bfunction\b|:\s*\bsub\b/i
               @ind += @count
+            elsif @prev_line =~ /=\s*\bfunction\b|=\s*\bsub\b/i
+              @ind += @count
             elsif @prev_line =~ /^\s*\bfunction\b|^\s*\bsub\b/i
               @ind += @count
             elsif @prev_line =~ /^\s*#?if\b|^\s*#?else\b/i
-              unless @prev_line =~ /\bthen\b[ \t ]*[^' \r\n']+.*$/i or @prev_line =~ /\breturn\b/i
+              unless @prev_line =~ /\bthen\b[ \t]*[^' \r\n]+.*$/i or @prev_line =~ /\breturn\b/i
                 @ind += @count
               end
             elsif @prev_line =~ /^\s*\bfor\b|^\s*\bwhile\b/i

data/lib/roku_builder/plugins/packager.rb

@@ -7,9 +7,9 @@ module RokuBuilder
 
     def self.commands
       {
-        package: {device: true, source: true, stage: true, exclude: true},
+        package: {device: true, source: true, stage: true, exclude: true, keyed: true},
         genkey: {device: true},
-        key: {device: true, source: true}
+        key: {device: true, source: true, keyed: true}
       }
     end
 

data/lib/roku_builder/plugins/profiler.rb

@@ -138,8 +138,8 @@ module RokuBuilder
       lines = get_command_response(command: "sgnodes all", start_reg: start_reg, end_reg: end_reg)
       xml_string = lines.join("\n")
       stats = {"Total" => {count: 0}}
-      doc = Oga.parse_xml(xml_string)
-      handle_node(stats: stats, node: doc.children.first)
+      doc = Nokogiri::XML(xml_string)
+      handle_node(stats: stats, node: doc.root)
       stats
     end
 
@@ -171,8 +171,7 @@ module RokuBuilder
 
     def handle_node(stats:,  node:)
       if node
-        node.children.each do |element|
-          next unless element.class == Oga::XML::Element
+        node.element_children.each do |element|
           attributes = element.attributes.map{|attr| {"#{attr.name}": attr.value}}.reduce({}, :merge)
           stats[element.name] ||= {count: 0, ids: []}
           stats[element.name][:count] += 1

data/lib/roku_builder/version.rb

@@ -2,5 +2,5 @@
 
 module RokuBuilder
   # Version of the RokuBuilder Gem
-  VERSION = "4.23.0"
+  VERSION = "4.24.1"
 end

data/lib/roku_builder.rb

@@ -13,7 +13,7 @@ require 'json'
 require 'pstore'
 require "git"
 #profiler
-require 'oga'
+require 'nokogiri'
 #navigator
 require 'io/console'
 #monitor

data/roku_builder.gemspec

@@ -18,7 +18,7 @@ Gem::Specification.new do |spec|
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ["lib"]
 
-  spec.required_ruby_version = "~> 2.3"
+  spec.required_ruby_version = "~> 3.0"
 
   spec.add_dependency "rubyzip",             "~> 1.2"
   spec.add_dependency "faraday",             "~> 0.13"
@@ -26,7 +26,7 @@ Gem::Specification.new do |spec|
   spec.add_dependency "git",                 "~> 1.3"
   spec.add_dependency "net-ping",            "~> 2.0"
   spec.add_dependency "net-telnet",          "~> 0.1"
-  spec.add_dependency "oga",                 "~> 2.11"
+  spec.add_dependency "nokogiri",            "~> 1.12"
   spec.add_dependency "win32-security",      "~> 0.5" # For windows compatibility
   spec.add_dependency "image_size",          "~> 2.0"
 

data/test/roku_builder/plugins/test_monitor.rb

@@ -138,7 +138,7 @@ module RokuBuilder
       all_text = "midline "
       txt = "split\nBrightScript Debugger> "
 
-      result = @monitor.send(:manage_text, {all_text: all_text, txt: txt})
+      result = @monitor.send(:manage_text, all_text: all_text, txt: txt)
 
       assert_equal "", result
 
@@ -148,7 +148,7 @@ module RokuBuilder
 
     def test_monitor_manage_text_connection_used
       assert_raises ExecutionError do
-        @monitor.send(:manage_text, {all_text: "", txt: "Console connection is already in use."})
+        @monitor.send(:manage_text, all_text: "", txt: "Console connection is already in use.")
       end
     end
   end

data/test/roku_builder/plugins/test_tester.rb

@@ -67,7 +67,7 @@ module RokuBuilder
       tester = Tester.new(config: config)
 
       text = "this\nis\na\ntest\nparagraph"
-      tester.send(:handle_text, {txt: text})
+      tester.send(:handle_text, txt: text)
 
       refute tester.instance_variable_get(:@in_tests)
     end
@@ -79,7 +79,7 @@ module RokuBuilder
 
       text = ["this","is","a","test","paragraph"]
 
-      tester.send(:handle_text, {txt: text.join("\n")})
+      tester.send(:handle_text, txt: text.join("\n"))
       assert_equal text, tester.instance_variable_get(:@logs)
       assert tester.instance_variable_get(:@in_tests)
     end
@@ -91,7 +91,7 @@ module RokuBuilder
       text = ["this","*Start testing*","is","a","test","*End testing*","paragraph"]
       verify_text = ["***************","***************","*Start testing*","is","a","test","*End testing*","*************","*************"]
 
-      tester.send(:handle_text, {txt: text.join("\n")})
+      tester.send(:handle_text, txt: text.join("\n"))
       refute tester.instance_variable_get(:@in_tests)
       assert_equal verify_text, tester.instance_variable_get(:@logs)
     end
@@ -103,7 +103,7 @@ module RokuBuilder
       text = ["connection already in use"]
 
       assert_raises IOError do
-        tester.send(:handle_text, {txt: text.join("\n")})
+        tester.send(:handle_text, txt: text.join("\n"))
       end
     end
   end

data/test/roku_builder/test_config_validator.rb

@@ -167,13 +167,6 @@ module RokuBuilder
       assert_equal [15, -1], validator.instance_variable_get(:@codes)
     end
 
-    def test_config_manager_validate_project_key
-      config = good_config
-      config[:projects][:project2][:stages][:production][:key] = "b"
-      validator = ConfigValidator.new(config: config)
-      assert_equal [22], validator.instance_variable_get(:@codes)
-    end
-
     def test_config_manager_validate_keys_pkg
       config = good_config
       config[:keys][:a][:keyed_pkg] = nil

data/test/roku_builder/test_logger.rb

@@ -42,7 +42,7 @@ module RokuBuilder
     def test_logger_testing
       Logger.set_testing
       logger = Logger.instance
-      assert_equal "/dev/null", logger.instance_variable_get(:@logdev).instance_variable_get(:@filename)
+      assert_nil logger.instance_variable_get(:@logdev).instance_variable_get(:@filename)
     end
   end
 end

data/test/roku_builder/test_options.rb

@@ -26,7 +26,7 @@ module RokuBuilder
       parse_stub = lambda{ count+= 1; {validate: true} }
       options = Options.allocate
       options.stub(:parse, parse_stub) do
-        options.send(:initialize, {options: {validate: true}})
+        options.send(:initialize, options: {validate: true})
       end
       assert_equal 0, count
     end
@@ -133,7 +133,7 @@ module RokuBuilder
       options = Options.allocate
       options.stub(:source_command?, true) do
         assert_raises InvalidOptions do
-          options.send(:initialize, {options: {validate: true}})
+          options.send(:initialize, options: {validate: true})
           options.validate
         end
       end
@@ -164,7 +164,7 @@ module RokuBuilder
       logger.expect(:warn, nil, ["Depricated"])
       Logger.class_variable_set(:@@instance, logger)
       options.stub(:depricated_options, {validate: "Depricated"}) do
-        options.send(:initialize, {options: {validate: true}})
+        options.send(:initialize, options: {validate: true})
         options.validate
       end
       logger.verify

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: roku_builder
 version: !ruby/object:Gem::Version
-  version: 4.23.0
+  version: 4.24.1
 platform: ruby
 authors:
 - greeneca
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-09-23 00:00:00.000000000 Z
+date: 2022-01-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubyzip
@@ -95,19 +95,19 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0.1'
 - !ruby/object:Gem::Dependency
-  name: oga
+  name: nokogiri
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.11'
+        version: '1.12'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.11'
+        version: '1.12'
 - !ruby/object:Gem::Dependency
   name: win32-security
   requirement: !ruby/object:Gem::Requirement
@@ -626,14 +626,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - "~>"
     - !ruby/object:Gem::Version
-      version: '2.3'
+      version: '3.0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.8
+rubygems_version: 3.2.3
 signing_key:
 specification_version: 4
 summary: Build Tool for Roku Apps