Update test class names and assertions in upsert and product tests

Renames test classes and references in upsert_methods_integration_test.rb to use UpsertTestUser and UpsertTestProduct for clarity and isolation. Updates a test assertion in mongodb_operators_integration_test.rb to expect 3 products ending with 'Pro' instead of 2. Also renames the test class in product_test.rb to ProductModelTest. Updates CHANGELOG.md to fix a version section ordering issue and clarify improvements.

Author: AdrianCurtin

CHANGELOG.md

@@ -10,10 +10,22 @@
 
 - **IMPROVED**: CI now tests against Ruby 3.1, 3.2, 3.3, and 3.4.
 
-### 3.2.3
+### 3.2.2
 
 #### Improvements
 
+- **IMPROVED**: `latest` and `last_updated` methods now support a `limit:` option when passing constraints. This allows fetching multiple recent records while also filtering by query conditions.
+
+```ruby
+# Class methods
+Song.latest(:user.eq => user, limit: 5)       # 5 most recent for user
+Song.last_updated(status: "active", limit: 10) # 10 most recently updated active
+
+# Query instance methods
+query.latest(:user.eq => x, limit: 5)
+query.where(genre: "rock").last_updated(limit: 3)
+```
+
 - **IMPROVED**: `PointerCollectionProxy#as_json` now supports the `pointers_only` option. By default it returns pointers (preserving backward compatibility), but you can set `pointers_only: false` to serialize objects with their fetched fields. This is useful when returning `has_many :through => :array` relationships in webhook responses.
 
   When `pointers_only: false`:
@@ -59,22 +71,6 @@ song.as_json(exclude_keys: [:acl, :created_at])
 song.as_json(exclude: [:acl, :created_at])
 ```
 
-### 3.2.2
-
-#### Improvements
-
-- **IMPROVED**: `latest` and `last_updated` methods now support a `limit:` option when passing constraints. This allows fetching multiple recent records while also filtering by query conditions.
-
-```ruby
-# Class methods
-Song.latest(:user.eq => user, limit: 5)       # 5 most recent for user
-Song.last_updated(status: "active", limit: 10) # 10 most recently updated active
-
-# Query instance methods
-query.latest(:user.eq => x, limit: 5)
-query.where(genre: "rock").last_updated(limit: 3)
-```
-
 ### 3.2.1
 
 #### New Features

Gemfile.lock

@@ -80,7 +80,7 @@ GEM
     pry-stack_explorer (0.6.1)
       binding_of_caller (~> 1.0)
       pry (~> 0.13)
-    rack (3.2.2)
+    rack (3.2.4)
     rake (13.3.0)
     redcarpet (3.6.1)
     redis (5.4.1)

test/lib/parse/hooks_and_validation_integration_test.rb

@@ -4,6 +4,7 @@
 
 # Test model with hooks, validations, and change tracking
 class TestProduct < Parse::Object
+  parse_class "HooksTestProduct"
   property :name, :string
   property :price, :float
   property :sku, :string

test/lib/parse/models/product_test.rb

@@ -1,6 +1,6 @@
 require_relative "../../../test_helper"
 
-class TestProduct < Minitest::Test
+class ProductModelTest < Minitest::Test
   CORE_FIELDS = Parse::Object.fields.merge({
     :id => :string,
     :created_at => :date,

test/lib/parse/mongodb_operators_integration_test.rb

@@ -102,7 +102,7 @@ def test_regex_and_string_operators_with_mongodb_direct
         puts "MongoDB Direct (:name.ends_with => 'Pro'): #{direct_names.inspect}"
 
         assert_equal parse_names, direct_names, "ends_with results should match"
-        assert_equal 2, direct_names.length, "Should find 2 products ending with Pro"
+        assert_equal 3, direct_names.length, "Should find 3 products ending with Pro"
 
         # --- Test 2c: ends_with with special characters ---
         puts "\n--- Test: ends_with with special characters ---"

test/lib/parse/query_pointers_contains_integration_test.rb

@@ -18,7 +18,7 @@ class QueryTestBook < Parse::Object
   property :isbn, :string
   property :price, :float
   property :publication_year, :integer
-  property :author, :object  # pointer to QueryTestAuthor
+  belongs_to :author, as: :query_test_author
   property :genres, :array
   property :awards, :array
   property :related_books, :array  # array of pointers to other QueryTestBook

test/lib/parse/upsert_methods_integration_test.rb

@@ -1,8 +1,8 @@
 require_relative "../../test_helper_integration"
 
 # Test models for upsert method integration testing
-class TestUser < Parse::Object
-  parse_class "TestUser"
+class UpsertTestUser < Parse::Object
+  parse_class "UpsertTestUser"
 
   property :email, :string
   property :name, :string
@@ -11,8 +11,8 @@ class TestUser < Parse::Object
   property :last_login, :date
 end
 
-class TestProduct < Parse::Object
-  parse_class "TestProduct"
+class UpsertTestProduct < Parse::Object
+  parse_class "UpsertTestProduct"
 
   property :sku, :string
   property :name, :string
@@ -40,12 +40,12 @@ def test_first_or_create_finds_existing_object_unchanged
         puts "\n=== Testing first_or_create Finds Existing Object Unchanged ==="
 
         # Create initial user
-        original_user = TestUser.new(email: "existing@example.com", name: "Original Name", age: 25)
+        original_user = UpsertTestUser.new(email: "existing@example.com", name: "Original Name", age: 25)
         assert original_user.save, "Original user should save"
         original_id = original_user.id
 
         # Use first_or_create with different resource_attrs
-        found_user = TestUser.first_or_create(
+        found_user = UpsertTestUser.first_or_create(
           { email: "existing@example.com" },
           { name: "Different Name", age: 30, status: "inactive" }
         )
@@ -70,7 +70,7 @@ def test_first_or_create_creates_new_object_unsaved
         puts "\n=== Testing first_or_create Creates New Object (Unsaved) ==="
 
         # Use first_or_create for non-existing user
-        new_user = TestUser.first_or_create(
+        new_user = UpsertTestUser.first_or_create(
           { email: "new@example.com" },
           { name: "New User", age: 35, status: "pending" }
         )
@@ -84,7 +84,7 @@ def test_first_or_create_creates_new_object_unsaved
         assert_nil new_user.id, "Unsaved object should not have ID"
 
         # Verify object is not yet in database
-        found_in_db = TestUser.first(email: "new@example.com")
+        found_in_db = UpsertTestUser.first(email: "new@example.com")
         assert_nil found_in_db, "Object should not be in database yet"
 
         puts "✅ first_or_create creates new unsaved object with combined attributes"
@@ -100,12 +100,12 @@ def test_first_or_create_bang_finds_existing_unchanged
         puts "\n=== Testing first_or_create! Finds Existing Object Unchanged ==="
 
         # Create initial product
-        original_product = TestProduct.new(sku: "PROD-001", name: "Original Product", price: 19.99)
+        original_product = UpsertTestProduct.new(sku: "PROD-001", name: "Original Product", price: 19.99)
         assert original_product.save, "Original product should save"
         original_id = original_product.id
 
         # Use first_or_create! with different resource_attrs
-        found_product = TestProduct.first_or_create!(
+        found_product = UpsertTestProduct.first_or_create!(
           { sku: "PROD-001" },
           { name: "Different Product", price: 29.99, category: "electronics" }
         )
@@ -130,7 +130,7 @@ def test_first_or_create_bang_creates_and_saves_new_object
         puts "\n=== Testing first_or_create! Creates and Saves New Object ==="
 
         # Use first_or_create! for non-existing product
-        new_product = TestProduct.first_or_create!(
+        new_product = UpsertTestProduct.first_or_create!(
           { sku: "PROD-NEW" },
           { name: "New Product", price: 49.99, category: "gadgets" }
         )
@@ -144,7 +144,7 @@ def test_first_or_create_bang_creates_and_saves_new_object
         assert_equal "gadgets", new_product.category, "Category should be set from resource_attrs"
 
         # Verify object is in database
-        found_in_db = TestProduct.first(sku: "PROD-NEW")
+        found_in_db = UpsertTestProduct.first(sku: "PROD-NEW")
         assert found_in_db, "Object should be in database"
         assert_equal new_product.id, found_in_db.id, "Should find the same object"
         assert_equal "New Product", found_in_db.name, "Database object should have correct name"
@@ -162,12 +162,12 @@ def test_create_or_update_bang_finds_and_updates_existing
         puts "\n=== Testing create_or_update! Finds and Updates Existing Object ==="
 
         # Create initial user
-        original_user = TestUser.new(email: "update@example.com", name: "Old Name", age: 25, status: "active")
+        original_user = UpsertTestUser.new(email: "update@example.com", name: "Old Name", age: 25, status: "active")
         assert original_user.save, "Original user should save"
         original_id = original_user.id
 
         # Use create_or_update! to update existing user
-        updated_user = TestUser.create_or_update!(
+        updated_user = UpsertTestUser.create_or_update!(
           { email: "update@example.com" },
           { name: "Updated Name", age: 30, last_login: Time.now }
         )
@@ -181,7 +181,7 @@ def test_create_or_update_bang_finds_and_updates_existing
         refute updated_user.new?, "Object should still be persisted"
 
         # Verify changes are persisted in database
-        found_in_db = TestUser.first(email: "update@example.com")
+        found_in_db = UpsertTestUser.first(email: "update@example.com")
         assert_equal "Updated Name", found_in_db.name, "Database should reflect name change"
         assert_equal 30, found_in_db.age, "Database should reflect age change"
 
@@ -198,7 +198,7 @@ def test_create_or_update_bang_creates_new_object
         puts "\n=== Testing create_or_update! Creates New Object ==="
 
         # Use create_or_update! for non-existing user
-        new_user = TestUser.create_or_update!(
+        new_user = UpsertTestUser.create_or_update!(
           { email: "create@example.com" },
           { name: "Created User", age: 28, status: "pending" }
         )
@@ -212,7 +212,7 @@ def test_create_or_update_bang_creates_new_object
         assert_equal "pending", new_user.status, "Status should be set from resource_attrs"
 
         # Verify object is in database
-        found_in_db = TestUser.first(email: "create@example.com")
+        found_in_db = UpsertTestUser.first(email: "create@example.com")
         assert found_in_db, "Object should be in database"
         assert_equal new_user.id, found_in_db.id, "Should find the same object"
 
@@ -229,15 +229,15 @@ def test_create_or_update_bang_no_save_when_no_changes
         puts "\n=== Testing create_or_update! No Save When No Changes ==="
 
         # Create initial product
-        original_product = TestProduct.new(sku: "NO-CHANGE", name: "Same Product", price: 15.50)
+        original_product = UpsertTestProduct.new(sku: "NO-CHANGE", name: "Same Product", price: 15.50)
         assert original_product.save, "Original product should save"
         original_updated_at = original_product.updated_at
 
         # Small delay to ensure updated_at would change if saved
         sleep(0.1)
 
         # Use create_or_update! with identical values
-        result_product = TestProduct.create_or_update!(
+        result_product = UpsertTestProduct.create_or_update!(
           { sku: "NO-CHANGE" },
           { name: "Same Product", price: 15.50 } # Identical values
         )
@@ -261,15 +261,15 @@ def test_create_or_update_bang_empty_resource_attrs
         puts "\n=== Testing create_or_update! with Empty resource_attrs ==="
 
         # Create initial user
-        original_user = TestUser.new(email: "empty@example.com", name: "Original", age: 40)
+        original_user = UpsertTestUser.new(email: "empty@example.com", name: "Original", age: 40)
         assert original_user.save, "Original user should save"
         original_updated_at = original_user.updated_at
 
         # Small delay to ensure updated_at would change if saved
         sleep(0.1)
 
         # Use create_or_update! with empty resource_attrs
-        result_user = TestUser.create_or_update!(
+        result_user = UpsertTestUser.create_or_update!(
           { email: "empty@example.com" },
           {} # Empty resource_attrs
         )
@@ -293,37 +293,37 @@ def test_performance_comparison_across_methods
         puts "\n=== Testing Performance Comparison Across Methods ==="
 
         # Create initial test data
-        test_user = TestUser.new(email: "perf@example.com", name: "Performance Test", age: 35)
+        test_user = UpsertTestUser.new(email: "perf@example.com", name: "Performance Test", age: 35)
         assert test_user.save, "Test user should save"
 
         # Test first_or_create performance (existing object)
         start_time = Time.now
         5.times do
-          result = TestUser.first_or_create({ email: "perf@example.com" }, { name: "Different" })
+          result = UpsertTestUser.first_or_create({ email: "perf@example.com" }, { name: "Different" })
           assert_equal "Performance Test", result.name, "Should find unchanged object"
         end
         first_or_create_time = Time.now - start_time
 
         # Test first_or_create! performance (existing object)
         start_time = Time.now
         5.times do
-          result = TestUser.first_or_create!({ email: "perf@example.com" }, { name: "Different" })
+          result = UpsertTestUser.first_or_create!({ email: "perf@example.com" }, { name: "Different" })
           assert_equal "Performance Test", result.name, "Should find unchanged object"
         end
         first_or_create_bang_time = Time.now - start_time
 
         # Test create_or_update! performance (no changes)
         start_time = Time.now
         5.times do
-          result = TestUser.create_or_update!({ email: "perf@example.com" }, { name: "Performance Test", age: 35 })
+          result = UpsertTestUser.create_or_update!({ email: "perf@example.com" }, { name: "Performance Test", age: 35 })
           assert_equal "Performance Test", result.name, "Should find unchanged object"
         end
         create_or_update_no_change_time = Time.now - start_time
 
         # Test create_or_update! performance (with changes)
         start_time = Time.now
         5.times do |i|
-          result = TestUser.create_or_update!({ email: "perf@example.com" }, { age: 35 + i })
+          result = UpsertTestUser.create_or_update!({ email: "perf@example.com" }, { age: 35 + i })
         end
         create_or_update_with_change_time = Time.now - start_time
 
@@ -355,7 +355,7 @@ def test_complex_upsert_workflow
         # Workflow: User registration and profile updates
 
         # Step 1: Try to find existing user, create if not found (unsaved)
-        user = TestUser.first_or_create(
+        user = UpsertTestUser.first_or_create(
           { email: "workflow@example.com" },
           { name: "Workflow User", age: 25, status: "pending" }
         )
@@ -368,7 +368,7 @@ def test_complex_upsert_workflow
         assert user.save, "Should save user after completing registration"
 
         # Step 3: Update profile information
-        updated_user = TestUser.create_or_update!(
+        updated_user = UpsertTestUser.create_or_update!(
           { email: "workflow@example.com" },
           { age: 26, last_login: Time.now }
         )
@@ -379,7 +379,7 @@ def test_complex_upsert_workflow
         assert updated_user.last_login.present?, "Should have last_login set"
 
         # Step 4: Subsequent login (no changes needed)
-        login_user = TestUser.create_or_update!(
+        login_user = UpsertTestUser.create_or_update!(
           { email: "workflow@example.com" },
           { age: 26 } # Same age, should not save
         )