Codeception / Codeception/module-db

Fix a confusion between 'cleanup' and 'skip_cleanup_if_failed' configuration params

Aberta
#67 0 comentários 0 reações 0 responsáveis Ver no GitHub
Linguagem predominante
PHP
Estrelas
23
Forks
31
Métricas de merge de PRs
Nenhum PR com merge em 30d

Descrição

#### Problem
Currently we have configuration params `cleanup` and `skip_cleanup_if_failed`.
Despite their names sound similar, they are responsible for completely different things: `cleanup` controls the database re-creation before each test, and `skip_cleanup_if_failed` controls whether records, added by `haveInDatabase()` should be erased after the test.
#### Suggested fix
It would be good to distinguish these two 'cleanups' in documentation and in the names of config params in next major versions.
For example, to rename `cleanup` to `repopulate` (since we have `populate` responsible for initial database creation).

Guia de contribuição

Nenhum guia de contribuição indexado para este repositório

Direção de pesquisa

A issue cita os parâmetros de configuração `cleanup` e `skip_cleanup_if_failed`, mas não identifica arquivos nem testes. Comece localizando as definições e a documentação deles; em seguida, determine como os dois comportamentos são descritos e onde uma futura renomeação afetaria os usuários. O trabalho estará concluído quando as duas responsabilidades de cleanup estiverem claramente diferenciadas e a mudança de nome proposta estiver resolvida para uma versão principal.

Escrita pelo modelo de indexação a partir do texto da issue.

Avaliação

Stack de tecnologia
php
Domínio
database, documentation
Tipo de issue
Documentação
Dificuldade
4/5
Tempo estimado
3-5 dias
Status de atividade
Estagnada
Clareza
Razoavelmente clara
Facilidade para iniciantes
35/100

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.