Complete all unit test in router/mod.rs

arn-the-long-beard / seed-routing

Prototype for having easy routing to use in Seed

MIT License

6 stars 2 forks source link

Complete all unit test in router/mod.rs #12

Closed arn-the-long-beard closed 3 years ago

arn-the-long-beard commented 3 years ago

From the very nice checklist made by @Ben-PH , 2 items are left

[ ] subscribe
[x] confirm_navigation

I am not sure yet on how we can test these 2 functions. It might be not possible to do it yet.

arn-the-long-beard commented 3 years ago

I am not sure if we can find a way to test the subscribe method.

Ben-PH commented 3 years ago

I'm thinking that subscribe testing comes under integration testing with seed itself. I'm thinking testing it would be within the scope of integrating it into seed?

arn-the-long-beard commented 3 years ago

Yes it is !!! Then I guess we can close this one.