MLops: instead of spying on `apolloClient` property, Bind a handler for the mutation and check it's called

The following discussion from !177110 (merged) should be addressed:

  • @ntepluhina started a discussion:

    suggestion: instead of spying on apolloClient property, it is recommended to bind a handler for the mutation and make sure the handler was called with the correct variables




File: experiment_tracking/routes/candidates/promote/promote_run_spec.js
191:12:      jest.spyOn(apolloProvider.defaultClient, 'mutate');

File: model_registry/components/model_version_create_spec.js
40:10:    jest.spyOn(Sentry, 'captureException').mockImplementation();
240:12:      jest.spyOn(apolloProvider.defaultClient, 'mutate');

File: model_registry/apps/show_ml_model_version_spec.js
301:10:    jest.spyOn(apolloProvider.defaultClient, 'mutate');

File: model_registry/components/model_version_edit_spec.js
121:12:      jest.spyOn(apolloProvider.defaultClient, 'mutate');
155:14:        jest.spyOn(apolloProvider.defaultClient, 'mutate');
179:14:        jest.spyOn(apolloProvider.defaultClient, 'mutate');

File: model_registry/components/model_create_spec.js
176:12:      jest.spyOn(apolloProvider.defaultClient, 'mutate');
193:14:        jest.spyOn(apolloProvider.defaultClient, 'mutate');
211:12:      jest.spyOn(apolloProvider.defaultClient, 'mutate').mockImplementation(() => {

File: model_registry/components/model_versions_table_spec.js
131:10:    jest.spyOn(apolloProvider.defaultClient, 'mutate');

File: model_registry/components/model_edit_spec.js
121:12:      jest.spyOn(apolloProvider.defaultClient, 'mutate');
146:12:      jest.spyOn(apolloProvider.defaultClient, 'mutate');

Ag finished at Fri Jan 10 14:44:38