chroma-core/chroma

[CLN] Break up test_api

Aberta

#2.586 aberto em 28 de jul. de 2024

Ver no GitHub
 (4 comentários) (0 reação) (1 responsável)Rust (2.247 forks)batch import
by-chromacleanupgood first issue

Métricas do repositório

Stars
 (27.958 estrelas)
Métricas de merge de PR
 (Mesclagem média 1d 16h) (80 fundiu PRs em 30d)

Description

test_api.py is a legacy of Chroma's launch, and provided some basic initial test coverage. We've continued to add to it as the project has evolved, and it's become a catch-all kitchen sink test file.

In practice a lot of the surface covered in test_api is better covered by our property tests. test_api.py should be broken up into separate files by API / object, and each object test should cover exceptional cases, rather than normal operation.

For example, we should save entropy in the property tests by not having them generate lists of IDs that are shorter than the lists of documents on a Collection.add call, but we should also be testing this path.

Guia do colaborador

Direção de pesquisa
A questão solicita a divisão de `test api.py` (localizado em `chromadb/test/test api.py`) em arquivos separados por objeto da API (por exemplo, `test collection.py`, `test database.py`). Cada novo arquivo deve focar em casos excepcionais em vez de operação normal. Os testes de propriedade existentes já cobrem os casos normais. Revise a estrutura atual dos testes de propriedade e a API do Chroma para identificar agrupamentos lógicos. Não há PRs vinculados, mas a questão tem um responsável atribuído, então confirme a disponibilidade antes de começar.
Pilha de tecnologia
python
Domain
testing
Tipo Issue
Teste
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Pythonpytest
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75