Skip to content

Conversation

@lposen
Copy link
Contributor

@lposen lposen commented Dec 12, 2025

🔹 JIRA Ticket(s) if any

✏️ Description

Please provide a brief description of what this pull request does.

lposen and others added 30 commits October 3, 2025 13:32
…to feature/MOB-12160-add-tests-for-usedeviceorientation
…-usedeviceorientation

[MOB-12160] add  unit tests for useDeviceOrientation
…-iterableutil

[MOB-12186] add-tests-for-iterableutil
…sses' into jwt/MOB-10947-task-3-android-retrypolicy-config-at-android-br
…oid-br' into jwt/MOB-11032-task-4-ios-bridge-retrypolicy-and-authfailure
…e-and-retrypolicy-ts-classes

[MOB-10946] Update JWT in ts
lposen and others added 25 commits November 18, 2025 19:25
…iterableinappmanager

[MOB-12189] add-tests-to-iterableinappmanager
…cidentally-left-in

[SDK-148] remove font references from Info.plist to streamline configuration
…-3.14.2

chore(deps): bump js-yaml from 3.14.1 to 3.14.2
…ative-community/cli-18.0.1

chore(deps-dev): bump @react-native-community/cli from 18.0.0 to 18.0.1
…al-not-respected-always-def

[SDK-196] Fix retryInterval not being able to be updated
@github-actions
Copy link

Lines Statements Branches Functions
Coverage: 47%
47.06% (369/784) 21.5% (80/372) 43.91% (119/271)

@qltysh
Copy link

qltysh bot commented Dec 12, 2025

Qlty

Coverage Impact

⬆️ Merging this pull request will increase total coverage on loren/embedded/MOB-12273-card-component by 5.07%.

Modified Files with Diff Coverage (4)

RatingFile% DiffUncovered Line #s
Coverage rating: A Coverage rating: A
src/core/classes/Iterable.ts96.2%1057
Coverage rating: A Coverage rating: A
src/core/classes/IterableConfig.ts100.0%
Coverage rating: A Coverage rating: A
src/core/classes/IterableLogger.ts100.0%
New file Coverage rating: A
src/inbox/components/HeaderBackButton.tsx100.0%
Total97.2%
🤖 Increase coverage with AI coding...

In the `loren/embedded/all-ios` branch, add test coverage for this new code:

- `src/core/classes/Iterable.ts` -- Line 1057

🚦 See full report on Qlty Cloud »

🛟 Help
  • Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

  • Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

  • File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

    • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

@qltysh
Copy link

qltysh bot commented Dec 12, 2025

5 new issues

Tool Category Rule Count
qlty Structure Deeply nested control flow (level = 4) 3
qlty Structure High total complexity (count = 67) 1
qlty Structure Function with high complexity (count = 44): toDict 1

dict["payload"] = payload
}

return dict
Copy link

Choose a reason for hiding this comment

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

Function with high complexity (count = 44): toDict [qlty:function-complexity]

var buttonDict = [AnyHashable: Any]()
buttonDict["id"] = button.id
if let title = button.title {
buttonDict["title"] = title
Copy link

Choose a reason for hiding this comment

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

Deeply nested control flow (level = 4) [qlty:nested-control-flow]

}
buttonDict["action"] = actionDict
} else {
buttonDict["action"] = NSNull()
Copy link

Choose a reason for hiding this comment

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

Deeply nested control flow (level = 4) [qlty:nested-control-flow]

var textDict = [AnyHashable: Any]()
textDict["id"] = textElement.id
if let textValue = textElement.text {
textDict["text"] = textValue
Copy link

Choose a reason for hiding this comment

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

Deeply nested control flow (level = 4) [qlty:nested-control-flow]

@lposen lposen added the embedded Issues/PRs related to Embedded Messages label Jan 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

embedded Issues/PRs related to Embedded Messages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants