ops_team 0.10.3 → 0.12.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a78e3661dac6cc0796d00dc840eeb9fedbbc7f358bf3c6af4609d37175e8d53e
-  data.tar.gz: f919b18cdcec154a9dac78020c20f229771b3bd870baf86ef020002c74394acc
+  metadata.gz: f4fd4661f8ea9e234bed8f61e205e625a232d301786557ce3c645519885c068a
+  data.tar.gz: af58d1ede9274d7a1aa884f036bc2dc6b93f7deff66307484bbcaae30806eea9
 SHA512:
-  metadata.gz: 54d21b402bce3e50963b679be2393a7cf1cd993fa51c2c73481c348c4fa019748d5511a4654d9c94c175a985bb7545580cf000594fd2a7c05c7a43d849895cdd
-  data.tar.gz: 64ef22c5c1a2264b3734166fef0b0752c2cc3ae18003ee376d1a86e0cba97d49fdb1c3de74606e0eb0aef050899d8e20c306165190d5cefc78dc030abe26997c
+  metadata.gz: bc505145b87e405a8a714196966ed2652bfd341fa8deec285c12ad6aa5fb07e01eb4797b888bc7ea5b253d90987b335e4fb6d45a5650af09b23fc74bac3086ed
+  data.tar.gz: 1227db2ce9be14ba4ee5bb1be55750fa6acbe92678a847b9d88a4e19bae6e465909bfa268b1700a73142946f4e0b751a69834630f8e6571860a6b8900403a8da

data/lib/builtins/init.rb

@@ -32,12 +32,18 @@ module Builtins
 
 		private
 
-		def template_name_arg
+		def template_name
 			@args[0]
 		end
 
 		def template_path
-			format(OPS_YML_TEMPLATE, template_name: template_name_arg || DEFAULT_TEMPLATE_NAME)
+			return template_name if template_name && File.exist?(template_name)
+
+			builtin_template_path
+		end
+
+		def builtin_template_path
+			format(OPS_YML_TEMPLATE, template_name: template_name || DEFAULT_TEMPLATE_NAME)
 		end
 
 		def template_name_list

data/lib/builtins/version.rb

@@ -1,13 +1,10 @@
 # frozen_string_literal: true
 
-require "rubygems"
-
 require "output"
+require "version"
 
 module Builtins
 	class Version < Builtin
-		GEMSPEC_FILE = "#{__dir__}/../../ops_team.gemspec"
-
 		class << self
 			def description
 				"prints the version of ops that is running"
@@ -15,18 +12,7 @@ module Builtins
 		end
 
 		def run
-			unless gemspec
-				Output.error("Unable to load gemspec at '#{GEMSPEC_FILE}")
-				return false
-			end
-
-			Output.out(gemspec.version)
-		end
-
-		private
-
-		def gemspec
-			@gemspec ||= Gem::Specification.load(GEMSPEC_FILE)
+			Output.out(::Version.version)
 		end
 	end
 end

data/lib/dependency.rb

@@ -4,11 +4,12 @@ require 'open3'
 require 'English'
 
 require 'output'
+require 'executor'
 
 class Dependency
 	DESCRIPTION_TYPE_WIDTH = 8
 
-	attr_reader :name, :output, :exit_code
+	attr_reader :name
 
 	def initialize(name)
 		@name = name
@@ -45,15 +46,23 @@ class Dependency
 	end
 
 	def success?
-		@exit_code.nil? ? true : @exit_code.zero?
+		@executor.nil? ? true : @executor.success?
+	end
+
+	def output
+		@executor&.output
+	end
+
+	def exit_code
+		@executor&.exit_code
 	end
 
 	private
 
 	def execute(cmd)
-		@output, status = Open3.capture2e(cmd)
-		@exit_code = status.exitstatus
+		@executor = Executor.new(cmd)
+		@executor.execute
 
-		success?
+		@executor.success?
 	end
 end

data/lib/executor.rb

@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+
+class Executor
+	attr_reader :output, :exit_code
+
+	class << self
+		def execute(command)
+			@output, status = Open3.capture2e(command)
+			@exit_code = status.exitstatus
+
+			[@output, @exit_code]
+		end
+	end
+
+	def initialize(command)
+		@command = command
+	end
+
+	def execute
+		@output, status = Open3.capture2e(@command)
+		@exit_code = status.exitstatus
+
+		success?
+	end
+
+	def success?
+		@exit_code.nil? ? true : @exit_code.zero?
+	end
+end

data/lib/hook_handler.rb

@@ -0,0 +1,32 @@
+# frozen_string_literal: true
+
+class HookHandler
+	class HookConfigError < StandardError; end
+	class HookExecError < StandardError; end
+
+	def initialize(config)
+		@config = config
+	end
+
+	def do_hooks(name)
+		raise HookConfigError, "'hooks.#{name}' must be a list" unless hooks(name).is_a?(Array)
+
+		execute_hooks(name)
+	end
+
+	private
+
+	def hooks(name)
+		@config.dig("hooks", name) || []
+	end
+
+	def execute_hooks(name)
+		hooks(name).each do |hook|
+			output, exit_code = Executor.execute(hook)
+
+			next if exit_code.zero?
+
+			raise HookExecError, "#{name} hook '#{hook}' failed with exit code #{exit_code}:\n#{output}"
+		end
+	end
+end

data/lib/ops.rb

@@ -3,11 +3,14 @@
 
 require 'yaml'
 require 'require_all'
+require "rubygems"
 
+require 'hook_handler'
 require 'action'
 require 'output'
 require 'options'
 require 'environment'
+require 'version'
 require_rel "builtins"
 
 # executes commands based on local `ops.yml`
@@ -19,6 +22,7 @@ class Ops
 	INVALID_SYNTAX_EXIT_CODE = 64
 	UNKNOWN_ACTION_EXIT_CODE = 65
 	ERROR_LOADING_APP_CONFIG_EXIT_CODE = 66
+	MIN_VERSION_NOT_MET_EXIT_CODE = 67
 
 	class << self
 		def project_name
@@ -34,7 +38,9 @@ class Ops
 	end
 
 	def run
+		# "return" is here to allow specs to stub "exit"
 		return exit(INVALID_SYNTAX_EXIT_CODE) unless syntax_valid?
+		return exit(MIN_VERSION_NOT_MET_EXIT_CODE) unless min_version_met?
 
 		run_action
 	rescue UnknownActionError => e
@@ -53,9 +59,23 @@ class Ops
 		end
 	end
 
+	def min_version_met?
+		return true unless min_version
+
+		if Version.min_version_met?(min_version)
+			true
+		else
+			Output.error("ops.yml specifies minimum version of #{min_version}, but ops version is #{Version.version}")
+			false
+		end
+	end
+
+	def min_version
+		config["min_version"]
+	end
+
 	def run_action
-		environment.set_variables
-		AppConfig.load
+		do_before_run_action
 
 		return builtin.run if builtin
 
@@ -66,6 +86,16 @@ class Ops
 		exit(ERROR_LOADING_APP_CONFIG_EXIT_CODE)
 	end
 
+	def do_before_run_action
+		environment.set_variables
+		AppConfig.load
+		hook_handler.do_hooks("before")
+	end
+
+	def hook_handler
+		@hook_handler ||= HookHandler.new(config)
+	end
+
 	def builtin
 		@builtin ||= Builtin.class_for(name: @action_name).new(@args, config)
 	rescue NameError

data/lib/version.rb

@@ -0,0 +1,38 @@
+# frozen_string_literal: true
+
+require "rubygems"
+
+class Version
+	GEMSPEC_FILE = "#{__dir__}/../ops_team.gemspec"
+
+	class << self
+		# these class methods exist because I don't want to have to call `Version.new.version` elsewhere
+		def version
+			new.send(:version)
+		end
+
+		def min_version_met?(min_version)
+			new.send(:min_version_met?, min_version)
+		end
+	end
+
+	private
+
+	# these instance methods exist so that I don't have to clear class variables between tests
+	def version
+		unless gemspec
+			Output.error("Unable to load gemspec at '#{GEMSPEC_FILE}")
+			return nil
+		end
+
+		gemspec.version
+	end
+
+	def min_version_met?(min_version)
+		Gem::Version.new(version) >= Gem::Version.new(min_version)
+	end
+
+	def gemspec
+		@gemspec ||= Gem::Specification.load(GEMSPEC_FILE)
+	end
+end

data/ops_team.gemspec

@@ -2,7 +2,7 @@
 
 Gem::Specification.new do |s|
 	s.name = 'ops_team'
-	s.version = '0.10.3'
+	s.version = '0.12.0'
 	s.authors = [
 		'nickthecook@gmail.com'
 	]

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ops_team
 version: !ruby/object:Gem::Version
-  version: 0.10.3
+  version: 0.12.0
 platform: ruby
 authors:
 - nickthecook@gmail.com
@@ -170,10 +170,13 @@ files:
 - lib/dependencies/sshkey.rb
 - lib/dependency.rb
 - lib/environment.rb
+- lib/executor.rb
+- lib/hook_handler.rb
 - lib/ops.rb
 - lib/options.rb
 - lib/output.rb
 - lib/secrets.rb
+- lib/version.rb
 - loader.rb
 - ops_team.gemspec
 homepage: https://github.com/nickthecook/ops