diff --git a/integration-tests/data/allTestsCases.ts b/integration-tests/data/allTestsCases.ts index 077697fedd..94e31fdede 100644 --- a/integration-tests/data/allTestsCases.ts +++ b/integration-tests/data/allTestsCases.ts @@ -48,85 +48,89 @@ interface TestCase { expected?: object; } -export const commonCases: TestCase[] = [ +export const nairobiCases: TestCase[] = [ + { - name: 'Delegation', + name: 'Set deposits limit 1000000', operation: { branch: 'BLzyjjHKEKMULtvkpSHxuZxx6ei6fpntH2BTkYZiLgs8zLVstvX', contents: [ { - kind: OpKind.DELEGATION, - delegate: 'tz1QZ6KY7d3BuZDT1d19dUxoQrtFPN2QJ3hn', + kind: OpKind.SET_DEPOSITS_LIMIT, counter: '1', source: 'tz1QZ6KY7d3BuZDT1d19dUxoQrtFPN2QJ3hn', fee: '10000', gas_limit: '10', storage_limit: '10', + limit: '1000000', }, ], }, }, { - name: 'Reveal', + name: 'Unset deposits limit', operation: { branch: 'BLzyjjHKEKMULtvkpSHxuZxx6ei6fpntH2BTkYZiLgs8zLVstvX', contents: [ { - kind: OpKind.REVEAL, + kind: OpKind.SET_DEPOSITS_LIMIT, counter: '1', source: 'tz1QZ6KY7d3BuZDT1d19dUxoQrtFPN2QJ3hn', - public_key: 'edpkvS5QFv7KRGfa3b87gg9DBpxSm3NpSwnjhUjNBQrRUUR66F7C9g', fee: '10000', gas_limit: '10', - storage_limit: '10', + storage_limit: '10' }, ], }, }, +] + +export const commonCases: TestCase[] = [ { - name: 'Ballot', + name: 'Delegation', operation: { branch: 'BLzyjjHKEKMULtvkpSHxuZxx6ei6fpntH2BTkYZiLgs8zLVstvX', contents: [ { - kind: OpKind.BALLOT, + kind: OpKind.DELEGATION, + delegate: 'tz1QZ6KY7d3BuZDT1d19dUxoQrtFPN2QJ3hn', + counter: '1', source: 'tz1QZ6KY7d3BuZDT1d19dUxoQrtFPN2QJ3hn', - period: -300, - ballot: 'yay', - proposal: 'PsCARTHAGazKbHtnKfLzQg3kms52kSRpgnDY982a9oYsSXRLQEb', + fee: '10000', + gas_limit: '10', + storage_limit: '10', }, ], }, }, { - name: 'Set deposits limit 1000000', + name: 'Reveal', operation: { branch: 'BLzyjjHKEKMULtvkpSHxuZxx6ei6fpntH2BTkYZiLgs8zLVstvX', contents: [ { - kind: OpKind.SET_DEPOSITS_LIMIT, + kind: OpKind.REVEAL, counter: '1', source: 'tz1QZ6KY7d3BuZDT1d19dUxoQrtFPN2QJ3hn', + public_key: 'edpkvS5QFv7KRGfa3b87gg9DBpxSm3NpSwnjhUjNBQrRUUR66F7C9g', fee: '10000', gas_limit: '10', storage_limit: '10', - limit: '1000000', }, ], }, }, { - name: 'Unset deposits limit', + name: 'Ballot', operation: { branch: 'BLzyjjHKEKMULtvkpSHxuZxx6ei6fpntH2BTkYZiLgs8zLVstvX', contents: [ { - kind: OpKind.SET_DEPOSITS_LIMIT, - counter: '1', + kind: OpKind.BALLOT, source: 'tz1QZ6KY7d3BuZDT1d19dUxoQrtFPN2QJ3hn', - fee: '10000', - gas_limit: '10', - storage_limit: '10' + period: -300, + ballot: 'yay', + proposal: 'PsCARTHAGazKbHtnKfLzQg3kms52kSRpgnDY982a9oYsSXRLQEb', }, ], }, diff --git a/integration-tests/local-forging.spec.ts b/integration-tests/local-forging.spec.ts index 228bac63b7..f8404a838c 100644 --- a/integration-tests/local-forging.spec.ts +++ b/integration-tests/local-forging.spec.ts @@ -1,12 +1,23 @@ import { CONFIGS } from "./config"; -import { commonCases } from './data/allTestsCases'; +import { commonCases, nairobiCases } from './data/allTestsCases'; import { LocalForger, ProtocolsHash } from '@taquito/local-forging' -import { TezosToolkit } from "@taquito/taquito"; +import { Protocols, TezosToolkit } from "@taquito/taquito"; CONFIGS().forEach(({ rpc, protocol }) => { const Tezos = new TezosToolkit(rpc); + const nairobinet = protocol === Protocols.PtNairobi ? it : it.skip; describe(`Test local forger: ${rpc}`, () => { + nairobiCases.forEach(({ name, operation }) => { + nairobinet(`Verify that .forge for local forge will return same result as for network forge for rpc: ${name} (${rpc})`, async done => { + const localForger = new LocalForger(protocol as unknown as ProtocolsHash); + const result = await localForger.forge(operation); + const rpcResult = await Tezos.rpc.forgeOperations(operation); + expect(result).toEqual(rpcResult); + + done(); + }); + }); // all protocols commonCases.forEach(({ name, operation, expected }) => { it(`Verify that .forge for local forge will return same result as for network forge for rpc: ${name} (${rpc})`, async done => { diff --git a/packages/taquito-local-forging/src/codec.ts b/packages/taquito-local-forging/src/codec.ts index 175f4d05e6..af5b9eb001 100644 --- a/packages/taquito-local-forging/src/codec.ts +++ b/packages/taquito-local-forging/src/codec.ts @@ -501,10 +501,6 @@ export const burnLimitDecoder = (value: Uint8ArrayConsumer) => { } }; -export const depositsLimitEncoder = (val: string) => { - return !val ? '00' : `ff${zarithEncoder(val)}`; -}; - export const depositsLimitDecoder = (value: Uint8ArrayConsumer) => { const prefix = value.consume(1); if (Buffer.from(prefix).toString('hex') !== '00') { diff --git a/packages/taquito-local-forging/src/encoder.ts b/packages/taquito-local-forging/src/encoder.ts index 3e23e11686..917f2ee26b 100644 --- a/packages/taquito-local-forging/src/encoder.ts +++ b/packages/taquito-local-forging/src/encoder.ts @@ -15,7 +15,6 @@ import { proposalEncoder, proposalsEncoder, publicKeyEncoder, - depositsLimitEncoder, pvmKindEncoder, smartContractAddressEncoder, smartRollupAddressEncoder, @@ -49,7 +48,6 @@ import { TransferTicketSchema, TxRollupOriginationSchema, TxRollupSubmitBatchSchema, - SetDepositsLimitSchema, SmartRollupOriginateSchema, SmartRollupExecuteOutboxMessageSchema, SmartRollupAddMessagesSchema, @@ -85,7 +83,6 @@ export const encoders: { [key: string]: Encoder } = { [CODEC.TX_ROLLUP_ID]: txRollupIdEncoder, [CODEC.TX_ROLLUP_BATCH_CONTENT]: txRollupBatchContentEncoder, [CODEC.BURN_LIMIT]: burnLimitEncoder, - [CODEC.DEPOSITS_LIMIT]: depositsLimitEncoder, [CODEC.PVM_KIND]: pvmKindEncoder, [CODEC.PADDED_BYTES]: paddedBytesEncoder, [CODEC.SMART_ROLLUP_MESSAGE]: smartRollupMessageEncoder, @@ -122,6 +119,4 @@ encoders[CODEC.OP_SMART_ROLLUP_ADD_MESSAGES] = (val: any) => encoders[CODEC.OP_SMART_ROLLUP_EXECUTE_OUTBOX_MESSAGE] = (val: any) => schemaEncoder(encoders)(SmartRollupExecuteOutboxMessageSchema)(val); encoders[CODEC.MANAGER] = schemaEncoder(encoders)(ManagerOperationSchema); -encoders[CODEC.OP_SET_DEPOSITS_LIMIT] = (val) => - schemaEncoder(encoders)(SetDepositsLimitSchema)(val); encoders[CODEC.OP_FAILING_NOOP] = (val: any) => schemaEncoder(encoders)(FailingNoopSchema)(val); diff --git a/packages/taquito-local-forging/src/taquito-local-forging.ts b/packages/taquito-local-forging/src/taquito-local-forging.ts index 49b51422b3..da0cf13e03 100644 --- a/packages/taquito-local-forging/src/taquito-local-forging.ts +++ b/packages/taquito-local-forging/src/taquito-local-forging.ts @@ -58,8 +58,6 @@ export class LocalForger implements Forger { continue; } else if (content.kind === 'transaction' && diff[0] === 'parameters') { continue; - } else if (content.kind === 'set_deposits_limit' && diff[0] === 'limit') { - continue; } else if ( content.kind === ('tx_rollup_submit_batch' as unknown) && diff[0] === 'burn_limit'