Skip to content

test: Skip analytics tests - Issue #263#265

Merged
maxrantil merged 1 commit intomasterfrom
hotfix/issue-263-skip-analytics-tests
Jan 2, 2026
Merged

test: Skip analytics tests - Issue #263#265
maxrantil merged 1 commit intomasterfrom
hotfix/issue-263-skip-analytics-tests

Conversation

@maxrantil
Copy link
Owner

Problem

Deployment is blocked because analytics provider tests are failing. Tests expect analytics script to load, but we intentionally disabled it in PR #264 due to analytics server outage.

Solution

Temporarily skip 7 analytics provider tests that expect script loading:

  • Production mode script loading tests (3 tests)
  • Script attributes validation tests (3 tests)
  • Script defer regression test (1 test)

All other tests (10 tests) still pass - they verify that analytics doesn't load in dev/test mode and that children render correctly.

Tests Skipped

○ skipped should load Umami script in production mode with env vars set
○ skipped should use requestIdleCallback for deferred loading in production
○ skipped should fallback to setTimeout when requestIdleCallback unavailable
○ skipped should set defer attribute on script element
○ skipped should construct correct script URL from environment variable  
○ skipped should set data-website-id attribute correctly
○ skipped should fail if script is not deferred

Impact

  • ✅ Unblocks deployment
  • ✅ 10/17 tests still running (verifying no-load scenarios)
  • ⚠️ Tests will be re-enabled when analytics is fixed

Related


Ready to merge immediately to unblock deployment

@maxrantil maxrantil merged commit 9338adc into master Jan 2, 2026
12 of 15 checks passed
@maxrantil maxrantil deleted the hotfix/issue-263-skip-analytics-tests branch January 2, 2026 18:25
Temporarily skip analytics provider tests since analytics script loading
is disabled due to server outage (Issue #263).

Tests will be re-enabled when analytics server is fixed and analytics
re-enabled.

Related to #263
@codecov-commenter
Copy link

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@github-actions
Copy link

github-actions bot commented Jan 2, 2026

🎯 Performance Budget Summary

Bundle Size Analysis

  • Status: warning
  • Total Size:
  • Budget: 1.5MB (1.45MB in CI)

Lighthouse Performance Validation

Desktop Results

  • Performance Score: Target 98%+
  • LCP: Target <1s
  • CLS: Target <0.05

Mobile Results

  • Performance Score: Target 98%+
  • LCP: Target <1s
  • CLS: Target <0.05

Performance Budget Status

  • Bundle Size: FAILED
  • Lighthouse Budget: PASSED

Next Steps

  • 🔧 Optimize bundle size to meet budget constraints

Generated at $(date)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants