fabiolamfleury commented 4 years ago

Descrição

Criação de rotas, views e serializer para criar, editar, apagar e listar árvores.

Resolve (Issues)

67 Cadastro de Árvore

PRs relacionados, se houver

branch	PR
Cadastro de Árvore (front)	link

Tarefas gerais realizadas

Criação da Model, View e Serializers para a Árvore e Mês de Colheita
Criação de testes

durvalcarvalho commented 4 years ago

Eu dei uma pesquisada rápida sobre esse error que está rolando no pipeline... Eu descofio que seja isso aqui link

shayanealcantara commented 4 years ago

Tentei reproduzir aqui puxando uma branch dessa feature/67 de vocês. Primeiro troquei a versão do postgres pra latest (pelo que li, as versões mais atuais conseguem adaptar os buffers necessários de acordo com a demanda de memória, como sugere o erro 137). Não gerou diferença na CI, como podem ver no PR de teste que criei #170. não fiz um fork pra isso porque né, não mais forks

Testei a dev, puxando uma branch de lá também para ver se a integração contínua passava com essa mudança na imagem do postgres. A dev está okay. log do git action

Então exclui todas as migrações de todos os apps da aplicação e buildei pra gerar de novo para começar do zero. Não recebi nenhum erro por isso. Ao rodar os testes recebi esse feedback: Vocês chegaram a ter esse problema? Esse erro é no app de users, no teste test_unique_email_validation. Se puderem tentar reproduzir pra ver se receberam também.. Sei que pode acontecer o efeito cascata em algumas situações de testes, mas não passei por isso ainda, então vou ler mais.

atualização: o teste que falei aqui em cima não interferiu na execução dos testes no CI :(

fabiolamfleury commented 4 years ago

@shayanealcantara esse erro ai é problema realmente dos testes em relação ao setup/teardown que não estão sendo usados corretamente, mas ai como você diz não interfere no ci, pq é depois que faz a conexão com o banco, já na execução dos testes mesmo... Eu só queria que o github actions desse um log mais completo pra gente tentar entender melhor.

codecov-io commented 4 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@ef2437b). Click here to learn what that means. The diff coverage is 100%.

@@            Coverage Diff             @@
##             develop     #164   +/-   ##
==========================================
  Coverage           ?   94.15%           
==========================================
  Files              ?       36           
  Lines              ?      804           
  Branches           ?        0           
==========================================
  Hits               ?      757           
  Misses             ?       47           
  Partials           ?        0

Flag	Coverage Δ
#unittests	`94.15% <100%> (?)`

Impacted Files	Coverage Δ
src/harvest/migrations/0001_initial.py	`100% <ø> (ø)`
src/property/migrations/0001_initial.py	`100% <ø> (ø)`
src/acacia/urls.py	`100% <ø> (ø)`
src/property/models.py	`100% <ø> (ø)`
src/users/migrations/0001_initial.py	`100% <ø> (ø)`
src/tree/urls.py	`100% <100%> (ø)`
src/tree/admin.py	`100% <100%> (ø)`
src/property/urls.py	`100% <100%> (ø)`
src/tree/migrations/0001_initial.py	`100% <100%> (ø)`
src/tree/viewsets.py	`100% <100%> (ø)`
... and 9 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update ef2437b...e76401b. Read the comment docs.

fga-eps-mds / 2019.2-Acacia