remocon 0.1.0 → 0.2.0

data/bin/get_access_token

@@ -3,8 +3,10 @@
 set -eu
 
 if type easy_install >/dev/null 2>&1; then
+    sudo easy_install --upgrade oauth2client
     sudo easy_install --upgrade google-api-python-client
 else
+    sudo pip install --upgrade oauth2client
     sudo pip install --upgrade google-api-python-client
 fi
 

data/bin/get_access_token.py

@@ -7,7 +7,7 @@ from oauth2client.service_account import ServiceAccountCredentials
 
 def print_access_token():
   argv = sys.argv
-	if (len(argv) != 2):
+  if (len(argv) < 1):
     print >> sys.stderr, 'a filepath to service-account.json must be specified as the 1st argument.'
     quit()
   credentials = ServiceAccountCredentials.from_json_keyfile_name(
@@ -15,4 +15,4 @@ def print_access_token():
   access_token_info = credentials.get_access_token()
   print(access_token_info.access_token)
 
-print_access_token
+print_access_token()

data/cmd/remocon

@@ -1,6 +1,6 @@
 #!/usr/bin/env ruby
 # frozen_string_literal: true
 
-require 'remocon'
+require "remocon"
 
 Remocon::CLI.start(ARGV)

data/lib/remocon.rb

@@ -1,47 +1,48 @@
 # frozen_string_literal: true
 
-require 'yaml'
-require 'json'
-require 'thor'
-require 'active_support'
-require 'active_support/core_ext'
-require 'singleton'
-require 'open-uri'
-require 'fileutils'
-require 'net/http'
-
-require 'remocon/util/array'
-require 'remocon/util/hash'
-require 'remocon/util/string'
-
-require 'remocon/version'
-require 'remocon/type'
-
-require 'remocon/error/unsupported_type_error'
-require 'remocon/error/validation_error'
-
-require 'remocon/sorter/condition_sorter'
-require 'remocon/sorter/parameter_sorter'
-
-require 'remocon/dumper/condition_file_dumper'
-require 'remocon/dumper/parameter_file_dumper'
-
-require 'remocon/interpreter/condition_file_interpreter'
-require 'remocon/interpreter/parameter_file_interpreter'
-
-require 'remocon/normalizer/normalizer'
-require 'remocon/normalizer/boolean_normalizer'
-require 'remocon/normalizer/integer_normalizer'
-require 'remocon/normalizer/json_normalizer'
-require 'remocon/normalizer/string_normalizer'
-require 'remocon/normalizer/void_normalizer'
-require 'remocon/normalizer/type_normalizer_factory'
-
-require 'remocon/command/lib/interpreter_helper'
-
-require 'remocon/command/create_command'
-require 'remocon/command/pull_command'
-require 'remocon/command/push_command'
-require 'remocon/command/validate_command'
-
-require 'remocon/cli'
+require "yaml"
+require "json"
+require "thor"
+require "active_support"
+require "active_support/core_ext"
+require "singleton"
+require "open-uri"
+require "fileutils"
+require "net/http"
+
+require "remocon/util/array"
+require "remocon/util/hash"
+require "remocon/util/string"
+
+require "remocon/version"
+require "remocon/type"
+
+require "remocon/error/unsupported_type_error"
+require "remocon/error/validation_error"
+
+require "remocon/sorter/condition_sorter"
+require "remocon/sorter/parameter_sorter"
+
+require "remocon/dumper/condition_file_dumper"
+require "remocon/dumper/parameter_file_dumper"
+
+require "remocon/interpreter/condition_file_interpreter"
+require "remocon/interpreter/parameter_file_interpreter"
+
+require "remocon/normalizer/normalizer"
+require "remocon/normalizer/boolean_normalizer"
+require "remocon/normalizer/integer_normalizer"
+require "remocon/normalizer/json_normalizer"
+require "remocon/normalizer/string_normalizer"
+require "remocon/normalizer/void_normalizer"
+require "remocon/normalizer/type_normalizer_factory"
+
+require "remocon/command/lib/config"
+require "remocon/command/lib/interpreter_helper"
+
+require "remocon/command/create_command"
+require "remocon/command/pull_command"
+require "remocon/command/push_command"
+require "remocon/command/validate_command"
+
+require "remocon/cli"

data/lib/remocon/cli.rb

@@ -2,32 +2,44 @@
 
 module Remocon
   class CLI < ::Thor
-    desc 'create', 'Create a json to be pushed'
-    option :parameters, type: :string, required: true, desc: 'a filepath of parameters file'
-    option :conditions, type: :string, required: true, desc: 'a filepath of conditions file'
-    option :dest, type: :string, desc: 'a filepath or directory path of destination'
+    desc "create", "Create a json to be pushed"
+    option :parameters, type: :string, desc: "Specify the filepath if you want to use a custom parameters file"
+    option :conditions, type: :string, desc: "Specify the filepath if you want to use a custom conditions file"
+    option :prefix, type: :string, desc: "the directory name which will contain project-related files"
+    option :id, type: :string, desc: "your project"
+    option :dest, type: :string, hide: true, desc: "[Deprecated] the same with --prefix"
     def create
       execute(Remocon::Command::Create)
     end
 
-    desc 'push', 'Upload remote configs based on a source json file'
-    class_option :source, type: :string, desc: 'a filepath of a source json file'
-    option :etag, type: :string, desc: 'a filepath or raw value of etag'
-    option :dest, type: :string, desc: 'a filepath or directory path of destination'
-    option :force, type: :boolean, default: false, desc: 'force to ignore some warnings'
+    desc "push", "Upload remote configs based on a source json file"
+    option :source, type: :string, desc: "the filepath of your config json file"
+    option :etag, type: :string, desc: "the file path of etag"
+    option :raw_etag, type: :string, desc: "the raw value of etag"
+    option :prefix, type: :string, desc: "the directory name which will contain project-related files"
+    option :force, type: :boolean, default: false, desc: "force to ignore some warnings"
+    option :token, type: :string, desc: "access token to your project"
+    option :id, type: :string, desc: "your project id"
+    option :dest, type: :string, hide: true, desc: "[Deprecated] the same with --prefix"
     def push
       execute(Remocon::Command::Push)
     end
 
-    desc 'pull', 'Pull remote configs'
-    option :dest, type: :string, desc: 'a filepath or directory path of destination'
+    desc "pull", "Pull remote configs"
+    option :prefix, type: :string, desc: "the directory name which will contain project-related files"
+    option :token, type: :string, desc: "access token to your project"
+    option :id, type: :string, desc: "your project id"
+    option :dest, type: :string, hide: true, desc: "[Deprecated] the same with --prefix"
     def pull
       execute(Remocon::Command::Pull)
     end
 
-    desc 'validate', 'Validate yml files'
-    option :parameters, type: :string, required: true, desc: 'a filepath of parameters file'
-    option :conditions, type: :string, required: true, desc: 'a filepath of conditions file'
+    desc "validate", "Validate yml files"
+    option :parameters, type: :string, desc: "Specify the filepath if you want to use a custom parameters file"
+    option :conditions, type: :string, desc: "Specify the filepath if you want to use a custom conditions file"
+    option :prefix, type: :string, desc: "the directory name which will contain project-related files"
+    option :id, type: :string, desc: "your project id"
+    option :token, type: :string, desc: "access token to your project"
     def validate
       execute(Remocon::Command::Validate)
     end
@@ -35,7 +47,7 @@ module Remocon
     private
 
     def execute(klass)
-      klass.new(options).run
+      exit(1) unless klass.new(options).run
     end
   end
 end

data/lib/remocon/command/create_command.rb

@@ -5,15 +5,19 @@ module Remocon
     class Create
       include Remocon::InterpreterHelper
 
+      attr_reader :config, :cmd_opts
+
       def initialize(opts)
-        @opts = opts
+        @config = Remocon::Config.new(opts)
+        @cmd_opts = { validate_only: false }
+      end
 
-        @project_id = ENV.fetch('FIREBASE_PROJECT_ID')
-        @conditions_filepath = @opts[:conditions]
-        @parameters_filepath = @opts[:parameters]
-        @dest_dir = File.join(@opts[:dest], @project_id) if @opts[:dest]
+      def require_parameters_file_path
+        config.parameters_file_path
+      end
 
-        @cmd_opts = { validate_only: false }
+      def require_conditions_file_path
+        config.conditions_file_path
       end
 
       def run
@@ -24,14 +28,10 @@ module Remocon
           parameters: parameter_hash
         }.skip_nil_values.stringify_values
 
-        if @dest_dir
-          File.open(File.join(@dest_dir, 'config.json'), 'w+') do |f|
-            # remote config allows only string values ;(
-            f.write(JSON.pretty_generate(artifact))
-            f.flush
-          end
-        else
-          STDOUT.puts JSON.pretty_generate(artifact)
+        File.open(config.config_json_file_path, "w+") do |f|
+          # remote config allows only string values ;(
+          f.write(JSON.pretty_generate(artifact))
+          f.flush
         end
 
         artifact
@@ -40,8 +40,8 @@ module Remocon
       private
 
       def validate_options
-        raise ValidationError, 'A condition file must exist' unless File.exist?(@conditions_filepath)
-        raise ValidationError, 'A parameter file must exist' unless File.exist?(@parameters_filepath)
+        raise ValidationError, "A condition file must exist" unless File.exist?(config.conditions_file_path)
+        raise ValidationError, "A parameter file must exist" unless File.exist?(config.parameters_file_path)
       end
     end
   end

data/lib/remocon/command/lib/config.rb

@@ -0,0 +1,82 @@
+# frozen_string_literal: true
+
+module Remocon
+  class Config
+    REMOCON_PROJECT_ID_KEY = "REMOCON_FIREBASE_PROJECT_ID"
+    REMOCON_ACCESS_TOKEN = "REMOCON_FIREBASE_ACCESS_TOKEN"
+
+    REMOCON_PREFIX_KEY = "REMOCON_PREFIX"
+
+    CONFIG_JSON_FILE = "config.json"
+    CONDITIONS_FILE_NAME = "conditions.yml"
+    PARAMETERS_FILE_NAME = "parameters.yml"
+    ETAG_FILE_NAME = "etag"
+
+    attr_reader :opts
+
+    def initialize(opts)
+      @opts = opts
+    end
+
+    def endpoint
+      @endpoint ||= "https://firebaseremoteconfig.googleapis.com/v1/projects/#{project_id}/remoteConfig"
+    end
+
+    def project_id
+      # FIREBASE_PROJECT_ID is for backward compatibility
+      @project_id ||= (opts[:id] || ENV[REMOCON_PROJECT_ID_KEY] || ENV["FIREBASE_PROJECT_ID"] || raise("--id or #{REMOCON_PROJECT_ID_KEY} env var is required"))
+    end
+
+    def token
+      # REMOTE_CONFIG_ACCESS_TOKEN is for backward compatibility
+      @token ||= (opts[:token] || ENV[REMOCON_ACCESS_TOKEN] || ENV["REMOTE_CONFIG_ACCESS_TOKEN"] || raise("--token or #{REMOCON_ACCESS_TOKEN} env var is required"))
+    end
+
+    def destination_dir_path
+      @destination_dir_path ||= (opts[:prefix] || opts[:dest] || ENV[REMOCON_PREFIX_KEY])
+    end
+
+    def project_dir_path
+      @project_dir_path ||= begin
+        dir_path = destination_dir_path
+        (dir_path ? File.join(dir_path, project_id) : project_id).tap do |dir|
+          FileUtils.mkdir_p(dir)
+        end
+      end
+    end
+
+    def config_json_file_path
+      @config_json_file_path ||= opts[:source] || begin
+        File.join(project_dir_path, CONFIG_JSON_FILE)
+      end
+    end
+
+    def conditions_file_path
+      @conditions_file_path ||= opts[:conditions] || begin
+        File.join(project_dir_path, CONDITIONS_FILE_NAME)
+      end
+    end
+
+    def parameters_file_path
+      @parameters_file_path ||= opts[:parameters] || begin
+        File.join(project_dir_path, PARAMETERS_FILE_NAME)
+      end
+    end
+
+    def etag_file_path
+      @etag_file_path ||= opts[:etag] || begin
+        File.join(project_dir_path, ETAG_FILE_NAME)
+      end
+    end
+
+    def etag
+      @etag ||= begin
+        if opts[:force] && opts[:raw_etag]
+          raise "--force and --raw_etag cannot be specified"
+        end
+
+        opts[:force] && "*" || opts[:raw_etag] || File.exist?(etag_file_path) && File.open(etag_file_path).read
+      end
+    end
+  end
+end

data/lib/remocon/command/lib/interpreter_helper.rb

@@ -5,11 +5,21 @@ module Remocon
     include Remocon::ConditionSorter
     include Remocon::ParameterSorter
 
-    attr_reader :parameters_filepath, :conditions_filepath, :cmd_opts
+    def cmd_opts
+      raise NotImplementedError
+    end
+
+    def require_parameters_file_path
+      raise NotImplementedError
+    end
+
+    def require_conditions_file_path
+      raise NotImplementedError
+    end
 
     def read_parameters
       @read_parameters ||= begin
-        parameter_interpreter = Remocon::ParameterFileInterpreter.new(parameters_filepath)
+        parameter_interpreter = Remocon::ParameterFileInterpreter.new(require_parameters_file_path)
         parameter_interpreter.read(condition_names, cmd_opts)
       end
     end
@@ -24,7 +34,7 @@ module Remocon
 
     def read_conditions
       @read_conditions ||= begin
-        condition_interpreter = Remocon::ConditionFileInterpreter.new(conditions_filepath)
+        condition_interpreter = Remocon::ConditionFileInterpreter.new(require_conditions_file_path)
         condition_interpreter.read(cmd_opts)
       end
     end

data/lib/remocon/command/pull_command.rb

@@ -5,61 +5,60 @@ module Remocon
     class Pull
       include Remocon::InterpreterHelper
 
+      attr_reader :config, :cmd_opts
+
       def initialize(opts)
-        @opts = opts
+        @config = Remocon::Config.new(opts)
+        @cmd_opts = { validate_only: false }
+      end
 
-        @project_id = ENV.fetch('FIREBASE_PROJECT_ID')
-        @token = ENV.fetch('REMOTE_CONFIG_ACCESS_TOKEN')
-        @url = "https://firebaseremoteconfig.googleapis.com/v1/projects/#{@project_id}/remoteConfig"
-        @dest_dir = File.join(@opts[:dest], @project_id) if @opts[:dest]
+      def require_parameters_file_path
+        config.parameters_file_path
+      end
 
-        @cmd_opts = { validate_only: false }
+      def require_conditions_file_path
+        config.conditions_file_path
       end
 
       def run
-        if @dest_dir
-          FileUtils.mkdir_p(@dest_dir)
+        raw_json, etag = do_request
 
-          raw_hash = JSON.parse(raw_json).with_indifferent_access
+        raw_hash = JSON.parse(raw_json).with_indifferent_access
 
-          raise 'etag cannot be fetched. please try again' unless @etag
+        raise "etag cannot be fetched. please try again" unless etag
 
-          conditions, parameters = [raw_hash[:conditions] || [], raw_hash[:parameters] || {}]
+        conditions = raw_hash[:conditions] || []
+        parameters = raw_hash[:parameters] || {}
 
-          File.open(File.join(@dest_dir, "conditions.yml"), 'w+') do |f|
-            f.write(JSON.parse(Remocon::ConditionFileDumper.new(sort_conditions(conditions)).dump.to_json).to_yaml)
-            f.flush
-          end
+        File.open(config.conditions_file_path, "w+") do |f|
+          f.write(JSON.parse(Remocon::ConditionFileDumper.new(sort_conditions(conditions)).dump.to_json).to_yaml)
+          f.flush
+        end
 
-          File.open(File.join(@dest_dir, "parameters.yml"), 'w+') do |f|
-            f.write(JSON.parse(Remocon::ParameterFileDumper.new(sort_parameters(parameters)).dump.to_json).to_yaml)
-            f.flush
-          end
+        File.open(config.parameters_file_path, "w+") do |f|
+          f.write(JSON.parse(Remocon::ParameterFileDumper.new(sort_parameters(parameters)).dump.to_json).to_yaml)
+          f.flush
+        end
 
-          File.open(File.join(@dest_dir, "config.json"), 'w+') do |f|
-            f.write(JSON.pretty_generate({ conditions: sort_conditions(conditions), parameters: sort_parameters(parameters) }))
-            f.flush
-          end
+        File.open(config.config_json_file_path, "w+") do |f|
+          f.write(JSON.pretty_generate({ conditions: sort_conditions(conditions), parameters: sort_parameters(parameters) }))
+          f.flush
+        end
 
-          File.open(File.join(@dest_dir, 'etag'), 'w+') do |f|
-            f.write(@etag)
-            f.flush
-          end
-        else
-          STDOUT.puts raw_json
+        File.open(config.etag_file_path, "w+") do |f|
+          f.write(etag)
+          f.flush
         end
       end
 
       private
 
-      def raw_json
-        return @raw_json if @raw_json
-
-        @raw_json, @etag = open(@url, 'Authorization' => "Bearer #{@token}") do |io|
-          [io.read, io.meta['etag']]
+      def do_request
+        raw_json, etag = open(config.endpoint, "Authorization" => "Bearer #{config.token}") do |io|
+          [io.read, io.meta["etag"]]
         end
 
-        @raw_json
+        [raw_json, etag]
       end
     end
   end