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