Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Graphics e2e tests syntax mode #1723

Merged
merged 4 commits into from
Nov 8, 2024

Conversation

SlicedSilver
Copy link
Contributor

@SlicedSilver SlicedSilver commented Nov 7, 2024

Type of PR: infrastructure

Overview of change:
Adds a mode for running the graphics test such that the golden case uses the test case code from the golden branch. This will be useful testing changes to API (such that the test case code is changed and can only run in the new branch).

Additionally fixes the graphics pipeline when running on the CI. Previously a single failing test would fail all the remaining test cases as well.

How to run:
Authorised users can trigger the pipeline on a specific branch with the run_branch_specific_graphics_tests parameter set to true through the CircleCI site.
Note: it will only work if the 'golden' branch has the code in this PR.

@SlicedSilver SlicedSilver self-assigned this Nov 7, 2024
Copy link

github-actions bot commented Nov 7, 2024

size-limit report 📦

Path Size
ESM 43.38 KB (+0.03% 🔺)
ESM createChart 40.32 KB (0%)
ESM createChartEx 39.04 KB (0%)
ESM Standalone 44.94 KB (+0.18% 🔺)
Standalone 44.81 KB (-0.04% 🔽)
Plugin: Text Watermark 1.87 KB (0%)
Plugin: Image Watermark 1.7 KB (0%)
Plugin: Series Markers 3.89 KB (0%)

@SlicedSilver SlicedSilver force-pushed the graphics-e2e-tests-syntax-mode branch 2 times, most recently from 2e8efd6 to 9a1c615 Compare November 7, 2024 16:49
@SlicedSilver SlicedSilver force-pushed the graphics-e2e-tests-syntax-mode branch from 9a1c615 to c67a4d4 Compare November 7, 2024 16:58
@SlicedSilver SlicedSilver requested a review from edew November 7, 2024 17:00
@SlicedSilver SlicedSilver marked this pull request as ready for review November 7, 2024 17:00
@SlicedSilver SlicedSilver added the infrastructure Infrastructure surrounding the development of the library (CI, tests, scripts). label Nov 7, 2024
Copy link
Contributor

@edew edew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice. LGTM 👍

@SlicedSilver SlicedSilver merged commit ceeb359 into v5-candidate Nov 8, 2024
33 checks passed
@SlicedSilver SlicedSilver deleted the graphics-e2e-tests-syntax-mode branch November 8, 2024 16:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infrastructure Infrastructure surrounding the development of the library (CI, tests, scripts).
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants