microsoft / microsoft/rag-experiment-accelerator

Using Classes in testing.

Open
#280 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

question refine
Dominant language
Python
Stars
311
Forks
111
PR merge metrics
No merged PRs in 30d

Description

There needs to be a consensus on whether we use classes while testing or not.

Currently, there are some tests that are in classes and some that are not. For uniformity should all tests be in Classes?
What advantages are gained from Classes vs not?

found some thoughts on it here:
https://stackoverflow.com/questions/50016862/grouping-tests-in-pytest-classes-vs-plain-functions

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Review the repository's existing tests to inventory which use classes and which use plain functions. Read the linked pytest discussion, then compare the advantages and tradeoffs for this project. Done means reaching and recording a clear decision on a uniform testing convention.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
testing-qa
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.