CID Test areas that need coverage

[anrossi]

These are

• [ ] QuicBindingAddAllSourceConnectionIDs - test that adds the same CID twice

• [ ] QuicBindingCreateConnection - test that creates connections as the binding is shutting down

• [ ] QuicConnGenerateNewSourceCid - test collision with new CID being the same as old.

• [ ] API Tests for QuicConnAddLocalAddress should test negatives for Server, not started, invalid address, address already in use (double-add), maximum path count.

• [ ] QuicConnAddLocalAddress - test for lacking binding, test for null CID alloc failure does this run in-line, or on a worker thread? if worker, need to add more log statements

• [ ] QuicConnAddLocalAddress - test PathsCount > 1

• [ ] QuicConnRemoveLocalAddress - test negative for server, not started, invalid address, remove address not existent, remove active path. And test positive. (no test currently)

• [ ] QuicConnSetParam(LOCAL_ADDRESS) test when it fails QuicConnRetireCurrentDestCid

• [ ] QuicConnSetParam(ADD_LOCAL_ADDRESS) should test invalid buffer length

• [ ] QuicConnSetParam(REMOVE_LOCAL_ADDRESS) should test invalid buffer length

• [ ] QuicConnSetParam(LOCAL_UNUSED_DEST_CID_COUNT) - test negative inputs.

• [ ] QuicLookupRebalance - test the case where PreviousPartitionCount != 0

• [ ] QuicLookupAddLocalCid - test failing to insert the Local Cid