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(e2e): detox tests #3080

Closed
wants to merge 21 commits into from
Closed

test(e2e): detox tests #3080

wants to merge 21 commits into from

Conversation

sandipndev
Copy link
Member

@sandipndev sandipndev commented Feb 17, 2024

This PR adds the following flows to Detox:

  • Login with Phone/Email Flow
  • Conversion Flow
  • Payment Send Flow
    • Intraledger using Username - BTC Amount
    • Intraledger using Username - Stablesats Amount
    • Intraledger using LN Invoice
    • External LN Invoice
    • Onchain Address
    • LNURL Pay
    • LNURL Withdraw
  • Payment Receive Flow
    • LN - BTC Amount
    • LN - BTC No Amount
    • LN - Stablesats Amount
    • LN - Stablesats No Amount
    • Onchain - BTC
    • Onchain - USD
    • Paycode

#3079 should be merged first

@sandipndev sandipndev force-pushed the e2e-tests-with-detox branch 3 times, most recently from 8fef5f3 to 3befcd9 Compare February 19, 2024 08:36
@sandipndev sandipndev marked this pull request as ready for review February 19, 2024 09:01
@sandipndev sandipndev changed the title test(e2e): detox tests test(e2e): detox tests - auth, intraledger and conversion Feb 20, 2024
@sandipndev sandipndev changed the title test(e2e): detox tests - auth, intraledger and conversion test(e2e): detox tests Feb 20, 2024
@sandipndev sandipndev marked this pull request as draft February 20, 2024 06:16
@sandipndev sandipndev removed the request for review from UncleSamtoshi February 20, 2024 06:25
Base automatically changed from detox to main February 23, 2024 14:05
@sandipndev sandipndev force-pushed the e2e-tests-with-detox branch 3 times, most recently from b901ea5 to f3c4612 Compare February 28, 2024 06:45
@sandipndev sandipndev force-pushed the e2e-tests-with-detox branch 5 times, most recently from dd085ef to 017fe15 Compare March 15, 2024 12:08
ci: test

chore: try macos

ci: test if everything works

ci: upload recordings

test

try emulator creation manually
export JAVA_HOME="/Library/Java/JavaVirtualMachines/zulu-11.jdk/Contents/Home"

ci: single e2e test
export JAVA_HOME="/Library/Java/JavaVirtualMachines/zulu-11.jdk/Contents/Home"
@sandipndev sandipndev force-pushed the e2e-tests-with-detox branch from 017fe15 to fa8e111 Compare March 17, 2024 18:12
@sandipndev
Copy link
Member Author

Blocked on this because somehow the Galoy API in quickstart reports zero balances after funds have been sent to user's btc/usd accounts.

Copy link

gitguardian bot commented Mar 19, 2024

⚠️ GitGuardian has uncovered 4 secrets following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

🔎 Detected hardcoded secrets in your pull request
GitGuardian id GitGuardian status Secret Commit Filename
7652397 Triggered Generic High Entropy Secret 58aac80 dev/vendor/galoy-quickstart/dev/core-bundle/serve-env.json View secret
7652397 Triggered Generic High Entropy Secret 58aac80 dev/vendor/galoy-quickstart/dev/core-bundle/integration-env.json View secret
7652397 Triggered Generic High Entropy Secret 58aac80 dev/vendor/galoy-quickstart/dev/Tiltfile View secret
9485054 Triggered Generic Private Key 58aac80 dev/vendor/galoy-quickstart/dev/config/notifications/fake_service_account.json View secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secrets safely. Learn here the best practices.
  3. Revoke and rotate these secrets.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

Our GitHub checks need improvements? Share your feedbacks!

@sandipndev sandipndev force-pushed the e2e-tests-with-detox branch 2 times, most recently from 7a7ab46 to 7936579 Compare March 20, 2024 07:12
chore: recv

test: destroy backend

save

check code

save

chore: update quickstart which also fix funding issue

fix: e2e

change ios device

more time
@sandipndev sandipndev force-pushed the e2e-tests-with-detox branch from 7936579 to 58aac80 Compare March 20, 2024 07:16
@sandipndev sandipndev force-pushed the e2e-tests-with-detox branch 2 times, most recently from b457cd9 to 36b2fff Compare March 20, 2024 14:39
@sandipndev sandipndev force-pushed the e2e-tests-with-detox branch from 36b2fff to 1c77dd0 Compare March 20, 2024 15:29
@sandipndev sandipndev force-pushed the e2e-tests-with-detox branch from 6d79e9c to ea54ecc Compare March 20, 2024 16:44
@sandipndev sandipndev force-pushed the e2e-tests-with-detox branch from ea54ecc to cd556bf Compare March 20, 2024 17:09
@sandipndev sandipndev closed this Mar 21, 2024
@sandipndev sandipndev deleted the e2e-tests-with-detox branch March 21, 2024 04:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants