test

Hi @NathanWalker, Thank you for the PR!

Caching the current frame, only if there is no already cached frame with the same name, will disrupt the logic that handles back navigation. It also breaks some of our tests that ensure back navigation inside modal view.

Can you share some info (sample project, playground or just steps) about the scenario that is fixed with your changes? We certainly have not covered all the scenarios with the latest router changes and your case could help a lot.

Note: We have some e2e test apps that could be helpful when contributing to nativescript-angular.

modal-navigation-ng app holds tests for modal views and your changes breaks some of them. You can run end-to-end tests locally by following these instructions.

@ADjenkov Let's hop on a brief hangout to demonstrate this - I sent you a message on {N} community Slack.

Closing in favor of this.