CCDM: change navigate signature to accept a config object

vaadin · Aug 12, 2019 · bbc61c6 · bbc61c6
1 parent 1c99f08
commit bbc61c6
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/flow-client/src/main/resources/META-INF/resources/frontend/Flow.ts b/flow-client/src/main/resources/META-INF/resources/frontend/Flow.ts
@@ -12,6 +12,10 @@ interface AppInitResponse {
     appConfig: AppConfig;
 }
 
+export interface NavigationParameters {
+    path: string;
+}
+
 /**
  * Client API for flow UI operations.
  */
@@ -53,9 +57,9 @@ export class Flow {
     /**
      * Go to a route defined in server.
      */
-    async navigate(path : string): Promise<HTMLElement> {
+    async navigate(params : NavigationParameters): Promise<HTMLElement> {
         await this.start();
-        return this.getFlowElement(path);
+        return this.getFlowElement(params.path);
     }
 
     private async getFlowElement(routePath : string): Promise<HTMLElement> {

diff --git a/flow-client/src/test/frontend/FlowTests.ts b/flow-client/src/test/frontend/FlowTests.ts
@@ -76,7 +76,7 @@ suite("Flow", () => {
 
     mockInitResponse();
     return new Flow()
-      .navigate("Foo/Bar.baz")
+      .navigate({path: "Foo/Bar.baz"})
       .then(() => {
         // Check that start() was called
         assert.isDefined((window as any).Vaadin.Flow.resolveUri);