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 CLI to regression testing #1048

Open
mihaiblaga89 opened this issue Jun 26, 2023 · 4 comments
Open

Add CLI to regression testing #1048

mihaiblaga89 opened this issue Jun 26, 2023 · 4 comments

Comments

@mihaiblaga89
Copy link
Contributor

mihaiblaga89 commented Jun 26, 2023

All renative CLI commands:

  • crypto decrypt
  • crypto encrypt
  • platform eject
  • platform connect
  • platform list
  • platform configure
  • platform setup
  • template add
  • template apply
  • template list
  • plugin add
  • plugin list
  • plugin update
  • workspace list
  • workspace add
  • workspace connect
  • workspace update
  • hooks list
  • hooks run
  • hooks pipes
  • clean
  • fastlane
  • publish
  • pkg
  • status
  • config
  • help
  • new
  • install
  • project configure
  • project upgrade
  • app configure
  • app create
  • workspace configure
  • configureSoft
  • kill
  • doctor
  • target list
  • target launch
  • link
  • unlink

Proposal:

  1. Aim to have a smoke test for all of them
  2. Start with regression tests for the most common ones:
  • crypto encrypt
  • crypto decrypt
  • new
  • template apply
  • hooks run
  • project configure
  • target list
@mihaiblaga89 mihaiblaga89 added this to the 1.0 milestone Jun 26, 2023
@mihaiblaga89
Copy link
Contributor Author

Current process

  • install latest version of rnv
  • rnv new
  • select matching template version and create project with default options
  • rnv run -p android/ios/web/macos/firetv/androidtv/tvos, optimistically on sims or devices otherwise just on one or the other

@mihaiblaga89
Copy link
Contributor Author

  • QA will start manually testing the following steps
    crypto encrypt
    crypto decrypt
    new
    hooks run
    target list - basic testing, no in-depth "why is this tablet emulator marked as a tv" testing for now
  • This week we'll plan a CLI automation session where we go through one CLI command testing example
  • @pauliusguzas will look if there are any interesting CLI automation testing libs, otherwise Jest

@pavjacko
Copy link
Member

@mihaiblaga89 @pauliusguzas another important thing to test is variant between local repo (linked) and rnv installed globally directly from npm

so the scope of testing would include:

  • rnv (via npm i -g) HIGHER PRIO as this is what MOST users would use
  • rnv (linked)
  • rnv (via project npx)
  • app-harness (to be created)
  • template-starter (via rnv new)

@GabrieleKaceviciute
Copy link
Collaborator

blocked by #1098

@mihaiblaga89 mihaiblaga89 modified the milestones: 0.37, 1.0 Oct 4, 2023
@pavjacko pavjacko removed this from the 1.0 milestone Feb 13, 2024
@GabrieleKaceviciute GabrieleKaceviciute removed their assignment Oct 1, 2024
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

No branches or pull requests

4 participants