-
Notifications
You must be signed in to change notification settings - Fork 57
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
feat: Oracle instrumentation update to support latest version #2721
Conversation
… initial Console MFA tests for Oracle
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2721 +/- ##
=======================================
Coverage 80.91% 80.92%
=======================================
Files 458 458
Lines 29147 29147
Branches 3209 3209
=======================================
+ Hits 23585 23587 +2
+ Misses 4771 4770 -1
+ Partials 791 790 -1
Flags with carried forward coverage won't be shown. Click here to find out more. |
Nice work! I think we need to add Oracle back to the integration tests workflow, right? |
No, that was done earlier this week via #2715 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have a style question but otherwise this looks good
With this PR, the .NET agent now instruments the latest
Oracle.ManagedDataAccess
andOracle.ManagedDataAccess.Core
libraries.Oracle unbounded integration tests were migrated to use the ConsoleMultiFunctionApplication pattern and updated to exercise a range of library versions for both .NET Framework and .NET 6+. Note that the existing
EnterpriseLibraryOracleTests
were not migrated.