cuducos / minha-receita

🏢 Sua API web para consulta de informações do CNPJ da Receita Federal
https://minhareceita.org
MIT License
1.29k stars 129 forks source link

Altera comando `sample` para adicionar arquivo `updated_at.txt` na pasta #159

Closed eu-ovictor closed 1 year ago

eu-ovictor commented 1 year ago

De início queria que a alteração fosse um simples if no método Sample, mas percebi que poderia extrair os comportamentos de criação de amostra da execução em si. Dessa forma conseguimos incluir ou remover mais facilmente arquivo(s) da amostra.

Fixes #152

eu-ovictor commented 1 year ago

Como a idéia é manter os valores hardcoded para simplificar o uso do sample.Sample por outros pacotes, realmente não faz sentido manter o sampler.Sampler. Minha proposta é dentro do sample.Sample incluir o path do arquivo updated_at.txt no append que também inclui o arquivo do Tesouro Nacional, e adicionar um case no makeSample para o novo arquivo.

Mas tenho uma dúvida sobre isso. No makeSample atual, ele faz o match para a função que vai ser executada para o arquivo com base na extensão (.zip, .csv), como temos um arquivo específico, vejo duas opções:

cuducos commented 1 year ago

Minha proposta é dentro do sample.Sample incluir o path do arquivo updated_at.txt no append que também inclui o arquivo do Tesouro Nacional, e adicionar um case no makeSample para o novo arquivo.

Boa.

No makeSample atual, ele faz o match para a função que vai ser executada para o arquivo com base na extensão (.zip, .csv), como temos um arquivo específico, vejo duas opções:

Prefiro a segunda. Antes do swicth a gente pode perguntar src == updatedAt e chamar uma outra função que faz a mágica: se existir, copia; se não existir, ou cria uma data aleatória, ou usa o valor passado como parâmetro : )

cuducos commented 1 year ago

Muito obrigado 💜