Refactor Parse Server startup and LiveQuery test setup

Moved Parse Server configuration to environment variables in the startup script and simplified server invocation. Updated LiveQuery integration test to use block-based configuration and improved setup/teardown for better test isolation.

Author: AdrianCurtin

scripts/start-parse.sh

@@ -4,17 +4,25 @@ set -e
 echo "=== Parse Server Startup Script ==="
 echo "Setting up environment..."
 
-# Write environment variables to a file for debugging
-echo "PARSE_SERVER_MASTER_KEY_IPS=0.0.0.0/0,::/0" >> /tmp/parse-env
-echo "Environment file contents:"
-cat /tmp/parse-env
-
-# Export the environment variable
+# Export environment variables for Parse Server
 export PARSE_SERVER_MASTER_KEY_IPS="0.0.0.0/0,::/0"
+export PARSE_SERVER_APPLICATION_ID="myAppId"
+export PARSE_SERVER_MASTER_KEY="myMasterKey"
+export PARSE_SERVER_REST_API_KEY="test-rest-key"
+export PARSE_SERVER_DATABASE_URI="mongodb://admin:password@mongo:27017/parse?authSource=admin"
+export PARSE_SERVER_MOUNT_PATH="/parse"
+export PARSE_SERVER_CLOUD="/parse-server/cloud/main.js"
+export PARSE_SERVER_LOG_LEVEL="info"
+export PARSE_SERVER_ALLOW_CLIENT_CLASS_CREATION="true"
+
+# LiveQuery configuration via environment variables
+export PARSE_SERVER_LIVE_QUERY='{"classNames":["Song","Album","User","_User","TestLiveQuery"]}'
+export PARSE_SERVER_START_LIVE_QUERY_SERVER="true"
 
-# Verify the variable is set
-echo "Environment variable check:"
-echo "PARSE_SERVER_MASTER_KEY_IPS: $PARSE_SERVER_MASTER_KEY_IPS"
+echo "Environment configured:"
+echo "  PARSE_SERVER_APPLICATION_ID: $PARSE_SERVER_APPLICATION_ID"
+echo "  PARSE_SERVER_LIVE_QUERY: $PARSE_SERVER_LIVE_QUERY"
+echo "  PARSE_SERVER_START_LIVE_QUERY_SERVER: $PARSE_SERVER_START_LIVE_QUERY_SERVER"
 
 # Start Parse Server
 echo "Starting Parse Server..."
@@ -23,48 +31,15 @@ echo "Looking for parse-server..."
 which node
 ls -la /parse-server/
 
-# LiveQuery classes to enable real-time updates
-LIVE_QUERY_CLASSES="Song,Album,User,_User,TestLiveQuery"
-
 # Try different ways to start parse-server
 if [ -f "/parse-server/bin/parse-server" ]; then
   echo "Using /parse-server/bin/parse-server"
-  exec /parse-server/bin/parse-server \
-    --appId myAppId \
-    --masterKey myMasterKey \
-    --restAPIKey test-rest-key \
-    --databaseURI mongodb://admin:password@mongo:27017/parse?authSource=admin \
-    --mountPath /parse \
-    --cloud /parse-server/cloud/main.js \
-    --logLevel info \
-    --allowClientClassCreation true \
-    --liveQuery.classNames "$LIVE_QUERY_CLASSES" \
-    --startLiveQueryServer true
+  exec /parse-server/bin/parse-server
 elif [ -f "/usr/src/app/bin/parse-server" ]; then
   echo "Using /usr/src/app/bin/parse-server"
-  exec /usr/src/app/bin/parse-server \
-    --appId myAppId \
-    --masterKey myMasterKey \
-    --restAPIKey test-rest-key \
-    --databaseURI mongodb://admin:password@mongo:27017/parse?authSource=admin \
-    --mountPath /parse \
-    --cloud /parse-server/cloud/main.js \
-    --logLevel info \
-    --allowClientClassCreation true \
-    --liveQuery.classNames "$LIVE_QUERY_CLASSES" \
-    --startLiveQueryServer true
+  exec /usr/src/app/bin/parse-server
 else
   echo "Trying with node and index.js"
   cd /parse-server
-  exec node ./bin/parse-server \
-    --appId myAppId \
-    --masterKey myMasterKey \
-    --restAPIKey test-rest-key \
-    --databaseURI mongodb://admin:password@mongo:27017/parse?authSource=admin \
-    --mountPath /parse \
-    --cloud /parse-server/cloud/main.js \
-    --logLevel info \
-    --allowClientClassCreation true \
-    --liveQuery.classNames "$LIVE_QUERY_CLASSES" \
-    --startLiveQueryServer true
+  exec node ./bin/parse-server
 fi

test/lib/parse/hooks_and_validation_integration_test.rb

@@ -30,7 +30,8 @@ class TestProduct < Parse::Object
   validate :custom_sku_format
 
   def custom_sku_format
-    if sku.present? && !sku.match?(/^[A-Z]{3}-\d{4}$/)
+    # Allow both upper and lowercase since normalize_data will uppercase it
+    if sku.present? && !sku.match?(/^[A-Za-z]{3}-\d{4}$/)
       errors.add(:sku, "must be in format XXX-0000 (e.g., ABC-1234)")
     end
   end

test/lib/parse/live_query_integration_test.rb

@@ -2,6 +2,7 @@
 # frozen_string_literal: true
 
 require_relative "../../test_helper_integration"
+require_relative "../../../lib/parse/live_query"
 
 # Define a test model for LiveQuery integration tests
 class TestLiveQueryModel < Parse::Object
@@ -17,14 +18,18 @@ class LiveQueryIntegrationTest < Minitest::Test
   LIVE_QUERY_URL = "ws://localhost:2337"
 
   def setup
-    super
+    # Setup Parse client connection first (this is normally done by the module)
+    Parse::Test::ServerHelper.setup
+
+    # Enable LiveQuery feature
+    Parse.live_query_enabled = true
 
     # Configure LiveQuery
-    Parse::LiveQuery.configure(
-      url: LIVE_QUERY_URL,
-      application_id: "myAppId",
-      client_key: "test-rest-key",
-    )
+    Parse::LiveQuery.configure do |config|
+      config.url = LIVE_QUERY_URL
+      config.application_id = "myAppId"
+      config.client_key = "test-rest-key"
+    end
 
     # Clean up any existing test data
     cleanup_test_objects
@@ -33,7 +38,6 @@ def setup
   def teardown
     Parse::LiveQuery.reset!
     cleanup_test_objects
-    super
   end
 
   def cleanup_test_objects

test/lib/parse/model_associations_integration_test.rb

@@ -30,7 +30,7 @@ class AssociationTestBook < Parse::Object
 
   # Belongs to association
   belongs_to :author, as: :association_test_author
-  belongs_to :publisher, required: true
+  belongs_to :publisher, as: :association_test_publisher, required: true
 end
 
 class AssociationTestPublisher < Parse::Object
@@ -468,6 +468,10 @@ def test_has_many_array_pointer_collections
         # Test 1: Set up data for array pointer collections
         puts "\n--- Test 1: Setting up data for array pointer collections ---"
 
+        # Create publisher for books (publisher is required)
+        publisher = AssociationTestPublisher.new(name: "Array Test Publisher", country: "USA")
+        assert publisher.save, "Publisher should save successfully"
+
         # Create authors
         author1 = AssociationTestAuthor.new(name: "Array Author 1", email: "array1@example.com")
         author2 = AssociationTestAuthor.new(name: "Array Author 2", email: "array2@example.com")
@@ -478,9 +482,9 @@ def test_has_many_array_pointer_collections
         assert author3.save, "Author 3 should save successfully"
 
         # Create books
-        book1 = AssociationTestBook.new(title: "Array Book 1", author: author1)
-        book2 = AssociationTestBook.new(title: "Array Book 2", author: author2)
-        book3 = AssociationTestBook.new(title: "Array Book 3", author: author3)
+        book1 = AssociationTestBook.new(title: "Array Book 1", author: author1, publisher: publisher)
+        book2 = AssociationTestBook.new(title: "Array Book 2", author: author2, publisher: publisher)
+        book3 = AssociationTestBook.new(title: "Array Book 3", author: author3, publisher: publisher)
 
         assert book1.save, "Book 1 should save successfully"
         assert book2.save, "Book 2 should save successfully"
@@ -862,6 +866,10 @@ def test_association_edge_cases_and_error_handling
         # Test 5: Test association queries with edge case data
         puts "\n--- Test 5: Testing association queries with edge case data ---"
 
+        # Create publisher for edge case tests
+        edge_publisher = AssociationTestPublisher.new(name: "Edge Publisher", country: "USA")
+        assert edge_publisher.save, "Edge publisher should save"
+
         # Create author with special characters
         special_author = AssociationTestAuthor.new(
           name: "Special Author àáâãäå",
@@ -873,6 +881,7 @@ def test_association_edge_cases_and_error_handling
         special_book = AssociationTestBook.new(
           title: "Special Title: Ñoël & Company",
           author: special_author,
+          publisher: edge_publisher,
           price: 99.99,
         )
         assert special_book.save, "Special character book should save"
@@ -899,11 +908,12 @@ def test_association_edge_cases_and_error_handling
         perf_author = AssociationTestAuthor.new(name: "Performance Author")
         assert perf_author.save, "Performance author should save"
 
-        # Create multiple books quickly
+        # Create multiple books quickly (reusing edge_publisher from above)
         (1..10).each do |i|
           book = AssociationTestBook.new(
             title: "Performance Book #{i}",
             author: perf_author,
+            publisher: edge_publisher,
             price: i * 10.0,
           )
           assert book.save, "Performance book #{i} should save"