fix(lib/provider.js): fix RecurlyProvider throwing error when fraud property is set

[mikeedwards]

When we set the fraud property on RecurlyProvider like so:

fraud={{kount: {dataCollector: true}}}

We get this error:

Error name:    "TypeError"
Error message: "Cannot read property 'get' of undefined"

in node_modules/recurly.js/lib/recurly/fraud.js:63:26

  attachDataCollector () {
    if (this.dataCollectorInitiated) return;
    this.dataCollectorInitiated = true;

    this.recurly.request.get({
      route: '/risk/info',
      done: (error, response) => {
        debug('risk info', error, response);

The problem seems to be with the way that recurly's constructor works—if you provide options in the constructor, then the configure event fires and the call to risk/info fails without a Request object being set yet. The way the recurly instance is set up here in this PR is more like your e2e tests in the recurly-js repo itself, so configuration happens after everything gets set up, including the request property.

[coveralls]

Pull Request Test Coverage Report for Build 690

Details

• 2 of 2 (100.0%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.004%) to 98.944%

---

Totals
Change from base Build 689:	0.004%
Covered Lines:	221
Relevant Lines:	221

---

💛 - Coveralls

[chrissrogers]

I think this is a good stopgap solution to the issue occurring in react-recurly and have made a note to investigate this is recurly-js as well. Thanks very much for the pull request!

[chrissrogers]

Thanks as well for these updates 👍