Skip to content

Add tests to WealthsimpleAsset and tweak asset error, test folder str…#259

Merged
Nef10 merged 1 commit intomainfrom
assets
Sep 2, 2025
Merged

Add tests to WealthsimpleAsset and tweak asset error, test folder str…#259
Nef10 merged 1 commit intomainfrom
assets

Conversation

@Nef10
Copy link
Owner

@Nef10 Nef10 commented Sep 2, 2025

…ucture

@Nef10 Nef10 added the tests Improvements to automated tests label Sep 2, 2025
@github-actions
Copy link

github-actions bot commented Sep 2, 2025

Minimum coverage required: 40%


Coverage after merging assets into main

44.17%

Coverage Report
FileStmtsFuncsLinesUncovered Lines
Sources/Wealthsimple
   Token.swift97.29%95.24%97.50%117–118, 137–138
   TransactionError.swift0%0%0%29–46
   URLConfiguration.swift100%100%100%
   WealthsimpleAccount.swift89.01%63.64%92.50%114–115
   WealthsimpleAsset.swift81.82%33.33%89.47%
   WealthsimpleDownloader.swift0%0%0%102–115, 122–135, 137–149, 48–51, 55–78, 82–95
   WealthsimplePosition.swift0%0%0%100–112, 114–137, 139–158, 51, 53–57, 66–92, 94–99
   WealthsimpleTransaction.swift0%0%0%127, 129–133, 152–196, 198–218, 220–243, 245–264
Sources/Wealthsimple/Extensions
   String+CamelCase.swift100%100%100%

@Nef10 Nef10 merged commit 919be1c into main Sep 2, 2025
7 of 12 checks passed
@Nef10 Nef10 deleted the assets branch September 2, 2025 07:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

tests Improvements to automated tests

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Nef10