mgid adapter: native support, minor changes

[Gaudeamus]

Type of change

• Feature
• Refactoring (no functional changes, no api changes)

Description of change

native support added, added currency and bidfloor optional parameters

• test parameters for validating bids

var adUnits = [{
  code: 'div-prebid',
  mediaTypes: {
    native: {
        image: {
            sendId: true,
            required: true,
            sizes: [80, 80]
        },
        clickUrl: {
            sendId: true,
            required: false
        },
        title: {
            required: true,
            len: 80
        },
        sponsored: {
            required: false
        }
    }
  },
  // Replace this object to test a new Adapter!
  bids: [{
    bidder: 'mgid',
    params : {
        accountId : "233", //test accountId, please replace after test
        placementId : "350971" // 300x250 test placementId, please replace after test
    }
  }]
}];

Be sure to test the integration with your adserver using the Hello World sample page.

• contact email of the adapter’s maintainer
  prebid@mgid.com

• official adapter submission

For any changes that affect user-facing APIs or example code documented on http://prebid.org, please provide:

• A link to a PR on the docs repo at https://github.com/prebid/prebid.github.io/
  native support & some optional params added prebid.github.io#1283

Other information

[mkendall07]

This pull request introduces 1 alert when merging a24a528 into 253cbf4 - view on LGTM.com

new alerts:

• 1 for Unneeded defensive code

---

Comment posted by LGTM.com

[Gaudeamus]

Hello, @mkendall07, please revise

[sumit116]

Hi,
Thanks for submitting your adaptor. I tested your adaptor on a demo page and found that functionality is working well and I am getting valid response for my request. But I also found that test coverage of the adaptor is below 80% (its 74.44% ). Can you please add missing test cases. Please Let me know if you have any questions.

[Gaudeamus]

Hi @sumit116, please review the last commit

[sumit116]

Tested the changes on hello_world page and all unit tests have passed. Test coverage of the adaptor is above 80%. The changes are working well against all parameters listed in https://github.com/prebid/Prebid.js/blob/master/PR_REVIEW.md, hence approving the PR.

[Gaudeamus]

Thanx @sumit116, could you please remove "needs unit tests" label