jobshop 0.0.167 → 0.0.179

data/db/migrate/support/temporal_tables.sql

@@ -0,0 +1,210 @@
+/* The MIT License (MIT)
+**
+** Copyright (c) 2016-2017 Nearform and contributors
+**
+** Contributors listed at https://github.com/nearform/temporal_tables#the-team
+** and in the README file.
+**
+** Permission is hereby granted, free of charge, to any person obtaining a copy
+** of this software and associated documentation files (the "Software"), to deal
+** in the Software without restriction, including without limitation the rights
+** to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+** copies of the Software, and to permit persons to whom the Software is
+** furnished to do so, subject to the following conditions:
+**
+** The above copyright notice and this permission notice shall be included in
+** all copies or substantial portions of the Software.
+**
+** THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+** IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+** FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+** AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+** LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+** OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+** SOFTWARE.
+*/
+
+CREATE OR REPLACE FUNCTION versioning() RETURNS TRIGGER AS $$
+DECLARE
+  sys_period text;
+  history_table text;
+  manipulate jsonb;
+  commonColumns text[];
+  time_stamp_to_use timestamptz := current_timestamp;
+  range_lower timestamptz;
+  transaction_info txid_snapshot;
+  existing_range tstzrange;
+  holder record;
+  holder2 record;
+  pg_version integer;
+BEGIN
+
+  IF (TG_WHEN != 'BEFORE') OR (TG_LEVEL != 'ROW' ) THEN
+    RAISE TRIGGER_PROTOCOL_VIOLATED USING
+    MESSAGE = 'function "versioning" must be fired BEFORE ROW';
+  END IF;
+
+  IF TG_OP != 'INSERT' AND TG_OP != 'UPDATE' AND TG_OP != 'DELETE' THEN
+    RAISE TRIGGER_PROTOCOL_VIOLATED USING
+    MESSAGE = 'function "versioning" must be fired for INSERT or UPDATE or DELETE';
+  END IF;
+
+  IF TG_NARGS != 3 THEN
+    RAISE INVALID_PARAMETER_VALUE USING
+    MESSAGE = 'wrong number of parameters for function "versioning"',
+    HINT = 'expected 3 parameters but got ' || TG_NARGS;
+  END IF;
+
+  sys_period := TG_ARGV[0];
+  history_table := TG_ARGV[1];
+
+  -- check if sys_period exists on original table
+  SELECT atttypid, attndims INTO holder FROM pg_attribute WHERE attrelid = TG_RELID AND attname = sys_period AND NOT attisdropped;
+  IF NOT FOUND THEN
+    RAISE 'column "%" of relation "%" does not exist', sys_period, TG_TABLE_NAME USING
+    ERRCODE = 'undefined_column';
+  END IF;
+  IF holder.atttypid != to_regtype('tstzrange') THEN
+    IF holder.attndims > 0 THEN
+      RAISE 'system period column "%" of relation "%" is not a range but an array', sys_period, TG_TABLE_NAME USING
+      ERRCODE = 'datatype_mismatch';
+    END IF;
+
+    SELECT rngsubtype INTO holder2 FROM pg_range WHERE rngtypid = holder.atttypid;
+    IF FOUND THEN
+      RAISE 'system period column "%" of relation "%" is not a range of timestamp with timezone but of type %', sys_period, TG_TABLE_NAME, format_type(holder2.rngsubtype, null) USING
+      ERRCODE = 'datatype_mismatch';
+    END IF;
+
+    RAISE 'system period column "%" of relation "%" is not a range but type %', sys_period, TG_TABLE_NAME, format_type(holder.atttypid, null) USING
+    ERRCODE = 'datatype_mismatch';
+  END IF;
+
+  IF TG_OP = 'UPDATE' OR TG_OP = 'DELETE' THEN
+    -- Ignore rows already modified in this transaction
+    transaction_info := txid_current_snapshot();
+    IF OLD.xmin::text >= (txid_snapshot_xmin(transaction_info) % (2^32)::bigint)::text
+    AND OLD.xmin::text <= (txid_snapshot_xmax(transaction_info) % (2^32)::bigint)::text THEN
+      IF TG_OP = 'DELETE' THEN
+        RETURN OLD;
+      END IF;
+
+      RETURN NEW;
+    END IF;
+
+    SELECT current_setting('server_version_num')::integer
+    INTO pg_version;
+
+    -- to support postgres < 9.6
+    IF pg_version < 90600 THEN
+      -- check if history table exits
+      IF to_regclass(history_table::cstring) IS NULL THEN
+        RAISE 'relation "%" does not exist', history_table;
+      END IF;
+    ELSE
+      IF to_regclass(history_table) IS NULL THEN
+        RAISE 'relation "%" does not exist', history_table;
+      END IF;
+    END IF;
+
+    -- check if history table has sys_period
+    IF NOT EXISTS(SELECT * FROM pg_attribute WHERE attrelid = history_table::regclass AND attname = sys_period AND NOT attisdropped) THEN
+      RAISE 'history relation "%" does not contain system period column "%"', history_table, sys_period USING
+      HINT = 'history relation must contain system period column with the same name and data type as the versioned one';
+    END IF;
+
+    EXECUTE format('SELECT $1.%I', sys_period) USING OLD INTO existing_range;
+
+    IF existing_range IS NULL THEN
+      RAISE 'system period column "%" of relation "%" must not be null', sys_period, TG_TABLE_NAME USING
+      ERRCODE = 'null_value_not_allowed';
+    END IF;
+
+    IF isempty(existing_range) OR NOT upper_inf(existing_range) THEN
+      RAISE 'system period column "%" of relation "%" contains invalid value', sys_period, TG_TABLE_NAME USING
+      ERRCODE = 'data_exception',
+      DETAIL = 'valid ranges must be non-empty and unbounded on the high side';
+    END IF;
+
+    IF TG_ARGV[2] = 'true' THEN
+      -- mitigate update conflicts
+      range_lower := lower(existing_range);
+      IF range_lower >= time_stamp_to_use THEN
+        time_stamp_to_use := range_lower + interval '1 microseconds';
+      END IF;
+    END IF;
+
+    WITH history AS
+      (SELECT attname, atttypid
+      FROM   pg_attribute
+      WHERE  attrelid = history_table::regclass
+      AND    attnum > 0
+      AND    NOT attisdropped),
+      main AS
+      (SELECT attname, atttypid
+      FROM   pg_attribute
+      WHERE  attrelid = TG_RELID
+      AND    attnum > 0
+      AND    NOT attisdropped)
+    SELECT
+      history.attname AS history_name,
+      main.attname AS main_name,
+      history.atttypid AS history_type,
+      main.atttypid AS main_type
+    INTO holder
+      FROM history
+      INNER JOIN main
+      ON history.attname = main.attname
+    WHERE
+      history.atttypid != main.atttypid;
+
+    IF FOUND THEN
+      RAISE 'column "%" of relation "%" is of type % but column "%" of history relation "%" is of type %',
+        holder.main_name, TG_TABLE_NAME, format_type(holder.main_type, null), holder.history_name, history_table, format_type(holder.history_type, null)
+      USING ERRCODE = 'datatype_mismatch';
+    END IF;
+
+    WITH history AS
+      (SELECT attname
+      FROM   pg_attribute
+      WHERE  attrelid = history_table::regclass
+      AND    attnum > 0
+      AND    NOT attisdropped),
+      main AS
+      (SELECT attname
+      FROM   pg_attribute
+      WHERE  attrelid = TG_RELID
+      AND    attnum > 0
+      AND    NOT attisdropped)
+    SELECT array_agg(quote_ident(history.attname)) INTO commonColumns
+      FROM history
+      INNER JOIN main
+      ON history.attname = main.attname
+      AND history.attname != sys_period;
+
+    EXECUTE ('INSERT INTO ' ||
+      CASE split_part(history_table, '.', 2)
+      WHEN '' THEN
+        quote_ident(history_table)
+      ELSE
+        quote_ident(split_part(history_table, '.', 1)) || '.' || quote_ident(split_part(history_table, '.', 2))
+      END ||
+      '(' ||
+      array_to_string(commonColumns , ',') ||
+      ',' ||
+      quote_ident(sys_period) ||
+      ') VALUES ($1.' ||
+      array_to_string(commonColumns, ',$1.') ||
+      ',tstzrange($2, $3, ''[)''))')
+       USING OLD, range_lower, time_stamp_to_use;
+  END IF;
+
+  IF TG_OP = 'UPDATE' OR TG_OP = 'INSERT' THEN
+    manipulate := jsonb_set('{}'::jsonb, ('{' || sys_period || '}')::text[], to_jsonb(tstzrange(time_stamp_to_use, null, '[)')));
+
+    RETURN jsonb_populate_record(NEW, manipulate);
+  END IF;
+
+  RETURN OLD;
+END;
+$$ LANGUAGE plpgsql;

data/db/seeds.rb

@@ -1,24 +1,22 @@
-# frozen_string_literals: true
+# frozen_string_literal: true
+
+ENV["RAILS_ENV"] ||= "development"
+
+require "benchmark"
 
 def factory_paths
   [ Jobshop::Engine.root.join("spec/factories") ]
 end
 
-private def clean_database
-  require "database_cleaner"
-  DatabaseCleaner.strategy = :truncation
-  DatabaseCleaner.clean
-end
-
 def do_with_spinner(name, &block)
+  retval = nil
   title = "[:spinner] #{name.ljust(40, '.')}"
   spinner = TTY::Spinner.new(title, format: :dots)
   spinner.auto_spin
-  t1 = Time.now
-  retval = yield
-  t2 = Time.now
-  delta = t2 - t1
-  spinner.success("done (#{delta}s)")
+  time = Benchmark.realtime do
+    retval = yield
+  end
+  spinner.success("done in #{(time * 1000).round(0)}ms")
   retval
 end
 
@@ -28,18 +26,51 @@ do_with_spinner("Loading factories") do
   FactoryBot.find_definitions if Rails.env.development?
 end
 
-do_with_spinner("Cleaning database") do
-  clean_database
+@org = do_with_spinner("Creating Organization") do
+  FactoryBot.create(:organization)
 end
 
-@org = do_with_spinner("Creating test Organization") do
-  FactoryBot.create(:organization)
+do_with_spinner("Creating Departments") do
+  @accounting = FactoryBot.create(:department, {
+    name: "Accounting", organization: @org
+  })
+
+  @engineering = FactoryBot.create(:department, {
+    name: "Engineering", organization: @org
+  })
+
+  @hr = FactoryBot.create(:department, {
+    name: "Human Resources", organization: @org
+  })
+
+  @manufacturing = FactoryBot.create(:department, {
+    name: "Manufacturing", organization: @org
+  })
+
+  @quality = FactoryBot.create(:department, {
+    name: "Quality", organization: @org
+  })
+
+  @sales = FactoryBot.create(:department, {
+    name: "Sales", organization: @org
+  })
+
+  @sr = FactoryBot.create(:department, {
+    name: "Sales", organization: @org
+  })
 end
 
-@ceo = do_with_spinner("Creating test Person") do
-  employee = FactoryBot.create(:employee, organization: @org)
-  employee.employments.create!({ started_on: rand(10..20).years.ago,
-                                 title: Faker::Job.title })
+do_with_spinner("Creating Employees") do
+  @salesmen = { }
+  @salesmen[:thomas] = FactoryBot.create(:salesman, organization: @org, departments: [ @sales ])
+  @salesmen[:richard] = FactoryBot.create(:salesman, organization: @org, departments: [ @sales ])
+  @salesmen[:harold] = FactoryBot.create(:salesman, organization: @org, departments: [ @sales ])
+
+  @engineers = { }
+  @engineers[:joe] = FactoryBot.create(:engineer, organization: @org, departments: [ @engineering ])
+  @engineers[:jim] = FactoryBot.create(:engineer, organization: @org, departments: [ @engineering ])
+
+  @hr1 = FactoryBot.create(:hr_specialist, organization: @org, departments: [ @hr ])
 end
 
 @places = do_with_spinner("Creating test Places") do
@@ -47,80 +78,77 @@ end
 end
 
 @customers = do_with_spinner("Creating test customers") do
-  FactoryBot.create_list(:customer, 10, organization: @org)#,
-  #created_by: @ceo)
+  FactoryBot.create_list(:customer, 10, organization: @org)
 end
 
 @products = do_with_spinner("Creating test Products") do
-  FactoryBot.create_list(:product, 10,
-                         organization: @org)#,
-  #created_by: @ceo)
+  FactoryBot.create_list(:product, 10, organization: @org)
 end
 
 do_with_spinner("Creating test Routers for Products") do
   @products.each do |product|
-    rp = FactoryBot.create(:routing_process, organization: @org,
-                           product: product)
-    FactoryBot.create_list(:routing_step, rand(1..4),organization: @org,
-                           routing_process: rp)
+    rp = FactoryBot.create(:routing_process, {
+      organization: @org, product: product
+    })
+
+    FactoryBot.create_list(:routing_step, rand(1..4), {
+      organization: @org, routing_process: rp
+    })
   end
 end
 
 do_with_spinner("Creating sample Orders") do
-  50.times do
-    ord = FactoryBot.create(:order, organization: @org,
-                            customer: @customers.sample)
-    #created_by: @ceo)
-    FactoryBot.create_list(:order_line, rand(1..4), organization: @org,
-                           order: ord,
-                           product: @products.sample)
-    #created_by: @ceo)
+  10.times do
+    ord = FactoryBot.create(:order, {
+      organization: @org, customer: @customers.sample
+    })
+
+    FactoryBot.create_list(:order_line, rand(1..4), {
+      organization: @org, order: ord, product: @products.sample
+    })
   end
 end
 
 @widgets = do_with_spinner("Creating Collection of widgets") do
-  wc = FactoryBot.create(:widget_collection,
-                         organization: @org)
-  FactoryBot.create_list(:widget_thing, 50, organization: @org,
-                         collection: wc)
+  wc = FactoryBot.create(:widget_collection, organization: @org)
+  FactoryBot.create_list(:widget_thing, 50, organization: @org, collection: wc)
 end
 
 @widget_packs = do_with_spinner("Creating Collection of widget packs") do
-  wpc = FactoryBot.create(:widget_pack_collection,
-                          organization: @org)
-  FactoryBot.create_list(:widget_pack_thing, 50, organization: @org,
-                         collection: wpc,
-                         widgets: @widgets)
+  wpc = FactoryBot.create(:widget_pack_collection, organization: @org)
+  FactoryBot.create_list(:widget_pack_thing, 50, {
+    organization: @org, collection: wpc, widgets: @widgets
+  })
 end
 
 do_with_spinner("Creating Inspection Reports") do
   5.times do
     report = FactoryBot.create(:inspection_report, organization: @org)
-    5.times do |index|
-      FactoryBot.create(Jobshop::Inspection::CRITERION_TYPES.sample,
-                        organization: @org, report: report,
-                        position: (index + 1))
-    end
-    5.times do |index|
-      tuple = FactoryBot.create(:inspection_tuple, organization: @org,
-                                report: report, position: (index + 1))
-      report.criteria.each do |criterion|
-        FactoryBot.create(:inspection_result, organization: @org,
-                          report: report, criterion: criterion,
-                          tuple: tuple)
-      end
+    Jobshop::Inspection::CRITERION_TYPES.each_with_index do |type, index|
+      FactoryBot.create(type, {
+        organization: @org, report: report, position: (index + 1)
+      })
     end
   end
 end
 
 do_with_spinner("Creating RFQ Mailman") do
-  @org.mailmen.create!(address: "rfq@jobshop.io",
-                       handler_type: "Jobshop::Mailroom::RFQHandler")
+  @org.mailmen.create({
+    address: "rfq@jobshop.io", handler_type: "Jobshop::Mailroom::RFQHandler"
+  })
 end
 
 do_with_spinner("Creating sample RFQs") do
-  50.times do
-    FactoryBot.create(:rfq, organization: @org,
-                      customer: @customers.sample)
+  10.times do
+    case rand(0..100) % 3
+    when 0
+      FactoryBot.create :rfq, :verified, organization: @org,
+        customer: @customers.sample
+    when 1
+      FactoryBot.create :rfq, :verifiable, organization: @org,
+        customer: @customers.sample
+    when 2
+      FactoryBot.create :rfq, :unverifiable, organization: @org
+    end
   end
 end

data/exe/jobshop

@@ -1,4 +1,5 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 
 require "jobshop/cli"
 

data/lib/generators/jobshop/app/app_generator.rb

@@ -29,6 +29,15 @@ module Jobshop
                        after: sentinel, verbose: true)
     end
 
+    def config_force_ssl
+      return if options[:pretend]
+
+      data = "config.force_ssl = true"
+      sentinel = /class [a-z_:]+ < Rails::Application/i
+      inject_into_file("config/application.rb", "\n    #{data}",
+                       after: sentinel, verbose: true)
+    end
+
     def mount_engine
       return if options[:pretend]
 
@@ -43,8 +52,8 @@ module Jobshop
       end
 
       def self.source_paths
-        [ Rails::Generators::AppGenerator.source_root,
-          Jobshop::Generators::AppGenerator.source_root ]
+        [ Jobshop::Generators::AppGenerator.source_root,
+          Rails::Generators::AppGenerator.source_root ]
       end
 
       hide!
@@ -52,7 +61,7 @@ module Jobshop
       class_option :help, type: :boolean, aliases: "-h", group: :other,
         desc: "Show this help message and quit"
       class_option :version, type: :boolean, aliases: "-v", group: :other,
-              desc: "Show Jobshop version number and quit"
+        desc: "Show Jobshop version number and quit"
 
       def initialize(*args)
         super
@@ -73,6 +82,7 @@ module Jobshop
 
       def finish_template
         build :config_schema_format
+        build :config_force_ssl
         build :mount_engine
         super
       end