-
Notifications
You must be signed in to change notification settings - Fork 404
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 Compass Runtime Agent acceptance tests #5270
Add Compass Runtime Agent acceptance tests #5270
Conversation
/test pre-master-kyma-gke-compass-integration |
c.assertEventAPIs(t, appId, compassEventAPIs, appCR) | ||
} | ||
|
||
func (c *K8sResourceChecker) AssertEventAPIResourcesDeleted(t *testing.T, applicationId, eventAPIId string) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This function is identical to "AssertAPIResourcesDeleted".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed
data: "{{.Data}}", | ||
{{- end}} | ||
{{- if .FetchRequest }} | ||
fetchRequest: {{- FetchRequesstInputToGQL .FetchRequest }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo: FetchRequesstInputToGQL.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
case *graphql.OAuthCredentialData: | ||
return fmt.Sprintf("%s/%s/%s", mock.OAuth, cred.ClientID, cred.ClientSecret) | ||
default: | ||
t.Fatalf("Failed to get path based on authentication: unkonw credentials type") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo: Unkonw
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
case *graphql.OAuthCredentialData: | ||
c.assertK8sOAuthSecret(t, secretName, cred, service) | ||
default: | ||
t.Fatalf("Unkonw credentials type") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo: Unkonw
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
// update no auth API to OAuth | ||
noAuthAPI, found := getAPIByName(application.APIs.Data, "no-auth-api") | ||
require.True(t, found) | ||
updatedInput := applications.NewAPI("basic-to-oauth", "", noAuthAPI.TargetURL).WithAuth(oauth) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"no-auth-to-oauth" instead of "basic-to-oauth"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
require.NoError(t, err) | ||
updatedAPIs = append(updatedAPIs, newBasicAuthAPI) | ||
|
||
// update OAuth API to Basic Auth |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"update BasicAuth to no auth"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
/retest |
@Szymongib: The following test failed, say
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
Description
Changes proposed in this pull request:
Related issue(s)