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
apolloClientproperty, 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