Adiciona feriado nacional do dia da consciência negra a partir de 2024 #567
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
@joaovitors1g , você tem o link de onde isso saiu no diário oficial? |
|
Claro, Diário Oficial |
Obrigado pelo feedback! Adicionei os testes. |
| expect(data).toHaveLength(12); | ||
| expect(data).toEqual( | ||
| expect.not.arrayContaining( | ||
| getHolidays(2024, ['Dia da consciência negra']) |
There was a problem hiding this comment.
Aqui não deveria ser 2023?
There was a problem hiding this comment.
Como estou verificando se o array não contém, uso o getHolidays pra buscar os feriados de 2024 e filtro somente o "Dia da consciência negra", não consegui pensar em outra maneira de validar.
There was a problem hiding this comment.
Pelo que entendi a lógica está correta, mas em 2024 o dia deveria estar presente, não?
Ou seja, em 2023 não teria o dia em questão e 2024 teria, me diga se eu estiver comendo bola... rs
There was a problem hiding this comment.
Pra validar eu fiz 2 testes, nesse caso de teste em específico estou buscando os feriados de 2023 na API e depois verifico se não contém o feriado "Dia da consciência negra", tem outro teste que verifica para esse caso de 2024 que você mencionou, onde eu busco os feriados de 2024 na API, filtro o "Dia da consciência negra" usando o getHolidays e uso o expect.arrayContaining para validar.
|
Poxa, eh uma pena que o PR ainda nao esteja aprovado, o codigo parece correto, estava precisando aplicar no meu projeto e notei que faltava esse feriado |
|
|
@joaovitors1g , muito obrigado pela contribuição!! @yuriaps , já está em produção, obrigado pelo remember |
Descrição
A partir de 2024 o dia da consciência negra será considerado feriado nacional, pensando nisso fiz a inclusão do feriado a partir de 2024 no endpoint holidays.