fix: logic for checking if the route can be established updated

[saledjenic]

toToken is used only when the user wants to swap, since Can function from the Bridge interface receives toToken which is nil for all but for the swap, we need to consider it and return appropriately from the function.

[status-im-auto]

Jenkins Builds

Click to see older builds (4)

❔	Commit	#️⃣	Finished (UTC)	Duration	Platform	Result
✖️	ad0f63c	#1	2024-04-18 10:26:26	~1 min	tests	📄log
✔️	ad0f63c	#1	2024-04-18 10:29:47	~4 min	linux	📦zip
✔️	ad0f63c	#1	2024-04-18 10:30:18	~5 min	ios	📦zip
✔️	ad0f63c	#1	2024-04-18 10:30:57	~5 min	android	📦aar

❔	Commit	#️⃣	Finished (UTC)	Duration	Platform	Result
✖️	6ac2d64	#2	2024-04-19 07:54:42	~1 min	tests	📄log
✔️	6ac2d64	#2	2024-04-19 07:57:33	~4 min	linux	📦zip
✔️	6ac2d64	#2	2024-04-19 07:58:43	~5 min	ios	📦zip
✔️	6ac2d64	#2	2024-04-19 07:59:11	~5 min	android	📦aar
✔️	c53b3a5	#3	2024-05-07 07:21:21	~3 min	linux	📦zip
✔️	c53b3a5	#3	2024-05-07 07:22:19	~4 min	ios	📦zip
✔️	c53b3a5	#3	2024-05-07 07:22:33	~5 min	android	📦aar
✔️	c53b3a5	#3	2024-05-07 08:00:31	~43 min	tests	📄log

[IvanBelyakoff]

@saledjenic, when can this happen that toToken is nil?

[saledjenic]

@saledjenic, when can this happen that toToken is nil?

@IvanBelyakoff toToken is used only for the Swap, thus for all other types will be nil.

[J-Son89]

@saledjenic - any chance you can include a description to this pr of what these changes are doing and why they were needed? 🙏
Also is there an associated issue for this?

[cammellos]

it'd be good also to add some level of tests, so that the behavior is clearer and we know it matches the dev's intention

[saledjenic]

@saledjenic - any chance you can include a description to this pr of what these changes are doing and why they were needed? 🙏
Also is there an associated issue for this?

@J-Son89 description updated. No special issue just for this, cause this was missed when swap was added to the router. We don't have any UI for it yet, just a backend logic, otherwise it would be noticed and logged as an issue. I noticed this and wanted to handle it.

it'd be good also to add some level of tests, so that the behavior is clearer and we know it matches the dev's intention

@cammellos I agree, maybe we can add them after we improve the router logic, cause I see we have many issues with the router.

[J-Son89]

thanks @saledjenic!