Skip to content

[SYCL][Graph][Test] Add native recording mock fixture - #23074

Open
mmichel11 wants to merge 3 commits into
intel:syclfrom
adamfidel:matt/unit_test_nr_fixture
Open

[SYCL][Graph][Test] Add native recording mock fixture#23074
mmichel11 wants to merge 3 commits into
intel:syclfrom
adamfidel:matt/unit_test_nr_fixture

Conversation

@mmichel11

Copy link
Copy Markdown
Contributor

Adds UR graph fixture for unit testing native recording. Basic tests are added to validate the native recording trace. The UR function overrides simulate successful graph scenarios tracked in a singleton state.

Adds UR graph fixture for unit testing native recording. Basic tests are
added to validate the native recording trace. The default function
overrides simulate the successful graph scenarios.
@mmichel11
mmichel11 force-pushed the matt/unit_test_nr_fixture branch from cefe7c1 to f892609 Compare September 1, 2026 19:29
@mmichel11
mmichel11 marked this pull request as ready for review September 2, 2026 13:27
@mmichel11
mmichel11 requested a review from a team as a code owner September 2, 2026 13:27
@mmichel11
mmichel11 requested a review from adamfidel September 2, 2026 13:27

@adamfidel adamfidel left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM.

Took me a bit to understand how the mock functions were all hooked up, but I think it's a good design.

The comment suggestion was just for me to understand it, but it might be useful for others.

Comment thread sycl/unittests/Extensions/CommandGraph/NativeRecordingMock.cpp Outdated
Comment thread sycl/unittests/Extensions/CommandGraph/NativeRecordingMock.cpp Outdated
Co-authored-by: Adam Fidel <adam.fidel@intel.com>
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