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 26 commits November 18, 2025 14:12
…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]

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