@applicaster/zapp-react-native-ui-components 13.0.0-rc.13 → 13.0.0-rc.130

package/Decorators/RiverResolver/__tests__/riverResolver.test.tsx

@@ -1,8 +1,8 @@
 import React from "react";
-import { shallow } from "enzyme";
 import { View } from "react-native";
-import { shallowToJson } from "enzyme-to-json";
-import * as navigationHooks from "@applicaster/zapp-react-native-utils/reactHooks/navigation";
+import { renderWithProviders } from "@applicaster/zapp-react-native-utils/testUtils";
+
+import { withRiver } from "../";
 
 const rivers = {
   A123: {
@@ -13,38 +13,38 @@ const rivers = {
   },
 };
 
-jest.doMock("@applicaster/zapp-react-native-redux/hooks", () => ({
-  usePickFromState: jest.fn(() => ({ rivers })),
-}));
-
-jest.mock("@applicaster/zapp-react-native-utils/reactHooks/layout", () => ({
-  useLayoutVersion: jest.fn(() => true),
-}));
-
-jest.spyOn(navigationHooks, "usePathname").mockReturnValue("/home");
-
-// mocks useNavigator to return currentRoute: "/home"
-jest.mock("@applicaster/zapp-react-native-utils/reactHooks", () => ({
-  useNavigation: jest.fn(() => ({ currentRoute: "/home" })),
-  useRoute: jest.fn(() => ({ pathname: "/home" })),
-}));
+jest.doMock(
+  "@applicaster/zapp-react-native-redux/hooks/usePickFromState",
+  () => ({
+    usePickFromState: jest.fn(() => ({ rivers })),
+  })
+);
 
-const { withRiver } = require("../index");
+jest.doMock(
+  "@applicaster/zapp-react-native-utils/reactHooks/navigation/usePathname",
+  () => ({
+    usePathname: jest.fn(() => "/home"),
+  })
+);
 
-const TestComponent = jest.fn(() => <View />);
+const TestComponent = jest.fn((props) => <View {...props} />);
 
 describe("withRiver HoC", () => {
   const WrappedComponent = withRiver(TestComponent);
 
-  const wrapper = shallow(
-    <WrappedComponent screenId="A123" rivers={rivers} otherProp="otherProp" />
+  const wrapper = renderWithProviders(
+    <WrappedComponent
+      testID="test-view"
+      screenId="A123"
+      otherProp="otherProp"
+    />,
+    { rivers: rivers }
   );
 
-  it("renders correctly", () => {
-    expect(shallowToJson(wrapper)).toMatchSnapshot();
-  });
-
   it("populates the river prop", () => {
-    expect(wrapper.find(TestComponent).prop("river")).toEqual(rivers.A123);
+    const targetView = wrapper.getByTestId("test-view");
+
+    expect(targetView.props.river).toEqual(rivers.A123);
+    expect(targetView.props.screenData).toEqual(rivers.A123);
   });
 });

package/Decorators/RiverResolver/index.tsx

@@ -1,8 +1,8 @@
 import * as React from "react";
 
-import { usePickFromState } from "@applicaster/zapp-react-native-redux/hooks";
 import { isTV } from "@applicaster/zapp-react-native-utils/reactUtils";
-import { usePathname } from "@applicaster/zapp-react-native-utils/reactHooks/navigation";
+import { usePathname } from "@applicaster/zapp-react-native-utils/reactHooks/navigation/usePathname";
+import { useRivers } from "@applicaster/zapp-react-native-utils/reactHooks";
 
 type Props = ZappUIComponentProps & {
   rivers: ZappRivers;
@@ -12,27 +12,25 @@ type Props = ZappUIComponentProps & {
 
 export function withRiver(Component: ZappComponent): ZappComponent {
   const RiverWithData = ({ screenId, ...otherProps }: Props) => {
-    const { rivers } = usePickFromState(["rivers"]);
+    const river = useRivers()?.[screenId];
     const pathname = usePathname();
-    const river = rivers[screenId];
 
     if (!river) {
       return null;
     }
 
-    if (!otherProps.screenData) {
-      otherProps.screenData = river;
-    }
-
     return (
       <Component
         screenId={screenId}
         river={river}
         key={isTV() ? pathname : undefined}
         {...otherProps}
+        screenData={otherProps.screenData || river}
       />
     );
   };
 
+  RiverWithData.displayName = `RiverWithData(${Component.displayName || Component.name})`;
+
   return RiverWithData;
 }

package/Decorators/ZappPipesDataConnector/__tests__/Hero.js

@@ -2,7 +2,7 @@ import React, { useEffect } from "react";
 import { View } from "react-native";
 
 type Props = {
-  onLoadFinished: () => void,
+  onLoadFinished: () => void;
 };
 
 export function Hero(props: Props) {

package/Decorators/ZappPipesDataConnector/__tests__/zappPipesDataConnector.test.js

@@ -1,6 +1,6 @@
 import React from "react";
-import { shallow } from "enzyme";
-import { shallowToJson } from "enzyme-to-json";
+import { renderWithProviders } from "@applicaster/zapp-react-native-utils/testUtils";
+
 import * as zappPipesRedux from "@applicaster/zapp-react-native-redux/ZappPipes";
 import configureStore from "redux-mock-store";
 import thunk from "redux-thunk";
@@ -8,6 +8,7 @@ import thunk from "redux-thunk";
 import { zappPipesDataConnector } from "../index";
 
 import { Hero } from "./Hero";
+import { render } from "ejs";
 
 const mockStore = configureStore([thunk]);
 
@@ -20,6 +21,15 @@ const initialState = {
   },
 };
 
+jest.mock(
+  "@applicaster/zapp-react-native-utils/reactHooks/navigation/useIsScreenActive",
+  () => ({
+    useIsScreenActive: () => ({
+      isScreenActive: true,
+    }),
+  })
+);
+
 const store = mockStore(initialState);
 
 const zappPipesSpy = jest
@@ -31,6 +41,7 @@ const WrappedComponent = zappPipesDataConnector(Hero);
 const componentProp = (data) => ({ data });
 
 // tests need to be re-written due to TS refactor
+// this test has old usage of enzyme API, that's need to be replaced with @testing-library/react-native
 describe.skip("zappPipesDataConnector", () => {
   beforeEach(() => {
     zappPipesSpy.mockClear();
@@ -43,7 +54,7 @@ describe.skip("zappPipesDataConnector", () => {
     it("loads the provided feed url", () => {
       expect(zappPipesDataConnector).toBeFunction();
 
-      const wrapper = shallow(
+      const wrapper = renderWithProviders(
         <WrappedComponent
           component={componentProp(data)}
           feedUrl={feedUrl}
@@ -51,8 +62,7 @@ describe.skip("zappPipesDataConnector", () => {
         />
       ).dive();
 
-      expect(shallowToJson(wrapper)).toMatchSnapshot();
-      expect(wrapper.first().shallow()).toMatchSnapshot();
+      expect(wrapper.toJSON()).toMatchSnapshot();
 
       expect(zappPipesSpy).toHaveBeenCalledWith(feedUrl, {
         clearCache: false,
@@ -71,12 +81,11 @@ describe.skip("zappPipesDataConnector", () => {
     it("is a HoC", () => {
       expect(zappPipesDataConnector).toBeFunction();
 
-      const wrapper = shallow(
+      const wrapper = renderWithProviders(
         <WrappedComponent component={componentProp(data)} store={store} />
       ).dive();
 
-      expect(shallowToJson(wrapper)).toMatchSnapshot();
-      expect(wrapper.first().shallow()).toMatchSnapshot();
+      expect(wrapper.toJSON()).toMatchSnapshot();
 
       expect(zappPipesSpy).toHaveBeenCalledWith(data.source, {
         clearCache: false,
@@ -99,12 +108,11 @@ describe.skip("zappPipesDataConnector", () => {
     it("calls clearCache true and silentRefresh with false", () => {
       expect(zappPipesDataConnector).toBeFunction();
 
-      const wrapper = shallow(
+      const wrapper = renderWithProviders(
         <WrappedComponent component={{ data, rules }} store={store} />
       ).dive();
 
-      expect(shallowToJson(wrapper)).toMatchSnapshot();
-      expect(wrapper.first().shallow()).toMatchSnapshot();
+      expect(wrapper.toJSON()).toMatchSnapshot();
 
       expect(zappPipesSpy).toHaveBeenCalledWith(data.source, {
         clearCache: true,
@@ -123,12 +131,11 @@ describe.skip("zappPipesDataConnector", () => {
     it("is a HoC", () => {
       expect(zappPipesDataConnector).toBeFunction();
 
-      const wrapper = shallow(
+      const wrapper = renderWithProviders(
         <WrappedComponent component={componentProp(data)} store={store} />
       ).dive();
 
-      expect(shallowToJson(wrapper)).toMatchSnapshot();
-      expect(wrapper.first().shallow()).toMatchSnapshot();
+      expect(wrapper.toJSON()).toMatchSnapshot();
 
       expect(zappPipesSpy).toHaveBeenCalledWith(
         "applicaster://fetchData?type=APPLICASTER_COLLECTION&collectionId=1234",
@@ -146,11 +153,11 @@ describe.skip("zappPipesDataConnector", () => {
     it("is a HoC", () => {
       expect(zappPipesDataConnector).toBeFunction();
 
-      const wrapper = shallow(
+      const wrapper = renderWithProviders(
         <WrappedComponent component={componentProp(data)} store={store} />
       ).dive();
 
-      expect(shallowToJson(wrapper)).toMatchSnapshot();
+      expect(wrapper.toJSON()).toMatchSnapshot();
       expect(wrapper.first().shallow()).toMatchSnapshot();
 
       expect(zappPipesSpy).toHaveBeenCalledWith(
@@ -171,11 +178,11 @@ describe.skip("zappPipesDataConnector", () => {
     };
 
     it("calls loadPipesData with favourites option", () => {
-      const wrapper = shallow(
+      const wrapper = renderWithProviders(
         <WrappedComponent component={componentProp(data)} store={store} />
       ).dive();
 
-      expect(shallowToJson(wrapper)).toMatchSnapshot();
+      expect(wrapper.toJSON()).toMatchSnapshot();
       expect(wrapper.first().shallow()).toMatchSnapshot();
 
       expect(zappPipesSpy).toHaveBeenCalledWith("FAVOURITES", {
@@ -188,7 +195,7 @@ describe.skip("zappPipesDataConnector", () => {
 
   describe("getDataSourceUrl", () => {
     it("returns null if data prop is empty", () => {
-      const wrapper = shallow(
+      const wrapper = render(
         <WrappedComponent component={componentProp({})} store={store} />
       ).dive();
 
@@ -199,7 +206,7 @@ describe.skip("zappPipesDataConnector", () => {
     });
 
     it("returns null if data.source is falsy", () => {
-      const wrapper = shallow(
+      const wrapper = renderWithProviders(
         <WrappedComponent
           component={componentProp({ source: null })}
           store={store}
@@ -222,7 +229,7 @@ describe.skip("zappPipesDataConnector", () => {
     it("reloads the app's datasource", () => {
       store.clearActions();
 
-      const wrapper = shallow(
+      const wrapper = renderWithProviders(
         <WrappedComponent component={componentProp(data)} store={store} />
       ).dive();
 
@@ -251,7 +258,7 @@ describe.skip("zappPipesDataConnector", () => {
         entry: [{ id: "test-entry", title: "test entry" }],
       }));
 
-      const wrapper = shallow(
+      const wrapper = renderWithProviders(
         <WrappedComponent
           component={{ data, getStaticComponentFeed }}
           store={store}

package/Decorators/ZappPipesDataConnector/index.tsx

@@ -1,10 +1,10 @@
 /// <reference types="@applicaster/applicaster-types" />
 /// <reference types="@applicaster/zapp-react-native-ui-components" />
 import React, { useEffect, useMemo } from "react";
+
 import * as R from "ramda";
 import { Platform } from "react-native";
 import Url from "url";
-
 import { favoritesListener } from "@applicaster/zapp-react-native-bridge/Favorites";
 import { usePickFromState } from "@applicaster/zapp-react-native-redux/hooks";
 import { useRoute } from "@applicaster/zapp-react-native-utils/reactHooks/navigation";
@@ -16,9 +16,10 @@ import {
 } from "@applicaster/zapp-react-native-utils/reactHooks";
 
 import { ZappPipesSearchContext } from "@applicaster/zapp-react-native-ui-components/Contexts";
-import { useScreenContext } from "@applicaster/zapp-react-native-utils/reactHooks/screen";
+import { useScreenContext } from "@applicaster/zapp-react-native-utils/reactHooks/screen/useScreenContext";
 
 import { isVerticalListOrGrid } from "./utils";
+import { subscribeForUrlContextKeyChanges } from "@applicaster/zapp-pipes-v2-client";
 
 type Props = {
   component: ZappUIComponent;
@@ -229,7 +230,8 @@ export function zappPipesDataConnector(
     const entryContext =
       (shouldUseNestedContext && screenContextData?.nested?.entry
         ? screenContextData?.nested?.entry
-        : screenContextData?.entry) || {};
+        : (screenContextData?.entry?.payload ?? screenContextData?.entry)) ||
+      {};
 
     const screenContext =
       (shouldUseNestedContext && screenContextData?.nested?.screen
@@ -291,6 +293,8 @@ export function zappPipesDataConnector(
         if (addListener) {
           return addListener(reloadData);
         }
+      } else {
+        return subscribeForUrlContextKeyChanges(dataSourceUrl, {}, reloadData);
       }
     }, [dataSourceUrl, reloadData]);
 

package/package.json

@@ -1,11 +1,10 @@
 {
   "name": "@applicaster/zapp-react-native-ui-components",
-  "version": "13.0.0-rc.13",
+  "version": "13.0.0-rc.130",
   "description": "Applicaster Zapp React Native ui components for the Quick Brick App",
   "main": "index.js",
   "types": "index.d.ts",
   "scripts": {
-    "test": "jest -w=2",
     "build": "echo skipping hybrid typescript build for now"
   },
   "publishConfig": {
@@ -29,25 +28,28 @@
   },
   "homepage": "https://github.com/applicaster/quickbrick#readme",
   "devDependencies": {
-    "@types/react": "17.0.2",
-    "@types/react-native": "0.69.6",
     "redux-mock-store": "^1.5.3"
   },
   "dependencies": {
-    "@applicaster/applicaster-types": "13.0.0-rc.13",
-    "@applicaster/zapp-react-native-bridge": "13.0.0-rc.13",
-    "@applicaster/zapp-react-native-redux": "13.0.0-rc.13",
-    "@applicaster/zapp-react-native-utils": "13.0.0-rc.13",
+    "@applicaster/applicaster-types": "13.0.0-rc.130",
+    "@applicaster/zapp-react-native-bridge": "13.0.0-rc.130",
+    "@applicaster/zapp-react-native-redux": "13.0.0-rc.130",
+    "@applicaster/zapp-react-native-utils": "13.0.0-rc.130",
     "promise": "^8.3.0",
     "react-router-native": "^5.1.2",
     "url": "^0.11.0",
     "uuid": "^3.3.2"
   },
   "peerDependencies": {
+    "@applicaster/zapp-pipes-v2-client": "*",
     "@react-native-community/netinfo": "*",
-    "react": "17.0.2",
-    "react-native": "0.68.6",
+    "immer": "*",
+    "react": "*",
+    "react-native": "*",
     "react-native-safe-area-context": "*",
-    "react-native-svg": "*"
+    "react-native-svg": "*",
+    "uglify-js": "*",
+    "validate-color": "*",
+    "zustand": "*"
   }
 }

package/tsconfig.json

@@ -2,8 +2,7 @@
   "extends": "../../tsconfig.base.json",
   "compilerOptions": {
     "outDir": "./lib",
-    "rootDir": "./src"
+    "rootDir": "./"
   },
-  "exclude": ["src/**/*.test.tsx"],
-  "include": ["src/**/*"]
+  "include": ["**/*"],
 }

package/.babelrc

@@ -1,8 +0,0 @@
-{
-  "presets": ["react-native", "flow", "react-native-stage-0/decorator-support"],
-  "env": {
-    "test": {
-      "presets": [["env", { "modules": "commonjs" }], "react-native", "flow"]
-    }
-  }
-}

package/Components/Cell/CellStyles/FallbackCellStyle/index.js

@@ -1,157 +0,0 @@
-import {
-  getColor,
-  ACTIVE_COLOR,
-  BACKGROUND_COLOR,
-  MAIN_TEXT_COLOR,
-  FOCUSED_TEXT_COLOR,
-} from "../colors";
-
-const Image = "Image";
-const View = "View";
-const Text = "Text";
-
-const containerStyles = (styles) => ({
-  flex: 1,
-  flexDirection: "column",
-  width: 384,
-  height: 436,
-  borderRadius: 4,
-  backgroundColor: getColor(BACKGROUND_COLOR, styles),
-  marginRight: 48,
-  marginBottom: 48,
-});
-
-const focusedContainerStyles = (styles) => ({
-  ...containerStyles(styles),
-  backgroundColor: getColor(ACTIVE_COLOR, styles),
-});
-
-const imageStyle = {
-  width: 384,
-  height: 216,
-  borderTopRightRadius: 4,
-  borderTopLeftRadius: 4,
-};
-
-const titleContainerStyles = {
-  width: 336,
-  height: 60,
-  marginHorizontal: 24,
-  marginTop: 24,
-  overflow: "hidden",
-};
-
-const titleStyles = (styles) => ({
-  fontSize: 26,
-  color: getColor(MAIN_TEXT_COLOR, styles),
-  fontWeight: "bold",
-  fontStyle: "normal",
-});
-
-const focusedTitleStyles = (styles) => ({
-  ...titleStyles(styles),
-  color: getColor(FOCUSED_TEXT_COLOR, styles),
-});
-
-const subtitleContainerStyles = {
-  width: 336,
-  height: 88,
-  marginHorizontal: 24,
-  marginTop: 24,
-  overflow: "hidden",
-};
-
-const subtitleStyles = (styles) => ({
-  fontSize: 24,
-  fontWeight: "normal",
-  opacity: 0.8,
-  color: getColor(MAIN_TEXT_COLOR, styles),
-  fontStyle: "normal",
-});
-
-const focusedSubtitleStyles = (styles) => ({
-  ...subtitleStyles(styles),
-  color: getColor(FOCUSED_TEXT_COLOR, styles),
-});
-
-const viewTree = (state, styles) => [
-  {
-    type: View,
-    style:
-      state === "focused"
-        ? focusedContainerStyles(styles)
-        : containerStyles(styles),
-    elements: [
-      {
-        type: Image,
-        style: imageStyle,
-        data: [
-          {
-            func: "image_src_from_media_item",
-            args: ["thumbnail-small"],
-            propName: "uri",
-          },
-        ],
-      },
-      {
-        type: View,
-        style: titleContainerStyles,
-        elements: [
-          {
-            type: Text,
-            style:
-              state === "focused"
-                ? focusedTitleStyles(styles)
-                : titleStyles(styles),
-            data: [
-              {
-                func: "path",
-                args: ["title"],
-                propName: "label",
-              },
-            ],
-            additionalProps: {
-              numberOfLines: 2,
-            },
-          },
-        ],
-      },
-      {
-        type: View,
-        style: subtitleContainerStyles,
-        elements: [
-          {
-            type: Text,
-            style:
-              state === "focused"
-                ? focusedSubtitleStyles(styles)
-                : subtitleStyles(styles),
-            data: [
-              {
-                func: "path",
-                args: ["summary"],
-                propName: "label",
-              },
-            ],
-            additionalProps: {
-              numberOfLines: 3,
-            },
-          },
-        ],
-      },
-    ],
-  },
-];
-
-export const fallbackCellStyle = (styles) => ({
-  content_types: {
-    default: {
-      states: {
-        default: viewTree("default", styles),
-        focused: viewTree("focused", styles),
-        selected: viewTree("selected", styles),
-        focused_selected: viewTree("focused_selected", styles),
-      },
-    },
-  },
-});

package/Components/Cell/CellStyles/Hero/index.js

@@ -1,111 +0,0 @@
-import {
-  getColor,
-  ACTIVE_COLOR,
-  BACKGROUND_COLOR,
-  MAIN_TEXT_COLOR,
-  FOCUSED_TEXT_COLOR,
-} from "../colors";
-
-const Image = "Image";
-const View = "View";
-const Text = "Text";
-
-const containerStyles = (styles) => ({
-  flex: 1,
-  flexDirection: "column",
-  width: 1280,
-  height: 436,
-  borderRadius: 4,
-  backgroundColor: getColor(BACKGROUND_COLOR, styles),
-  marginRight: 48,
-  marginBottom: 48,
-});
-
-const focusedContainerStyles = (styles) => ({
-  ...containerStyles(styles),
-  backgroundColor: getColor(ACTIVE_COLOR, styles),
-});
-
-const imageStyle = {
-  width: 1280,
-  height: 352,
-  borderTopRightRadius: 4,
-  borderTopLeftRadius: 4,
-};
-
-const titleContainerStyles = {
-  width: 1184,
-  height: 60,
-  marginHorizontal: 24,
-  marginTop: 24,
-  overflow: "hidden",
-};
-
-const titleStyles = (styles) => ({
-  fontSize: 26,
-  color: getColor(MAIN_TEXT_COLOR, styles),
-  fontWeight: "bold",
-  fontStyle: "normal",
-});
-
-const focusedTitleStyles = (styles) => ({
-  ...titleStyles(styles),
-  color: getColor(FOCUSED_TEXT_COLOR, styles),
-});
-
-const viewTree = (state, styles) => [
-  {
-    type: View,
-    style:
-      state === "focused"
-        ? focusedContainerStyles(styles)
-        : containerStyles(styles),
-    elements: [
-      {
-        type: Image,
-        style: imageStyle,
-        data: [
-          {
-            func: "image_src_from_media_item",
-            args: ["thumbnail-small"],
-            propName: "uri",
-          },
-        ],
-      },
-      {
-        type: View,
-        style: titleContainerStyles,
-        elements: [
-          {
-            type: Text,
-            style:
-              state === "focused"
-                ? focusedTitleStyles(styles)
-                : titleStyles(styles),
-            data: [
-              {
-                func: "path",
-                args: ["title"],
-                propName: "label",
-              },
-            ],
-            additionalProps: {
-              numberOfLines: 2,
-            },
-          },
-        ],
-      },
-    ],
-  },
-];
-
-export const hero = (styles) => ({
-  content_types: {
-    default: {
-      states: {
-        default: viewTree("default", styles),
-        focused: viewTree("focused", styles),
-      },
-    },
-  },
-});