kithe 2.0.0.pre.alpha2 → 2.0.2

data/app/models/kithe/derivative.rb

@@ -1,15 +0,0 @@
-module Kithe
-
-  # Derivatives by default will be stored in Shrine storage :kithe_derivatives, so
-  # that should be registered in your app.
-  #
-  # Only one deriv can exist for a given asset_id/key pair, enforced by db constraint.
-  class Derivative < ApplicationRecord
-    # the fk is to kithe_models STI table, but we only intend for assets
-    belongs_to :asset, class_name: "Kithe::Asset"
-
-    include Kithe::DerivativeUploader::Attachment.new(:file, store: :kithe_derivatives)
-
-    delegate :content_type, :size, :height, :width, :url, to: :file, allow_nil: true
-  end
-end

data/app/uploaders/kithe/derivative_uploader.rb

@@ -1,48 +0,0 @@
-require 'mini_mime'
-
-module Kithe
-  # The derivative uploader doesn't have to do too much, we don't even use
-  # promotion for derivatives, just writing directly to a storage.
-  #
-  # But it needs activerecord integration, and limited metadata automatic extraction.
-  class DerivativeUploader < Shrine
-    plugin :activerecord
-
-    plugin :determine_mime_type, analyzer: :marcel
-
-    # ignore error, often from storing a non-image file which can't have dimensions
-    # extracted. behavior consistent with shrine 2.x.
-    plugin :store_dimensions, on_error: :ignore
-
-    # Useful in case consumers want it, and doesn't harm anything to be available.
-    # https://github.com/shrinerb/shrine/blob/master/doc/plugins/rack_response.md
-    plugin :rack_response
-
-    # should this be in a plugin? location in file system based on original asset
-    # id and derivative key, as well as unique random file id from shrine.
-    def generate_location(io, context)
-      # assumes we're only used with Derivative model, that has an asset_id and key
-      asset_id = context[:record].asset_id
-      key = context[:record].key
-      original = super
-      [asset_id, key, original].compact.join("/")
-    end
-
-
-    # Override to fix "filename" metadata to be something reasonable, regardless
-    # of what if anything was the filename of the IO being attached. shrine S3 will
-    # insist on setting a default content-disposition with this filename.
-    def extract_metadata(io, context = {})
-      result = super
-
-      if context[:kithe_derivative_key] &&
-         context[:record]
-        extension = MiniMime.lookup_by_content_type(result["mime_type"] || "")&.extension
-        result["filename"] = "#{context[:record].asset.friendlier_id}_#{context[:kithe_derivative_key]}.#{extension}"
-        result["kithe_derivative_key"] = context[:kithe_derivative_key]
-      end
-
-      return result
-    end
-  end
-end

data/spec/dummy/db/structure.sql

@@ -1,309 +0,0 @@
-SET statement_timeout = 0;
-SET lock_timeout = 0;
-SET idle_in_transaction_session_timeout = 0;
-SET client_encoding = 'UTF8';
-SET standard_conforming_strings = on;
-SELECT pg_catalog.set_config('search_path', '', false);
-SET check_function_bodies = false;
-SET client_min_messages = warning;
-SET row_security = off;
-
---
--- Name: pgcrypto; Type: EXTENSION; Schema: -; Owner: -
---
-
-CREATE EXTENSION IF NOT EXISTS pgcrypto WITH SCHEMA public;
-
-
---
--- Name: EXTENSION pgcrypto; Type: COMMENT; Schema: -; Owner: -
---
-
-COMMENT ON EXTENSION pgcrypto IS 'cryptographic functions';
-
-
---
--- Name: kithe_models_friendlier_id_gen(bigint, bigint); Type: FUNCTION; Schema: public; Owner: -
---
-
-CREATE FUNCTION public.kithe_models_friendlier_id_gen(min_value bigint, max_value bigint) RETURNS text
-    LANGUAGE plpgsql
-    AS $$
-  DECLARE
-    new_id_int bigint;
-    new_id_str character varying := '';
-    done bool;
-    tries integer;
-    alphabet char[] := ARRAY['0','1','2','3','4','5','6','7','8','9',
-      'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n',
-      'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z'];
-    alphabet_length integer := array_length(alphabet, 1);
-
-  BEGIN
-    done := false;
-    tries := 0;
-    WHILE (NOT done) LOOP
-      tries := tries + 1;
-      IF (tries > 3) THEN
-        RAISE 'Could not find non-conflicting friendlier_id in 3 tries';
-      END IF;
-
-      new_id_int := trunc(random() * (max_value - min_value) + min_value);
-
-      -- convert bigint to a Base-36 alphanumeric string
-      -- see https://web.archive.org/web/20130420084605/http://www.jamiebegin.com/base36-conversion-in-postgresql/
-      -- https://gist.github.com/btbytes/7159902
-      WHILE new_id_int != 0 LOOP
-        new_id_str := alphabet[(new_id_int % alphabet_length)+1] || new_id_str;
-        new_id_int := new_id_int / alphabet_length;
-      END LOOP;
-
-      done := NOT exists(SELECT 1 FROM kithe_models WHERE friendlier_id=new_id_str);
-    END LOOP;
-    RETURN new_id_str;
-  END;
-  $$;
-
-
-SET default_tablespace = '';
-
-SET default_with_oids = false;
-
---
--- Name: ar_internal_metadata; Type: TABLE; Schema: public; Owner: -
---
-
-CREATE TABLE public.ar_internal_metadata (
-    key character varying NOT NULL,
-    value character varying,
-    created_at timestamp without time zone NOT NULL,
-    updated_at timestamp without time zone NOT NULL
-);
-
-
---
--- Name: kithe_derivatives; Type: TABLE; Schema: public; Owner: -
---
-
-CREATE TABLE public.kithe_derivatives (
-    id bigint NOT NULL,
-    key character varying NOT NULL,
-    file_data jsonb,
-    asset_id uuid NOT NULL,
-    created_at timestamp without time zone NOT NULL,
-    updated_at timestamp without time zone NOT NULL
-);
-
-
---
--- Name: kithe_derivatives_id_seq; Type: SEQUENCE; Schema: public; Owner: -
---
-
-CREATE SEQUENCE public.kithe_derivatives_id_seq
-    START WITH 1
-    INCREMENT BY 1
-    NO MINVALUE
-    NO MAXVALUE
-    CACHE 1;
-
-
---
--- Name: kithe_derivatives_id_seq; Type: SEQUENCE OWNED BY; Schema: public; Owner: -
---
-
-ALTER SEQUENCE public.kithe_derivatives_id_seq OWNED BY public.kithe_derivatives.id;
-
-
---
--- Name: kithe_model_contains; Type: TABLE; Schema: public; Owner: -
---
-
-CREATE TABLE public.kithe_model_contains (
-    containee_id uuid,
-    container_id uuid
-);
-
-
---
--- Name: kithe_models; Type: TABLE; Schema: public; Owner: -
---
-
-CREATE TABLE public.kithe_models (
-    id uuid DEFAULT public.gen_random_uuid() NOT NULL,
-    title character varying NOT NULL,
-    type character varying NOT NULL,
-    "position" integer,
-    json_attributes jsonb,
-    created_at timestamp without time zone NOT NULL,
-    updated_at timestamp without time zone NOT NULL,
-    parent_id uuid,
-    friendlier_id character varying DEFAULT public.kithe_models_friendlier_id_gen('2821109907456'::bigint, '101559956668415'::bigint) NOT NULL,
-    file_data jsonb,
-    representative_id uuid,
-    leaf_representative_id uuid,
-    kithe_model_type integer NOT NULL
-);
-
-
---
--- Name: schema_migrations; Type: TABLE; Schema: public; Owner: -
---
-
-CREATE TABLE public.schema_migrations (
-    version character varying NOT NULL
-);
-
-
---
--- Name: kithe_derivatives id; Type: DEFAULT; Schema: public; Owner: -
---
-
-ALTER TABLE ONLY public.kithe_derivatives ALTER COLUMN id SET DEFAULT nextval('public.kithe_derivatives_id_seq'::regclass);
-
-
---
--- Name: ar_internal_metadata ar_internal_metadata_pkey; Type: CONSTRAINT; Schema: public; Owner: -
---
-
-ALTER TABLE ONLY public.ar_internal_metadata
-    ADD CONSTRAINT ar_internal_metadata_pkey PRIMARY KEY (key);
-
-
---
--- Name: kithe_derivatives kithe_derivatives_pkey; Type: CONSTRAINT; Schema: public; Owner: -
---
-
-ALTER TABLE ONLY public.kithe_derivatives
-    ADD CONSTRAINT kithe_derivatives_pkey PRIMARY KEY (id);
-
-
---
--- Name: kithe_models kithe_models_pkey; Type: CONSTRAINT; Schema: public; Owner: -
---
-
-ALTER TABLE ONLY public.kithe_models
-    ADD CONSTRAINT kithe_models_pkey PRIMARY KEY (id);
-
-
---
--- Name: schema_migrations schema_migrations_pkey; Type: CONSTRAINT; Schema: public; Owner: -
---
-
-ALTER TABLE ONLY public.schema_migrations
-    ADD CONSTRAINT schema_migrations_pkey PRIMARY KEY (version);
-
-
---
--- Name: index_kithe_derivatives_on_asset_id; Type: INDEX; Schema: public; Owner: -
---
-
-CREATE INDEX index_kithe_derivatives_on_asset_id ON public.kithe_derivatives USING btree (asset_id);
-
-
---
--- Name: index_kithe_derivatives_on_asset_id_and_key; Type: INDEX; Schema: public; Owner: -
---
-
-CREATE UNIQUE INDEX index_kithe_derivatives_on_asset_id_and_key ON public.kithe_derivatives USING btree (asset_id, key);
-
-
---
--- Name: index_kithe_model_contains_on_containee_id; Type: INDEX; Schema: public; Owner: -
---
-
-CREATE INDEX index_kithe_model_contains_on_containee_id ON public.kithe_model_contains USING btree (containee_id);
-
-
---
--- Name: index_kithe_model_contains_on_container_id; Type: INDEX; Schema: public; Owner: -
---
-
-CREATE INDEX index_kithe_model_contains_on_container_id ON public.kithe_model_contains USING btree (container_id);
-
-
---
--- Name: index_kithe_models_on_friendlier_id; Type: INDEX; Schema: public; Owner: -
---
-
-CREATE UNIQUE INDEX index_kithe_models_on_friendlier_id ON public.kithe_models USING btree (friendlier_id);
-
-
---
--- Name: index_kithe_models_on_parent_id; Type: INDEX; Schema: public; Owner: -
---
-
-CREATE INDEX index_kithe_models_on_parent_id ON public.kithe_models USING btree (parent_id);
-
-
---
--- Name: index_kithe_models_on_representative_id; Type: INDEX; Schema: public; Owner: -
---
-
-CREATE INDEX index_kithe_models_on_representative_id ON public.kithe_models USING btree (representative_id);
-
-
---
--- Name: kithe_model_contains fk_rails_091010187b; Type: FK CONSTRAINT; Schema: public; Owner: -
---
-
-ALTER TABLE ONLY public.kithe_model_contains
-    ADD CONSTRAINT fk_rails_091010187b FOREIGN KEY (container_id) REFERENCES public.kithe_models(id);
-
-
---
--- Name: kithe_derivatives fk_rails_3dac8b4201; Type: FK CONSTRAINT; Schema: public; Owner: -
---
-
-ALTER TABLE ONLY public.kithe_derivatives
-    ADD CONSTRAINT fk_rails_3dac8b4201 FOREIGN KEY (asset_id) REFERENCES public.kithe_models(id);
-
-
---
--- Name: kithe_models fk_rails_403cce5c0d; Type: FK CONSTRAINT; Schema: public; Owner: -
---
-
-ALTER TABLE ONLY public.kithe_models
-    ADD CONSTRAINT fk_rails_403cce5c0d FOREIGN KEY (leaf_representative_id) REFERENCES public.kithe_models(id);
-
-
---
--- Name: kithe_model_contains fk_rails_490c1158f7; Type: FK CONSTRAINT; Schema: public; Owner: -
---
-
-ALTER TABLE ONLY public.kithe_model_contains
-    ADD CONSTRAINT fk_rails_490c1158f7 FOREIGN KEY (containee_id) REFERENCES public.kithe_models(id);
-
-
---
--- Name: kithe_models fk_rails_90130a9780; Type: FK CONSTRAINT; Schema: public; Owner: -
---
-
-ALTER TABLE ONLY public.kithe_models
-    ADD CONSTRAINT fk_rails_90130a9780 FOREIGN KEY (parent_id) REFERENCES public.kithe_models(id);
-
-
---
--- Name: kithe_models fk_rails_afa93b7b5d; Type: FK CONSTRAINT; Schema: public; Owner: -
---
-
-ALTER TABLE ONLY public.kithe_models
-    ADD CONSTRAINT fk_rails_afa93b7b5d FOREIGN KEY (representative_id) REFERENCES public.kithe_models(id);
-
-
---
--- PostgreSQL database dump complete
---
-
-SET search_path TO "$user", public;
-
-INSERT INTO "schema_migrations" (version) VALUES
-('20181015143259'),
-('20181015143413'),
-('20181015143737'),
-('20181031190647'),
-('20181128185658'),
-('20190103144947'),
-('20190109192252'),
-('20190404144551');
-
-

data/spec/models/kithe/asset/asset_create_derivatives_spec.rb

@@ -1,320 +0,0 @@
-# Make a new test file cause it's a buncha func
-require 'rails_helper'
-
-# Not sure how to get our
-describe "Kithe::Asset derivative definitions", queue_adapter: :test do
-  let(:a_jpg_deriv_file) { Kithe::Engine.root.join("spec/test_support/images/2x2_pixel.jpg") }
-
-  temporary_class("TestAssetSubclass") do
-    deriv_src_path = a_jpg_deriv_file
-    Class.new(Kithe::Asset) do
-      define_derivative(:some_data) do |original_file|
-        StringIO.new("some one data")
-      end
-
-      define_derivative(:a_jpg) do |original_file|
-        FileUtils.cp(deriv_src_path,
-             Kithe::Engine.root.join("spec/test_support/images/2x2_pixel-TEMP.jpg"))
-
-        File.open(Kithe::Engine.root.join("spec/test_support/images/2x2_pixel-TEMP.jpg"))
-      end
-    end
-  end
-
-  let(:asset) do
-    TestAssetSubclass.create(title: "test",
-      file: File.open(Kithe::Engine.root.join("spec/test_support/images/1x1_pixel.jpg"))
-    ).tap do |a|
-      # We want to promote without create_derivatives being automatically called
-      # as usual, so we can test create_derivatives manually.
-      a.file_attacher.set_promotion_directives(create_derivatives: false)
-      a.promote
-
-      # Precondition assumptions for our test setup to be valid
-      expect(a.file_attacher.stored?).to be(true)
-      expect(a.derivatives).to be_empty
-    end
-  end
-
-  it "builds derivatives" do
-    asset.create_derivatives
-
-    one_deriv = asset.derivatives.find { |d| d.key == "some_data" }
-    expect(one_deriv).to be_present
-    expect(one_deriv.file.read).to eq("some one data")
-
-    jpg_deriv = asset.derivatives.find {|d| d.key == "a_jpg"}
-    expect(jpg_deriv.file.read).to eq(File.read(a_jpg_deriv_file, encoding: "BINARY"))
-  end
-
-  it "sets #derivatives_created?" do
-    expect(asset.derivatives_created?).to be(false)
-    asset.create_derivatives
-    asset.reload
-    expect(asset.derivatives_created?).to be(true)
-  end
-
-
-  describe "Original deleted before derivatives can be created", queue_adapter: :inline do
-    let(:short_lived_asset) do
-      TestAssetSubclass.create!(title: "test",
-        file: File.open(Kithe::Engine.root.join("spec/test_support/images/1x1_pixel.jpg")))
-    end
-    it """catches the ActiveJob::DeserializationError
-      if the asset is no longer in the database
-      once the derivative creation job starts up.""" do
-      id_to_delete = short_lived_asset.id
-      Kithe::Derivative.where(asset_id: id_to_delete).delete_all
-      Kithe::Model.where(id: id_to_delete).delete_all
-
-      # short_lived_asset is no longer in the DB.
-      # Let's try and create derivatives for it:
-      expect do
-        Kithe::CreateDerivativesJob.perform_later(short_lived_asset)
-      end.not_to raise_error
-    end
-  end
-
-  describe "under normal operation", queue_adapter: :inline do
-    let(:asset) do
-      TestAssetSubclass.create!(title: "test",
-        file: File.open(Kithe::Engine.root.join("spec/test_support/images/1x1_pixel.jpg")))
-    end
-    it "automatically creates derivatives" do
-      expect(asset.derivatives.count).to eq(2)
-    end
-  end
-
-  it "extracts limited metadata from derivative" do
-    asset.create_derivatives
-
-    jpg_deriv = asset.derivatives.find {|d| d.key == "a_jpg"}
-    expect(jpg_deriv.size).to eq(File.size(Kithe::Engine.root.join("spec/test_support/images/2x2_pixel.jpg")))
-    expect(jpg_deriv.width).to eq(2)
-    expect(jpg_deriv.height).to eq(2)
-    expect(jpg_deriv.content_type).to eq("image/jpeg")
-  end
-
-  it "deletes derivative file returned by block" do
-    asset.create_derivatives
-
-    expect(File.exist?(Kithe::Engine.root.join("spec/test_support/images/2x2_pixel-TEMP.jpg"))).not_to be(true)
-  end
-
-  it "by default saves in :kithe_derivatives storage" do
-    asset.create_derivatives
-
-    jpg_deriv = asset.derivatives.find {|d| d.key == "a_jpg"}
-    expect(jpg_deriv.file.storage_key).to eq(:kithe_derivatives)
-  end
-
-
-  describe "block arguments" do
-    let(:monitoring_proc) do
-      proc do |original_file, record:|
-        expect(original_file.kind_of?(File) || original_file.kind_of?(Tempfile)).to be(true)
-        expect(original_file.path).to be_present
-        expect(original_file.read).to eq(asset.file.read)
-
-        expect(record).to eq(asset)
-
-        nil
-      end
-    end
-
-    temporary_class("TestAssetSubclass") do
-      our_proc = monitoring_proc
-      Class.new(Kithe::Asset) do
-        define_derivative(:some_data, &our_proc)
-      end
-    end
-
-    it "as expected" do
-      expect(monitoring_proc).to receive(:call).and_call_original
-
-      asset.create_derivatives
-      expect(asset.derivatives.length).to eq(0)
-    end
-  end
-
-  describe "custom storage_key" do
-    temporary_class("TestAssetSubclass") do
-      Class.new(Kithe::Asset) do
-        define_derivative(:some_data, storage_key: :store) do |original_file|
-          StringIO.new("some one data")
-        end
-      end
-    end
-    it "saves appropriately" do
-      asset.create_derivatives
-
-      deriv = asset.derivatives.first
-
-      expect(deriv).to be_present
-      expect(deriv.file.storage_key).to eq(:store)
-    end
-  end
-
-  describe "default_create false" do
-    let(:monitoring_proc) { proc { |asset| } }
-
-    temporary_class("TestAssetSubclass") do
-      p = monitoring_proc
-      Class.new(Kithe::Asset) do
-        define_derivative(:some_data, default_create: false, &p)
-      end
-    end
-
-    it "is not run automatically" do
-      expect(monitoring_proc).not_to receive(:call)
-      asset.create_derivatives
-    end
-  end
-
-  describe "only/except" do
-    let(:monitoring_proc1) { proc { |asset| StringIO.new("one") } }
-    let(:monitoring_proc2) { proc { |asset| StringIO.new("two") } }
-    let(:monitoring_proc3) { proc { |asset| StringIO.new("three") } }
-
-    temporary_class("TestAssetSubclass") do
-      p1, p2, p3 = monitoring_proc1, monitoring_proc2, monitoring_proc3
-      Class.new(Kithe::Asset) do
-        define_derivative(:one, default_create: false, &p1)
-        define_derivative(:two, &p2)
-        define_derivative(:three, &p3)
-      end
-    end
-
-    it "can call with only" do
-      expect(monitoring_proc1).to receive(:call).and_call_original
-      expect(monitoring_proc2).to receive(:call).and_call_original
-      expect(monitoring_proc3).not_to receive(:call)
-
-      asset.create_derivatives(only: [:one, :two])
-
-      expect(asset.derivatives.collect(&:key)).to eq(["one", "two"])
-    end
-
-    it "can call with except" do
-      expect(monitoring_proc1).not_to receive(:call)
-      expect(monitoring_proc2).to receive(:call).and_call_original
-      expect(monitoring_proc3).not_to receive(:call)
-
-      asset.create_derivatives(except: [:three])
-
-      expect(asset.derivatives.collect(&:key)).to eq(["two"])
-    end
-
-    it "can call with only and except" do
-      expect(monitoring_proc1).to receive(:call).and_call_original
-      expect(monitoring_proc2).not_to receive(:call)
-      expect(monitoring_proc3).not_to receive(:call)
-
-      asset.create_derivatives(only: [:one, :two], except: :two)
-
-      expect(asset.derivatives.collect(&:key)).to eq(["one"])
-    end
-  end
-
-  describe "content_type filters" do
-    temporary_class("TestAssetSubclass") do
-      Class.new(Kithe::Asset) do
-        define_derivative(:never_called, content_type: "nothing/nothing") { |o| StringIO.new("never") }
-        define_derivative(:gated_positive, content_type: "image/jpeg") { |o| StringIO.new("gated positive") }
-        define_derivative(:gated_positive_main_type, content_type: "image") { |o| StringIO.new("gated positive") }
-      end
-    end
-
-    it "does not call if content type does not match" do
-      asset.create_derivatives
-      expect(asset.derivatives.collect(&:key)).not_to include("never_called")
-    end
-
-    it "calls for exact content type match" do
-      asset.create_derivatives
-      expect(asset.derivatives.collect(&:key)).to include("gated_positive")
-    end
-
-    it "calls for main content type match" do
-      asset.create_derivatives
-      expect(asset.derivatives.collect(&:key)).to include("gated_positive_main_type")
-    end
-
-    describe "as array" do
-      temporary_class("TestAssetSubclass") do
-        Class.new(Kithe::Asset) do
-          define_derivative(:never_called, content_type: ["nothing/nothing", "also/nothing"]) { |o| StringIO.new("never") }
-          define_derivative(:gated_positive, content_type: ["image/jpeg", "something/else"]) { |o| StringIO.new("gated positive") }
-        end
-      end
-      it "calls for one match" do
-        asset.create_derivatives
-        expect(asset.derivatives.collect(&:key)).to eq(["gated_positive"])
-      end
-    end
-
-    describe "conflicting types" do
-      let(:unfiltered) { proc { |asset| StringIO.new("unfiltered") } }
-      let(:image) { proc { |asset| StringIO.new("image") } }
-      let(:image_jpeg) { proc { |asset| StringIO.new("image/jpeg") } }
-
-
-      temporary_class("TestAssetSubclass") do
-        u, i, ij = unfiltered, image, image_jpeg
-        Class.new(Kithe::Asset) do
-          define_derivative(:key, &u)
-          define_derivative(:key, content_type: "image/jpeg", &ij)
-          define_derivative(:key, content_type: "image", &i)
-        end
-      end
-
-      it "takes most specific" do
-        expect(unfiltered).not_to receive(:call)
-        expect(image).not_to receive(:call)
-        expect(image_jpeg).to receive(:call).and_call_original
-
-        asset.create_derivatives
-        expect(asset.derivatives.count).to eq(1)
-
-        deriv = asset.derivatives.first
-        expect(deriv.key).to eq("key")
-        expect(deriv.file.read). to eq("image/jpeg")
-      end
-    end
-  end
-
-  describe "lazy creation" do
-    before do
-      asset.class.derivative_definitions.collect(&:key).each do |key|
-        asset.update_derivative(key, StringIO.new("#{key} original"))
-      end
-    end
-
-    it "does not re-create" do
-      derivatives_pre_creation = asset.derivatives.collect(&:attributes)
-
-      asset.create_derivatives(lazy: true)
-      derivatives_post_creation = asset.derivatives.reload.collect(&:attributes)
-
-      expect(derivatives_post_creation).to eq(derivatives_pre_creation)
-    end
-  end
-
-  describe "#remove_derivative_definition!" do
-    it "can remove by string" do
-      original_keys = TestAssetSubclass.defined_derivative_keys
-      TestAssetSubclass.remove_derivative_definition!(original_keys.first.to_s)
-      expect(TestAssetSubclass.defined_derivative_keys).to eq(original_keys.slice(1..original_keys.length))
-    end
-    it "can remove by symbol" do
-      original_keys = TestAssetSubclass.defined_derivative_keys
-      TestAssetSubclass.remove_derivative_definition!(original_keys.first.to_sym)
-      expect(TestAssetSubclass.defined_derivative_keys).to eq(original_keys.slice(1..original_keys.length))
-    end
-    it "can remove multiple args" do
-      original_keys = TestAssetSubclass.defined_derivative_keys
-      TestAssetSubclass.remove_derivative_definition!(*original_keys)
-      expect(TestAssetSubclass.defined_derivative_keys).to eq([])
-    end
-  end
-end