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

test: some branches in transaction-outbox service #2507

Merged
merged 2 commits into from
Oct 16, 2023
Merged

Conversation

suyashpatil78
Copy link
Contributor

@suyashpatil78 suyashpatil78 commented Oct 11, 2023

workerB

Description

🤖 Generated by Copilot at b176bac

This pull request improves the testing coverage and quality of the transactions-outbox.service, which handles offline transactions in the Fyle mobile app. It adds and updates test cases for the data extraction and sync features, and creates mock data for parsed receipts. It affects the files transactions-outbox.service.spec.ts and parsed-receipt.data.ts.

🤖 Generated by Copilot at b176bac

Sing, O Muse, of the outbox service, the cunning work of code
That extracts from receipts the hidden data, like Hermes, the herald of the gods
And syncs the offline transactions with the cloud, the mighty realm of Zeus
And tests its logic with mock data, the faithful image of the truth

Walkthrough

🤖 Generated by Copilot at b176bac

  • Add mock data object for parsed receipt data (link)
  • Import additional modules and mock data objects for testing transactions outbox service (link)
  • Add spy methods for DateService, TransactionService, and FileService (link)
  • Assign spy methods to service properties of transactions outbox service instance (link)
  • Refactor test case for addEntryAndSync method into two scenarios with applyMagic parameter (link)
  • Modify test case for deleteBulkOfflineExpenses method to test multiple expenses (link)
  • Add test cases for matchIfRequired, restoreQueue, and processDataExtractionEntry methods (link)

Clickup

app.clickup.com

Code Coverage

Please add code coverage here

UI Preview

Please add screenshots for UI change

@github-actions github-actions bot added the size/M Medium PR label Oct 11, 2023
@github-actions
Copy link

Unit Test Coverage % values
Statements 95.43% ( 17182 / 18003 )
Branches 93% ( 8666 / 9318 )
Functions 93.57% ( 5301 / 5665 )
Lines 95.59% ( 16364 / 17118 )

@suyashpatil78 suyashpatil78 merged commit 3beb643 into master Oct 16, 2023
3 of 4 checks passed
@suyashpatil78 suyashpatil78 deleted the tx-outbox-test branch October 17, 2023 07:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/M Medium PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants