Fix unit test failures where we were using workflow action toooltip item id's

This commit is contained in:
mabashian 2020-02-20 12:57:38 -05:00
parent 23677b4963
commit 764c0b2e15
3 changed files with 49 additions and 25 deletions

View File

@ -171,7 +171,7 @@ describe('Visualizer', () => {
wrapper.update();
expect(wrapper.find('LinkEditModal').length).toBe(0);
wrapper.find('g#link-2-3').simulate('mouseenter');
wrapper.find('#link-edit').simulate('click');
wrapper.find('WorkflowActionTooltipItem#link-edit').simulate('click');
expect(wrapper.find('LinkEditModal').length).toBe(1);
act(() => {
wrapper

View File

@ -96,13 +96,17 @@ describe('VisualizerLink', () => {
.find('g')
.first()
.simulate('mouseenter');
wrapper.find('#link-add-node').simulate('mouseenter');
wrapper
.find('WorkflowActionTooltipItem#link-add-node')
.simulate('mouseenter');
expect(updateHelpText).toHaveBeenCalledWith(
'Add a new node between these two nodes'
);
wrapper.find('#link-add-node').simulate('mouseleave');
wrapper
.find('WorkflowActionTooltipItem#link-add-node')
.simulate('mouseleave');
expect(updateHelpText).toHaveBeenCalledWith(null);
wrapper.find('#link-add-node').simulate('click');
wrapper.find('WorkflowActionTooltipItem#link-add-node').simulate('click');
expect(dispatch).toHaveBeenCalledWith({
type: 'START_ADD_NODE',
sourceNodeId: 2,
@ -116,11 +120,11 @@ describe('VisualizerLink', () => {
.find('g')
.first()
.simulate('mouseenter');
wrapper.find('#link-edit').simulate('mouseenter');
wrapper.find('WorkflowActionTooltipItem#link-edit').simulate('mouseenter');
expect(updateHelpText).toHaveBeenCalledWith('Edit this link');
wrapper.find('#link-edit').simulate('mouseleave');
wrapper.find('WorkflowActionTooltipItem#link-edit').simulate('mouseleave');
expect(updateHelpText).toHaveBeenCalledWith(null);
wrapper.find('#link-edit').simulate('click');
wrapper.find('WorkflowActionTooltipItem#link-edit').simulate('click');
expect(dispatch).toHaveBeenCalledWith({
type: 'SET_LINK_TO_EDIT',
value: link,
@ -133,11 +137,15 @@ describe('VisualizerLink', () => {
.find('g')
.first()
.simulate('mouseenter');
wrapper.find('#link-delete').simulate('mouseenter');
wrapper
.find('WorkflowActionTooltipItem#link-delete')
.simulate('mouseenter');
expect(updateHelpText).toHaveBeenCalledWith('Delete this link');
wrapper.find('#link-delete').simulate('mouseleave');
wrapper
.find('WorkflowActionTooltipItem#link-delete')
.simulate('mouseleave');
expect(updateHelpText).toHaveBeenCalledWith(null);
wrapper.find('#link-delete').simulate('click');
wrapper.find('WorkflowActionTooltipItem#link-delete').simulate('click');
expect(dispatch).toHaveBeenCalledWith({
type: 'START_DELETE_LINK',
link,

View File

@ -86,11 +86,11 @@ describe('VisualizerNode', () => {
test('Add tooltip action hover/click updates help text and dispatches properly', () => {
wrapper.find('VisualizerNode').simulate('mouseenter');
wrapper.find('#node-add').simulate('mouseenter');
wrapper.find('WorkflowActionTooltipItem#node-add').simulate('mouseenter');
expect(updateHelpText).toHaveBeenCalledWith('Add a new node');
wrapper.find('#node-add').simulate('mouseleave');
wrapper.find('WorkflowActionTooltipItem#node-add').simulate('mouseleave');
expect(updateHelpText).toHaveBeenCalledWith(null);
wrapper.find('#node-add').simulate('click');
wrapper.find('WorkflowActionTooltipItem#node-add').simulate('click');
expect(dispatch).toHaveBeenCalledWith({
type: 'START_ADD_NODE',
sourceNodeId: 2,
@ -100,11 +100,15 @@ describe('VisualizerNode', () => {
test('Edit tooltip action hover/click updates help text and dispatches properly', () => {
wrapper.find('VisualizerNode').simulate('mouseenter');
wrapper.find('#node-edit').simulate('mouseenter');
wrapper
.find('WorkflowActionTooltipItem#node-edit')
.simulate('mouseenter');
expect(updateHelpText).toHaveBeenCalledWith('Edit this node');
wrapper.find('#node-edit').simulate('mouseleave');
wrapper
.find('WorkflowActionTooltipItem#node-edit')
.simulate('mouseleave');
expect(updateHelpText).toHaveBeenCalledWith(null);
wrapper.find('#node-edit').simulate('click');
wrapper.find('WorkflowActionTooltipItem#node-edit').simulate('click');
expect(dispatch).toHaveBeenCalledWith({
type: 'SET_NODE_TO_EDIT',
value: nodeWithJT,
@ -114,11 +118,15 @@ describe('VisualizerNode', () => {
test('Details tooltip action hover/click updates help text and dispatches properly', () => {
wrapper.find('VisualizerNode').simulate('mouseenter');
wrapper.find('#node-details').simulate('mouseenter');
wrapper
.find('WorkflowActionTooltipItem#node-details')
.simulate('mouseenter');
expect(updateHelpText).toHaveBeenCalledWith('View node details');
wrapper.find('#node-details').simulate('mouseleave');
wrapper
.find('WorkflowActionTooltipItem#node-details')
.simulate('mouseleave');
expect(updateHelpText).toHaveBeenCalledWith(null);
wrapper.find('#node-details').simulate('click');
wrapper.find('WorkflowActionTooltipItem#node-details').simulate('click');
expect(dispatch).toHaveBeenCalledWith({
type: 'SET_NODE_TO_VIEW',
value: nodeWithJT,
@ -128,11 +136,15 @@ describe('VisualizerNode', () => {
test('Link tooltip action hover/click updates help text and dispatches properly', () => {
wrapper.find('VisualizerNode').simulate('mouseenter');
wrapper.find('#node-link').simulate('mouseenter');
wrapper
.find('WorkflowActionTooltipItem#node-link')
.simulate('mouseenter');
expect(updateHelpText).toHaveBeenCalledWith('Link to an available node');
wrapper.find('#node-link').simulate('mouseleave');
wrapper
.find('WorkflowActionTooltipItem#node-link')
.simulate('mouseleave');
expect(updateHelpText).toHaveBeenCalledWith(null);
wrapper.find('#node-link').simulate('click');
wrapper.find('WorkflowActionTooltipItem#node-link').simulate('click');
expect(dispatch).toHaveBeenCalledWith({
type: 'SELECT_SOURCE_FOR_LINKING',
node: nodeWithJT,
@ -142,11 +154,15 @@ describe('VisualizerNode', () => {
test('Delete tooltip action hover/click updates help text and dispatches properly', () => {
wrapper.find('VisualizerNode').simulate('mouseenter');
wrapper.find('#node-delete').simulate('mouseenter');
wrapper
.find('WorkflowActionTooltipItem#node-delete')
.simulate('mouseenter');
expect(updateHelpText).toHaveBeenCalledWith('Delete this node');
wrapper.find('#node-delete').simulate('mouseleave');
wrapper
.find('WorkflowActionTooltipItem#node-delete')
.simulate('mouseleave');
expect(updateHelpText).toHaveBeenCalledWith(null);
wrapper.find('#node-delete').simulate('click');
wrapper.find('WorkflowActionTooltipItem#node-delete').simulate('click');
expect(dispatch).toHaveBeenCalledWith({
type: 'SET_NODE_TO_DELETE',
value: nodeWithJT,