Merge pull request #335 from StakerDAO/sane-defaults

Add Gas and Storage Limit Defaults for `WXTZ` Operations
Cryptonomic · Dec 1, 2020 · caaa22a · caaa22a
2 parents a41b1d7 + d359d4c
commit caaa22a
Showing 1 changed file with 16 additions and 16 deletions.
diff --git a/src/chain/tezos/contracts/WrappedTezosHelper.ts b/src/chain/tezos/contracts/WrappedTezosHelper.ts
@@ -192,8 +192,8 @@ export namespace WrappedTezosHelper {
         sourceAddress: string,
         destinationAddress: string,
         amount: number,
-        gasLimit: number,
-        storageLimit: number
+        gasLimit: number = 51_300,
+        storageLimit: number = 70
     ): Promise<string> {
         const parameters = `Pair "${sourceAddress}" (Pair "${destinationAddress}" ${amount})`;
 
@@ -237,8 +237,8 @@ export namespace WrappedTezosHelper {
         ovenAddress: string,
         fee: number,
         amountMutez: number,
-        gasLimit: number,
-        storageLimit: number
+        gasLimit: number = 126_500,
+        storageLimit: number = 10
     ): Promise<string> {
         const parameters = 'Unit'
 
@@ -284,8 +284,8 @@ export namespace WrappedTezosHelper {
         ovenAddress: string,
         fee: number,
         amountMutez: number,
-        gasLimit: number,
-        storageLimit: number
+        gasLimit: number = 121_000,
+        storageLimit: number = 0
     ): Promise<string> {
         const parameters = `${amountMutez}`
 
@@ -380,8 +380,8 @@ export namespace WrappedTezosHelper {
         keystore: KeyStore,
         fee: number,
         coreAddress: string,
-        gasLimit: number,
-        storageLimit: number
+        gasLimit: number = 115_000,
+        storageLimit: number = 1100
     ): Promise<OpenOvenResult> {
         const entryPoint = 'runEntrypointLambda'
         const lambdaName = 'createOven'
@@ -419,21 +419,21 @@ export namespace WrappedTezosHelper {
      * @param signer A Signer for the sourceAddress.
      * @param keystore A Keystore for the sourceAddress.
      * @param fee The fee to use.
-     * @param gasLimit The gas limit to use.
-     * @param storageLimit The storage limit to use. 
      * @param ovenAddress The address of the oven contract. 
      * @param bakerAddress The address of the baker for the oven.
+     * @param gasLimit The gas limit to use.
+     * @param storageLimit The storage limit to use. 
      * @returns A string representing the operation hash.
      */
     export async function setOvenBaker(
         nodeUrl: string,
         signer: Signer,
         keystore: KeyStore,
         fee: number,
-        gasLimit: number,
-        storageLimit: number,
         ovenAddress: string,
-        bakerAddress: string
+        bakerAddress: string,
+        gasLimit: number = 19_500,
+        storageLimit: number = 0,
     ): Promise<string> {
         const parameters = `Some "${bakerAddress}"`
 
@@ -464,19 +464,19 @@ export namespace WrappedTezosHelper {
      * @param signer A Signer for the sourceAddress.
      * @param keystore A Keystore for the sourceAddress.
      * @param fee The fee to use.
+     * @param ovenAddress The address of the oven contract. 
      * @param gasLimit The gas limit to use.
      * @param storageLimit The storage limit to use. 
-     * @param ovenAddress The address of the oven contract. 
      * @returns A string representing the operation hash.
      */
     export async function clearOvenBaker(
         nodeUrl: string,
         signer: Signer,
         keystore: KeyStore,
         fee: number,
-        gasLimit: number,
-        storageLimit: number,
         ovenAddress: string,
+        gasLimit: number = 19_500,
+        storageLimit: number = 0,
     ): Promise<string> {
         const parameters = `None`