fix: tests e2e

[wisley7l]

No description provided.

[leomp12]

O que comentei com você no Slack, até tem um lugar aqui setando env

cloud-commerce/.github/workflows/test-e2e.yml

Lines 81 to 92 in f88629c

	- shell: bash
	env:
	ECOM_STORE_ID: ${{ secrets.ECOM_STORE_ID }}
	ECOM_AUTHENTICATION_ID: ${{ secrets.ECOM_AUTHENTICATION_ID }}
	ECOM_API_KEY: ${{ secrets.ECOM_API_KEY }}
	MERCADOPAGO_TOKEN: ${{ secrets.MERCADOPAGO_TOKEN }}
	FRENET_TOKEN: ${{ secrets.FRENET_TOKEN }}
	GALAXPAY_ID: ${{ secrets.GALAXPAY_ID }}
	GALAXPAY_HASH: ${{ secrets.GALAXPAY_HASH }}
	INFINITEPAY_ID: ${{ secrets.INFINITEPAY_ID }}
	INFINITEPAY_SECRET: ${{ secrets.INFINITEPAY_SECRET }}
	run: pnpm build

mas no build (acho que nem precisava de envs dos apps aí inclusive).

Me parece válido criar um build só pros apps inclusive (talvez só sem packages/storefront e stores) e usar aqui, ex:

cloud-commerce/package.json

Line 19 in f88629c

"test:apps": "turbo run test --filter='./packages/{apps/**,modules}'",

No teste em si não tá setando env vars:

cloud-commerce/.github/workflows/test-e2e.yml

Lines 125 to 129 in f88629c

	- name: Run tests
	shell: bash
	run: |
	sleep 20
	pnpm test:e2e

Então vai emular as functions sem as variáveis de ambiente. Adicionalmente, também não sei se setar elas em env da sessão do bash (como você fez pra build) seria suficiente, o que sabemos é que firebase-tools lê .env, acho que não pega todas as variáveis da sessão não por isso crio o .env aqui por exemplo

cloud-commerce/action.yml

Line 284 in ac4d67d

" > functions/.env

[leomp12]

Isso aqui não é bom não...
Precisa mesmo?

[wisley7l]

Não precisa não, coloquei so pra verificar umas coisas. Mas já vou voltar.

[leomp12]

try catch em teste ? 🤔

[leomp12]

O que comentei na outra PR é que o scope do commit é test, btw não é o subject 😄
btw é "by the way", o que eu disse foi tipo "a propósito, escopo de commit sobre testes é test", tipo test: Adding first apps e2e tests

[leomp12]

Isso daqui não deve ser necessário, se estiver sendo necessário outro comando (do @cloudcommerce/cli) deve rolar antes do teste

[leomp12]

cloud-commerce/packages/cli/src/build.ts

Line 16 in aad3bf4

const filesToCopy = ['.env', 'config.json', 'ssr/content/settings.json'];

[leomp12]

cloud-commerce/packages/cli/src/cli.ts

Lines 97 to 99 in aad3bf4

	if (argv._.includes('deploy')) {
	return $firebase('deploy');
	}

[leomp12]

 if (argv._.includes('prepare')) { 
   // prepareCodebase
   // return
 }

[leomp12]

cloudcommerce prepare && test...

[wisley7l]

@leomp12 estava observando o pipeline do workflow dos teste, os secrets estão mesmo setados?
Pq nos steps que busca os secrets, quando setados eles ficam com ** mas não está aparecendo assim. Digo isso pq fiz outros teste em um fork do cloudcommerce e lá aparece assim.

[leomp12]

Suggested change

	await prepareCodebases(true);
	return $`echo 'prepare'`;
	return prepareCodebases(false);

Tem nada de echo 😄
Esse argumento booleano aí é o isDev, ele faz preparar o codebase SSR pra dev server, acho que não precisa aqui (e faria o setup dos testes demorar mais)

[leomp12]

@leomp12 estava observando o pipeline do workflow dos teste, os secrets estão mesmo setados?

Tão não, só no https://github.com/ecomplus/store 🤡

[leomp12]

Vou colocar os ECOM_* , mas esses específicos de apps aí ainda não temos...

[leomp12]

mas esses específicos de apps aí ainda não temos

Vamos convencionar um console.warn (== um alerta em stdout em bash) quando não houver a variável de ambiente do app sendo testado, sem quebrar a execução (exit 1) ou falhar o teste.

[wisley7l]

Acabei adicionando o teste para FRENET aqui também, depois que me toquei que deveria ser outra PR.

[leomp12]

valeu @wisley7l 👍🏾