Skip to content

Conversation

@Ayyanchira
Copy link
Member

🔹 Jira Ticket(s) if any

Patch on #411

✏️ Description

@codecov
Copy link

codecov bot commented Dec 22, 2021

Codecov Report

Merging #414 (d980fce) into jay/MOB-3769-allowed-protocols (27a74d5) will increase coverage by 0.13%.
The diff coverage is 60.00%.

❗ Current head d980fce differs from pull request most recent head 0044686. Consider uploading reports for the commit 0044686 to get more accurate results
Impacted file tree graph

@@                        Coverage Diff                         @@
##           jay/MOB-3769-allowed-protocols     #414      +/-   ##
==================================================================
+ Coverage                           66.99%   67.12%   +0.13%     
==================================================================
  Files                                  63       63              
  Lines                                3833     3833              
  Branches                              447      447              
==================================================================
+ Hits                                 2568     2573       +5     
+ Misses                               1003      997       -6     
- Partials                              262      263       +1     
Impacted Files Coverage Δ
...ain/java/com/iterable/iterableapi/IterableApi.java 65.02% <ø> (-0.26%) ⬇️
...com/iterable/iterableapi/IterableActionRunner.java 43.18% <60.00%> (ø)
...in/java/com/iterable/iterableapi/IterableUtil.java 70.53% <0.00%> (+5.35%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6109022...0044686. Read the comment docs.

1. InAppManager no more holds array of allowedProtocols. Its always accessed from IterableConfig.
2. `isUrlOpenAllowed` is now a common utility method in IterableUtil
3. Modified test methods to accomodate the changes
@Ayyanchira Ayyanchira closed this Dec 22, 2021
@Ayyanchira Ayyanchira deleted the Patch-for-allowedProtocols branch December 22, 2021 21:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants