microsoft / semantic-kernel

Integrate cutting-edge LLM technology quickly and easily into your apps
https://aka.ms/semantic-kernel
MIT License
21.48k stars 3.16k forks source link

Python: double check if unit tests cover all creation paths #6700

Open eavanvalkenburg opened 3 months ago

eavanvalkenburg commented 3 months ago

Describe the bug A clear and concise description of what the bug is. We had a regression in the settings of the AI search connector, this want caught with unit tests, hence we want to make sure that we do have that for all ways of setting up. Will partially be solved when we further expand test coverage to include all memory services.

To Reproduce Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

Expected behavior A clear and concise description of what you expected to happen.

Screenshots If applicable, add screenshots to help explain your problem.

Platform

Additional context Add any other context about the problem here.

github-actions[bot] commented 1 week ago

This issue is stale because it has been open for 90 days with no activity.