Remover o script "test:ci" do package.json e mover para dentro da Action

[filipedeschamps]

Descrição

Como mantenedor da infraestrutura de testes, eu gostaria de tirar do package.json scripts que não agregam em nada, como o test:ci e repassar o comando do jest de dentro desse script diretamente para a Action/Workflow tests.yml.

Motivo

Me parece uma abstração completamente desnecessária ter um comando que é usado exclusivamente pela Action, que no final das contas roda apenas jest --runInBand. Sugiro economizar essa linha dentro do package.json e não ter que fazer a pessoa que está lendo a Action ter que voltar para o package.json para entender o que o comando faz.

[geovani-brito]

Essa issue é simples de resolver, certo?

Eu tentei mas não sou autorizado a alterar um workflow do Actions. Acho que só você vai poder resolver essa.

refusing to allow a Personal Access Token to create or update workflow .github/workflows/tests.yml without workflow scope

[filipedeschamps]

@geovani-brito é bem simples, correto!

E estranho, eu não entendi exatamente onde está recebendo esse erro. Você está usando um token para alterar o workflow?

Sugestão: faça um clone desse repositório aqui (e não um fork), altere o package.json e a Action, e envie um PR aqui para esse repositório. Ele deveria rodar a Action com as novas condições já 🤝 👍

[geovani-brito]

Sugestão: faça um clone desse repositório aqui (e não um fork), altere o package.json e a Action, e envie um PR aqui para esse repositório. Ele deveria rodar a Action com as novas condições já

Eu já tava trabalhando assim. Eu nunca fiz um fork desse repositório.
Então eu pensei o seguinte: já que eu já tava trabalhando com um clone vamos tentar com um fork.

Bem... fiz com o fork e recusou. Pensei que pudesse ser por alguma razão com a branch que criei. Então mesclei a branch na main e tentei dar o push. Mesmo erro.

Voltei para o clone e fiz o mesmo procedimento. Mesclei a branch do trabalho na main. Como a main aqui é protegida imagino que dando um push vai ser solicitado um pull request. Tentei. Mesmo erro.

A mensagem muda um pouco caso eu tenha feito com o fork ou com a branch ou com o clone direto... seja como for, aquela mensagem refusing to allow a Personal Access Token to create or update workflow '.github/workflows/tests.yml' without 'workflow' scope sempre aparece.

Counting objects: 9, done. Delta compression using up to 4 threads. Compressing objects: 100% (8/8), done. Writing objects: 100% (9/9), 943 bytes | 943.00 KiB/s, done. Total 9 (delta 6), reused 0 (delta 0) remote: Resolving deltas: 100% (6/6), completed with 4 local objects. To https://github.com/filipedeschamps/tabnews.com.br.git ! [remote rejected] main -> main (refusing to allow a Personal Access Token to create or update workflow '.github/workflows/tests.yml' without 'workflow' scope) error: failed to push some refs to 'https://geovani-brito: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@github.com/filipedeschamps/tabnews.com.br.git'

Depois de falhar tudo, fui por outra estratégia. Fiz as alterações aqui mesmo no Github. Pelo menos aqui eu consegui criar a branch o que possibilitou rodar os testes. Só que dessa vez o que aconteceu é que ele não encontrou o Jest como mostra a mensagem de saída do CI:

Run npm start & jest --runInBand npm start & jest --runInBand shell: /usr/bin/bash -e {0} /home/runner/work/_temp/7a9f98f8-7b70-41da-89a8-6f3c4568efde.sh: line 1: jest: command not found

Eu não sei porque isso acontece mas já tive esse problema. Eu só conseguia rodar Jest usando o npm run. Escrever o comando jest diretamente no terminal dava "comando não encontrado". Se alguém me explicar porque isso acontece eu prometo guardar em uma gaveta especial no meu cérebro. 😆

[filipedeschamps]

Sensacional @geovani-brito acho que entendi o que aconteceu:

Voltei para o clone e fiz o mesmo procedimento. Mesclei a branch do trabalho na main. Como a main aqui é protegida imagino que dando um push vai ser solicitado um pull request. Tentei. Mesmo erro.

Na verdade não, você não deve mesclar sua branch de trabalho na main. O que você deve empurrar para o origin é a sua branch de trabalho, assim como fez com a branch issue_94.

Eu não sei porque isso acontece mas já tive esse problema. Eu só conseguia rodar Jest usando o npm run. Escrever o comando jest diretamente no terminal dava "comando não encontrado". Se alguém me explicar porque isso acontece eu prometo guardar em uma gaveta especial no meu cérebro. 😆

Perfeito, faz sentido dar esse erro, pois de fato o jest não é instalado globalmente e não vai estar disponível diretamente na linha de comando. E ao rodar com o npm, pelo fato dele saber resolver o local da dependência dentro do node_modules ele consegue rodar sem problemas.

Então para não precisar instalar globalmente o pacote, e nem resolver na unha onde está o binário do jest, vamos usar o npx, porque ele vai saber resolver isso para nós.

Lá na Action deverá ficar algo como:

- run: npm start & npx jest --runInBand

[geovani-brito]

Na verdade não, você não deve mesclar sua branch de trabalho na main. O que você deve empurrar para o origin é a sua branch de trabalho, assim como fez com a branch issue_94.

Não se preocupe, esse sempre foi o meu procedimento padrão. Qualquer alteração que faço em um projeto eu crio uma branch pra isso. Eu tentei essa lambança aí porque eu não tava conseguindo nem mesmo enviar a branch a partir do meu computador e porque imaginei que o Github não deixaria fazer esse merge e forçaria a criação de uma branch como ele faz em algumas situações colocando o nome do usuário seguido da palavra "patch".

Então para não precisar instalar globalmente o pacote, e nem resolver na unha onde está o binário do jest, vamos usar o npx, porque ele vai saber resolver isso para nós.

A maravilha é essa mesmo. Fiz a alteração e os testes passaram. Só falta você verificar lá se tá tudo certinho. Eu não sabia que o npx resolveria isso. Mais um aprendizado pra guardar em uma gavetinha especial do cérebro. Valeu!!

[filipedeschamps]

Sensacionaaaal meu caro!!! Feito o merge! 🤝 👍