Using eth-gas-reporter with OpenZeppelin SDK proxies

Archive SDK
1

Did anyone here were able to use eth-gas-reporter in a project using openzeppelin SDK proxies?

Their documentation mentions that using proxies introduces an issue for resolving names, which I do confirm: https://github.com/cgewecke/eth-gas-reporter/blob/master/docs/advanced.md#resolving-method-identities-when-using-proxy-contracts

But there is no solution out-of-box apparently. Does anyone have a walkthrough for making it work?

1 Like
2

Hi @wmitsuda,

I tried using eth-gas-reporter with truffle test and a very simple contract and it appeared to work. (see my contract, tests and output below)
What errors do you get?

A work around could be to test the contracts just as logic contracts, (though this doesn’t take into account the overhead of using a Proxy).

Counter.sol

pragma solidity ^0.5.0;

contract Counter {
  uint256 public value;

  function increase() public {
    value++;
  }
}

Counter.test.js

const { TestHelper } = require('@openzeppelin/cli');
const { Contracts, ZWeb3 } = require('@openzeppelin/upgrades');
const {
  BN,           // Big Number support
  constants,    // Common constants, like the zero address and largest integers
  expectEvent,  // Assertions for emitted events
  expectRevert, // Assertions for transactions that should fail
} = require('@openzeppelin/test-helpers');


ZWeb3.initialize(web3.currentProvider);

const Counter = Contracts.getFromLocal('Counter');

require('chai').should();

contract('Counter (upgradeable)', function () {

  beforeEach(async function () {
    this.project = await TestHelper();
    this.proxy = await this.project.createProxy(Counter);
  })

  it('should have a value', async function () {
    const result = await this.proxy.methods.value().call();
    result.should.be.bignumber.equal('0');
  })

  it('should increase value', async function () {
    await this.proxy.methods.increase().send();
    const result = await this.proxy.methods.value().call();
    result.should.be.bignumber.equal('1');
  })
})

Counter.logic.test.js

const {
  BN,           // Big Number support
  constants,    // Common constants, like the zero address and largest integers
  expectEvent,  // Assertions for emitted events
  expectRevert, // Assertions for transactions that should fail
} = require('@openzeppelin/test-helpers');

const Counter = artifacts.require('Counter');

require('chai').should();

contract('Counter (logic)', function () {

  beforeEach(async function () {
    this.counter = await Counter.new();
  })

  it('should have a value', async function () {
    const result = await this.counter.value();
    result.should.be.bignumber.equal('0');
  })

  it('should increase value', async function () {
    await this.counter.increase();
    const result = await this.counter.value();
    result.should.be.bignumber.equal('1');
  })
})

Truffle Test

$ truffle test
Using network 'test'.


Compiling your contracts...
===========================
> Everything is up to date, there is nothing to compile.


  Contract: Counter (logic)
    ✓ should have a value
    ✓ should increase value (41653 gas)

  Contract: Counter (upgradeable)
    ✓ should have a value
    ✓ should increase value (43218 gas)

·----------------------------------------|----------------------------|-------------|----------------------------·
|  Solc version: 0.5.12+commit.7709ece9  ·  Optimizer enabled: false  ·  Runs: 200  ·  Block limit: 6721975 gas  │
·········································|····························|·············|·····························
|  Methods                               ·                1 gwei/gas                ·       124.42 eur/eth       │
·····················|···················|··············|·············|·············|··············|··············
|  Contract          ·  Method           ·  Min         ·  Max        ·  Avg        ·  # calls     ·  eur (avg)  │
·····················|···················|··············|·············|·············|··············|··············
|  Counter           ·  increase         ·       41653  ·      43218  ·      42436  ·           2  ·       0.01  │
·····················|···················|··············|·············|·············|··············|··············
|  Deployments                           ·                                          ·  % of limit  ·             │
·········································|··············|·············|·············|··············|··············
|  Counter                               ·           -  ·          -  ·      99651  ·       1.5 %  ·       0.01  │
·----------------------------------------|--------------|-------------|-------------|--------------|-------------·

  4 passing (13s)
3

That’s weird… I’m using buidler, it seems it does not identify any function call, see:

·------------------------|----------------------------|-------------|----------------------------·
|  Solc version: 0.5.11  ·  Optimizer enabled: false  ·  Runs: 200  ·  Block limit: 6721975 gas  │
·························|····························|·············|·····························
|  Methods               ·                1 gwei/gas                ·       130.40 eur/eth       │
··············|··········|··············|·············|·············|·············|···············
|  Contract   ·  Method  ·  Min         ·  Max        ·  Avg        ·  # calls    ·  eur (avg)   │
·-------------|----------|--------------|-------------|-------------|-------------|--------------·

  16 passing (29s)

I’ll try to debug a little more and narrow the issue…

1 Like
4

Hi @wmitsuda,

It could be that my very simple contract was too simple. You could try my simple contract on builder and see if you get the same results.

1 Like
5

I’ve used your example and created an issue on buidler-gas-reporter github: https://github.com/cgewecke/buidler-gas-reporter/issues/12

1 Like
6

A post was split to a new topic: Eth-gas-reporter with proxies