Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add error tests #76

Merged
merged 1 commit into from
Sep 18, 2021
Merged

Add error tests #76

merged 1 commit into from
Sep 18, 2021

Conversation

Nef10
Copy link
Owner

@Nef10 Nef10 commented Sep 18, 2021

Refs #1

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

Coverage after merging tests into main

20.89%

Coverage Report
FileStmtsFuncsLinesUncovered Lines
Sources/SwiftBeanCountWealthsimpleMapper
   LedgerLookup.swift57.55%57.69%57.52%115–127, 67–73, 75–84, 86–92
   Wealthsimple.Transaction.swift0%0%0%13–30, 32–34, 36–38
   WealthsimpleConversionError.swift100%100%100%
   WealthsimpleLedgerMapper.swift0%0%0%100–129, 143–178, 187–199, 202–235, 237–244, 246–251, 253–268, 270–276, 278–290, 292–298, 300–305, 307–309, 31, 310–314, 317–319, 32, 320–325, 327–329, 33, 330–333, 34–37, 40–43, 46–50, 52–56, 61, 67–69, 71–81, 95–99

@Nef10 Nef10 merged commit f77ea1d into main Sep 18, 2021
@Nef10 Nef10 deleted the tests branch September 18, 2021 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tests Improvements to automated tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant