Skip to content

Add Token Tests and small refactoring in token class#256

Merged
Nef10 merged 8 commits intomainfrom
tests
Sep 2, 2025
Merged

Add Token Tests and small refactoring in token class#256
Nef10 merged 8 commits intomainfrom
tests

Conversation

@Nef10
Copy link
Owner

@Nef10 Nef10 commented Sep 2, 2025

No description provided.

@Nef10 Nef10 added maintenance Internal, non-code related changes tests Improvements to automated tests labels Sep 2, 2025
@Nef10 Nef10 enabled auto-merge (rebase) September 2, 2025 04:46
@Nef10 Nef10 added refactoring Code refactoring and removed maintenance Internal, non-code related changes labels Sep 2, 2025
@Nef10 Nef10 merged commit 90cfcee into main Sep 2, 2025
11 of 20 checks passed
@Nef10 Nef10 deleted the tests branch September 2, 2025 05:02
@github-actions
Copy link

github-actions bot commented Sep 2, 2025

Minimum coverage required: 30%


Coverage after merging tests into main

31.77%

Coverage Report
FileStmtsFuncsLinesUncovered Lines
Sources/Wealthsimple
   Token.swift98.17%100%97.99%117–118, 137–138
   TransactionError.swift0%0%0%29–46
   URLConfiguration.swift100%100%100%
   WealthsimpleAccount.swift0%0%0%100–109, 111–134, 136–155, 75, 82–97, 99
   WealthsimpleAsset.swift0%0%0%52–68
   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 added the major This change should bump the major version of the next release label Sep 2, 2025
@Nef10 Nef10 removed the major This change should bump the major version of the next release label Nov 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactoring Code refactoring tests Improvements to automated tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant