89luca89 / 89luca89/terrible

[IMPROVEMENT] Improve Asserts

Aberta
#87 3 comentários 0 reações 1 responsável Reivindicada por @89luca89 Ver no GitHub
bug enhancement experimental help wanted
Linguagem predominante
YAML
Estrelas
259
Forks
29
Métricas de merge de PRs
Nenhum PR com merge em 30d

Descrição

Some assertions should be improved, for example:

```yaml
host-vm-1:
disk_source: "/mnt/iso/debian10-terraform.qcow2"
os_family: Debian
cpu: 1
memory: 512
hypervisor: hypervisor_1
ansible_host: 192.168.122.9
network_interfaces:
iface_1:
name: default
type: nat
ip: 192.168.122.9
gw: 192.168.122.1
dns:
- 192.168.122.1
default_route: True <<<--- note this is unindented
```

This snippets passes the assertions even if it's not valid
We should review and evaluate more corner cases to include in the assertions

Also we could think of an external tool/linter as @alegrey91 suggests to validate inventories

Guia de contribuição

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

Direção de pesquisa

Examine a lógica de asserção no playbook que valida a estrutura YAML do inventário, provavelmente em tasks ou em uma role de validação. O exemplo mostra uma chave 'default_route' sem indentação que deveria ser detectada. Revise os arquivos de teste ou módulos de asserção existentes para entender o escopo da validação. Adicionar mais casos extremos requer entender o esquema do inventário e os erros comuns.

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

Avaliação

Domínio
infrastructure, testing, tooling
Tipo de issue
Bug
Dificuldade
3/5
Tempo estimado
1-2 dias
Status de atividade
Estagnada
Clareza
Razoavelmente clara
Facilidade para iniciantes
45/100

Receba novas issues na sua caixa de entrada

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