panda-core 0.10.2 → 0.10.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f1ca00630306c920fd7c8b7a5c8703532d44c6f24e1960b5a070f5083ee9d467
-  data.tar.gz: aaebba3e769ccba28420bb69680d461558b6371e88977c38846e8180d045acce
+  metadata.gz: b76ce50771d377d8b8434339c205494d1b3994bd45c1e381dfa700b71a2656bd
+  data.tar.gz: 2d0e4f33307b5dca31db3464ff17d8dc1dd70c35849e688cca30b904d30d2bec
 SHA512:
-  metadata.gz: 9b4235040de9088b17e648d1085c3740129dd62a0d319ff4548d32e979bf425caf1b3fe80a4491898fef8a42d61517cb5fe2d3c55f1f949f707f8708e8eabf27
-  data.tar.gz: c4afb07c75306e2e6a2a689f3fc24c89bc62c274b71f6d5d8490d6874f5776afd318fc4dbb74bcca4f67e74ab1555d4bbb72d5704daa640eb4e6b58167ba9231
+  metadata.gz: 2cf27e3c3bf6199a57320a5d6351125c704bf0d2a87de6790d698caf4dc86351661af8fa75164c3139deac6383ae0f63c5e527e896e56017ad8bff5eee5a9b39
+  data.tar.gz: f6f374ae824ce953f2ebbe100a9bcdfb5e11db6efe7154b098b1c0cb74475f0b1d77e99c6f6b491c519c216e731f8b96b6161bb6b1da5b38b6702b77d1411200

data/app/javascript/panda/core/application.js

@@ -1,8 +1,13 @@
-import { Application } from "@hotwired/stimulus"
-import "@fortawesome/fontawesome-free"
+import { Application } from '@hotwired/stimulus'
+import '@fortawesome/fontawesome-free'
 
 const application = Application.start()
 
+// Always propagate Stimulus controller errors for better debugging
+application.handleError = (error, message, detail) => {
+  throw error
+}
+
 // Configure Stimulus development experience
 application.debug = false
 window.Stimulus = application
@@ -14,4 +19,4 @@ export { application }
 
 // Tailwind Plus Elements can be loaded by importing "panda/core/tailwindplus-elements"
 // or by adding the script tag directly to your HTML:
-// <script src="https://cdn.jsdelivr.net/npm/@tailwindplus/elements@1" type="module"></script>
+// <script src="https://cdn.jsdelivr.net/npm/@tailwindplus/elements@1" type="module"></script>

data/lib/panda/core/engine.rb

@@ -24,7 +24,6 @@ require_relative "shared/inflections_config"
 require_relative "shared/generator_config"
 
 # Load engine configuration modules
-require_relative "engine/test_config"
 require_relative "engine/autoload_config"
 require_relative "engine/middleware_config"
 require_relative "engine/importmap_config"
@@ -45,7 +44,6 @@ module Panda
       include Shared::GeneratorConfig
 
       # Include engine-specific configuration modules
-      include TestConfig
       include AutoloadConfig
       include MiddlewareConfig
       include ImportmapConfig

data/lib/panda/core/testing/rails_helper.rb

@@ -61,8 +61,11 @@ RSpec.configure do |config|
   if ENV["GITHUB_ACTIONS"] == "true"
     require "rspec/github"
     config.add_formatter RSpec::Github::Formatter
-    # Also add documentation formatter for colored real-time output in CI logs
-    config.add_formatter RSpec::Core::Formatters::DocumentationFormatter, $stdout
+    # Also add documentation formatter for colored real-time output in CI logs,
+    # but avoid double-printing when one is already configured (e.g. via CLI).
+    unless config.formatters.any? { |formatter| formatter.is_a?(RSpec::Core::Formatters::DocumentationFormatter) }
+      config.add_formatter RSpec::Core::Formatters::DocumentationFormatter, $stdout
+    end
   end
 
   # Controller testing support (if rails-controller-testing is available)

data/lib/panda/core/testing/support/authentication_test_helpers.rb

@@ -134,13 +134,13 @@ module Panda
         visit "/admin/test_login/#{user.id}?return_to=#{return_path}"
 
         # Wait briefly for session to be set
-        sleep 0.2
+        # sleep 0.2
 
         # Manually visit the destination since Cuprite doesn't reliably follow redirects
         if expect_success
           visit return_path
           # Wait for page to load
-          sleep 0.2
+          # sleep 0.2
 
           # Verify we're on the expected path
           expect(page).to have_current_path(return_path, wait: 2)

data/lib/panda/core/testing/support/system/better_system_tests.rb

@@ -114,7 +114,7 @@ module Panda
                 end
 
                 # Wait for DOM to be ready
-                sleep 0.5
+                # sleep 0.5
 
                 # Get comprehensive page info
                 page_html = begin

data/lib/panda/core/testing/support/system/browser_console_logger.rb

@@ -9,7 +9,15 @@ module BrowserConsoleLogger
 
       if respond_to?(:page) && page.driver.is_a?(Capybara::Cuprite::Driver)
         begin
-          console_logs = page.driver.browser.console_messages
+          # Access the console logger via the CupriteSetup module
+          console_logger = Panda::Core::Testing::CupriteSetup.console_logger
+
+          unless console_logger
+            puts "\n⚠️  Console logger not available"
+            next
+          end
+
+          console_logs = console_logger.logs
 
           if console_logs.any?
             puts "\n" + "=" * 80
@@ -17,17 +25,15 @@ module BrowserConsoleLogger
             puts "=" * 80
 
             console_logs.each_with_index do |msg, index|
-              type_icon = case msg["type"]
+              level = msg.level.downcase
+              type_icon = case level
               when "error" then "❌"
               when "warning" then "⚠️"
               when "info" then "ℹ️"
               else "📝"
               end
 
-              puts "#{index + 1}. [#{msg["type"].upcase}] #{type_icon}"
-              puts "   Message: #{msg["message"]}"
-              puts "   Source: #{msg["source"]}" if msg["source"]
-              puts "   Line: #{msg["line"]}" if msg["line"]
+              puts "#{index + 1}. #{type_icon} #{msg}"
               puts ""
             end
 
@@ -37,6 +43,7 @@ module BrowserConsoleLogger
           end
         rescue => e
           puts "\n⚠️  Failed to capture console logs: #{e.message}"
+          puts "   #{e.class}: #{e.backtrace.first(3).join("\n   ")}" if ENV["DEBUG"]
         end
       end
     end

data/lib/panda/core/testing/support/system/ci_capybara_config.rb

@@ -47,13 +47,16 @@ Capybara.register_server :puma_ci do |app, port, host|
     raise e
   end
 
+  # Allow conditional silencing of server logs via CAPYBARA_SERVER_VERBOSE
+  verbose = ENV.fetch("CAPYBARA_SERVER_VERBOSE", "false") == "true"
+
   options = {
     Host: host,
     Port: port,
     Threads: "#{min_threads}:#{max_threads}",
     Workers: 0,
-    Silent: false,  # Always verbose in CI to catch startup errors
-    Verbose: true,
+    Silent: !verbose,
+    Verbose: verbose,
     PreloadApp: false
   }
 

data/lib/panda/core/testing/support/system/cuprite_helpers.rb

@@ -44,7 +44,7 @@ module Panda
               ready = page.evaluate_script("document.readyState")
               break if ready == "complete"
 
-              sleep 0.1
+              # sleep 0.1
             end
           end
         rescue Timeout::Error
@@ -64,7 +64,7 @@ module Panda
               end
               break if loaded
 
-              sleep 0.1
+              # sleep 0.1
             end
           end
           true
@@ -82,7 +82,7 @@ module Panda
           while Time.now - start_time < timeout
             return true if page.has_css?(selector, visible: true)
 
-            sleep 0.1
+            # sleep 0.1
           end
           false
         end
@@ -96,7 +96,7 @@ module Panda
           while Time.now - start_time < timeout
             return true if page.has_text?(text)
 
-            sleep 0.1
+            # sleep 0.1
           end
           false
         end
@@ -122,7 +122,7 @@ module Panda
           while Time.now - start_time < timeout
             return true if page.html != initial_dom
 
-            sleep 0.1
+            # sleep 0.1
           end
           false
         end
@@ -147,7 +147,7 @@ module Panda
         def click_on_selectors(*css_selectors)
           css_selectors.each do |selector|
             find(selector).click
-            sleep 0.1 # Add a small delay to allow JavaScript to run
+            # sleep 0.1 # Add a small delay to allow JavaScript to run
           end
         end
 
@@ -160,7 +160,7 @@ module Panda
           while Time.now - start_time < timeout
             return true if page.has_field?(field_name, with: value)
 
-            sleep 0.1
+            # sleep 0.1
           end
           false
         end
@@ -179,7 +179,7 @@ module Panda
 
             if retries <= 2 && elapsed < max_duration && ENV["GITHUB_ACTIONS"]
               puts "[CI] Element not found on fill_in '#{locator}', retry #{retries}/2 (#{elapsed.round(1)}s elapsed)"
-              sleep 0.5
+              # sleep 0.5
               retry
             else
               puts "[CI] Giving up on fill_in '#{locator}' after #{retries} retries and #{elapsed.round(1)}s" if ENV["GITHUB_ACTIONS"]
@@ -201,7 +201,7 @@ module Panda
 
             if retries <= 2 && elapsed < max_duration && ENV["GITHUB_ACTIONS"]
               puts "[CI] Element not found on select '#{value}' from '#{from}', retry #{retries}/2 (#{elapsed.round(1)}s elapsed)"
-              sleep 0.5
+              # sleep 0.5
               retry
             else
               puts "[CI] Giving up on select '#{value}' from '#{from}' after #{retries} retries and #{elapsed.round(1)}s" if ENV["GITHUB_ACTIONS"]
@@ -223,7 +223,7 @@ module Panda
 
             if retries <= 2 && elapsed < max_duration && ENV["GITHUB_ACTIONS"]
               puts "[CI] Element not found on click_button '#{locator}', retry #{retries}/2 (#{elapsed.round(1)}s elapsed)"
-              sleep 0.5
+              # sleep 0.5
               retry
             else
               puts "[CI] Giving up on click_button '#{locator}' after #{retries} retries and #{elapsed.round(1)}s" if ENV["GITHUB_ACTIONS"]

data/lib/panda/core/testing/support/system/cuprite_setup.rb

@@ -2,6 +2,7 @@
 
 require "ferrum"
 require "capybara/cuprite"
+require_relative "ferrum_console_logger"
 
 # Shared Cuprite driver configuration for all Panda gems
 # This provides standard Cuprite setup with sensible defaults that work across gems
@@ -17,6 +18,14 @@ module Panda
   module Core
     module Testing
       module CupriteSetup
+        # Class variable to store the console logger instance
+        # This allows tests to access console logs after they run
+        @console_logger = nil
+
+        class << self
+          attr_accessor :console_logger
+        end
+
         # Base Cuprite options shared across all drivers
         def self.base_options
           default_timeout = 2
@@ -67,8 +76,15 @@ module Panda
           # Add CI-specific options
           if ENV["GITHUB_ACTIONS"] == "true"
             options[:browser_options].merge!(ci_browser_options)
+
+            # Ensure CI uses xvfb to run the browser
+            options[:browser_options][:xvfb] = true
           end
 
+          # Create console logger for capturing browser console messages
+          self.console_logger = Panda::Core::Testing::Support::System::FerrumConsoleLogger.new
+          options[:logger] = console_logger
+
           Capybara.register_driver :cuprite do |app|
             Capybara::Cuprite::Driver.new(app, **options)
           end
@@ -83,6 +99,9 @@ module Panda
             options[:browser_options].merge!(ci_browser_options)
           end
 
+          # Use the same console logger instance for mobile driver
+          options[:logger] = console_logger if console_logger
+
           Capybara.register_driver :cuprite_mobile do |app|
             Capybara::Cuprite::Driver.new(app, **options)
           end

data/lib/panda/core/testing/support/system/ferrum_console_logger.rb

@@ -0,0 +1,121 @@
+# frozen_string_literal: true
+
+require "json"
+
+module Panda
+  module Core
+    module Testing
+      module Support
+        module System
+          # Custom logger for capturing browser console messages via Ferrum/Cuprite
+          # Ferrum doesn't provide a direct API for console messages - instead it uses
+          # Chrome DevTools Protocol (CDP) events that are sent to a logger object
+          class FerrumConsoleLogger
+            attr_reader :logs
+
+            def initialize
+              @logs = []
+            end
+
+            # Ferrum calls this method with CDP protocol events
+            # Format: "SEND message_id {json}" or "RECV message_id {json}"
+            def puts(log_str)
+              return unless log_str.is_a?(String)
+
+              parts = log_str.strip.split(" ", 3)
+              return if parts.size < 3
+
+              # Parse the JSON data from CDP event
+              data = JSON.parse(parts[2])
+
+              # Only capture console-related events
+              if console_event?(data)
+                @logs << Message.new(log_str)
+              end
+            rescue JSON::ParserError
+              # Silently ignore malformed JSON
+            end
+
+            def clear
+              @logs.clear
+            end
+
+            private
+
+            def console_event?(data)
+              %w[
+                Runtime.exceptionThrown
+                Log.entryAdded
+                Runtime.consoleAPICalled
+              ].include?(data["method"])
+            end
+
+            # Wrapper for a single console message
+            class Message
+              def initialize(log_str)
+                parts = log_str.strip.split(" ", 3)
+                @raw = log_str
+                @data = JSON.parse(parts[2])
+              end
+
+              # Get the log level (error, warning, info, etc.)
+              def level
+                # Different CDP events structure level differently
+                @data.dig("params", "entry", "level") ||
+                  @data.dig("params", "type") ||
+                  "log"
+              end
+
+              # Get the message text
+              def message
+                # Handle different CDP event types
+                if @data["method"] == "Runtime.exceptionThrown"
+                  exception = @data.dig("params", "exceptionDetails")
+                  if exception
+                    text = exception.dig("exception", "description") || exception["text"]
+                    location = "#{exception.dig("url")}:#{exception.dig("lineNumber")}"
+                    "#{text} (#{location})"
+                  else
+                    "Unknown exception"
+                  end
+                elsif @data["method"] == "Log.entryAdded"
+                  @data.dig("params", "entry", "text") || ""
+                elsif @data["method"] == "Runtime.consoleAPICalled"
+                  # Console API calls have arguments that need to be extracted
+                  args = @data.dig("params", "args") || []
+                  args.map { |arg| format_argument(arg) }.join(" ")
+                else
+                  @raw
+                end
+              end
+
+              # Format a CDP RemoteObject argument
+              def format_argument(arg)
+                case arg["type"]
+                when "string"
+                  arg["value"]
+                when "number", "boolean"
+                  arg["value"].to_s
+                when "undefined"
+                  "undefined"
+                when "object"
+                  if arg["subtype"] == "null"
+                    "null"
+                  else
+                    arg["description"] || "[Object]"
+                  end
+                else
+                  arg["description"] || arg.inspect
+                end
+              end
+
+              def to_s
+                "[#{level.upcase}] #{message}"
+              end
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/panda/core/testing/support/system/system_test_helpers.rb

@@ -102,7 +102,7 @@ RSpec.configure do |config|
         nil
       end
 
-      sleep 0.5
+      # sleep 0.5
 
       # Get comprehensive page info
       page_html = begin

data/lib/panda/core/version.rb

@@ -2,7 +2,7 @@
 
 module Panda
   module Core
-    # Version 0.10.2 - Fix JavaScript middleware fallback for CI environments
-    VERSION = "0.10.2"
+    # Version 0.10.4 - Database configuration improvements for CI tests
+    VERSION = "0.10.4"
   end
 end

data/lib/panda/core.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require "rails"
+# require "active_support/core_ext/module/attribute_accessors"
 
 module Panda
   module Core

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: panda-core
 version: !ruby/object:Gem::Version
-  version: 0.10.2
+  version: 0.10.4
 platform: ruby
 authors:
 - Otaina Limited
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-11-16 00:00:00.000000000 Z
+date: 2025-11-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: image_processing
@@ -500,7 +500,6 @@ files:
 - lib/panda/core/engine/middleware_config.rb
 - lib/panda/core/engine/omniauth_config.rb
 - lib/panda/core/engine/phlex_config.rb
-- lib/panda/core/engine/test_config.rb
 - lib/panda/core/media.rb
 - lib/panda/core/module_registry.rb
 - lib/panda/core/notifications.rb
@@ -527,6 +526,7 @@ files:
 - lib/panda/core/testing/support/system/cuprite_helpers.rb
 - lib/panda/core/testing/support/system/cuprite_setup.rb
 - lib/panda/core/testing/support/system/database_connection_helpers.rb
+- lib/panda/core/testing/support/system/ferrum_console_logger.rb
 - lib/panda/core/testing/support/system/system_test_helpers.rb
 - lib/panda/core/version.rb
 - lib/tasks/panda/core/migrations.rake

data/lib/panda/core/engine/test_config.rb

@@ -1,20 +0,0 @@
-# frozen_string_literal: true
-
-module Panda
-  module Core
-    class Engine < ::Rails::Engine
-      # Test environment configuration
-      module TestConfig
-        extend ActiveSupport::Concern
-
-        included do
-          # For testing: Don't expose engine migrations since we use "copy to host app" strategy
-          # In test environment, migrations should be copied to the host app
-          if Rails.env.test?
-            config.paths["db/migrate"] = []
-          end
-        end
-      end
-    end
-  end
-end