Skip to content

Conversation

@Mierdin
Copy link
Member

@Mierdin Mierdin commented Feb 16, 2017

This adds a test case for st2 login command and subsequent commands using it's provided authentication configuration.

I have validated that this detects the current st2 login failure, and also that it succeeds when applying the fix in StackStorm/st2#3226. See full output here

Matt Oswalt added 2 commits February 16, 2017 12:13
Signed-off-by: Matt Oswalt <oswaltm@brocade.com>
Signed-off-by: mierdin <oswaltm@brocade.com>
@Mierdin Mierdin changed the title Added st2 login test (first attempt) Add st2 login test Feb 16, 2017
Matt Oswalt added 3 commits February 16, 2017 15:30
Signed-off-by: Matt Oswalt <oswaltm@brocade.com>
Signed-off-by: Matt Oswalt <oswaltm@brocade.com>
Signed-off-by: Matt Oswalt <oswaltm@brocade.com>
@Kami
Copy link
Member

Kami commented Feb 17, 2017

So far LGTM, but I think we also need a set up phase which ensures the environment is clean and we start from a clean consistent start before each run:

  1. Removes ~/.st2/config (set up or test should write config we want to test, we want to ensure there is no default config messing up with the tests)
  2. Remove ~/.st2/token* (to ensure there are no cached tokens on the disk)

@Kami
Copy link
Member

Kami commented Jun 12, 2017

@Mierdin bump, how is this looking, can we merge it?

@Mierdin
Copy link
Member Author

Mierdin commented Oct 24, 2018

Closing this for now, as this is a bit stale. Feel free to pick up where I left off on this branch.

@Mierdin Mierdin closed this Oct 24, 2018
@arm4b arm4b deleted the test-st2-login branch October 25, 2018 10:20
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.

4 participants