scenic-oracle_adapter 1.3.3 → 1.3.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: dc29d851c72aab3e3ff74f1a51fe0afc03f6acb9d1e4f9619027c231ad1f2729
-  data.tar.gz: 303764a51df609bcec295a8b495a0beb0dd3575d678060a26a5d8ed5a7b61232
+  metadata.gz: 453bc175d2fa922a314a781064fb63a6415dc3be36aa096baa9d52c77f1542ed
+  data.tar.gz: 3d5ed519bc3ce673c934bc32d310b298c8d525e12fd089abf8a1201674036d4e
 SHA512:
-  metadata.gz: 5584138f239fa8d5df30bcefa0fa6ccf25ee870c4759bf578aed41ea381d429a8ae5bf379a1103fedbe68de645e17d055f4797cb1f596e38f25e6d437edb2ae5
-  data.tar.gz: c255d823fec8652adedb46ccd1c3638670cfdde2431ab392391fc52f0eb0b570ede4c416a77767b7ec68b0de2db0ad4a1eab301defd31f3a5a65afcb7899d3d1
+  metadata.gz: cb0722b1989112a9b5f48c7fdbd7b303d4fd7baf5b577dc002bcd9d174e795b120715b725ffdd72cc562f7522b09a9eea13cc8e4984dccd53d929bdea2b1b0c4
+  data.tar.gz: df2366ae4965f197b205f7148ecef307f17695f3c05815b17ee681bf230b53c2bb1edfc7831b854d590022422fbfc42c3ce00e526ff7d14f719ecc8399ae5ce6

data/docker-compose.yml

@@ -11,8 +11,6 @@ services:
     build: .
     volumes:
       - .:/app
-    environment:
-      - TERM=dumb
     stdin_open: true
     tty: true
     depends_on:

data/lib/scenic/adapters/oracle.rb

@@ -13,13 +13,11 @@ module Scenic
       end
 
       def views
-        all_view_objects.sort_by do |view_object|
-          dependency_order.index(view_object.name)
-        end
+        sorted_dependency_views + sorted_missing_dependency_views
       end
 
       def create_view(name, definition)
-        execute("create view #{quote_table_name(name)} as #{definition}")
+        execute("create view #{quote_table_name(name)} as #{trimmed_definition(definition)}")
       end
 
       def drop_view(name)
@@ -27,7 +25,7 @@ module Scenic
       end
 
       def replace_view(name, definition)
-        execute("create or replace view #{quote_table_name(name)} as #{definition}")
+        execute("create or replace view #{quote_table_name(name)} as #{trimmed_definition(definition)}")
       end
 
       def update_view(name, definition)
@@ -36,7 +34,7 @@ module Scenic
       end
 
       def create_materialized_view(name, definition, no_data: false)
-        execute("create materialized view #{quote_table_name(name)} #{'build deferred' if no_data} as #{definition}")
+        execute("create materialized view #{quote_table_name(name)} #{'build deferred' if no_data} as #{trimmed_definition(definition)}")
       end
 
       def update_materialized_view(name, definition, no_data: false)
@@ -71,7 +69,7 @@ module Scenic
       private
 
       def view_dependencies
-        select_all(<<~EOSQL)
+        @view_dependencies ||= select_all(<<~EOSQL)
           select lower(uo.object_name) as name, lower(ud.referenced_name) as dependency
           from user_objects uo
             left join user_dependencies ud on
@@ -88,6 +86,7 @@ module Scenic
               and ud.referenced_name in (select object_name from user_objects)
               and ud.referenced_owner = user
           where uo.object_type in ('VIEW', 'MATERIALIZED VIEW')
+          order by lower(uo.object_name), lower(ud.referenced_name)
         EOSQL
       end
 
@@ -102,6 +101,22 @@ module Scenic
         views_hash.tsort
       end
 
+      def sorted_dependency_views
+        all_view_objects.filter do |view|
+          dependency_order.include?(view.name)
+        end.sort_by do |view|
+          dependency_order.index(view.name)
+        end
+      end
+
+      def sorted_missing_dependency_views
+        all_view_objects.filter do |view|
+          dependency_order.exclude?(view.name)
+        end.sort_by do |view|
+          view.name
+        end
+      end
+
       def all_views
         select_all("select lower(view_name) name, text definition from user_views").map do |view|
           Scenic::View.new(name: view["name"], definition: view["definition"], materialized: false)
@@ -121,6 +136,10 @@ module Scenic
       def refresh_dependencies_for(name)
         Scenic::Adapters::Oracle::RefreshDependencies.call(name, self, connection)
       end
+
+      def trimmed_definition(sql)
+        sql.strip.sub(/;$/, "").strip
+      end
     end
   end
 end

data/lib/scenic/oracle_adapter/version.rb

@@ -2,6 +2,6 @@
 
 module Scenic
   module OracleAdapter
-    VERSION = "1.3.3"
+    VERSION = "1.3.4"
   end
 end

data/lib/tsortable_hash.rb

@@ -4,7 +4,7 @@ class TSortableHash < Hash
   include TSort
 
   alias_method :tsort_each_node, :each_key
-  def tsort_each_child(node, &)
-    fetch(node).each(&)
+  def tsort_each_child(node, &block)
+    fetch(node).each(&block)
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: scenic-oracle_adapter
 version: !ruby/object:Gem::Version
-  version: 1.3.3
+  version: 1.3.4
 platform: ruby
 authors:
 - Chris Dinger
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-07-18 00:00:00.000000000 Z
+date: 2024-07-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: scenic