feat(Postgres Node): Batching warning for executeQuery operation insert query (#14287)

packages/nodes-base/nodes/Postgres/test/v2/utils.test.ts

@@ -1,4 +1,4 @@
-import type { IDataObject, INode } from 'n8n-workflow';
+import type { IDataObject, IExecuteFunctions, INode } from 'n8n-workflow';
 import { NodeOperationError } from 'n8n-workflow';
 import pgPromise from 'pg-promise';
 
@@ -18,6 +18,7 @@ import {
 	convertValuesToJsonWithPgp,
 	hasJsonDataTypeInSchema,
 	evaluateExpression,
+	addExecutionHints,
 } from '../../v2/helpers/utils';
 
 const node: INode = {
@@ -535,3 +536,38 @@ describe('Test PostgresV2, convertArraysToPostgresFormat', () => {
 		});
 	});
 });
+
+describe('Test PostgresV2, addExecutionHints', () => {
+	it('should add batching insert hint to executeQuery operation', () => {
+		const context = {
+			getNodeParameter: (parameterName: string) => {
+				if (parameterName === 'options.queryBatching') {
+					return 'single';
+				}
+				if (parameterName === 'query') {
+					return 'INSERT INTO my_test_table VALUES (`{{ $json.name }}`)';
+				}
+			},
+			addExecutionHints: jest.fn(),
+		} as unknown as IExecuteFunctions;
+
+		addExecutionHints(context, [{ json: {} }, { json: {} }], 'executeQuery', false);
+		expect(context.addExecutionHints).toHaveBeenCalledWith({
+			message:
+				"Inserts were batched for performance. If you need to preserve item matching, consider changing 'Query batching' to 'Independent' in the options.",
+			location: 'outputPane',
+		});
+	});
+	it('should add run per item hint to select operation', () => {
+		const context = {
+			addExecutionHints: jest.fn(),
+		} as unknown as IExecuteFunctions;
+
+		addExecutionHints(context, [{ json: {} }, { json: {} }], 'select', false);
+		expect(context.addExecutionHints).toHaveBeenCalledWith({
+			location: 'outputPane',
+			message:
+				"This node ran 2 times, once for each input item. To run for the first item only, enable 'execute once' in the node settings",
+		});
+	});
+});