mudhorn/n8n
1
0
Fork 0
mirror of https://github.com/n8n-io/n8n.git synced 2025-03-05 20:50:17 -08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

fix(MySQL Node): Set paired items correctly in single query batch mode (#8940)

Browse source
This commit is contained in:
Iván Ovejero 2024-03-21 12:17:43 +01:00 committed by GitHub
parent 34b79333fc
commit 89df277b80
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 34 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

32
packages/nodes-base/nodes/MySql/test/v2/runQueries.test.ts
View file

@ -42,6 +42,38 @@ describe('Test MySql V2, runQueries', () => {
jest.clearAllMocks();
});
describe('in single query batch mode', () => {
it('should set paired items correctly', async () => {
const nodeOptions = { queryBatching: BATCH_MODE.SINGLE, nodeVersion: 2 };
const pool = createFakePool(fakeConnection);
const mockExecuteFns = createMockExecuteFunction({}, mySqlMockNode);
pool.query = jest.fn(async () => [
[[{ finishedAt: '2023-12-30' }], [{ finishedAt: '2023-12-31' }]],
]);
const result = await configureQueryRunner.call(
mockExecuteFns,
nodeOptions,
pool,
)([
{ query: 'SELECT finishedAt FROM my_table WHERE id = ?', values: [123] },
{ query: 'SELECT finishedAt FROM my_table WHERE id = ?', values: [456] },
]);
expect(result).toEqual([
{
json: { finishedAt: '2023-12-30' },
pairedItem: { item: 0 },
},
{
json: { finishedAt: '2023-12-31' },
pairedItem: { item: 1 },
},
]);
});
});
it('should execute in "Single" mode, should return success true', async () => {
const nodeOptions: IDataObject = { queryBatching: BATCH_MODE.SINGLE, nodeVersion: 2 };

4
packages/nodes-base/nodes/MySql/v2/helpers/utils.ts
View file

@ -159,9 +159,9 @@ export function prepareOutput(
} else {
response
.filter((entry) => Array.isArray(entry))
.forEach((entry) => {
.forEach((entry, index) => {
const executionData = constructExecutionHelper(wrapData(entry), {
itemData,
itemData: Array.isArray(itemData) ? itemData[index] : itemData,
});
returnData.push(...executionData);