Skip to content
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

Make it easy to run functional tests with multiple combinations of configurations. #801

Closed
3 tasks
superhindupur opened this issue May 2, 2024 · 1 comment
Labels
enhancement New feature or request subtask A subtask

Comments

@superhindupur
Copy link
Collaborator

Motivation

Currently, the functional tests spin up a new instance of the mock frontend API service for each combination of app configuration.

For e.g., we have one group of tests running for SHOULD_USE_DATA= true and another for false.

With the addition of functional tests for Semantic Kernel in #800, it became apparent that there are multiple combinations of app configs that we want to test, and the current method of spinning up a new mock server instance will not scale. Making it easy to write functional tests for multiple combinations of configs will encourage developers to write more of them, ultimately increasing the stability of CWYD.

One way of solving this problem would be to:

  • Allow declaring app config per test file rather than per test folder.
  • Allow reloading app config while the server is running, removing the need for restarting a server when there is a new config combination.

How would you feel if this feature request was implemented?

everything

Requirements

A list of requirements to consider this feature delivered

  • The ability to run functional tests with multiple app config combinations without needing to run each combination on a new mock server instance.

Tasks

To be filled in by the engineer picking up the issue

  • Task 1
  • Task 2
  • ...
@superhindupur superhindupur added the enhancement New feature or request label May 2, 2024
@reechamishra reechamishra mentioned this issue May 8, 2024
4 tasks
@ross-p-smith ross-p-smith added the subtask A subtask label May 15, 2024
@ross-p-smith
Copy link
Collaborator

Closing this as this is a nice to have, but not been prioritised

@ross-p-smith ross-p-smith closed this as not planned Won't fix, can't repro, duplicate, stale May 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request subtask A subtask
Projects
None yet
Development

No branches or pull requests

2 participants