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

feat(Airtop Node): Implement windows list API and other improvements (#16748)

Browse Source 
Co-authored-by: Eugene <eugene@n8n.io>
Co-authored-by: Daria <daria.staferova@n8n.io>
Co-authored-by: Alex Grozav <alex@grozav.com>
Co-authored-by: Tomi Turtiainen <10324676+tomi@users.noreply.github.com>
Co-authored-by: Guillaume Jacquart <jacquart.guillaume@gmail.com>
Co-authored-by: Charlie Kolb <charlie@n8n.io>
Co-authored-by: Elias Meire <elias@meire.dev>
Co-authored-by: Michael Kret <88898367+michael-radency@users.noreply.github.com>
Co-authored-by: shortstacked <declan@n8n.io>
Co-authored-by: oleg <me@olegivaniv.com>
Co-authored-by: Csaba Tuncsik <csaba@n8n.io>
Co-authored-by: Jaakko Husso <jaakko@n8n.io>
Co-authored-by: Raúl Gómez Morales <raul00gm@gmail.com>
Co-authored-by: Suguru Inoue <suguru@n8n.io>
Co-authored-by: Milorad FIlipović <milorad@n8n.io>
Co-authored-by: Danny Martini <danny@n8n.io>
Co-authored-by: Iván Ovejero <ivov.src@gmail.com>
Co-authored-by: RomanDavydchuk <roman.davydchuk@n8n.io>
Co-authored-by: Mutasem Aldmour <4711238+mutdmour@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
Co-authored-by: Ricardo Espinoza <ricardo@n8n.io>
Co-authored-by: Dana <152518854+dana-gill@users.noreply.github.com>
Co-authored-by: Michael Kret <michael.k@radency.com>
Co-authored-by: Mutasem Aldmour <mutasem@n8n.io>
This commit is contained in:
Cesar Sanchez
2025-07-04 18:56:14 +01:00
committed by GitHub
parent bb9679c4fa
commit 621745e291
21 changed files with 587 additions and 163 deletions
Download Patch File Download Diff File Expand all files Collapse all files

93
packages/nodes-base/nodes/Airtop/test/node/file/helpers.test.ts
View File

@@ -10,10 +10,6 @@ const mockFileCreateResponse = {
},
};
const mockFileEvent = `
event: fileEvent
data: {"event":"file_upload_status","status":"available","fileId":"file-123"}`;
// Mock the transport and other dependencies
jest.mock('../../../transport', () => {
const originalModule = jest.requireActual<typeof transport>('../../../transport');
@@ -30,6 +26,7 @@ describe('Test Airtop file helpers', () => {
afterEach(() => {
jest.clearAllMocks();
(transport.apiRequest as jest.Mock).mockReset();
});
describe('requestAllFiles', () => {
@@ -198,42 +195,33 @@ describe('Test Airtop file helpers', () => {
});
describe('waitForFileInSession', () => {
it('should resolve when file is available', async () => {
// Create a mock stream
const mockStream = {
on: jest.fn().mockImplementation((event, callback) => {
if (event === 'data') {
callback(mockFileEvent);
}
}),
removeAllListeners: jest.fn(),
};
const mockHttpRequestWithAuthentication = jest.fn().mockResolvedValueOnce(mockStream);
const mockExecuteFunction = createMockExecuteFunction({});
mockExecuteFunction.helpers.httpRequestWithAuthentication = mockHttpRequestWithAuthentication;
await helpers.waitForFileInSession.call(mockExecuteFunction, 'session-123', 'file-123', 100);
expect(mockHttpRequestWithAuthentication).toHaveBeenCalledWith('airtopApi', {
method: 'GET',
url: `${BASE_URL}/sessions/session-123/events?all=true`,
encoding: 'stream',
it('should resolve when file is available in session', async () => {
const apiRequestMock = transport.apiRequest as jest.Mock;
apiRequestMock.mockResolvedValueOnce({
data: {
sessionIds: ['session-123', 'other-session'],
},
});
expect(mockStream.removeAllListeners).toHaveBeenCalled();
const mockExecuteFunction = createMockExecuteFunction({});
await helpers.waitForFileInSession.call(mockExecuteFunction, 'session-123', 'file-123', 1000);
expect(apiRequestMock).toHaveBeenCalledTimes(1);
expect(apiRequestMock).toHaveBeenCalledWith('GET', `${BASE_URL}/files/file-123`);
});
it('should timeout if no event is received', async () => {
// Create a mock stream
const mockStream = {
on: jest.fn().mockImplementation(() => {}),
removeAllListeners: jest.fn(),
};
const mockHttpRequestWithAuthentication = jest.fn().mockResolvedValueOnce(mockStream);
it('should timeout if file never becomes available in session', async () => {
const apiRequestMock = transport.apiRequest as jest.Mock;
// Mock to always return file not in session
apiRequestMock.mockResolvedValue({
data: {
sessionIds: ['other-session'],
},
});
const mockExecuteFunction = createMockExecuteFunction({});
mockExecuteFunction.helpers.httpRequestWithAuthentication = mockHttpRequestWithAuthentication;
const waitPromise = helpers.waitForFileInSession.call(
mockExecuteFunction,
@@ -244,6 +232,24 @@ describe('Test Airtop file helpers', () => {
await expect(waitPromise).rejects.toThrow();
});
it('should resolve immediately if file is already in session', async () => {
const apiRequestMock = transport.apiRequest as jest.Mock;
// Mock to return file already in session
apiRequestMock.mockResolvedValueOnce({
data: {
sessionIds: ['session-123', 'other-session'],
},
});
const mockExecuteFunction = createMockExecuteFunction({});
await helpers.waitForFileInSession.call(mockExecuteFunction, 'session-123', 'file-123', 1000);
expect(apiRequestMock).toHaveBeenCalledTimes(1);
expect(apiRequestMock).toHaveBeenCalledWith('GET', `${BASE_URL}/files/file-123`);
});
});
describe('pushFileToSession', () => {
@@ -280,17 +286,22 @@ describe('Test Airtop file helpers', () => {
const mockWindowId = 'window-123';
const mockSessionId = 'session-123';
await helpers.triggerFileInput.call(
createMockExecuteFunction({}),
mockFileId,
mockWindowId,
mockSessionId,
);
await helpers.triggerFileInput.call(createMockExecuteFunction({}), {
fileId: mockFileId,
windowId: mockWindowId,
sessionId: mockSessionId,
elementDescription: 'test',
includeHiddenElements: false,
});
expect(apiRequestMock).toHaveBeenCalledWith(
'POST',
`/sessions/${mockSessionId}/windows/${mockWindowId}/file-input`,
{ fileId: mockFileId },
{
fileId: mockFileId,
elementDescription: 'test',
includeHiddenElements: false,
},
);
});
});