Fix Status bar height and update test

percy · Sep 5, 2023 · fbf8cd2 · fbf8cd2
1 parent 42dbeef
commit fbf8cd2
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/percy/providers/appAutomateProvider.js b/percy/providers/appAutomateProvider.js
@@ -148,9 +148,9 @@ class AppAutomateProvider extends GenericProvider {
 
     JSON.parse(response.result).forEach(tileData => {
       tiles.push(new Tile({
-        statBarHeight,
-        navBarHeight,
-        fullscreen,
+        statusBarHeight: statBarHeight,
+        navBarHeight: navBarHeight,
+        fullscreen: fullscreen,
         headerHeight: tileData.header_height,
         footerHeight: tileData.footer_height,
         sha: tileData.sha.split('-')[0] // drop build id

diff --git a/test/percy/providers/appAutomateProvider.test.mjs b/test/percy/providers/appAutomateProvider.test.mjs
@@ -153,7 +153,9 @@ describe('AppAutomateProvider', () => {
           args['options']['deviceHeight'] = screenSize['height'];
           appAutomate.metadata = { statusBarHeight: () => 100, navigationBarHeight: () => 200, scaleFactor: () => 1, screenSize: () => screenSize };
 
-          await appAutomate.getTiles(true, false, null, null, null);
+          let tiles = await appAutomate.getTiles(true, false, null, null, null);
+          expect(tiles[0].statusBarHeight).toEqual(100);
+          expect(tiles[0].navBarHeight).toEqual(200);
           expect(browserstack_executorSpy).toHaveBeenCalledWith('percyScreenshot', args);
         });
       });