Skip to content

Create tests for SemanticKernelPool #23

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
Open
SubZeroX9 wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Create tests for SemanticKernelPool #23

SubZeroX9 wants to merge 12 commits into from

Conversation

@SubZeroX9
Copy link
Collaborator

@SubZeroX9 SubZeroX9 commented Jan 5, 2025

PR Summary

  • Created Tests to get Kernel By Provider Type
  • Created Tests to get Kernel By Name
  • Created Tests to Get Kernel By Scope
  • Created Test for Initialization of Kernels pre and post
  • fixed Get kernel By provider throwing Linq error instead of our own
  • fixed when registering to pre and post actions by scope we pass scopes correctly

@SubZeroX9
Added Mock Kernel pool and configuration and Mock Http Handler for mo…
9d2e395 
…ck responses
@SubZeroX9
Added Test project and created KernelPoolTestContext for the Fixture
5e26bec
@SubZeroX9
Updated Mock Http Send with Logs
f81c492
@SubZeroX9
added appsettings and TestOutputHelpers with XUnit logger and logger …
72aed6d 
…provider
@SubZeroX9
renamed context to fixture and update Fixture and created 3 simple Tests
f0162af
@SubZeroX9
updated GetKernelByName linq method to FirstOrDefault instead of Firs…
2d70880 
…t so we throw our own exception instead of linq's
@SubZeroX9
removed old tes class sepreated test classes By Get, added tests to G…
57d3005 
…etByNameTests class
@SubZeroX9
Added Tests for GetByService
63a9b2b 
- added unique name prop in wrapper
- added 1 more configs for each provider type
- added a second test-scope
@SubZeroX9
Added tests to GetByScope
0b1ce99
@SubZeroX9
fix scopes not passed for pre and after register
41df407
@SubZeroX9
Added tests for initializations pre and post registration
28dca69
@SubZeroX9
removed mock http handler, removed unnecessery method and property
0bfd318
@SubZeroX9 SubZeroX9 self-assigned this Jan 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alonf alonf Awaiting requested review from alonf

@VenyaBrodetskiy VenyaBrodetskiy Awaiting requested review from VenyaBrodetskiy

Assignees

@SubZeroX9 SubZeroX9

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@SubZeroX9