onnxruntime
Add unit test for fix of plugin EP library handle leak in PR 28396
#28430
Merged
Overview
Commits
18
Changes
View On GitHub

Add unit test for fix of plugin EP library handle leak in PR 28396 #28430

adrianlizarraga merged 18 commits into main from adrianl/EpLoadRefCountLeak_UnitTest
adrianlizarraga
adrianlizarraga Add unit test for fix in PR #28396
9b3a2422
adrianlizarraga Add unit test for fix in PR #28396
5773f534
adrianlizarraga adrianlizarraga requested a review from copilot-pull-request-reviewer copilot-pull-request-reviewer 35 days ago
copilot-pull-request-reviewer
copilot-pull-request-reviewer commented on 2026-05-09
adrianlizarraga Address PR review: use baseline comparison instead of drain loop
55ae8a55
adrianlizarraga Use GTEST_SKIP when library is already loaded
6a88946e
adrianlizarraga Move handle leak test to separate binary for reliable detection
c7c3c3fa
adrianlizarraga Clean up
1a1d4231
adrianlizarraga Merge branch 'main' into adrianl/EpLoadRefCountLeak_UnitTest
6bad1604
adrianlizarraga adrianlizarraga requested a review from copilot-pull-request-reviewer copilot-pull-request-reviewer 32 days ago
copilot-pull-request-reviewer
copilot-pull-request-reviewer commented on 2026-05-11
adrianlizarraga Exclude test_handle_leak.cc from main autoep test binary
712b1ff2
adrianlizarraga adrianlizarraga requested a review from copilot-pull-request-reviewer copilot-pull-request-reviewer 32 days ago
copilot-pull-request-reviewer
copilot-pull-request-reviewer commented on 2026-05-11
adrianlizarraga Revert to GTEST_SKIP: remove separate binary to fix ARM64 Debug OOM
9e529b25
adrianlizarraga adrianlizarraga requested a review from copilot-pull-request-reviewer copilot-pull-request-reviewer 32 days ago
copilot-pull-request-reviewer
copilot-pull-request-reviewer commented on 2026-05-11
adrianlizarraga Address PR review: add missing headers and guard RTLD_NOLOAD
2583a176
adrianlizarraga Return std::optional<bool> from IsLibraryLoaded for clarity
e8fbb5ef
adrianlizarraga Address more comments
98f2469f
adrianlizarraga adrianlizarraga requested a review from copilot-pull-request-reviewer copilot-pull-request-reviewer 32 days ago
copilot-pull-request-reviewer
copilot-pull-request-reviewer commented on 2026-05-12
adrianlizarraga Merge branch 'main' into adrianl/EpLoadRefCountLeak_UnitTest
00446154
adrianlizarraga Use temp-copied library in handle leak test for process-state indepen…
404a1874
adrianlizarraga Clean up
fd63fc92
adrianlizarraga adrianlizarraga requested a review from copilot-pull-request-reviewer copilot-pull-request-reviewer 30 days ago
copilot-pull-request-reviewer
copilot-pull-request-reviewer commented on 2026-05-13
adrianlizarraga adrianlizarraga changed the title Add unit test for PR 28396 Add unit test for fix of plugin EP library handle leak in PR 28396 30 days ago
adrianlizarraga adrianlizarraga marked this pull request as ready for review 30 days ago
adrianlizarraga adrianlizarraga requested a review from adrastogi adrastogi 30 days ago
adrianlizarraga adrianlizarraga requested a review from edgchen1 edgchen1 30 days ago
edgchen1
edgchen1 commented on 2026-05-13
adrianlizarraga Merge branch 'main' into adrianl/EpLoadRefCountLeak_UnitTest
37fdf7a3
adrianlizarraga Clean up lib registration with RAII; Use static_cast
798f7742
adrianlizarraga adrianlizarraga requested a review from edgchen1 edgchen1 25 days ago
edgchen1
edgchen1 commented on 2026-05-18
adrianlizarraga Use smaller scope
98c8ab63
edgchen1
edgchen1 commented on 2026-05-18
edgchen1
edgchen1 approved these changes on 2026-05-18
adrianlizarraga adrianlizarraga merged 40e30488 into main 25 days ago
adrianlizarraga adrianlizarraga deleted the adrianl/EpLoadRefCountLeak_UnitTest branch 25 days ago

Login to write a write a comment.

Login via GitHub

Reviewers
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Terms Of Use Privacy Policy Refund Policy Impressum