Merge pull request #1279 from GetStream/develop

Next Release

Author: vishalnarkhede

docusaurus/docs/reactnative/customization/custom_components.mdx

@@ -207,7 +207,7 @@ It also gives insight into what data the out of the box component is using, and
 
 ### Equality
 
-It is import to consider equality when writing `areEqual` checks.
+It is important to consider equality when writing `areEqual` checks.
 Trying to compare two functions, objects, or arrays during a re-render cycle, unless they are themselves memoized or in another way referentially stable, will fail a shallow comparison.
 For this reason in the out of the box components it is often considered _when_ the component should re-render, and the equality checks are created based on that logic.
 

examples/SampleApp/ios/SampleApp.xcodeproj/project.pbxproj

@@ -495,7 +495,7 @@
 				ASSETCATALOG_COMPILER_APPICON_NAME = AppIcon;
 				CLANG_ENABLE_MODULES = YES;
 				CODE_SIGN_IDENTITY = "iPhone Distribution";
-				CURRENT_PROJECT_VERSION = 84;
+				CURRENT_PROJECT_VERSION = 90;
 				DEVELOPMENT_TEAM = EHV7XZLAHA;
 				ENABLE_BITCODE = NO;
 				INFOPLIST_FILE = SampleApp/Info.plist;
@@ -525,7 +525,7 @@
 				CLANG_ENABLE_MODULES = YES;
 				CODE_SIGN_IDENTITY = "iPhone Distribution";
 				CODE_SIGN_STYLE = Manual;
-				CURRENT_PROJECT_VERSION = 84;
+				CURRENT_PROJECT_VERSION = 90;
 				DEVELOPMENT_TEAM = EHV7XZLAHA;
 				INFOPLIST_FILE = SampleApp/Info.plist;
 				LD_RUNPATH_SEARCH_PATHS = (

examples/SampleApp/ios/SampleApp/Info.plist

@@ -17,11 +17,11 @@
 	<key>CFBundlePackageType</key>
 	<string>APPL</string>
 	<key>CFBundleShortVersionString</key>
-	<string>1.2.0</string>
+	<string>1.3.0</string>
 	<key>CFBundleSignature</key>
 	<string>????</string>
 	<key>CFBundleVersion</key>
-	<string>84</string>
+	<string>90</string>
 	<key>LSRequiresIPhoneOS</key>
 	<true />
 	<key>NSAppTransportSecurity</key>

examples/SampleApp/ios/SampleAppTests/Info.plist

@@ -15,10 +15,10 @@
 	<key>CFBundlePackageType</key>
 	<string>BNDL</string>
 	<key>CFBundleShortVersionString</key>
-	<string>1.2.0</string>
+	<string>1.3.0</string>
 	<key>CFBundleSignature</key>
 	<string>????</string>
 	<key>CFBundleVersion</key>
-	<string>84</string>
+	<string>90</string>
 </dict>
 </plist>

package/CHANGELOG.md

@@ -908,13 +908,13 @@ We are going to incrementally implement these changes and will try to keep every
 - Upgrading @stream-io/react-native-simple-markdown to 1.2.1 [d89f012](https://github.com/GetStream/react-native-simple-markdown/commit/d89f0128e6c4f179f2afe9f9a896b289288e2afd)
 
   It fixes the issue with markdown where text like "#stream" was treated as heading, thus disallowing anyone to use hashtags in chat.
-  
-- Internal directory restructuring of components 
+
+- Internal directory restructuring of components
 
 ## [0.14.0] 2020-07-16
 
 - Fixing compatibility issues with react-native 0.63 [88721a0](https://github.com/GetStream/stream-chat-react-native/commit/88721a077b3bb3369c3d8dbde9451d84f6761d87)
-- Updating stream-chat to 0.13.x [4af1b1d](https://github.com/GetStream/stream-chat-react-native/commit/4af1b1d63a3c7dec662f8c75426ea7a7082f26a4) 
+- Updating stream-chat to 0.13.x [4af1b1d](https://github.com/GetStream/stream-chat-react-native/commit/4af1b1d63a3c7dec662f8c75426ea7a7082f26a4)
 
 ## [0.13.1] 2020-07-14
 
@@ -1263,7 +1263,7 @@ From now on, autocomplete input (specifically mentions autocomplete) will be api
 
 ## [0.7.0] 2020-03-17
 
-- Introducing internationalization (i18n) support for the sdk https://github.com/GetStream/stream-chat-react-native/blob/master/src/components/docs/Streami18n.md
+- Introducing internationalization (i18n) support for the sdk https://github.com/GetStream/stream-chat-react-native/blob/master/package/src/components/docs/Streami18n.md
 
 ## [0.6.6] 2020-02-20
 

package/src/components/Attachment/Giphy.tsx

@@ -201,6 +201,7 @@ const GiphyWithContext = <
         { backgroundColor: white, borderColor: `${black}0D` },
         selectionContainer,
       ]}
+      testID='giphy-action-attachment'
     >
       <View style={[styles.header, header]}>
         <GiphyIcon />
@@ -229,6 +230,7 @@ const GiphyWithContext = <
               { borderColor: grey_gainsboro, borderRightWidth: 1 },
               buttonContainer,
             ]}
+            testID={`${actions?.[2].value}-action-button`}
           >
             <Text style={[styles.cancel, { color: grey }, cancel]}>{actions?.[2].text}</Text>
           </TouchableOpacity>
@@ -243,6 +245,7 @@ const GiphyWithContext = <
               { borderColor: grey_gainsboro, borderRightWidth: 1 },
               buttonContainer,
             ]}
+            testID={`${actions?.[1].value}-action-button`}
           >
             <Text style={[styles.shuffle, { color: grey }, shuffle]}>{actions?.[1].text}</Text>
           </TouchableOpacity>
@@ -253,6 +256,7 @@ const GiphyWithContext = <
               }
             }}
             style={[styles.buttonContainer, { borderColor: grey_gainsboro }, buttonContainer]}
+            testID={`${actions?.[0].value}-action-button`}
           >
             <Text style={[styles.send, { color: accent_blue }, send]}>{actions?.[0].text}</Text>
           </TouchableOpacity>

package/src/components/Attachment/__tests__/Attachment.test.js

@@ -10,9 +10,7 @@ import {
   generateAudioAttachment,
   generateCardAttachment,
   generateFileAttachment,
-  generateGiphyAttachment,
   generateImageAttachment,
-  generateImgurAttachment,
 } from '../../../mock-builders/generator/attachment';
 import { generateMessage } from '../../../mock-builders/generator/message';
 import { Attachment } from '../Attachment';
@@ -53,65 +51,6 @@ describe('Attachment', () => {
     });
   });
 
-  it('should render Card component for "imgur" type attachment', async () => {
-    const attachment = generateImgurAttachment();
-    const { getByTestId } = render(getAttachmentComponent({ attachment }));
-
-    await waitFor(() => {
-      expect(getByTestId('giphy-attachment')).toBeTruthy();
-    });
-  });
-
-  it('should render Card component for "giphy" type attachment', async () => {
-    const attachment = generateGiphyAttachment();
-    const { getByTestId } = render(getAttachmentComponent({ attachment }));
-
-    await waitFor(() => {
-      expect(getByTestId('giphy-attachment')).toBeTruthy();
-    });
-  });
-
-  it('"giphy" attachment size should be customisable', async () => {
-    const attachment = generateGiphyAttachment();
-    attachment.giphy = {
-      fixed_height: {
-        height: '200',
-        url: 'https://media1.giphy.com/media/test/fixed_height.gif',
-        width: '375',
-      },
-      original: {
-        height: '256',
-        url: 'https://media1.giphy.com/media/test/original.gif',
-        width: '480',
-      },
-    };
-    const { getByTestId: getByTestIdFixedHeight } = render(
-      getAttachmentComponent({ attachment, giphyVersion: 'fixed_height' }),
-    );
-    const { getByTestId: getByTestIdOriginal } = render(
-      getAttachmentComponent({ attachment, giphyVersion: 'original' }),
-    );
-    await waitFor(() => {
-      const checkImageProps = (imageProps, specificSizedGiphyData) => {
-        let imageStyle = imageProps.style;
-        if (Array.isArray(imageStyle)) {
-          imageStyle = Object.assign({}, ...imageStyle);
-        }
-        expect(imageStyle.height).toBe(parseFloat(specificSizedGiphyData.height));
-        expect(imageStyle.width).toBe(parseFloat(specificSizedGiphyData.width));
-        expect(imageProps.source.uri).toBe(specificSizedGiphyData.url);
-      };
-      checkImageProps(
-        getByTestIdFixedHeight('giphy-attachment-image').props,
-        attachment.giphy.fixed_height,
-      );
-      checkImageProps(
-        getByTestIdOriginal('giphy-attachment-image').props,
-        attachment.giphy.original,
-      );
-    });
-  });
-
   it('should render UrlPreview component if attachment has title_link or og_scrape_url', async () => {
     const attachment = generateImageAttachment({
       og_scrape_url: uuidv4(),