uspdev / copaco

COntrole de PArque COmputacional
8 stars 14 forks source link

Agora a persistência é feita num método só. #330

Closed wgnann closed 5 years ago

wgnann commented 5 years ago

Aproveitei para corrigir um bug com encoding que apareceu nos testes. Não há testes automatizados nesse commit, contudo, o comportamento foi verificado compatível nos seguintes casos:

1) Atualização do equipamento não reclama do MAC Address já estar cadastrado. Foi um merge das regras do store e do update.

2) Cadastros (fixo, ip, rede)

3) O restante é essencialmente um cut & paste do original.

wgnann commented 5 years ago

Eu aprovaria com uma pá de sal, pois desisti de fazer testes automatizados para verificar se o funcionamento do controller se manteve. No fim das contas, acabei correndo atrás de fazer os testes automatizados no braço. (o ideal seria ver um jeito de usar um banco de dados diferente para rodar os testes automatizados e verificar se esse tipo de controlador está fazendo direito seu trabalho a partir das entradas no banco, acho que o django fazia isso usando o sqlite)

thiagogomesverissimo commented 5 years ago

Mas ficou bom. Vc até organizou o código colocando as linhas em ordem que aparecem no form.