debug ci failures

Signed-off-by: Andrei Gherghescu <[email protected]>

Author: andrei-ng

.github/workflows/ci.yml

@@ -113,13 +113,33 @@ jobs:
       - name: Test plotly and plotly_static
         shell: bash
         run: |
+          echo "=== Platform Information ==="
+          echo "OS: $(uname -a)"
+          echo "Architecture: $(uname -m)"
+          echo "Current directory: $(pwd)"
+          
+          echo "=== Checking for existing processes ==="
+          ps aux | grep -E "(chromedriver|geckodriver|firefox|chrome)" || echo "No browser processes found"
+          
+          echo "=== Checking port availability ==="
+          netstat -an | grep -E ":(4444|4445|4446|4447|4448|4449|4450)" || echo "No WebDriver ports in use"
+          
+          echo "=== Running tests ==="
           if [ "${{ matrix.browser }}" = "firefox" ]; then
-            cargo test -p plotly --features static_export_default --lib
-            cargo test -p plotly_static --features geckodriver,webdriver_download
+            echo "Testing with Firefox/geckodriver..."
+            timeout 600 cargo test -p plotly --features static_export_default --lib -- --nocapture || echo "plotly tests failed"
+            timeout 600 cargo test -p plotly_static --features geckodriver,webdriver_download -- --nocapture || echo "plotly_static tests failed"
           else
-            cargo test -p plotly --features static_export_chromedriver,static_export_downloader --lib
-            cargo test -p plotly_static --features chromedriver,webdriver_download
+            echo "Testing with Chrome/chromedriver..."
+            timeout 600 cargo test -p plotly --features static_export_chromedriver,static_export_downloader --lib -- --nocapture || echo "plotly tests failed"
+            timeout 600 cargo test -p plotly_static --features chromedriver,webdriver_download -- --nocapture || echo "plotly_static tests failed"
           fi
+          
+          echo "=== Cleanup after tests ==="
+          pkill -f chromedriver || echo "No chromedriver processes to kill"
+          pkill -f geckodriver || echo "No geckodriver processes to kill"
+          pkill -f firefox || echo "No firefox processes to kill"
+          pkill -f chrome || echo "No chrome processes to kill"
 
   test-macos-compatibility:
     name: Test macOS Compatibility
@@ -186,6 +206,15 @@ jobs:
           
           echo "=== Testing user data directory management ==="
           cargo test -p plotly_static --features chromedriver,webdriver_download --lib macos_tests::macos_tests::test_user_data_directory_management -- --nocapture || echo "User data directory management test failed"
+          
+          echo "=== Testing PDF debugging ==="
+          cargo test -p plotly_static --features chromedriver,webdriver_download --lib macos_tests::macos_tests::test_pdf_debugging_macos -- --nocapture || echo "PDF debugging test failed"
+          
+          echo "=== Testing PDF with alternative flags ==="
+          cargo test -p plotly_static --features chromedriver,webdriver_download --lib macos_tests::macos_tests::test_pdf_with_alternative_flags -- --nocapture || echo "PDF alternative flags test failed"
+          
+          echo "=== Testing html2pdf library availability ==="
+          cargo test -p plotly_static --features chromedriver,webdriver_download --lib macos_tests::macos_tests::test_html2pdf_library_loading -- --nocapture || echo "html2pdf library test failed"
       
       - name: Run all plotly_static tests on macOS
         run: |

plotly_static/src/lib.rs

@@ -261,7 +261,8 @@ use webdriver::WebDriver;
 
 use crate::template::{IMAGE_EXPORT_JS_SCRIPT, PDF_EXPORT_JS_SCRIPT};
 
-const DEFAULT_CAPS_ARRAY: [&str; 20] = [
+#[cfg(feature = "chromedriver")]
+const CHROME_DEFAULT_CAPS: [&str; 24] = [
     "--headless",
     "--no-sandbox",
     "--disable-gpu-sandbox",
@@ -283,6 +284,18 @@ const DEFAULT_CAPS_ARRAY: [&str; 20] = [
     "--allow-file-access-from-files",
     "--no-first-run",
     "--no-default-browser-check",
+    // Additional flags for better PDF generation support
+    "--disable-backgrounding-occluded-windows",
+    "--disable-ipc-flooding-protection",
+    "--enable-logging",
+    "--v=1",
+];
+
+#[cfg(feature = "geckodriver")]
+const FIREFOX_DEFAULT_CAPS: [&str; 3] = [
+    "-headless", // Note: single dash for Firefox
+    "--no-remote",
+    "--disable-gpu", // Firefox does support this flag
 ];
 
 mod template;
@@ -292,6 +305,10 @@ mod webdriver;
 #[cfg(target_os = "macos")]
 mod macos_tests;
 
+#[cfg(test)]
+#[cfg(target_os = "windows")]
+mod windows_tests;
+
 /// Supported image formats for static image export.
 ///
 /// This enum defines all the image formats that can be exported from Plotly
@@ -438,7 +455,14 @@ impl Default for StaticExporterBuilder {
             webdriver_url: webdriver::WEBDRIVER_URL.to_string(),
             spawn_webdriver: true,
             offline_mode: false,
-            webdriver_browser_caps: DEFAULT_CAPS_ARRAY.iter().map(|s| s.to_string()).collect(),
+            webdriver_browser_caps: {
+                #[cfg(feature = "chromedriver")]
+                { CHROME_DEFAULT_CAPS.iter().map(|s| s.to_string()).collect() }
+                #[cfg(feature = "geckodriver")]
+                { FIREFOX_DEFAULT_CAPS.iter().map(|s| s.to_string()).collect() }
+                #[cfg(not(any(feature = "chromedriver", feature = "geckodriver")))]
+                { Vec::new() }
+            },
         }
     }
 }