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

lagu: Add end to end tests #430

Merged
merged 92 commits into from
Jul 31, 2018
Merged

lagu: Add end to end tests #430

merged 92 commits into from
Jul 31, 2018

Conversation

karlhorky
Copy link
Contributor

@karlhorky karlhorky commented Jun 4, 2018

TODO

  • Upgrade to latest Jest
  • Review other CLI tools to develop testing strategy
  • Adapt file structure and executable running to more standard approach
  • Choose executable spawning strategy
  • lagu - failing test (not enough options)
  • Write more config values from lagu init (api, ssh)
  • Remove reliance on global token for login, logout and other commands (option for token file path?)
  • Make yarn execute work again without building first
  • Change cwd to fixtures directory
  • Figure out a way to deal with changing created times in snapshots
  • Add VS Code extension settings and recommendations
  • Update CLI GraphQL schema
  • General
    • lagu init
    • lagu login
      • Configure local token path
      • Test token
    • lagu logout
  • Queries
    • lagu customer
    • lagu environment - add data to GraphQL fixtures
    • lagu project
    • lagu projects

Bonus tasks:

@karlhorky karlhorky added blocked and removed blocked labels Jun 4, 2018
@karlhorky karlhorky changed the title lagu: Add end to end tests [WIP] lagu: Add end to end tests Jun 4, 2018
@karlhorky karlhorky requested a review from Schnitzel July 10, 2018 14:50
@karlhorky karlhorky changed the title [WIP] lagu: Add end to end tests lagu: Add end to end tests Jul 11, 2018
Copy link
Contributor

@Schnitzel Schnitzel left a comment

Choose a reason for hiding this comment

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

looking good @karlhorky, thanks a lot.

Schnitzel added a commit that referenced this pull request Jul 31, 2018
@Schnitzel Schnitzel merged commit 5b47dea into master Jul 31, 2018
@karlhorky karlhorky deleted the add-e2e-tests-for-cli branch August 1, 2018 07:44
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.

2 participants