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.

Sign up for GitHub

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

Jump to bottom

test_runner: refactor snapshots to support multiple files in the same process #53853

Closed
wants to merge 2 commits into from
Closed

test_runner: refactor snapshots to support multiple files in the same process #53853

wants to merge 2 commits into from

Commits on Jul 15, 2024

  1. test_runner: add context.filePath 

    This commit adds a filePath getter to the TestContext and
SuiteContext classes. This allows a context to be mapped back to
the original test file that created it, even if it was imported
from another file. This is useful for mapping features like test
snapshots to the correct test file. This is also prep work for
supporting running test files in the test runner process.
    @cjihrig
    cjihrig committed Jul 15, 2024
    Configuration menu
    Copy the full SHA
    e5de45a View commit details
    Browse the repository at this point in the history

  2. test_runner: refactor snapshots to get file from context 

    This commit refactors the internals of snapshot tests to get the
name of the test file from the test context instead of passing
it to the SnapshotManager constructor. This is prep work for
supporting running test files in the test runner process.
    @cjihrig
    cjihrig committed Jul 15, 2024
    Configuration menu
    Copy the full SHA
    f4bff7d View commit details
    Browse the repository at this point in the history