@2bad/bitrix 2.5.1 → 3.0.0

package/build/module/client/index.d.ts

@@ -1,15 +0,0 @@
-import { ExtendOptions } from 'got';
-/**
- * Construct a Bitrix client with generic methods
- * @param restURI REST endpoint, like a `https://hello.bitrix24.ru/rest` or an inbound webhook endpoint,
- *                like a `https://hello.bitrix24.ru/rest/1/WEBHOOK_TOKEN`.
- * @param accessToken Bitrix application Access Token. Do not specify in case inbound webhook endpoint used.
- * @param options an object that will overwrite underlying configuration for HTTP client,
- *                see `https://github.com/sindresorhus/got/blob/main/documentation/2-options.md`.
- */
-declare const _default: (restURI: string, accessToken?: string, options?: ExtendOptions) => {
-    call: import("./methods/call").Call;
-    batch: import("./methods/batch").Batch;
-    list: import("./methods/list").List;
-};
-export default _default;

package/build/module/client/index.js

@@ -1,48 +0,0 @@
-import got from 'got';
-import Queue from 'p-queue';
-import addAccessToken from './hooks/addAccessToken';
-import Batch from './methods/batch';
-import Call from './methods/call';
-import List from './methods/list';
-const BITRIX_API_RATE_LIMIT = 2;
-const BITRIX_API_RATE_INTERVAL = 1000; // 1 second
-/**
- * Construct a Bitrix client with generic methods
- * @param restURI REST endpoint, like a `https://hello.bitrix24.ru/rest` or an inbound webhook endpoint,
- *                like a `https://hello.bitrix24.ru/rest/1/WEBHOOK_TOKEN`.
- * @param accessToken Bitrix application Access Token. Do not specify in case inbound webhook endpoint used.
- * @param options an object that will overwrite underlying configuration for HTTP client,
- *                see `https://github.com/sindresorhus/got/blob/main/documentation/2-options.md`.
- */
-export default (restURI, accessToken, options) => {
-    const client = got.extend({
-        prefixUrl: restURI,
-        headers: {
-            'user-agent': '@2bad/bitrix'
-        },
-        responseType: 'json',
-        hooks: {
-            beforeRequest: [
-                addAccessToken(accessToken)
-            ]
-        },
-        ...options ?? {}
-    });
-    const queue = new Queue({
-        intervalCap: BITRIX_API_RATE_LIMIT,
-        interval: BITRIX_API_RATE_INTERVAL
-    });
-    const queuedGet = (...args) => queue.add(() => client.get(...args));
-    // eslint-disable-next-line @typescript-eslint/ban-ts-comment
-    // @ts-expect-error @todo remove after issue is resolved (https://github.com/sindresorhus/got/issues/954)
-    const call = Call({ get: queuedGet });
-    // eslint-disable-next-line @typescript-eslint/ban-ts-comment
-    // @ts-expect-error @todo remove after issue is resolved (https://github.com/sindresorhus/got/issues/954)
-    const batch = Batch({ get: queuedGet });
-    const list = List({ call, batch });
-    return {
-        call,
-        batch,
-        list
-    };
-};

package/build/module/client/methods/batch.d.ts

@@ -1,40 +0,0 @@
-import got from 'got';
-import { Commands } from '../../commands';
-import { Method, MethodPayloadType } from '../../methods';
-import { BatchPayload } from '../../payloads';
-export declare const MAX_COMMANDS_PER_BATCH = 50;
-/**
- * Split any number of commands into the chunks with max commands per batch in each chunk
- */
-export declare const chunkCommands: <C extends Commands>(commands: C, size?: number) => readonly C[];
-/**
- * Prepares list of commands to be used in requests to the Bitrix
- * @note We could avoid that function... if only Bitrix API would support posting of the batch commands as
- *       plain objects, like it does with other methods. Instead, it should be a dict or array of string queries
- */
-export declare const prepareCommandsQueries: <C extends Commands, R = { [K in keyof C]: string; }>(commands: C) => R;
-/**
- * Merges list of batch responses into a single batch
- * @todo Generics inference is complicated here and might be not super accurate
- */
-export declare const mergeBatchPayloads: <B extends BatchPayload<unknown>, P = B extends BatchPayload<infer U> ? U : never>(payloads: readonly B[]) => BatchPayload<P>;
-export declare type CommandsPayloads<C extends Commands, CM = {
-    [K in keyof C]: C[K] extends {
-        readonly method: infer M;
-    } ? M : never;
-}> = {
-    [K in keyof CM]: MethodPayloadType<CM[K] extends Method ? CM[K] : never>;
-};
-export declare type Batch = <C extends Commands, P = CommandsPayloads<C>>(commands: C, commandsPerRequest?: number) => Promise<BatchPayload<P>>;
-declare type Dependencies = {
-    readonly get: typeof got.get;
-};
-/**
- * Dispatches a batch request with specified commands. Will fill figure out payload type based on the Methods.
- * Supports unlimited number of commands. If they do exceed max amount of commands per batch, will dispatch
- * multiple request and merge result into single batch payload
- * @todo For now due to issues with types it won't check `params` of command, so any Method can be used
- *       with any (even wrong) params
- */
-declare const _default: ({ get }: Dependencies) => Batch;
-export default _default;

package/build/module/client/methods/batch.js

@@ -1,76 +0,0 @@
-import chunk from 'lodash.chunk';
-import fromPairs from 'lodash.frompairs';
-import { stringify as toQuery } from 'qs';
-import { Method } from '../../methods';
-import isArray from '../../utils/isArray';
-import { handlePayload } from './call';
-export const MAX_COMMANDS_PER_BATCH = 50;
-/**
- * Split any number of commands into the chunks with max commands per batch in each chunk
- */
-export const chunkCommands = (commands, size = MAX_COMMANDS_PER_BATCH) => {
-    const commandsIsArray = isArray(commands);
-    const chunks = chunk(Object.entries(commands), size);
-    // @todo Generated return type isn't very accurate since it says that it will make array of <C>, while
-    //       in fact <C> will be distributed among chunks. That gives wrong hints with named commands and arrays as const.
-    return chunks.map((c) => commandsIsArray
-        // @todo Figure out how to avoid unsafe casting to <C> here
-        //       Without `<C>` generic it won't work with complex functions like `batch`
-        ? c.map(([_key, command]) => command)
-        : fromPairs(c));
-};
-/**
- * Prepares list of commands to be used in requests to the Bitrix
- * @note We could avoid that function... if only Bitrix API would support posting of the batch commands as
- *       plain objects, like it does with other methods. Instead, it should be a dict or array of string queries
- */
-export const prepareCommandsQueries = (commands) => Object.entries(commands).reduce((calls, [name, { method, params }]) => {
-    const stringifiedParams = params ? `?${toQuery(params)}` : '';
-    return {
-        ...calls,
-        [`cmd[${name}]`]: `${method}${stringifiedParams}`
-    };
-    // eslint-disable-next-line @typescript-eslint/prefer-reduce-type-parameter
-}, {});
-/**
- * Merges list of batch responses into a single batch
- * @todo Generics inference is complicated here and might be not super accurate
- */
-export const mergeBatchPayloads = (payloads) => {
-    const merge = (a, b) => isArray(a) && isArray(b)
-        // @todo Gotta find way to avoid unsafe casting here
-        ? [...a, ...b]
-        : ({ ...a, ...b });
-    return payloads.reduce((merged, payload) => ({
-        result: {
-            result: merge(merged.result.result || [], payload.result.result),
-            result_error: merge(merged.result.result_error || [], payload.result.result_error),
-            result_next: merge(merged.result.result_next || [], payload.result.result_next),
-            result_time: merge(merged.result.result_time || [], payload.result.result_time),
-            result_total: merge(merged.result.result_total || [], payload.result.result_total)
-        },
-        time: { ...merged.time, ...payload.time }
-        // eslint-disable-next-line @typescript-eslint/prefer-reduce-type-parameter
-    }), { result: {}, time: {} });
-};
-/**
- * Dispatches a batch request with specified commands. Will fill figure out payload type based on the Methods.
- * Supports unlimited number of commands. If they do exceed max amount of commands per batch, will dispatch
- * multiple request and merge result into single batch payload
- * @todo For now due to issues with types it won't check `params` of command, so any Method can be used
- *       with any (even wrong) params
- */
-export default ({ get }) => {
-    const batch = async (commands, commandsPerRequest = MAX_COMMANDS_PER_BATCH) => {
-        const call = (c) => get(Method.BATCH, { searchParams: prepareCommandsQueries(c) })
-            .then(({ body }) => body);
-        const calls = chunkCommands(commands, commandsPerRequest)
-            .map(call);
-        return await Promise.all(calls)
-            .then((chunkedPayloads) => {
-            const payloads = mergeBatchPayloads(chunkedPayloads);
-            return handlePayload(payloads);
-        });
-    };
-    return batch;
-};

package/build/module/client/methods/batch.unit.test.d.ts

@@ -1 +0,0 @@
-export {};

package/build/module/client/methods/batch.unit.test.js

@@ -1,466 +0,0 @@
-import got from 'got';
-import range from 'lodash.range';
-import nock from 'nock';
-import { Method } from '../../methods';
-import Batch, { chunkCommands, MAX_COMMANDS_PER_BATCH, mergeBatchPayloads, prepareCommandsQueries } from './batch';
-describe('Client `chunkCommands` method', () => {
-    it('should chunk named commands', () => {
-        const chunkSize = 2;
-        const commands = {
-            a: { method: Method.CRM_DEAL_GET },
-            b: { method: Method.CRM_LEAD_GET },
-            c: { method: Method.CRM_DEAL_LIST },
-            d: { method: Method.CRM_LEAD_LIST },
-            e: { method: Method.CRM_STATUS_LIST }
-        };
-        expect(chunkCommands(commands, chunkSize)).toMatchSnapshot();
-    });
-    it('should chunk array of commands', () => {
-        const chunkSize = 2;
-        const commands = [
-            { method: Method.CRM_DEAL_GET },
-            { method: Method.CRM_LEAD_GET },
-            { method: Method.CRM_DEAL_LIST },
-            { method: Method.CRM_LEAD_LIST },
-            { method: Method.CRM_STATUS_LIST }
-        ];
-        expect(chunkCommands(commands, chunkSize)).toMatchSnapshot();
-    });
-    it('should by default chunk with a size of `MAX_COMMANDS_PER_BATCH`', () => {
-        const commandsSets = 2;
-        const commands = range(0, MAX_COMMANDS_PER_BATCH * commandsSets).map(() => ({
-            method: Method.CRM_DEAL_GET
-        }));
-        const chunked = chunkCommands(commands);
-        expect(chunked.length).toBe(commandsSets);
-        // @ts-ignore we have asserted it length
-        expect(chunked[0].length).toBe(MAX_COMMANDS_PER_BATCH);
-    });
-});
-describe('Client `prepareCommandsQueries` method', () => {
-    it('should transform dict of the commands into the query object', () => {
-        const testDealId = 11111;
-        const commands = {
-            one: {
-                method: Method.CRM_DEAL_GET,
-                params: { ID: testDealId }
-            },
-            two: {
-                method: Method.CRM_DEAL_LIST,
-                params: {
-                    filter: { '>PROBABILITY': 50 },
-                    order: { STAGE_ID: 'ASC' },
-                    select: ['ID', 'TITLE'],
-                    start: 200
-                }
-            }
-        };
-        expect(prepareCommandsQueries(commands)).toMatchSnapshot();
-    });
-    it('should work with numbered commands', () => {
-        const commands = {
-            0: { method: Method.CRM_DEAL_GET },
-            1: { method: Method.CRM_DEAL_LIST }
-        };
-        expect(prepareCommandsQueries(commands)).toMatchSnapshot();
-    });
-    it('should work with array of commands', () => {
-        const commands = [{ method: Method.CRM_DEAL_GET }, { method: Method.CRM_DEAL_LIST }];
-        expect(prepareCommandsQueries(commands)).toMatchSnapshot();
-    });
-    it('should return empty query object when no commands provided', () => {
-        expect(prepareCommandsQueries({})).toMatchSnapshot();
-    });
-});
-describe('Client `mergeBatchPayloads` method', () => {
-    it('should merge named payloads', () => {
-        const batch1 = {
-            result: {
-                result: {
-                    a: [{ ID: '1' }, { ID: '2' }],
-                    b: [{ ID: '3' }, { ID: '4' }]
-                },
-                result_error: [],
-                result_total: { a: 8, b: 8 },
-                result_next: { a: 2, b: 4 },
-                result_time: {
-                    a: {
-                        start: 1567196891.008149,
-                        finish: 1567196891.022234,
-                        duration: 0.014085054397583008,
-                        processing: 0.013998985290527344,
-                        date_start: '2019-08-30T23:28:11+03:00',
-                        date_finish: '2019-08-30T23:28:11+03:00'
-                    },
-                    b: {
-                        start: 1567196891.022316,
-                        finish: 1567196891.03225,
-                        duration: 0.009933948516845703,
-                        processing: 0.009846210479736328,
-                        date_start: '2019-08-30T23:28:11+03:00',
-                        date_finish: '2019-08-30T23:28:11+03:00'
-                    }
-                }
-            },
-            time: {
-                start: 1567196890.959017,
-                finish: 1567196891.223739,
-                duration: 0.2647218704223633,
-                processing: 0.21567082405090332,
-                date_start: '2019-08-30T23:28:10+03:00',
-                date_finish: '2019-08-30T23:28:11+03:00'
-            }
-        };
-        const batch2 = {
-            result: {
-                result: {
-                    c: [{ ID: '5' }, { ID: '6' }],
-                    d: [{ ID: '7' }, { ID: '8' }]
-                },
-                result_error: [],
-                result_total: { c: 8, d: 8 },
-                result_next: { c: 6, d: 8 },
-                result_time: {
-                    c: {
-                        start: 1567196891.032315,
-                        finish: 1567196891.035297,
-                        duration: 0.002981901168823242,
-                        processing: 0.002897024154663086,
-                        date_start: '2019-08-30T23:28:11+03:00',
-                        date_finish: '2019-08-30T23:28:11+03:00'
-                    },
-                    d: {
-                        start: 1567196891.03536,
-                        finish: 1567196891.039936,
-                        duration: 0.004575967788696289,
-                        processing: 0.00450897216796875,
-                        date_start: '2019-08-30T23:28:11+03:00',
-                        date_finish: '2019-08-30T23:28:11+03:00'
-                    }
-                }
-            },
-            time: {
-                start: 1567196890.959517,
-                finish: 1567196891.225739,
-                duration: 0.2647218704223683,
-                processing: 0.21567082405090832,
-                date_start: '2019-08-30T23:28:12+03:00',
-                date_finish: '2019-08-30T23:28:13+03:00'
-            }
-        };
-        expect(mergeBatchPayloads([batch1, batch2])).toMatchSnapshot();
-    });
-    it('should merge array payloads', () => {
-        const batch1 = {
-            result: {
-                result: [
-                    [{ ID: '1' }, { ID: '2' }],
-                    [{ ID: '3' }, { ID: '4' }]
-                ],
-                result_error: [],
-                result_total: [8, 8],
-                result_next: [2, 4],
-                result_time: [
-                    {
-                        start: 1567196891.008149,
-                        finish: 1567196891.022234,
-                        duration: 0.014085054397583008,
-                        processing: 0.013998985290527344,
-                        date_start: '2019-08-30T23:28:11+03:00',
-                        date_finish: '2019-08-30T23:28:11+03:00'
-                    },
-                    {
-                        start: 1567196891.022316,
-                        finish: 1567196891.03225,
-                        duration: 0.009933948516845703,
-                        processing: 0.009846210479736328,
-                        date_start: '2019-08-30T23:28:11+03:00',
-                        date_finish: '2019-08-30T23:28:11+03:00'
-                    }
-                ]
-            },
-            time: {
-                start: 1567196890.959017,
-                finish: 1567196891.223739,
-                duration: 0.2647218704223633,
-                processing: 0.21567082405090332,
-                date_start: '2019-08-30T23:28:10+03:00',
-                date_finish: '2019-08-30T23:28:11+03:00'
-            }
-        };
-        const batch2 = {
-            result: {
-                result: [
-                    [{ ID: '5' }, { ID: '6' }],
-                    [{ ID: '7' }, { ID: '8' }]
-                ],
-                result_error: [],
-                result_total: [8, 8],
-                result_next: [6, 8],
-                result_time: [
-                    {
-                        start: 1567196891.032315,
-                        finish: 1567196891.035297,
-                        duration: 0.002981901168823242,
-                        processing: 0.002897024154663086,
-                        date_start: '2019-08-30T23:28:11+03:00',
-                        date_finish: '2019-08-30T23:28:11+03:00'
-                    },
-                    {
-                        start: 1567196891.03536,
-                        finish: 1567196891.039936,
-                        duration: 0.004575967788696289,
-                        processing: 0.00450897216796875,
-                        date_start: '2019-08-30T23:28:11+03:00',
-                        date_finish: '2019-08-30T23:28:11+03:00'
-                    }
-                ]
-            },
-            time: {
-                start: 1567196890.959517,
-                finish: 1567196891.225739,
-                duration: 0.2647218704223683,
-                processing: 0.21567082405090832,
-                date_start: '2019-08-30T23:28:12+03:00',
-                date_finish: '2019-08-30T23:28:13+03:00'
-            }
-        };
-        expect(mergeBatchPayloads([batch1, batch2])).toMatchSnapshot();
-    });
-});
-const TEST_URI = 'https://test.com/rest';
-const batch = Batch({
-    get: got.extend({ prefixUrl: TEST_URI, responseType: 'json' }).get
-});
-const RESPONSE_200 = 200;
-describe('Client `batch` method', () => {
-    it('should form a proper request', async () => {
-        const payload = { result: { result: { one: 'done', two: 'done' }, result_error: [] } };
-        const dealId = 999;
-        const commands = {
-            one: { method: Method.CRM_DEAL_GET, params: { ID: dealId } },
-            two: { method: Method.CRM_DEAL_LIST }
-        };
-        const scope = nock(TEST_URI)
-            // @todo We'd want to use `query` object here as it is much more readable, but nock for some reason
-            //       fails to match request when it contains `cmd[someName]`. The issue definitely connected
-            //       to the `[]`, since it does not appear when only one bracket present
-            .get(`/${Method.BATCH}?cmd%5Bone%5D=${commands.one.method}%3FID%3D${dealId}&cmd%5Btwo%5D=${commands.two.method}`)
-            .reply(RESPONSE_200, payload);
-        await batch(commands);
-        expect(scope.done()).toBe(undefined);
-    });
-    it('should form a proper request with numbered commands', async () => {
-        const payload = { result: { result: ['done1', 'done2'], result_error: [] } };
-        const dealId = 999;
-        const commands = {
-            0: { method: Method.CRM_DEAL_GET, params: { ID: dealId } },
-            1: { method: Method.CRM_DEAL_LIST }
-        };
-        const scope = nock(TEST_URI)
-            // @todo We'd want to use `query` object here as it is much more readable, but nock for some reason
-            //       fails to match request when it contains `cmd[someName]`. The issue definitely
-            //       connected to the `[]` since it does not appear when only one bracket present
-            .get(`/${Method.BATCH}?cmd%5B0%5D=${commands[0].method}%3FID%3D${dealId}&cmd%5B1%5D=${commands[1].method}`)
-            .reply(RESPONSE_200, payload);
-        await batch(commands);
-        expect(scope.done()).toBe(undefined);
-    });
-    it('should form a proper request with array of commands', async () => {
-        const payload = { result: { result: ['done1', 'done2'], result_error: [] } };
-        const dealId = 999;
-        const commands = [
-            { method: Method.CRM_DEAL_GET, params: { ID: dealId } },
-            { method: Method.CRM_DEAL_LIST }
-        ];
-        const scope = nock(TEST_URI)
-            // @todo We'd want to use `query` object here as it is much more readable, but nock for some reason
-            //       fails to match request when it contains `cmd[someName]`. The issue definitely
-            //       connected to the `[]` since it does not appear when only one bracket present
-            .get(`/${Method.BATCH}?cmd%5B0%5D=${commands[0].method}%3FID%3D${dealId}&cmd%5B1%5D=${commands[1].method}`)
-            .reply(RESPONSE_200, payload);
-        await batch(commands);
-        expect(scope.done()).toBe(undefined);
-    });
-    it('should form a proper requests with more than max allowed commands per batch', async () => {
-        const maxCommandsPerBatch = 1;
-        const payload = { result: { result: { one: 'done', two: 'done' }, result_error: [] } };
-        const dealId = 999;
-        const commands = {
-            one: { method: Method.CRM_DEAL_GET, params: { ID: dealId } },
-            two: { method: Method.CRM_DEAL_LIST }
-        };
-        const scope = nock(TEST_URI)
-            // @todo We'd want to use `query` object here as it is much more readable, but nock for some reason
-            //       fails to match request when it contains `cmd[someName]`. The issue definitely connected
-            //       to the `[]`, since it does not appear when only one bracket present
-            .get(`/${Method.BATCH}?cmd%5Bone%5D=${commands.one.method}%3FID%3D${dealId}`)
-            .reply(RESPONSE_200, payload)
-            .get(`/${Method.BATCH}?cmd%5Btwo%5D=${commands.two.method}`)
-            .reply(RESPONSE_200, payload);
-        await batch(commands, maxCommandsPerBatch);
-        expect(scope.done()).toBe(undefined);
-    });
-    it('should form a proper request with more than allowed max numbered commands', async () => {
-        const maxCommandsPerBatch = 1;
-        const payload = { result: { result: ['done1', 'done2'], result_error: [] } };
-        const dealId = 999;
-        const commands = {
-            0: { method: Method.CRM_DEAL_GET, params: { ID: dealId } },
-            1: { method: Method.CRM_DEAL_LIST }
-        };
-        const scope = nock(TEST_URI)
-            // @todo We'd want to use `query` object here as it is much more readable, but nock for some reason
-            //       fails to match request when it contains `cmd[someName]`. The issue definitely
-            //       connected to the `[]` since it does not appear when only one bracket present
-            .get(`/${Method.BATCH}?cmd%5B0%5D=${commands[0].method}%3FID%3D${dealId}`)
-            .reply(RESPONSE_200, payload)
-            .get(`/${Method.BATCH}?cmd%5B1%5D=${commands[1].method}`)
-            .reply(RESPONSE_200, payload);
-        await batch(commands, maxCommandsPerBatch);
-        expect(scope.done()).toBe(undefined);
-    });
-    it('should form a proper request with more than max allowed array of commands', async () => {
-        const maxCommandsPerBatch = 1;
-        const payload = { result: { result: ['done1', 'done2'], result_error: [] } };
-        const dealId = 999;
-        const commands = [
-            { method: Method.CRM_DEAL_GET, params: { ID: dealId } },
-            { method: Method.CRM_DEAL_LIST }
-        ];
-        const scope = nock(TEST_URI)
-            // @todo We'd want to use `query` object here as it is much more readable, but nock for some reason
-            //       fails to match request when it contains `cmd[someName]`. The issue definitely
-            //       connected to the `[]` since it does not appear when only one bracket present
-            .get(`/${Method.BATCH}?cmd%5B0%5D=${commands[0].method}%3FID%3D${dealId}`)
-            .reply(RESPONSE_200, payload)
-            .get(`/${Method.BATCH}?cmd%5B0%5D=${commands[1].method}`)
-            .reply(RESPONSE_200, payload);
-        await batch(commands, maxCommandsPerBatch);
-        expect(scope.done()).toBe(undefined);
-    });
-    it('should return body as payload', async () => {
-        const commands = {
-            0: { method: Method.CRM_DEAL_GET },
-            1: { method: Method.CRM_DEAL_LIST }
-        };
-        const payload = {
-            result: {
-                result: ['done'],
-                result_error: [],
-                result_next: [],
-                result_time: [],
-                result_total: []
-            },
-            time: {}
-        };
-        nock(TEST_URI)
-            // @todo We'd want to use `query` object here as it is much more readable, but nock for some reason
-            //       fails to match request when it contains `cmd[someName]`. The issue definitely connected
-            //       to the `[]`, since it does not appear when only one bracket present
-            .get(`/${Method.BATCH}?cmd%5B0%5D=${commands[0].method}&cmd%5B1%5D=${commands[1].method}`)
-            .reply(RESPONSE_200, payload);
-        expect(await batch(commands)).toEqual(payload);
-    });
-    // @todo Test this also for named and numbered commands
-    it('should merge payloads when more then max allowed commands provided', async () => {
-        const maxAllowedCommands = 1;
-        const commands = {
-            0: { method: Method.CRM_DEAL_GET },
-            1: { method: Method.CRM_DEAL_LIST }
-        };
-        const payload1 = {
-            result: {
-                result: ['done1'],
-                result_error: [],
-                result_next: [1],
-                result_time: [],
-                result_total: [2]
-            },
-            time: {}
-        };
-        const payload2 = {
-            result: {
-                result: ['done2'],
-                result_error: [],
-                result_next: [2],
-                result_time: [],
-                result_total: [2]
-            },
-            time: {}
-        };
-        nock(TEST_URI)
-            // @todo We'd want to use `query` object here as it is much more readable, but nock for some reason
-            //       fails to match request when it contains `cmd[someName]`. The issue definitely connected
-            //       to the `[]`, since it does not appear when only one bracket present
-            .get(`/${Method.BATCH}?cmd%5B0%5D=${commands[0].method}`)
-            .reply(RESPONSE_200, payload1)
-            .get(`/${Method.BATCH}?cmd%5B1%5D=${commands[1].method}`)
-            .reply(RESPONSE_200, payload2);
-        expect(await batch(commands, maxAllowedCommands)).toMatchSnapshot();
-    });
-    it.todo('should cast payload to the <P>');
-    it('should throw when getting errors in payload', () => {
-        const payload = {
-            result: {
-                result: { one: 'done', two: 'done' },
-                result_error: {
-                    one: 'Expected error from numbered `batch` one',
-                    two: 'Expected error from numbered `batch` two'
-                }
-            }
-        };
-        const commands = {
-            one: { method: Method.CRM_DEAL_GET },
-            two: { method: Method.CRM_DEAL_LIST }
-        };
-        nock(TEST_URI)
-            // @todo We'd want to use `query` object here as it is much more readable, but nock for some reason
-            //       fails to match request when it contains `cmd[someName]`. The issue definitely connected
-            //       to the `[]`, since it does not appear when only one bracket present
-            .get(`/${Method.BATCH}?cmd%5Bone%5D=${commands.one.method}&cmd%5Btwo%5D=${commands.two.method}`)
-            .reply(RESPONSE_200, payload);
-        return expect(batch(commands)).rejects.toMatchSnapshot();
-    });
-    it('should throw when getting errors in numbered commands payload', () => {
-        const payload = {
-            result: {
-                result: ['done'],
-                result_error: [
-                    'Expected error from numbered `batch` 0',
-                    'Expected error from numbered `batch` 1'
-                ]
-            }
-        };
-        const commands = {
-            0: { method: Method.CRM_DEAL_GET },
-            1: { method: Method.CRM_DEAL_LIST }
-        };
-        nock(TEST_URI)
-            // @todo We'd want to use `query` object here as it is much more readable, but nock for some reason
-            //       fails to match request when it contains `cmd[someName]`. The issue definitely connected
-            //       to the `[]`, since it does not appear when only one bracket present
-            .get(`/${Method.BATCH}?cmd%5B0%5D=${commands[0].method}&cmd%5B1%5D=${commands[1].method}`)
-            .reply(RESPONSE_200, payload);
-        return expect(batch(commands)).rejects.toMatchSnapshot();
-    });
-    it('should throw when getting errors in array of commands payload', () => {
-        const payload = {
-            result: {
-                result: ['done'],
-                result_error: [
-                    'Expected error from array of command `batch` 0',
-                    'Expected error from array of command `batch` 1'
-                ]
-            }
-        };
-        const commands = [{ method: Method.CRM_DEAL_GET }, { method: Method.CRM_DEAL_LIST }];
-        nock(TEST_URI)
-            // @todo We'd want to use `query` object here as it is much more readable, but nock for some reason
-            //       fails to match request when it contains `cmd[someName]`. The issue definitely connected
-            //       to the `[]`, since it does not appear when only one bracket present
-            .get(`/${Method.BATCH}?cmd%5B0%5D=${commands[0].method}&cmd%5B1%5D=${commands[1].method}`)
-            .reply(RESPONSE_200, payload);
-        return expect(batch(commands)).rejects.toMatchSnapshot();
-    });
-});

package/build/module/client/methods/call.d.ts

@@ -1,17 +0,0 @@
-import got from 'got';
-import { Method, MethodParams, MethodPayload } from '../../methods';
-import { Payload } from '../../payloads';
-/**
- * Checks wether payload have any errors and if it does — throws them
- * Bitrix payload types do not provide discriminators, so we're forced to type cast them
- */
-export declare const handlePayload: <P extends Payload<unknown>>(payload: P) => P;
-export declare type Call = <M extends Method>(method: M, params: MethodParams<M>) => Promise<MethodPayload<M>>;
-declare type Dependencies = {
-    readonly get: typeof got.get;
-};
-/**
- * Dispatches a request with specified method and params. Will fill figure out payload type based on the Method
- */
-declare const _default: ({ get }: Dependencies) => Call;
-export default _default;