Skip to content

Add support for validating FDL links from custom domains. - #1962

Merged
dmandar merged 10 commits into
masterfrom
md-fdl
Nov 12, 2018
Merged

Add support for validating FDL links from custom domains. #1962
dmandar merged 10 commits into
masterfrom
md-fdl

Conversation

@dmandar

@dmandar dmandar commented Oct 16, 2018

Copy link
Copy Markdown
Contributor

This feature is not yet available for public consumption.

@paulb777 paulb777 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please address the two failing unit tests on travis:

https://travis-ci.org/firebase/firebase-ios-sdk/jobs/442453702

Looks like an extra call to [FIRApp configure]?
Done.

FIRDynamicLinksTest
testInvalidCustomDomainNames, failed: caught "com.firebase.core", "Default app has already been configured."
/Users/travis/build/firebase/firebase-ios-sdk/Example/DynamicLinks/Tests/FIRDynamicLinksTest.m:1054

[FIRApp configure];
NSArray<NSString *> *urlStrings = @[

testValidCustomDomainNames, failed: caught "com.firebase.core", "Default app has already been configured."
/Users/travis/build/firebase/firebase-ios-sdk/Example/DynamicLinks/Tests/FIRDynamicLinksTest.m:1027

[FIRApp configure];
NSArray<NSString *> *urlStrings = @[
 Executed 129 tests, with 2 failures (2 unexpected) in 1.578 (1.952) seconds

2018-10-17 00:56:17.268 xcodebuild[8624:17626] [MT] IDETestOperationsObserverDebug: 157.318 elapsed -- Testing started completed.
2018-10-17 00:56:17.268 xcodebuild[8624:17626] [MT] IDETestOperationsObserverDebug: 0.000 sec, +0.000 sec -- start
2018-10-17 00:56:17.268 xcodebuild[8624:17626] [MT] IDETestOperationsObserverDebug: 157.318 sec, +157.318 sec -- end
Failing tests:
-[FIRDynamicLinksTest testInvalidCustomDomainNames]
-[FIRDynamicLinksTest testValidCustomDomainNames]
** TEST FAILED **

@dmandar dmandar self-assigned this Nov 10, 2018

@dmandar dmandar left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed. PTAL.

<string>https://mydomain.com</string>
<string>https://mydomain2.com</string>
<string>https://google.com</string>
<string>https://google.com</string>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

duplicate intentional?

&lt;string&gt;https://mydomain.com&lt;/string&gt;
&lt;string&gt;https://mydomain2.com&lt;/string&gt;
&lt;string&gt;https://google.com&lt;/string&gt;
&lt;string&gt;https://google.com&lt;/string&gt;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

duplicate

@dmandar
dmandar merged commit bf1ce88 into master Nov 12, 2018
@paulb777
paulb777 deleted the md-fdl branch March 20, 2019 21:52
@firebase firebase locked and limited conversation to collaborators Oct 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants