Core: Skip module bids during mediaType eligibility checks (prebid#14058)

* Core: Skip module bids during mediaType eligibility checks

Module bids do not have a `bidder` property. They should not be considered for mediaType checks.

This only works as-is for banner units because we default `supportedMediaTypes` to `['banner']'` for an `undefined` bidder.

Closes prebid#14057

* add test case

---------

Co-authored-by: Demetrio Girardi <dgirardi@prebid.org>

Author: gwhigs

src/prebid.ts

@@ -850,7 +850,7 @@ export const startAuction = hook('async', function ({ bidsBackHandler, timeout:
     const adUnitMediaTypes = Object.keys(adUnit.mediaTypes || { 'banner': 'banner' });
 
     // get the bidder's mediaTypes
-    const allBidders = adUnit.bids.map(bid => bid.bidder);
+    const allBidders = adUnit.bids.map(bid => bid.bidder).filter(Boolean);
     const bidderRegistry = adapterManager.bidderRegistry;
 
     const bidders = allBidders.filter(bidder => !s2sBidders.has(bidder));

test/spec/unit/pbjs_api_spec.js

@@ -2849,6 +2849,18 @@ describe('Unit: Prebid Module', function () {
         // only appnexus supports native
         expect(biddersCalled.length).to.equal(1);
       });
+
+      it('module bids should not be filtered out', async () => {
+        delete adUnits[0].mediaTypes.banner;
+        adUnits[0].bids.push({
+          module: 'pbsBidAdapter',
+          ortb2Imp: {}
+        });
+
+        pbjs.requestBids({adUnits});
+        await auctionStarted;
+        expect(adapterManager.callBids.getCall(0).args[0][0].bids.length).to.eql(2);
+      })
     });
 
     describe('part 2', function () {