-
Notifications
You must be signed in to change notification settings - Fork 693
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Test/exp run: Handler and service tests for experiment run package (#…
…4126) * feat: seperate exp_run and exp grapql schemas Signed-off-by: SohamRatnaparkhi <[email protected]> * feat: seperate exp_run and exp graphql resolvers Signed-off-by: SohamRatnaparkhi <[email protected]> * feat: add generated types Signed-off-by: SohamRatnaparkhi <[email protected]> * feat: add types for exp_run Signed-off-by: SohamRatnaparkhi <[email protected]> * feat: add exp_run services Signed-off-by: SohamRatnaparkhi <[email protected]> * feat: add exp_run handler Signed-off-by: SohamRatnaparkhi <[email protected]> * refractor: remove functions in experiments service related to runs Signed-off-by: SohamRatnaparkhi <[email protected]> * refractor: remove functions in experiments handler related to runs Signed-off-by: SohamRatnaparkhi <[email protected]> * fix: issues due to handler and resolver types Signed-off-by: SohamRatnaparkhi <[email protected]> * fix: breaking changes in generated and service files Signed-off-by: SohamRatnaparkhi <[email protected]> * fix: bugs in experiment handlers and roles Signed-off-by: SohamRatnaparkhi <[email protected]> * fix: add exp run operator and update handler Signed-off-by: SohamRatnaparkhi <[email protected]> * fix: update infra opr to return exp-run Signed-off-by: SohamRatnaparkhi <[email protected]> * feat: add gitops mocks Signed-off-by: SohamRatnaparkhi <[email protected]> * feat: add mongo operations Signed-off-by: SohamRatnaparkhi <[email protected]> * refractor: schemas for new types Signed-off-by: SohamRatnaparkhi <[email protected]> * feat: mocks service for exp-run Signed-off-by: SohamRatnaparkhi <[email protected]> * feat: handler tests for exp-run Signed-off-by: SohamRatnaparkhi <[email protected]> * fix: package update to match latest changes Signed-off-by: SohamRatnaparkhi <[email protected]> * feat: test for exp-run service Signed-off-by: SohamRatnaparkhi <[email protected]> * feat: infra service mocks Signed-off-by: SohamRatnaparkhi <[email protected]> * feat: test for exp handlers(incomplete) Signed-off-by: SohamRatnaparkhi <[email protected]> * fix: local mongo opr usage instead of global Signed-off-by: SohamRatnaparkhi <[email protected]> * feat: incomplete GetExpRun and ReRunExp tests Signed-off-by: SohamRatnaparkhi <[email protected]> * feat: complete services tests Signed-off-by: SohamRatnaparkhi <[email protected]> * feat: complete handler tests Signed-off-by: SohamRatnaparkhi <[email protected]> --------- Signed-off-by: SohamRatnaparkhi <[email protected]>
- Loading branch information
1 parent
2d64b25
commit fcc8382
Showing
22 changed files
with
3,536 additions
and
1,543 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
35 changes: 35 additions & 0 deletions
35
chaoscenter/graphql/definitions/shared/chaos_experiment_run.graphqls
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,35 @@ | ||
| extend type Query { | ||
| """ | ||
| Returns experiment run based on experiment run ID | ||
| """ | ||
| getExperimentRun(projectID: ID!, experimentRunID: String!): ExperimentRun! | ||
|
|
||
| """ | ||
| Returns the list of experiment run based on various filter parameters | ||
| """ | ||
| listExperimentRun( | ||
| projectID: ID! | ||
| request: ListExperimentRunRequest! | ||
| ): ListExperimentRunResponse! | ||
|
|
||
| """ | ||
| Query to get experiment run stats | ||
| """ | ||
| getExperimentRunStats(projectID: ID!): GetExperimentRunStatsResponse! | ||
| } | ||
|
|
||
| extend type Mutation { | ||
| """ | ||
| Creates a new experiment run and sends it to subscriber | ||
| """ | ||
| # authorized directive not required | ||
| chaosExperimentRun(request: ExperimentRunRequest!): String! | ||
|
|
||
| """ | ||
| Run the chaos experiment (used by frontend) | ||
| """ | ||
| runChaosExperiment( | ||
| experimentID: String! | ||
| projectID: ID! | ||
| ): RunChaosExperimentResponse! | ||
| } |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
117 changes: 117 additions & 0 deletions
117
chaoscenter/graphql/server/graph/chaos_experiment_run.resolvers.go
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,117 @@ | ||
| package graph | ||
|
|
||
| // This file will be automatically regenerated based on the schema, any resolver implementations | ||
| // will be copied through when generating and any unknown code will be moved to the end. | ||
|
|
||
| import ( | ||
| "context" | ||
| "errors" | ||
|
|
||
| "github.com/litmuschaos/litmus/chaoscenter/graphql/server/graph/model" | ||
| "github.com/litmuschaos/litmus/chaoscenter/graphql/server/pkg/authorization" | ||
| data_store "github.com/litmuschaos/litmus/chaoscenter/graphql/server/pkg/data-store" | ||
| "github.com/sirupsen/logrus" | ||
| "go.mongodb.org/mongo-driver/bson" | ||
| ) | ||
|
|
||
| func (r *mutationResolver) ChaosExperimentRun(ctx context.Context, request model.ExperimentRunRequest) (string, error) { | ||
| return r.chaosExperimentRunHandler.ChaosExperimentRunEvent(request) | ||
| } | ||
|
|
||
| func (r *mutationResolver) RunChaosExperiment(ctx context.Context, experimentID string, projectID string) (*model.RunChaosExperimentResponse, error) { | ||
| logFields := logrus.Fields{ | ||
| "projectId": projectID, | ||
| "chaosExperimentId": experimentID, | ||
| } | ||
|
|
||
| logrus.WithFields(logFields).Info("request received to run chaos experiment") | ||
| err := authorization.ValidateRole(ctx, projectID, | ||
| authorization.MutationRbacRules[authorization.CreateChaosWorkFlow], | ||
| model.InvitationAccepted.String()) | ||
| if err != nil { | ||
| return nil, err | ||
| } | ||
|
|
||
| query := bson.D{ | ||
| {"experiment_id", experimentID}, | ||
| {"is_removed", false}, | ||
| } | ||
|
|
||
| experiment, err := r.chaosExperimentHandler.GetDBExperiment(query) | ||
| if err != nil { | ||
| return nil, errors.New("could not get experiment run, error: " + err.Error()) | ||
| } | ||
|
|
||
| var uiResponse *model.RunChaosExperimentResponse | ||
|
|
||
| uiResponse, err = r.chaosExperimentRunHandler.RunChaosWorkFlow(ctx, projectID, experiment, data_store.Store) | ||
| if err != nil { | ||
| logrus.WithFields(logFields).Error(err) | ||
| return nil, err | ||
| } | ||
|
|
||
| return &model.RunChaosExperimentResponse{NotifyID: uiResponse.NotifyID}, err | ||
| } | ||
|
|
||
| func (r *queryResolver) GetExperimentRun(ctx context.Context, projectID string, experimentRunID string) (*model.ExperimentRun, error) { | ||
| logFields := logrus.Fields{ | ||
| "projectId": projectID, | ||
| "chaosExperimentRunId": experimentRunID, | ||
| } | ||
| logrus.WithFields(logFields).Info("request received to fetch chaos experiment run") | ||
| err := authorization.ValidateRole(ctx, projectID, | ||
| authorization.MutationRbacRules[authorization.GetWorkflowRun], | ||
| model.InvitationAccepted.String()) | ||
| if err != nil { | ||
| return nil, err | ||
| } | ||
|
|
||
| expRunResponse, err := r.chaosExperimentRunHandler.GetExperimentRun(ctx, projectID, experimentRunID) | ||
| if err != nil { | ||
| logrus.WithFields(logFields).Error(err) | ||
| return nil, err | ||
| } | ||
| return expRunResponse, err | ||
| } | ||
|
|
||
| func (r *queryResolver) ListExperimentRun(ctx context.Context, projectID string, request model.ListExperimentRunRequest) (*model.ListExperimentRunResponse, error) { | ||
| logFields := logrus.Fields{ | ||
| "projectId": projectID, | ||
| "chaosExperimentIds": request.ExperimentIDs, | ||
| "chaosExperimentRunIds": request.ExperimentRunIDs, | ||
| } | ||
| logrus.WithFields(logFields).Info("request received to list chaos experiment run") | ||
|
|
||
| err := authorization.ValidateRole(ctx, projectID, | ||
| authorization.MutationRbacRules[authorization.ListWorkflowRuns], | ||
| model.InvitationAccepted.String()) | ||
| if err != nil { | ||
| return nil, err | ||
| } | ||
| uiResponse, err := r.chaosExperimentRunHandler.ListExperimentRun(projectID, request) | ||
| if err != nil { | ||
| logrus.WithFields(logFields).Error(err) | ||
| return nil, err | ||
| } | ||
| return uiResponse, err | ||
| } | ||
|
|
||
| func (r *queryResolver) GetExperimentRunStats(ctx context.Context, projectID string) (*model.GetExperimentRunStatsResponse, error) { | ||
| logFields := logrus.Fields{ | ||
| "projectId": projectID, | ||
| } | ||
| logrus.WithFields(logFields).Info("request received to get chaos experiment run stats") | ||
| err := authorization.ValidateRole(ctx, projectID, | ||
| authorization.MutationRbacRules[authorization.ListWorkflowRuns], | ||
| model.InvitationAccepted.String()) | ||
| if err != nil { | ||
| return nil, err | ||
| } | ||
|
|
||
| uiResponse, err := r.chaosExperimentRunHandler.GetExperimentRunStats(ctx, projectID) | ||
| if err != nil { | ||
| logrus.WithFields(logFields).Error(err) | ||
| return nil, err | ||
| } | ||
| return uiResponse, err | ||
| } |
Oops, something went wrong.