Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build: Improve e2e script #14980

Merged
merged 7 commits into from
May 20, 2021
Merged

Build: Improve e2e script #14980

merged 7 commits into from
May 20, 2021

Conversation

gaetanmaisse
Copy link
Member

@gaetanmaisse gaetanmaisse commented May 18, 2021

What I did

  • Improve CLI with prompts
  • Clean some duplicated code (use function from SB CLI directly)
  • Reduce log and improve e2e output with a sum-up of the passing/failing tests

How to test

  • Locally: yarn local-registry --port 6000 --open --publish and yarn test:e2e-framework --clean --use-local-sb-cli cra
  • On CI: the e2e steps should be way more understandable

@nx-cloud
Copy link

nx-cloud bot commented May 18, 2021

Nx Cloud Report

CI ran the following commands for commit ee559b0. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch

Status Command
#000000 nx run-many --target=prepare --all --parallel --max-parallel=15

Sent with 💌 from NxCloud.

@gaetanmaisse gaetanmaisse added the maintenance User-facing maintenance tasks label May 18, 2021
@gaetanmaisse gaetanmaisse marked this pull request as ready for review May 19, 2021 18:19
@gaetanmaisse gaetanmaisse requested review from yannbf and a team May 19, 2021 18:20
logger.info(`🧮 E2E Results`);

Object.entries(e2eResult).forEach(([configName, result]) => {
logger.info(`${configName}: ${result ? 'OK' : 'KO'}`);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is nice! But KO reads very similar to OK, maybe another word would be better?

Copy link
Member

@yannbf yannbf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@shilman shilman merged commit 22803ef into next May 20, 2021
@shilman shilman deleted the feat/improved-e2e-scripts branch May 20, 2021 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance User-facing maintenance tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants