-
Notifications
You must be signed in to change notification settings - Fork 31
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
Integration test for send transfers #1211
Comments
Blocked by #1307 |
One thought: As we have got receiving merged now, we could test the transfer like this If #1290 is merged we could also use webrtc? |
Yes that should work
Yup I think so, only need to check whether the matrix server in the integration test image supports STUN etc |
We DO handle mediation fees properly in the light-client, for sending, receiving and mediating. |
That's exactly what I meant |
We need to create a new issue about enabling STUN on the integration image |
We already have a hardcoded fallback google STUN server. If the container doesn't restrict outbound connections it probably works just fine |
It shouldn't but for the tests I would still like not to rely on anything external as it can introduce uncessary flakiness. |
The recommended matrix TURN server is |
Making something configurable doesn't magically make it not hardcoded |
Yes, it does, at least in my definition of hardcoded, which interprets the "hard" as "not changeable" without meddling with source code. That line you linked is in the function |
I know, yet it's hardcoded :) |
I did a quick test and a direct transfer worked with both nodes. We have the following nodes:
When attempting a mediated tansfer us I'm getting an error from the PFS:
I remember having the same issue when the PFS didn't pick up the presence updates from Matrix correctly |
@nephix we should probably enhance the script to pull the service/node logs from the container before shutting it down at least with some debug flag going on |
do you have a draft somewhere? |
With the merge of #1341 PFS should work properly outside the container. From the debugging done on the existing test, the steps needed before the transfer are the following:
|
My 2cents, as a side-node: no need to cache |
Description
As a follow up for #348 we need to test a send transfer. To keep the number of tests low, we should test a transfer with the highest complexity (mediated, fees, fees for PFS).
Acceptance criteria
Tasks
The text was updated successfully, but these errors were encountered: