Skip to content

Conversation

@eldhosembabu
Copy link
Contributor

@eldhosembabu eldhosembabu commented Mar 23, 2022

Adding more unit tests to validate custom domain logic

#no-changelog

Related issue : #6978

@eldhosembabu eldhosembabu requested a review from paulb777 March 23, 2022 19:35
@google-oss-bot
Copy link

google-oss-bot commented Mar 23, 2022

Size Report 1

Affected Products

  • FirebaseDynamicLinks

    TypeBase (6911d52)Merge (c83379d)Diff
    CocoaPods?403 kB? (?)

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/jdw0ZnICjd.html

@google-oss-bot
Copy link

google-oss-bot commented Mar 23, 2022

Coverage Report 1

Affected Products

  • FirebaseDynamicLinks-iOS-FirebaseDynamicLinks.framework

    Overall coverage changed from ? (6911d52) to 60.39% (c83379d) by ?.

    15 individual files with coverage change

    FilenameBase (6911d52)Merge (c83379d)Diff
    FDLDeviceHeuristicsHelper.m?0.00%?
    FDLLogging.m?58.62%?
    FDLURLComponents.m?81.95%?
    FDLUtilities.m?73.19%?
    FIRDLDefaultRetrievalProcessV2.m?14.04%?
    FIRDLJavaScriptExecutor.m?23.26%?
    FIRDLRetrievalProcessFactory.m?100.00%?
    FIRDLRetrievalProcessResult.m?100.00%?
    FIRDLScionLogging.m?100.00%?
    FIRDynamicLink.m?77.78%?
    FIRDynamicLinkComponentsKeyProvider.m?66.67%?
    FIRDynamicLinkNetworking.m?44.10%?
    FIRDynamicLinks.m?69.10%?
    GINArgument.m?0.00%?
    GINInvocation.m?0.00%?

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/fIYEes88Bs.html

@eldhosembabu eldhosembabu merged commit 6efe4f5 into master Mar 23, 2022
@eldhosembabu eldhosembabu deleted the fdl-unit-tests branch March 23, 2022 20:36
@firebase firebase locked and limited conversation to collaborators Apr 23, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants