roku_builder 4.12.3 → 4.13.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b477e5e27ca5a3829a2fa7a599996a770f381515
-  data.tar.gz: 186cb75c575ff093aa0d0909502e9719fc74af79
+  metadata.gz: 4e77383c33c69d2858c263393e4e4ae038e2a8c9
+  data.tar.gz: 4ab74464ad9fcc48e0b2dd991d0799220de1802c
 SHA512:
-  metadata.gz: c88aa67dac26c6eeac9ee3488e75eae5a5a96e9c838f575fc9e1119f177311b6a071a191f35a94191013523d9ba8baf218da1e02fec45bc186d568fcdb2762eb
-  data.tar.gz: f1c9e106d0c567c02b99e27e22af9c309e68e3ac12ab93ac8fcf6cf54b06ce34fe56d796d629e52f109f432fa6a1d8d298900120c2708eb997b4fae827612914
+  metadata.gz: 5e3df3b2a6f7f138b6600c04d48de8bf46213e37fea263fd0c8090b53530690659c79aeddfa165a84ccc1bfa848698fc6bd4d0a5f0d4cfdb425bee4b423c8046
+  data.tar.gz: 3700e5da2d298fe907dd4638213daa4c128ace3c740de0b5db31f44295d750c507eb82cd2c3195c923ea95e8a4f1f9e7c534d4ce594722613be5414949ad3d47

data/CHANGELOG

@@ -1,3 +1,13 @@
+ = 4.13.0 =
+
+ - Add combined source_files project config key
+ - Depricate files and folders keys
+ - Pass options to post_config_load hook
+
+ = 4.12.3 =
+
+ - Fix Windows incompatibility
+
  = 4.12.2 =
 
  - Fix Windows incompatibility

data/config.json.example

@@ -12,8 +12,7 @@
     "project_dir": "<optional parent directory for all projects>",
     "<project id>": {
       "directory": "<path/to/repo>",
-      "folders": ["resources","source"],
-      "files": ["manifest"],
+      "source_files": ["resources","source", "manifest"],
       "excludes": [],
       "app_name": "<app name>",
       "stage_method": "<git|script>",

data/lib/roku_builder/config.rb

@@ -97,7 +97,7 @@ module RokuBuilder
       merge_local_config
       expand_repeatable_stages
       fix_config_symbol_values
-      RokuBuilder.process_hook(hook: "post_config_load", params: {config: @config})
+      RokuBuilder.process_hook(hook: "post_config_load", params: {config: @config, options: @options})
     end
 
     def read_config(io)

data/lib/roku_builder/config_parser.rb

@@ -120,6 +120,7 @@ module RokuBuilder
         @parsed[:project] = @config[:projects][@options[:project].to_sym].dup
         raise ParseError, "Unknown Project: #{@options[:project]}" unless @parsed[:project]
         set_project_directory
+        convert_to_source_files
         check_for_working
       end
     end
@@ -130,8 +131,7 @@ module RokuBuilder
       raise ParseError, "Missing Manifest: #{manifest}" unless File.exist?(manifest)
       @parsed[:project] = {
         directory: pwd,
-        folders: nil,
-        files: nil,
+        source_files: nil,
         stage_method: :current
       }
     end
@@ -145,6 +145,14 @@ module RokuBuilder
       end
     end
 
+    def convert_to_source_files
+      unless @parsed[:project][:source_files]
+        @parsed[:project][:source_files] = @parsed[:project][:files] + @parsed[:project][:folders]
+        @parsed[:project].delete(:files)
+        @parsed[:project].delete(:folders)
+      end
+    end
+
     def check_for_working
       @parsed[:project][:stage_method] = :working if @options[:working]
     end

data/lib/roku_builder/config_validator.rb

@@ -3,6 +3,7 @@
 module RokuBuilder
   class ConfigValidator
 
+    DEPRICATED_FILES_FOLDERS  = -1
     VALID_CONFIG              = 0
     MISSING_DEVICES           = 1
     MISSING_DEVICES_DEFAULT   = 2
@@ -15,7 +16,7 @@ module RokuBuilder
     DEVICE_MISSING_PASSWORD   = 9
     PROJECT_MISSING_APP_NAME  = 10
     PROJECT_MISSING_DIRECTORY = 11
-    PROJECT_MISSING_FOLDERS   = 12
+    #                         = 12
     PROJECT_FOLDERS_BAD       = 13
     PROJECT_MISSING_FILES     = 14
     PROJECT_FILES_BAD         = 15
@@ -138,12 +139,12 @@ module RokuBuilder
       errors= [
         [PROJECT_MISSING_APP_NAME, (!project[:app_name])],
         [PROJECT_MISSING_DIRECTORY, (!project[:directory])],
-        [PROJECT_MISSING_FOLDERS, (!project[:folders])],
         [PROJECT_FOLDERS_BAD, (project[:folders] and !project[:folders].is_a?(Array))],
-        [PROJECT_MISSING_FILES, (!project[:files])],
+        [PROJECT_MISSING_FILES, (!project[:source_files] and !(project[:files] and project[:folders]))],
         [PROJECT_FILES_BAD, (project[:files] and !project[:files].is_a?(Array))],
         [MISSING_STAGE_METHOD, ( !project[:stage_method])],
-        [PROJECT_STAGE_METHOD_BAD, (![:git, :script, nil].include?(project[:stage_method]))]
+        [PROJECT_STAGE_METHOD_BAD, (![:git, :script, nil].include?(project[:stage_method]))],
+        [DEPRICATED_FILES_FOLDERS, (project[:files] or project[:folders])],
       ]
       process_errors(errors: errors)
     end
@@ -195,7 +196,7 @@ module RokuBuilder
         "A device config is missing its password.",
         "A project config is missing its app_name.", #10
         "A project config is missing its directorty.",
-        "A project config is missing its folders.",
+        "",
         "A project config's folders is not an array.",
         "A project config is missing its files.",
         "A project config's files is not an array.", #15
@@ -206,7 +207,9 @@ module RokuBuilder
         "A key is missing its password.", #20
         "A input mapping is invalid",
         "A key is missing from the keys section",
-        "A project is missing its stage method."
+        "A project is missing its stage method.",
+        #===============Warnings===============#
+        "Using depricated files/folders projects keys. Use the combined source_files instead" #-1
       ]
     end
   end

data/lib/roku_builder/plugins/loader.rb

@@ -104,14 +104,13 @@ module RokuBuilder
     end
 
     def setup_build_content()
-      content = {}
-      content[:excludes] = []
+      content = {
+        excludes: []
+      }
       if @options[:current]
-        content[:folders] = Dir.entries(@config.root_dir).select {|entry| File.directory? File.join(@config.root_dir, entry) and !(entry =='.' || entry == '..') }
-        content[:files] = Dir.entries(@config.root_dir).select {|entry| File.file? File.join(@config.root_dir, entry)}
+        content[:source_files] = Dir.entries(@config.root_dir).select {|entry| !(entry =='.' || entry == '..') }
       else
-        content[:folders] = @config.project[:folders]
-        content[:files] = @config.project[:files]
+        content[:source_files] = @config.project[:source_files]
         content[:excludes] = @config.project[:excludes] if @config.project[:excludes] and (@options[:exclude] or @options.exclude_command?)
       end
       content
@@ -121,20 +120,7 @@ module RokuBuilder
       path = file_path(:out)
       File.delete(path) if File.exist?(path)
       io = Zip::File.open(path, Zip::File::CREATE)
-      # Add folders to zip
-      content[:folders].each do |folder|
-        base_folder = File.join(@config.root_dir, folder)
-        if File.exist?(base_folder)
-          entries = Dir.entries(base_folder)
-          entries.delete(".")
-          entries.delete("..")
-          writeEntries(@config.root_dir, entries, folder, content[:excludes], io)
-        else
-          @logger.warn "Missing Folder: #{base_folder}"
-        end
-      end
-      # Add file to zip
-      writeEntries(@config.parsed[:root_dir], content[:files], "", content[:excludes], io)
+      writeEntries(@config.parsed[:root_dir], content[:source_files], "", content[:excludes], io)
       io.close()
     end
 
@@ -168,13 +154,11 @@ module RokuBuilder
       }
     end
     def copy_channel_files(content)
-      [:files, :folders].each do |type|
-        content[type].each do |entity|
-          begin
-            FileUtils.copy_entry(File.join(@config.parsed[:root_dir], entity), File.join(@target, entity))
-          rescue Errno::ENOENT
-            @logger.warn "Missing Entry: #{entity}"
-          end
+      content[:source_files].each do |entity|
+        begin
+          FileUtils.copy_entry(File.join(@config.parsed[:root_dir], entity), File.join(@target, entity))
+        rescue Errno::ENOENT
+          @logger.warn "Missing Entry: #{entity}"
         end
       end
     end

data/lib/roku_builder/version.rb

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

data/test/roku_builder/plugins/test_analyzer.rb

@@ -260,7 +260,7 @@ module RokuBuilder
     end
     def test_macosx_directory
       config = good_config(AnalyzerTest)
-      config[:projects][:project1][:folders].push("Test__MACOSX")
+      config[:projects][:project1][:source_files].push("Test__MACOSX")
       @config, @options = build_config_options_objects(AnalyzerTest, {analyze: true, working: true}, false, config)
       folder = File.join(@root_dir, "Test__MACOSX")
       Dir.mkdir(folder) unless File.exist?(folder)

data/test/roku_builder/test_config_validator.rb

@@ -137,21 +137,32 @@ module RokuBuilder
 
     def test_config_manager_validate_project_folders
       config = good_config
+      config[:projects][:project1][:files] = ["manifest"]
       config[:projects][:project1][:folders] = nil
+      config[:projects][:project1][:source_files] = nil
       validator = ConfigValidator.new(config: config)
-      assert_equal [12], validator.instance_variable_get(:@codes)
+      assert_equal [14, -1], validator.instance_variable_get(:@codes)
     end
 
     def test_config_manager_validate_project_folders_is_array
       config = good_config
       config[:projects][:project1][:folders] = "Folders"
       validator = ConfigValidator.new(config: config)
-      assert_equal [13], validator.instance_variable_get(:@codes)
+      assert_equal [13, -1], validator.instance_variable_get(:@codes)
     end
 
     def test_config_manager_validate_project_files
       config = good_config
       config[:projects][:project1][:files] = nil
+      config[:projects][:project1][:folders] = ["images", "source"]
+      config[:projects][:project1][:source_files] = nil
+      validator = ConfigValidator.new(config: config)
+      assert_equal [14, -1], validator.instance_variable_get(:@codes)
+    end
+
+    def test_config_manager_validate_project_source_files
+      config = good_config
+      config[:projects][:project1][:source_files] = nil
       validator = ConfigValidator.new(config: config)
       assert_equal [14], validator.instance_variable_get(:@codes)
     end
@@ -160,7 +171,7 @@ module RokuBuilder
       config = good_config
       config[:projects][:project1][:files] = "Files"
       validator = ConfigValidator.new(config: config)
-      assert_equal [15], validator.instance_variable_get(:@codes)
+      assert_equal [15, -1], validator.instance_variable_get(:@codes)
     end
 
     def test_config_manager_validate_project_key

data/test/roku_builder/test_helper.rb

@@ -77,8 +77,7 @@ def good_config(klass=nil)
     default: :project1,
     project1: {
     directory: root_dir,
-    folders: ["images","source"],
-    files: ["manifest"],
+    source_files: ["manifest", "images", "source"],
     app_name: "<app name>",
     stage_method: :git,
     stages:{
@@ -93,8 +92,7 @@ def good_config(klass=nil)
   },
     project2: {
     directory: root_dir,
-    folders: ["images","source"],
-    files: ["manifest"],
+    source_files: ["images","source","manifest"],
     app_name: "<app name>",
     stage_method: :script,
     stages:{

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: roku_builder
 version: !ruby/object:Gem::Version
-  version: 4.12.3
+  version: 4.13.0
 platform: ruby
 authors:
 - greeneca