Skip to content

test: resolve issues #1837, #1840, #1839, #1838 with integration test… - #1873

Merged
sublime247 merged 1 commit into
sublime247:mainfrom
Favourof:feat/testing-1837-1838-1839-1840
Aug 27, 2026
Merged

test: resolve issues #1837, #1840, #1839, #1838 with integration test…#1873
sublime247 merged 1 commit into
sublime247:mainfrom
Favourof:feat/testing-1837-1838-1839-1840

Conversation

@Favourof

Copy link
Copy Markdown
Contributor

Description

This PR resolves four related testing, resilience, and security issues across the codebase:

Changes

  • src/tests/integration/mtnPayoutConfirmation.test.ts
  • src/utils/csvExporter.ts & src/utils/__tests__/csvExporter.test.ts
  • src/routes/export.ts
  • .github/workflows/security-scan.yml & scripts/security-scan.ts
  • src/tests/security/securityPipeline.test.ts
  • src/tests/chaos/networkFailureRecovery.test.ts
  • package.json

Closes #1837, Closes #1838, Closes #1839, Closes #1840

@drips-wave

drips-wave Bot commented Aug 26, 2026

Copy link
Copy Markdown

@Favourof Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@github-advanced-security

Copy link
Copy Markdown

You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool.

What Enabling Code Scanning Means:

  • The 'Security' tab will display more code scanning analysis results (e.g., for the default branch).
  • Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results.
  • You will be able to see the analysis results for the pull request's branch on this overview once the scans have completed and the checks have passed.

For more information about GitHub Code Scanning, check out the documentation.

Comment thread src/tests/security/securityPipeline.test.ts Fixed
Comment thread src/utils/csvExporter.ts Fixed
Comment thread src/utils/csvExporter.ts Fixed
Comment thread src/utils/csvExporter.ts Fixed
Comment thread src/utils/csvExporter.ts Fixed
@Favourof
Favourof force-pushed the feat/testing-1837-1838-1839-1840 branch from 487d277 to 605d948 Compare August 26, 2026 19:03
sublime247#1840

Resolve testing, resilience, and security issues:
- sublime247#1837: MTN payout confirmation routines integration tests
- sublime247#1838: Transaction CSV exporter utility and unit tests
- sublime247#1839: Security scanning tools and pipeline tests
- sublime247#1840: Error propagation and network failure recovery loops
@Favourof
Favourof force-pushed the feat/testing-1837-1838-1839-1840 branch from 605d948 to 7891214 Compare August 27, 2026 07:19
@sublime247
sublime247 merged commit e3791c8 into sublime247:main Aug 27, 2026
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants