deploy icon indicating copy to clipboard operation
deploy copied to clipboard

Inspect validation

Open diaabadr opened this issue 2 years ago • 4 comments

@viferga I have added the new updates on the master branch with the ensureDeploymentsExist

diaabadr avatar Apr 27 '23 08:04 diaabadr

I don't know why but many tests are failing, you should review them, otherwise we cannot merge this. I'm setting this PR as a draft until it passes the tests.

viferga avatar Apr 28 '23 15:04 viferga

@viferga I think they are failing because of the secrets. workflows on a pull requests doesn't have access to the secrets. when i run it on my forked repo it passes the tests that doesn't require plans such as login, inspect, etc. my account doesn't have plans to test deploy, delete, etc. that's the reason they are failing here

image

It sees my secrets in my forked repo

image

but it doesn't see base repo secrets in the pr workflow

image

I think that's the problem. what do you see?

diaabadr avatar Apr 29 '23 04:04 diaabadr

I have added extra validation at the login with email and password flags. if the email and password are empty. the CLI asks for login with the auth choices (token or email). it doesn't consider them as invalid credentials so it doesn't throw an error to catch it and fail the login test. so i added a check on the info messages if it includes login successful or not.

diaabadr avatar Apr 29 '23 21:04 diaabadr

@diaabadr I re-run the tests by doing a small change in the code, and they failed. You can review now and solve the issues if you want, meanwhile I am going to leave this PR as a draft.

viferga avatar Aug 27 '24 21:08 viferga