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

Add tests selection menu. #59

Merged
merged 9 commits into from
Jan 16, 2025
Merged

Add tests selection menu. #59

merged 9 commits into from
Jan 16, 2025

Conversation

agrojean-ledger
Copy link
Contributor

@agrojean-ledger agrojean-ledger commented Nov 22, 2024

  • Add tests selection menu
  • Update packages :
    • Update ESLint to latest version :
      • Use flat config file (js syntax),
      • Update precommit hooks,
      • Indent/format/lint ts/js files following latest ESLint rules.
      • Rewrite index.ts test file to match latest glob package syntax
  • Optimize task activation :
    • use only one forEach loop in checkDisabledTasks
    • Allow to regenerate a subset of tasks with regenerateSubset

Tested on the following platforms :

  • Linux
  • Windows
  • macOS

Tested on the following apps:

  • app-boilerplate
  • app-zilliqa
  • app-bitcoin-new
  • app-radix-babylon

@agrojean-ledger agrojean-ledger force-pushed the add-tests-selection branch 3 times, most recently from 28a52cc to 98d2ef6 Compare November 22, 2024 11:07
@cedelavergne-ledger
Copy link
Contributor

Missing the information in the UI that a test selection has been performed, especially because they are stored in the settings.
Should be interesting to modify the UI entry with something like Run tests [Selected only]

@agrojean-ledger
Copy link
Contributor Author

agrojean-ledger commented Nov 28, 2024

Missing the information in the UI that a test selection has been performed, especially because they are stored in the settings. Should be interesting to modify the UI entry with something like Run tests [Selected only]

I've added a visual indicator in the "Functional tests" tree view item.

TestsListRefresh

@agrojean-ledger agrojean-ledger merged commit db40f5c into main Jan 16, 2025
10 checks passed
@agrojean-ledger agrojean-ledger deleted the add-tests-selection branch January 16, 2025 12:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants