$param was not being used in mockResponse() #12882

Merged
merged 1 commit into from Jan 18, 2017

Projects

None yet

3 participants

@CViniciusSDias CViniciusSDias $param was not being used in mockResponse()
There was a parameter ($param) not being used in the mockResponse() method.
410fa2a
@codecov-io

Current coverage is 54.21% (diff: 100%)

Merging #12882 into master will not change coverage

@@             master     #12882   diff @@
==========================================
  Files           465        465          
  Lines         69647      69647          
  Methods        2128       2128          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits          37758      37758          
  Misses        31889      31889          
  Partials          0          0          

Powered by Codecov. Last update b40e9c1...410fa2a

@nijel nijel merged commit 9b490f7 into phpmyadmin:master Jan 18, 2017

3 of 4 checks passed

Scrutinizer Errored
Details
codecov/patch Coverage not affected when comparing b40e9c1...410fa2a
Details
codecov/project 54.21% (+0.00%) compared to b40e9c1
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@nijel
Member
nijel commented Jan 18, 2017

Thanks, indeed it's not needed since we use func_get_args().

@nijel nijel self-assigned this Jan 18, 2017
@nijel nijel added this to the 4.7.0 milestone Jan 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment