admin/n8n-enterprise-unlocked
1
0
Fork 0
mirror of https://github.com/Abdulazizzn/n8n-enterprise-unlocked.git synced 2025-12-16 09:36:44 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(core): Account for itemless case on restoring binary data ID (#7305)

Browse Source 
https://linear.app/n8n/issue/PAY-862
This commit is contained in:
Iván Ovejero
2023-10-02 09:13:55 +02:00
committed by GitHub
parent d1b6c7fd79
commit 1691223789
2 changed files with 12 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
packages/cli/src/executionLifecycleHooks/restoreBinaryDataId.ts
View File

@@ -25,7 +25,7 @@ export async function restoreBinaryDataId(run: IRun, executionId: string) {
const { runData } = run.data.resultData;
const promises = Object.keys(runData).map(async (nodeName) => {
const binaryDataId = runData[nodeName]?.[0]?.data?.main?.[0]?.[0].binary?.data.id;
const binaryDataId = runData[nodeName]?.[0]?.data?.main?.[0]?.[0]?.binary?.data.id;
if (!binaryDataId || !isMissingExecutionId(binaryDataId)) return;

12
packages/cli/test/unit/execution.lifecycle.test.ts
View File

@@ -3,7 +3,7 @@ import { BinaryDataService } from 'n8n-core';
import { mockInstance } from '../integration/shared/utils/mocking';
import type { IRun } from 'n8n-workflow';
function toIRun(item: object) {
function toIRun(item?: object) {
return {
data: {
resultData: {
@@ -84,4 +84,14 @@ describe('restoreBinaryDataId()', () => {
expect(binaryDataService.rename).not.toHaveBeenCalled();
expect(getDataId(run, 'json')).toBe(dataId);
});
it('should do nothing on itemless case', async () => {
const executionId = '999';
const promise = restoreBinaryDataId(toIRun(), executionId);
await expect(promise).resolves.not.toThrow();
expect(binaryDataService.rename).not.toHaveBeenCalled();
});
});