Skip to content

Upgrade Allure Report from 2.39.0 to 2.43.0#6730

Draft
valfirst with Copilot wants to merge 1 commit into
masterfrom
copilot/review-allure-versions
Draft

Upgrade Allure Report from 2.39.0 to 2.43.0#6730
valfirst with Copilot wants to merge 1 commit into
masterfrom
copilot/review-allure-versions

Conversation

Copilot AI commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Allure 2.40.0 introduced a completely new frontend ("new web") built with Vite/TypeScript, replacing the Backbone.Marionette SPA. This breaks all existing plugin JS that used allure.api.* APIs. CSS is now inlined into a single hashed JS bundle (assets/index-[hash].js) with CSS variables for theming.

Changes

Core

  • Version bump allureReportVersion 2.39.0 → 2.43.0
  • FreeMarker template rewritten for new template variables (coreStyleUrls/pluginStyleUrls/coreJsUrls/pluginJsUrls) and __allureCoreLoaded promise-based plugin loading
  • AllureReportGenerator.patchAllureFiles() — patches single hashed JS bundle (no separate CSS file); uses backtick-delimited patterns (unknown:\Unknown`, `` failed,broken... ``); swaps --color-status-broken-bg--color-status-unknown-bg` palette variables directly

Plugins

  • BrandTitlePlugin — targets .side-nav__brand-text + CSS variables (--font-size-xl, --color-text-primary); handles collapsed sidebar state
  • BrandLogoPlugin — targets .side-nav__brand-icon instead of .side-nav__brand
  • LayoutConfiguringPlugin — rewritten from JS (allure.api.tabs.filter) to CSS (:has() selectors on .side-nav__item, [data-widget-id], [data-chart-id] with display: none)
  • CustomTranslationsPlugin — stores translations on window.__vividusCustomTranslations via IIFE (old allure.api.addTranslation no longer exists)
  • CustomTabPlugin index.js — vanilla JS that injects nav link into sidebar and renders <zero-md> on hash change
  • Behaviors plugin — removed index.js reference from Spring config (behaviors tab is now built into core; only the Java data aggregator is needed)

Tests

  • AllureReportPatchingConsistencyTests — reads entry JS from vite-manifest.json instead of hardcoded app.js/styles.css
  • All plugin test expectations updated to match new CSS/JS output

@codecov

codecov Bot commented Jul 1, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 91.83673% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 97.77%. Comparing base (3e2340c) to head (1a6ed0f).
⚠️ Report is 2 commits behind head on master.

Files with missing lines Patch % Lines
...g/vividus/report/allure/AllureReportGenerator.java 73.33% 2 Missing and 2 partials ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #6730      +/-   ##
============================================
- Coverage     97.79%   97.77%   -0.02%     
- Complexity     7325     7329       +4     
============================================
  Files          1015     1015              
  Lines         21508    21522      +14     
  Branches       1416     1416              
============================================
+ Hits          21033    21043      +10     
- Misses          359      361       +2     
- Partials        116      118       +2     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown

Qodana for JVM

It seems all right 👌

No new problems were found according to the checks applied

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Contact Qodana team

Contact us at qodana-support@jetbrains.com

@sonarqubecloud

sonarqubecloud Bot commented Jul 1, 2026

Copy link
Copy Markdown

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