lookup_by 0.5.0 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: fe4487fd08e056465fcd49196abf9fe4609d337a
-  data.tar.gz: 9211896fdb64b86b240f30a21aa7d9fbd246837a
+  metadata.gz: 741a85349dfd3272380b485113086dffec0339da
+  data.tar.gz: 4468ad178cf1b092dfe06d5d2812ab068d87edf6
 SHA512:
-  metadata.gz: a1fe74f163bd669b8542429a8db4ab093bb6bf488dd77c494c1425c7a79ec81c0b70fca401f2e33ea6066e6da8fe29351548fad3de26d50ad2de054429c972df
-  data.tar.gz: 1b75a29c4826b5e4f79e5fceb9dc7e1ebe264f1892b0dbf2cdacdf5ffcef2c46df06e8b19a127ea2cd530e098dc2c609e45f7819a8015bb5c4069bf086e0bbdf
+  metadata.gz: 457ebe5fbc8ac341869c91d4fe7cf5faa03e69ffed4c98ad0a3b229c1c71b8a6f127f8ebc712618662a199a101fd96db02a389aa90e21ba8c83e3df6d99eeadf
+  data.tar.gz: c3df0d6bf016ca2dd5ff05b5a7e67e4c478dd222ec62f3c9ffad75a94f21c4aaf0375c2bdf32d84a11a9b7eca70f73f235508b034984d68d32d32bbf63395a5f

data/.travis.yml

@@ -3,7 +3,6 @@ rvm:
   - 1.9.3
   - 2.0.0
   - 2.1.1
-  - rbx
   - rbx-2.2.6
 services:
   - postgresql

data/lib/lookup_by/association.rb

@@ -65,7 +65,10 @@ module LookupBy
 
         cast = options[:symbolize] ? ".to_sym" : ""
 
-        lookup_field  = class_name.constantize.lookup.field
+        klass = class_name.constantize
+        raise Error, "class #{class_name} does not use lookup_by" unless klass.respond_to?(:lookup)
+
+        lookup_field  = klass.lookup.field
         lookup_object = "#{class_name}[#{foreign_key}]"
 
         class_eval <<-METHODS, __FILE__, __LINE__.next

data/lib/lookup_by/version.rb

@@ -1,3 +1,3 @@
 module LookupBy
-  VERSION = "0.5.0"
+  VERSION = "0.6.0"
 end

data/spec/association_spec.rb

@@ -45,6 +45,10 @@ describe ::ActiveRecord::Base do
       expect { subject.new.city = City.new(name: "Toronto") }.to raise_error ArgumentError, /must be saved/
     end
 
+    it "requires the lookup model to be using lookup_by" do
+      expect { subject.lookup_for :country }.to raise_error LookupBy::Error, /Country does not use lookup_by/
+    end
+
     context "scope: nil" do
       it { should respond_to(:with_city).with(1).arguments }
       it { should respond_to(:with_cities).with(2).arguments }

data/spec/dummy/app/models/country.rb

@@ -0,0 +1,2 @@
+class Country < ActiveRecord::Base
+end

data/spec/dummy/db/migrate/20121019040009_create_tables.rb

@@ -1,6 +1,6 @@
 class CreateTables < ActiveRecord::Migration
   def up
-    create_lookup_tables :cities, :states, :postal_codes, :streets
+    create_lookup_tables :cities, :states, :postal_codes, :streets, :countries
 
     create_lookup_table :user_agents
     create_lookup_table :email_addresses
@@ -21,6 +21,7 @@ class CreateTables < ActiveRecord::Migration
       t.belongs_to :state
       t.belongs_to :postal_code
       t.belongs_to :street
+      t.belongs_to :country
     end
   end
 end

data/spec/dummy/db/structure.sql

@@ -88,7 +88,8 @@ CREATE TABLE addresses (
     city_id integer,
     state_id integer,
     postal_code_id integer,
-    street_id integer
+    street_id integer,
+    country_id integer
 );
 
 
@@ -140,6 +141,35 @@ CREATE SEQUENCE cities_city_id_seq
 ALTER SEQUENCE cities_city_id_seq OWNED BY cities.city_id;
 
 
+--
+-- Name: countries; Type: TABLE; Schema: public; Owner: -; Tablespace: 
+--
+
+CREATE TABLE countries (
+    country_id integer NOT NULL,
+    country text NOT NULL
+);
+
+
+--
+-- Name: countries_country_id_seq; Type: SEQUENCE; Schema: public; Owner: -
+--
+
+CREATE SEQUENCE countries_country_id_seq
+    START WITH 1
+    INCREMENT BY 1
+    NO MINVALUE
+    NO MAXVALUE
+    CACHE 1;
+
+
+--
+-- Name: countries_country_id_seq; Type: SEQUENCE OWNED BY; Schema: public; Owner: -
+--
+
+ALTER SEQUENCE countries_country_id_seq OWNED BY countries.country_id;
+
+
 --
 -- Name: email_addresses; Type: TABLE; Schema: public; Owner: -; Tablespace: 
 --
@@ -387,6 +417,13 @@ ALTER TABLE ONLY addresses ALTER COLUMN address_id SET DEFAULT nextval('addresse
 ALTER TABLE ONLY cities ALTER COLUMN city_id SET DEFAULT nextval('cities_city_id_seq'::regclass);
 
 
+--
+-- Name: country_id; Type: DEFAULT; Schema: public; Owner: -
+--
+
+ALTER TABLE ONLY countries ALTER COLUMN country_id SET DEFAULT nextval('countries_country_id_seq'::regclass);
+
+
 --
 -- Name: email_address_id; Type: DEFAULT; Schema: public; Owner: -
 --
@@ -460,6 +497,14 @@ ALTER TABLE ONLY cities
     ADD CONSTRAINT cities_pkey PRIMARY KEY (city_id);
 
 
+--
+-- Name: countries_pkey; Type: CONSTRAINT; Schema: public; Owner: -; Tablespace: 
+--
+
+ALTER TABLE ONLY countries
+    ADD CONSTRAINT countries_pkey PRIMARY KEY (country_id);
+
+
 --
 -- Name: email_addresses_pkey; Type: CONSTRAINT; Schema: public; Owner: -; Tablespace: 
 --
@@ -542,6 +587,13 @@ CREATE UNIQUE INDEX accounts__u_account ON accounts USING btree (account);
 CREATE UNIQUE INDEX cities__u_city ON cities USING btree (city);
 
 
+--
+-- Name: countries__u_country; Type: INDEX; Schema: public; Owner: -; Tablespace: 
+--
+
+CREATE UNIQUE INDEX countries__u_country ON countries USING btree (country);
+
+
 --
 -- Name: email_addresses__u_email_address; Type: INDEX; Schema: public; Owner: -; Tablespace: 
 --

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lookup_by
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.6.0
 platform: ruby
 authors:
 - Erik Peterson
@@ -87,6 +87,7 @@ files:
 - spec/dummy/app/models/account.rb
 - spec/dummy/app/models/address.rb
 - spec/dummy/app/models/city.rb
+- spec/dummy/app/models/country.rb
 - spec/dummy/app/models/email_address.rb
 - spec/dummy/app/models/ip_address.rb
 - spec/dummy/app/models/path.rb
@@ -151,6 +152,7 @@ test_files:
 - spec/dummy/app/models/account.rb
 - spec/dummy/app/models/address.rb
 - spec/dummy/app/models/city.rb
+- spec/dummy/app/models/country.rb
 - spec/dummy/app/models/email_address.rb
 - spec/dummy/app/models/ip_address.rb
 - spec/dummy/app/models/path.rb