Adding specs to confirm #906. Seems to be fine. Closes #906

db/migrations/20230805212959_add_next_id_to_tokens.cr

@@ -0,0 +1,13 @@
+class AddNextIdToTokens::V20230805212959 < Avram::Migrator::Migration::V1
+  def migrate
+    alter table_for(Token) do
+      add next_id : Int32, default: 0
+    end
+  end
+
+  def rollback
+    alter table_for(Token) do
+      remove :next_id
+    end
+  end
+end

spec/avram/operations/save_operation_spec.cr

@@ -74,6 +74,10 @@ private class UpsertUserOperation < User::SaveOperation
   upsert_lookup_columns :name, :nickname
 end
 
+private class UpsertToken < Token::SaveOperation
+  upsert_lookup_columns :next_id
+end
+
 private class OverrideDefaults < ModelWithDefaultValues::SaveOperation
   permit_columns :greeting, :drafted_at, :published_at, :admin, :age, :money
 end
@@ -234,6 +238,17 @@ describe "Avram::SaveOperation" do
         end
       end
 
+      it "updates the record using the same default value" do
+        TokenFactory.create(&.name("special").scopes(["name"]).next_id(4))
+
+        UpsertToken.upsert(next_id: 4, name: "Secret", scopes: ["red", "blue"]) do |op, token|
+          op.valid?.should eq(true)
+          token.should_not eq(nil)
+          token.as(Token).name.should eq("Secret")
+          token.as(Token).scopes.should eq(["red", "blue"])
+        end
+      end
+
       it "creates a new record if match one doesn't exist" do
         user_with_different_nickname =
           UserFactory.create &.name("Rich").nickname(nil).age(20)

spec/support/factories/token_factory.cr

@@ -0,0 +1,7 @@
+class TokenFactory < BaseFactory
+  def initialize
+    name "Secret"
+    scopes ["email"]
+    next_id 0
+  end
+end

spec/support/models/token.cr

@@ -2,6 +2,7 @@ class Token < BaseModel
   table do
     column name : String = "Secret"
     column scopes : Array(String) = ["email"]
+    column next_id : Int32 = 0
   end
 end