tree 74cdcce9a6510e4f4c9b601525f004641707c1de
parent c8828ebf59318b049610048de8da610d4060118c
author martinduke <martinduke@google.com> 1779316576 -0700
committer Copybara-Service <copybara-worker@google.com> 1779316601 -0700

Remove unnecessary tests from MoqtSessionTest. Putting the logic in OutgoingSubgroupStreamTest and SubscriptionPublisherTest is more compact and cleaner conceptually.

Includes a small fix to allow opening several streams at once.

Here is a mapping of the deleted tests and what covers them:
OSST = OutgoingSubgroupStreamTest
SPT = SubscriptionPublisherTest

CreateOutgoingSubgroupStreamAndSend -> CreateStream via
                                       SPT::UpdatePriorityWithActiveStreams
FinDataStreamFromCache -> OSST::OnCanWriteSetsAlarm
SendFragmentedObject -> OSST::SendFragmentedObject
GroupAbandonedNoDeliveryTimeout -> SPT::OnGroupAbandonedTooFarBehind
GroupAbandondedDeliveryTimeout is the same as GroupAbandonedNoDeliveryTimeout. This is an identical test!
GroupAbandoned -> SPT::OnGroupAbandonedWithStreams
LateFinInDataStream -> OSST::Fin
SeparateFinForFutureObject -> OSST::FinForFutureObject
PublisherAbandonsSubgrpup -> SPT::OnSubgroupAbandonedWithStream
UnidirectionalStreamCannotBeOpened ->
                         SPT::OnNewObjectAvailableStreamCreationBlocked,
                         SPT::OnCanCreateNewUniStreamSuccess
QueuedStreamIsCleared -> SPT::OnCanCreateNewUniStreamSuccess,
                         SPT::OnCanCreateNewUniStreamPendingCleanup
OutgoingStreamDisappears -> SPT::OnDataStreamDestroyed
SendDatagram -> SPT::OnNewObjectAvailableDatagram
QueuedStreamsOpenedInOrder -> SPT::PendingStreamsInOrder
StreamQueuedForSubscriptionThatDoesntExist ->
                        SPT::OnCanCreateNewUniStreamPendingCleanup
QueuedStreamPriorityChanged -> MoqtSessionTest::UpdateTrackPriority,
                               SPT::PendingStreamsInOrder,
                               SPT::UpdatePriorityWithPendingStreams
DeliveryTimeoutExpiredOnArrival -> OSST::OnCanWriteTimeout
DeliveryTimeoutAfterIntegratedFin -> OSST::OnCanWriteSetsAlarm
DeliveryTimeoutAfterSeparateFin -> OSST::Fin
DeliveryTimeoutAlternateDesign -> AlternateDeliveryTimeout
IncomingRequestUpdateTruncatesSubscription ->
  MoqtSessionTest::IncomingRequestUpdateTriggersRequestOk,
  MoqtSessionTest::IncomingRequestUpdateTriggersRequestError,
  SPT::IncomingUpdateTruncatesSubscription

PiperOrigin-RevId: 918686567