mirror of
https://github.com/n8n-io/n8n.git
synced 2024-12-24 04:04:06 -08:00
fix(Google Sheets Node): Unit tests (no-changelog)
This commit is contained in:
Michael Kret
GitHub
parent
a1306c6903
commit
082c61701c
|
|
@ -0,0 +1,284 @@
|
|||
import type { IExecuteFunctions, INode } from 'n8n-workflow';
|
||||
import { GoogleSheet } from '../../../v2/helpers/GoogleSheet';
|
||||
import {
|
||||
addRowNumber,
|
||||
autoMapInputData,
|
||||
prepareSheetData,
|
||||
removeEmptyColumns,
|
||||
removeEmptyRows,
|
||||
trimLeadingEmptyRows,
|
||||
trimToFirstEmptyRow,
|
||||
} from '../../../v2/helpers/GoogleSheets.utils';
|
||||
|
||||
describe('Test Google Sheets, addRowNumber', () => {
|
||||
it('should add row nomber', () => {
|
||||
const data = [
|
||||
['id', 'col1', 'col2', 'col3'],
|
||||
[0, 'A', 'B', 'C'],
|
||||
[1, 'a', 'b', 'c'],
|
||||
[2, 'd', 'e', 'f'],
|
||||
[3, 'g', 'h', 'i'],
|
||||
];
|
||||
const result = addRowNumber(data, 0);
|
||||
|
||||
expect(result).toBeDefined();
|
||||
expect(result).toEqual([
|
||||
['row_number', 'id', 'col1', 'col2', 'col3'],
|
||||
[2, 0, 'A', 'B', 'C'],
|
||||
[3, 1, 'a', 'b', 'c'],
|
||||
[4, 2, 'd', 'e', 'f'],
|
||||
[5, 3, 'g', 'h', 'i'],
|
||||
]);
|
||||
});
|
||||
});
|
||||
|
||||
describe('Test Google Sheets, trimToFirstEmptyRow', () => {
|
||||
it('should trimToFirstEmptyRow without row numbers', () => {
|
||||
const data = [
|
||||
['id', 'col1', 'col2', 'col3'],
|
||||
[0, 'A', 'B', 'C'],
|
||||
['', '', '', ''],
|
||||
[2, 'd', 'e', 'f'],
|
||||
[3, 'g', 'h', 'i'],
|
||||
];
|
||||
|
||||
const result = trimToFirstEmptyRow(data, false);
|
||||
|
||||
expect(result).toBeDefined();
|
||||
expect(result).toEqual([
|
||||
['id', 'col1', 'col2', 'col3'],
|
||||
[0, 'A', 'B', 'C'],
|
||||
]);
|
||||
});
|
||||
it('should trimToFirstEmptyRow with row numbers', () => {
|
||||
const data = [
|
||||
['row_number', 'id', 'col1', 'col2', 'col3'],
|
||||
[2, 0, 'A', 'B', 'C'],
|
||||
[3, 1, 'a', 'b', 'c'],
|
||||
[4, '', '', '', ''],
|
||||
[5, 3, 'g', 'h', 'i'],
|
||||
];
|
||||
|
||||
const result = trimToFirstEmptyRow(data);
|
||||
|
||||
expect(result).toBeDefined();
|
||||
expect(result).toEqual([
|
||||
['row_number', 'id', 'col1', 'col2', 'col3'],
|
||||
[2, 0, 'A', 'B', 'C'],
|
||||
[3, 1, 'a', 'b', 'c'],
|
||||
]);
|
||||
});
|
||||
});
|
||||
|
||||
describe('Test Google Sheets, removeEmptyRows', () => {
|
||||
it('should removeEmptyRows without row numbers', () => {
|
||||
const data = [
|
||||
['id', 'col1', 'col2', 'col3'],
|
||||
[0, 'A', 'B', 'C'],
|
||||
['', '', '', ''],
|
||||
[2, 'd', 'e', 'f'],
|
||||
['', '', '', ''],
|
||||
];
|
||||
|
||||
const result = removeEmptyRows(data, false);
|
||||
|
||||
expect(result).toBeDefined();
|
||||
expect(result).toEqual([
|
||||
['id', 'col1', 'col2', 'col3'],
|
||||
[0, 'A', 'B', 'C'],
|
||||
[2, 'd', 'e', 'f'],
|
||||
]);
|
||||
});
|
||||
it('should removeEmptyRows with row numbers', () => {
|
||||
const data = [
|
||||
['row_number', 'id', 'col1', 'col2', 'col3'],
|
||||
[2, 0, 'A', 'B', 'C'],
|
||||
[3, 1, 'a', 'b', 'c'],
|
||||
[4, '', '', '', ''],
|
||||
[5, 3, 'g', 'h', 'i'],
|
||||
];
|
||||
|
||||
const result = removeEmptyRows(data);
|
||||
|
||||
expect(result).toBeDefined();
|
||||
expect(result).toEqual([
|
||||
['row_number', 'id', 'col1', 'col2', 'col3'],
|
||||
[2, 0, 'A', 'B', 'C'],
|
||||
[3, 1, 'a', 'b', 'c'],
|
||||
[5, 3, 'g', 'h', 'i'],
|
||||
]);
|
||||
});
|
||||
});
|
||||
|
||||
describe('Test Google Sheets, trimLeadingEmptyRows', () => {
|
||||
it('should trimLeadingEmptyRows without row numbers', () => {
|
||||
const data = [
|
||||
['', '', '', ''],
|
||||
['', '', '', ''],
|
||||
[2, 'd', 'e', 'f'],
|
||||
['', '', '', ''],
|
||||
];
|
||||
|
||||
const result = trimLeadingEmptyRows(data, false);
|
||||
|
||||
expect(result).toBeDefined();
|
||||
expect(result).toEqual([
|
||||
[2, 'd', 'e', 'f'],
|
||||
['', '', '', ''],
|
||||
]);
|
||||
});
|
||||
it('should trimLeadingEmptyRows with row numbers', () => {
|
||||
const data = [
|
||||
[1, '', '', '', ''],
|
||||
['row_number', 'id', 'col1', 'col2', 'col3'],
|
||||
[2, 0, 'A', 'B', 'C'],
|
||||
[3, 1, 'a', 'b', 'c'],
|
||||
[5, 3, 'g', 'h', 'i'],
|
||||
];
|
||||
|
||||
const result = trimLeadingEmptyRows(data, true);
|
||||
|
||||
expect(result).toBeDefined();
|
||||
expect(result).toEqual([
|
||||
['row_number', 'id', 'col1', 'col2', 'col3'],
|
||||
[2, 0, 'A', 'B', 'C'],
|
||||
[3, 1, 'a', 'b', 'c'],
|
||||
[5, 3, 'g', 'h', 'i'],
|
||||
]);
|
||||
});
|
||||
});
|
||||
|
||||
describe('Test Google Sheets, removeEmptyColumns', () => {
|
||||
it('should removeEmptyColumns without row numbers', () => {
|
||||
const data = [
|
||||
['id', 'col1', '', 'col3'],
|
||||
[0, 'A', '', 'C'],
|
||||
[1, 'a', '', 'c'],
|
||||
[2, 'd', '', 'f'],
|
||||
[3, 'g', '', 'i'],
|
||||
];
|
||||
|
||||
const result = removeEmptyColumns(data);
|
||||
|
||||
expect(result).toBeDefined();
|
||||
expect(result).toEqual([
|
||||
['id', 'col1', 'col3'],
|
||||
['', 'A', 'C'], // TODO:should be [0, 'A', 'C'] ?
|
||||
[1, 'a', 'c'],
|
||||
[2, 'd', 'f'],
|
||||
[3, 'g', 'i'],
|
||||
]);
|
||||
});
|
||||
});
|
||||
|
||||
describe('Test Google Sheets, prepareSheetData', () => {
|
||||
it('should prepareSheetData without row numbers', () => {
|
||||
const data = [
|
||||
['id', 'col1', 'col2', 'col3'],
|
||||
[1, 'A', 'B', 'C'],
|
||||
['', '', '', ''],
|
||||
[2, 'd', 'e', 'f'],
|
||||
['', '', '', ''],
|
||||
];
|
||||
|
||||
const result = prepareSheetData(data, { rangeDefinition: 'detectAutomatically' }, true);
|
||||
|
||||
expect(result).toBeDefined();
|
||||
expect(result).toEqual({
|
||||
data: [
|
||||
['row_number', 'id', 'col1', 'col2', 'col3'],
|
||||
[2, 1, 'A', 'B', 'C'],
|
||||
[4, 2, 'd', 'e', 'f'],
|
||||
],
|
||||
firstDataRow: 1,
|
||||
headerRow: 0,
|
||||
});
|
||||
});
|
||||
});
|
||||
|
||||
describe('Test Google Sheets, autoMapInputData', () => {
|
||||
it('should autoMapInputData', async () => {
|
||||
const node: INode = {
|
||||
id: '1',
|
||||
name: 'Postgres node',
|
||||
typeVersion: 2,
|
||||
type: 'n8n-nodes-base.postgres',
|
||||
position: [60, 760],
|
||||
parameters: {
|
||||
operation: 'executeQuery',
|
||||
},
|
||||
};
|
||||
|
||||
const items = [
|
||||
{
|
||||
json: {
|
||||
id: 1,
|
||||
name: 'Jon',
|
||||
data: 'A',
|
||||
},
|
||||
},
|
||||
{
|
||||
json: {
|
||||
id: 2,
|
||||
name: 'Sam',
|
||||
data: 'B',
|
||||
},
|
||||
},
|
||||
{
|
||||
json: {
|
||||
id: 3,
|
||||
name: 'Ron',
|
||||
data: 'C',
|
||||
info: 'some info',
|
||||
},
|
||||
},
|
||||
];
|
||||
|
||||
const fakeExecuteFunction = {
|
||||
getNode() {
|
||||
return node;
|
||||
},
|
||||
} as unknown as IExecuteFunctions;
|
||||
|
||||
const getData = (GoogleSheet.prototype.getData = jest.fn().mockResolvedValue([[]]));
|
||||
|
||||
const updateRows = (GoogleSheet.prototype.updateRows = jest.fn().mockResolvedValue(true));
|
||||
|
||||
const googleSheet = new GoogleSheet('spreadsheetId', fakeExecuteFunction);
|
||||
|
||||
const result = await autoMapInputData.call(
|
||||
fakeExecuteFunction,
|
||||
'foo 1',
|
||||
googleSheet,
|
||||
items,
|
||||
{},
|
||||
);
|
||||
|
||||
expect(getData).toHaveBeenCalledTimes(1);
|
||||
expect(getData).toHaveBeenCalledWith('foo 1!1:1', 'FORMATTED_VALUE');
|
||||
|
||||
expect(updateRows).toHaveBeenCalledTimes(2);
|
||||
expect(updateRows).toHaveBeenCalledWith('foo 1', [['id', 'name', 'data']], 'RAW', 1);
|
||||
expect(updateRows).toHaveBeenCalledWith('foo 1', [['id', 'name', 'data', 'info']], 'RAW', 1);
|
||||
|
||||
expect(result).toBeDefined();
|
||||
expect(result).toEqual([
|
||||
{
|
||||
id: 1,
|
||||
name: 'Jon',
|
||||
data: 'A',
|
||||
},
|
||||
{
|
||||
id: 2,
|
||||
name: 'Sam',
|
||||
data: 'B',
|
||||
},
|
||||
{
|
||||
id: 3,
|
||||
name: 'Ron',
|
||||
data: 'C',
|
||||
info: 'some info',
|
||||
},
|
||||
]);
|
||||
});
|
||||
});
|
||||
Loading…
Reference in a new issue