chair 1.1.1 → 1.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 708b6c777ef65da883a84fea3b82efc0fab65d04
-  data.tar.gz: 6487ea6b33e0a3580ed3dee015e3dee76ef84bf0
+  metadata.gz: 5a6ec94854da3e028f8aea6f564c9cf63c9b9011
+  data.tar.gz: 8d9c2bb02e94815fcaa7ace97ab291bc0c266e90
 SHA512:
-  metadata.gz: ad6141ebfb731b5c310b0f6f3180ad84881418de08e5d5b78d928aa3702c9cdbeacdc39142443cfc7696c74d61bf587be10a37d74e9ee0f24f5e0b9db84b10a0
-  data.tar.gz: f53d8948344d3dff2afa298c0501b74084590d03301720c94d87ef49e66d0e933380b3c37858bcaf7a73a6c7b494fe508e467c0e9c3b39d2ebef45f6682db355
+  metadata.gz: 805790821f8a261f79c94c494b8431f700f552ad6aa5df2adc9886c9ad9673db3ff322811b638dabd0d397c02dfe5acbc9bbaaba5083923efcb38bab1f506eeb
+  data.tar.gz: 7828bdc15695f86d5448bff224bd1eeeee9cb42e7dea3662da18f1134fd4ac147eec524c5b4b125a1d219ad62dd66c38d2e907fe6c5eb0d3c94c9fd414b1bc42

data/lib/chair/chair.rb

@@ -86,11 +86,15 @@ class Chair
   def insert!(args)
     args = process_incoming_data(args)
     if has_primary_key?
-      if args.include? @primary_key
-        @pk_map[args[@primary_key]] = @table.size
-      else # If our table has a primary key, but can't find it in the data
+      unless args.include? @primary_key
+        # If our table has a primary key, but can't find it in the data
         raise ArgumentError, 'Missing primary key in record to be inserted'
       end
+      val = args[@primary_key]
+      if @pk_map.has_key?(val)
+        raise RuntimeError, "Primary key #{val.inspect} already exists in table"
+      end
+      @pk_map[val] = @table.size
     end
     row = Row.new(self, @table.size, args)
     @table << row

data/lib/chair/version.rb

@@ -1,3 +1,3 @@
 class Chair
-  VERSION = "1.1.1"
+  VERSION = "1.1.2"
 end

data/spec/table_spec.rb

@@ -56,7 +56,7 @@ describe Chair do
     }.to change {table.columns.count}.by(2)
   end
 
-  describe 'inserts a row' do
+  describe 'inserting a row' do
     it 'from a hash' do
       table = Chair.new :title
       expect {
@@ -70,6 +70,15 @@ describe Chair do
         table.insert!(['Gone with the Wind'])
       }.to change { table.size }.by(1)
     end
+
+    it 'with the same primary key raises RuntimeError' do
+      table = Chair.new :title
+      table.set_primary_key! :title
+      table.insert! title: 'Gone with the Wind'
+      expect {
+        table.insert!(['Gone with the Wind'])
+      }.to raise_error RuntimeError, 'Primary key "Gone with the Wind" already exists in table'
+    end
   end
 
   describe 'setting primary key' do
@@ -84,7 +93,7 @@ describe Chair do
       table = Chair.new
       expect{
         table.set_primary_key! :title
-      }.to raise_error ArgumentError, "Column :title does not exist in table"
+      }.to raise_error ArgumentError, 'Column :title does not exist in table'
     end
 
     it "fails if there's already a primary key" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: chair
 version: !ruby/object:Gem::Version
-  version: 1.1.1
+  version: 1.1.2
 platform: ruby
 authors:
 - Katherine Whitlock