Converted the rest of the tests to es6

[dacarley]

No description provided.

[coveralls]

Coverage increased (+0.08%) to 85.006% when pulling f0efdef on blitzpredict:feature/es6-all-tests into 6a133ab on ethereum:1.0ES6.

[frozeman]

@dacarley please change mocha reporters to dot, it gives better overview errors show the actual problems anyway.

You have one test failing:

  1) web3-providers-http
       send
         should send basic async request:

      AssertionError: expected 'undefined' to equal 'object'
      + expected - actual

      -undefined
      +object

[coveralls]

Coverage increased (+0.07%) to 85.0% when pulling e5b05f4 on blitzpredict:feature/es6-all-tests into 6a133ab on ethereum:1.0ES6.

[dacarley]

I switched to the "dot" reporter for Mocha, and removed a stray "console.error" that shouldn't have been in the code at all.

I don't see the test failure you're referring to, either on my machine, or in Travis.

What node and npm version are you using when you see the failure?

[frozeman]

I use node 6 I think. Not on my comp to check.

[frozeman]

I will try tomorrow again. Thanks for the great work!

[dacarley]

I just tried a fresh enlistment on Node 6.11.5 / NPM 3.10.10, and all is well.

[frozeman]

There seems to be a left over in mocha, because when i run it standalone the test passes: mocha test/httpprovider.js.

I removed the node modules folder and re-run all and now it passes :) Thanks.

Currently the listing is not running, you removed that from the npm test call? Is it because it wouldn't pass yet?

[dacarley]

I'm not sure what you mean by "the listing is not running"?

[frozeman]

Linting

[dacarley]

No, that was an oversight on my part -- I'll address that in my next PR.