xmigra 1.5.1 → 1.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b8cc262267c7f2ccdb80a17bb68f76da30ea4bcb
-  data.tar.gz: d5245127f534c80e96a983e0222f855e872c7e4d
+  metadata.gz: 6bd9e09df1d03a0bb23ee61fd21644669c459315
+  data.tar.gz: 2467c3c1de5236f6e27f87706b7d1b091c774dbd
 SHA512:
-  metadata.gz: dcde456a4403f7ec18a273c27550293e66032445ca62df0bf25c42160e21daacc16e37a56ae0ad8a7dfad44631e27179a1d589188b9c00748525c0c5aabb14b3
-  data.tar.gz: 8426367f6962b891e919df73cde7ca086b6524a70e10344390f83e53cdaaac0ca36f61c4082d50188d18e6e9a0e0528bbd4cf2ad2cd17deabbc7b818a762212b
+  metadata.gz: 4cdf662d14d79137b156324376dcb3725183712354ecbe0a2e3ecd453f3a6b331f322f46204d9e5a5290f356062e9f6125a2571754d3ee75d439ca22c9cb21f6
+  data.tar.gz: 75c686b17ad25eb2a117a25d30ced42e501944facef9cb8446461ff9715556a7b99f7049bfea154a0cf83e12ebe9caea5922b4fc239c9c3fdf2e34da5b1d8941

data/lib/xmigra/db_support/mssql.rb

@@ -1153,6 +1153,49 @@ INSERT INTO [xmigra].[branch_upgrade] ([Current]) VALUES (#{branch_id_literal});
       return parts.join("\n")
     end
     
+    def index_template_sql
+      <<-"END_OF_SQL"
+CREATE INDEX [{filename}]
+ON <<<table>>> (<<<columns>>>);
+END_OF_SQL
+    end
+    
+    def view_definition_template_sql
+      <<-"END_OF_SQL"
+CREATE VIEW [{filename}]
+AS SELECT <<<query>>>;
+END_OF_SQL
+    end
+    
+    def procedure_definition_template_sql
+      <<-"END_OF_SQL"
+CREATE PROCEDURE [{filename}]
+  <<<parameters>>>
+AS BEGIN
+  <<<statements>>>
+END
+END_OF_SQL
+    end
+    
+    def function_definition_template_sql
+      <<-"END_OF_SQL"
+CREATE FUNCTION [{filename}] (
+  <<<parameters>>>
+) RETURNS <<<return-type>>>
+AS BEGIN
+  <<<statements>>>
+  RETURN <<<return-value-expression>>>;
+END
+END_OF_SQL
+    end
+    
+    def alter_table_columns_sql_statements(col_pairs)
+      col_pairs.map do |_, col|
+        nullability = (col.nullable? ? "" : "NOT ") + "NULL"
+        "ALTER TABLE #{name} ALTER COLUMN #{col.name} #{col.type} #{nullability};"
+      end
+    end
+    
     class << self
       def strip_identifier_quoting(s)
         case

data/lib/xmigra/db_support/psql.rb

@@ -704,6 +704,51 @@ module XMigra
       return parts.join("\n")
     end
     
+    def index_template_sql
+      XMigra.dedent(%Q{
+        CREATE INDEX [{filename}]
+        ON <<<table>>> (<<<columns>>>);
+      })
+    end
+    
+    def view_definition_template_sql
+      XMigra.dedent(%Q{
+        CREATE VIEW [{filename}]
+        AS SELECT <<<query>>>;
+      })
+    end
+    
+    def procedure_definition_template_sql
+      raise XMigra::NewAccessArtifactAdder::UnsupportedArtifactType.new(:procedure, SYSTEM_NAME)
+    end
+    
+    def function_definition_template_sql
+      XMigra.dedent(%Q{
+        CREATE FUNCTION [{filename}] (
+          <<<parameters>>>
+        ) RETURNS <<<return-type>>>
+        AS $$
+          <<<function-body>>>
+        $$ LANGUAGE plpgsql;
+      })
+    end
+    
+    def alter_table_columns_sql_statements(col_pairs)
+      col_pairs.flat_map do |old_col, col|
+        [].tap do |parts|
+          if !old_col.nullable? && col.nullable?
+            parts << "ALTER TABLE #{name} ALTER COLUMN #{col.name} DROP NOT NULL;"
+          end
+          if old_col.type != col.type
+            parts << "ALTER TABLE #{name} ALTER COLUMN #{col.name} TYPE #{col.type};"
+          end
+          if old_col.nullable? && !col.nullable?
+            parts << "ALTER TABLE #{name} ALTER COLUMN #{col.name} SET NOT NULL;"
+          end
+        end
+      end
+    end
+    
     class <<self
       def in_plpgsql(*args)
         variables = args[0].kind_of?(Hash) ? args.shift : {}

data/lib/xmigra/declarative_migration.rb

@@ -0,0 +1,160 @@
+
+module XMigra
+  module DeclarativeMigration
+    VALID_GOALS = %w{creation adoption revision renunciation destruction}
+    GOAL_KEY = 'does'
+    TARGET_KEY = 'of object'
+    DECLARATION_VERSION_KEY = 'to realize'
+    QUALIFICATION_KEY = 'implementation qualification'
+    SUBDIR = 'declarative'
+    
+    class MissingImplementationError < Error
+      COMMAND_LINE_HELP = "The '%prog impdecl' command may help resolve this error."
+    end
+    
+    class QuestionableImplementationError < Error; end
+    
+    Missing = Class.new do
+      def goal
+        :newly_managed_object
+      end
+      
+      def declarative_status
+        :unimplemented
+      end
+      
+      def delta(file_path)
+        Pathname(file_path).readlines.map {|l| '+' + l}.join('')
+      end
+    end.new
+    
+    module ChainSupport
+      def latest_declarative_implementations
+        @latest_declarative_implementations ||= Hash.new do |h, file_path|
+          ext_path = Pathname(file_path).expand_path
+          if h.has_key? ext_path
+            next h[ext_path]
+          end
+          raise Error, (
+            "Unexpected file path '#{file_path}', known file paths:" +
+            h.keys.collect {|kp| "    #{kp}\n"}.join('')
+          )
+        end.tap do |files|
+          each do |migration|
+            # Skip non-declarative migrations
+            next unless migration.is_a? DeclarativeMigration
+            if (
+              [:renunciation, :destruction].include?(migration.goal) &&
+              migration.declarative_status == :missing
+            )
+              files.delete(migration.declarative_file_path)
+            else
+              files[migration.declarative_file_path] = migration
+            end
+          end
+          
+          Dir.glob(Pathname(path).join(SUBDIR, '*.yaml').to_s) do |decl_file|
+            decl_file_path = Pathname(decl_file).expand_path
+            next if files.has_key?(decl_file_path)
+            files[decl_file_path] = DeclarativeMigration::Missing
+          end
+          
+          files.freeze
+        end
+      end
+      
+      def unimplemented_declaratives
+        @unimplemented_declaratives ||= latest_declarative_implementations.reject do |file_path, migration|
+          [:equal, :older].include? migration.declarative_status
+        end.keys
+      end
+      
+      def check_declaratives_current!
+        unless unimplemented_declaratives.empty?
+          raise(
+            MissingImplementationError,
+            "Declaratives without migration implementing current state:\n" +
+            unimplemented_declaratives.collect {|df| "    #{df.basename('.yaml')}\n"}.join("")
+          )
+        end
+        
+        questionable_migrations = latest_declarative_implementations.values.select {|m| m.questionable?}
+        unless questionable_migrations.empty?
+          raise(
+            QuestionableImplementationError,
+            "Implementing migrations with questionable SQL:\n" +
+            questionable_migrations.collect {|m| "    #{m.file_path}\n"}.join("")
+          )
+        end
+        
+        questionable_migrations = latest_declarative_implementations.values.each do |m|
+          next unless m.management_migration?
+          raise(
+            QuestionableImplementationError,
+            "#{m.file_path} cannot execute SQL for a declarative #{m.goal}"
+          ) unless m.sql.nil? || m.sql.empty?
+        end
+      end
+    end
+    
+    def goal
+      @declarative_goal ||= @all_info[GOAL_KEY].tap do |val|
+        raise(ArgumentError, "'#{GOAL_KEY}' must be one of: #{VALID_GOALS.join(', ')}") unless VALID_GOALS.include?(val)
+      end.to_sym
+    end
+    
+    def affected_object
+      @declarative_target ||= @all_info[TARGET_KEY].dup.freeze
+    end
+    
+    # Override the way the base class handles changes -- this integrates with
+    # the "history" command
+    def changes
+      if management_migration?
+        []
+      else
+        [affected_object]
+      end
+    end
+    
+    def sql
+      if management_migration?
+        ''
+      else
+        super()
+      end
+    end
+    
+    def management_migration?
+      [:adoption, :renunciation].include? goal
+    end
+    
+    # This method is only used when the declarative file has uncommitted
+    # modifications and the migration file is uncommitted
+    def implemented_version
+      @declarative_implemented_ver ||= (@all_info[DECLARATION_VERSION_KEY].dup.freeze if vcs_uncommitted?)
+    end
+    
+    def implementation_of?(file_path)
+      XMigra.secure_digest(file_path.read) == implemented_version
+    end
+    
+    def declarative_file_path
+      @declarative_file_path ||= Pathname(file_path).dirname.join(SUBDIR, affected_object + '.yaml')
+    end
+    
+    def declarative_status
+      @declarative_status ||= begin
+        vcs_comparator(:expected_content_method=>:implementation_of?).relative_version(declarative_file_path)
+      end
+    end
+    
+    def delta(file_path)
+      vcs_changes_from(vcs_latest_revision, file_path)
+    end
+    
+    def questionable?
+      @all_info.has_key? QUALIFICATION_KEY
+    end
+  end
+end