From 2af76ebcf99b6cd2c4445e0b76a86af3ce17f95f Mon Sep 17 00:00:00 2001
From: paweldomas <pawel.domas@jitsi.org>
Date: Wed, 18 Apr 2018 13:37:33 -0500
Subject: [PATCH] fix(testing): add TestHint for LargeVideo

Since the main conference container is no longer "clickable" there must
be a way for clicking on the "large video". A clickable TestHint nested
in ParticipantView makes it easier for dealing with the fact that the
click handler is not always on the same component (required for the
pinch and zoom feature to work correctly).
---
 .../components/ParticipantView.native.js      | 19 +++++++++++++++++++
 .../components/Conference.native.js           |  5 +----
 .../components/LargeVideo.native.js           |  1 +
 3 files changed, 21 insertions(+), 4 deletions(-)

diff --git a/react/features/base/participants/components/ParticipantView.native.js b/react/features/base/participants/components/ParticipantView.native.js
index 0c9d12420..3e9394b0c 100644
--- a/react/features/base/participants/components/ParticipantView.native.js
+++ b/react/features/base/participants/components/ParticipantView.native.js
@@ -13,6 +13,7 @@ import {
 } from '../../media';
 import { prefetch } from '../../../mobile/image-cache';
 import { Container, TintedView } from '../../react';
+import { TestHint } from '../../testing/components';
 import { getTrackByMediaTypeAndParticipant } from '../../tracks';
 
 import Avatar from './Avatar';
@@ -106,6 +107,14 @@ type Props = {
      */
     t: Function,
 
+    /**
+     * The test hint id which can be used to locate the {@code ParticipantView}
+     * on the jitsi-meet-torture side. If not provided, the
+     * {@code participantId} with the following format will be used:
+     * {@code `org.jitsi.meet.Participant#${participantId}`}
+     */
+    testHintId: ?string,
+
     /**
      * Indicates if the connectivity info label should be shown, if appropriate.
      * It will be shown in case the connection is interrupted.
@@ -227,6 +236,11 @@ class ParticipantView extends Component<Props> {
                 || connectionStatus
                     === JitsiParticipantConnectionStatus.INTERRUPTED;
 
+        const testHintId
+            = this.props.testHintId
+                ? this.props.testHintId
+                : `org.jitsi.meet.Participant#${this.props.participantId}`;
+
         return (
             <Container
                 onClick = { renderVideo ? undefined : onPress }
@@ -236,6 +250,11 @@ class ParticipantView extends Component<Props> {
                 }}
                 touchFeedback = { false }>
 
+                <TestHint
+                    id = { testHintId }
+                    onPress = { onPress }
+                    value = '' />
+
                 { renderVideo
                     && <VideoTrack
                         onPress = { renderVideo ? onPress : undefined }
diff --git a/react/features/conference/components/Conference.native.js b/react/features/conference/components/Conference.native.js
index 63c273ca2..1bfeaf2fc 100644
--- a/react/features/conference/components/Conference.native.js
+++ b/react/features/conference/components/Conference.native.js
@@ -188,10 +188,7 @@ class Conference extends Component<Props> {
      */
     render() {
         return (
-            <Container
-                accessibilityLabel = 'Conference'
-                accessible = { false }
-                style = { styles.conference }>
+            <Container style = { styles.conference }>
                 <StatusBar
                     barStyle = 'light-content'
                     hidden = { true }
diff --git a/react/features/large-video/components/LargeVideo.native.js b/react/features/large-video/components/LargeVideo.native.js
index a86c36624..f0bd5dd2a 100644
--- a/react/features/large-video/components/LargeVideo.native.js
+++ b/react/features/large-video/components/LargeVideo.native.js
@@ -125,6 +125,7 @@ class LargeVideo extends Component<Props, State> {
                     onPress = { onPress }
                     participantId = { _participantId }
                     style = { styles.largeVideo }
+                    testHintId = 'org.jitsi.meet.LargeVideo'
                     useConnectivityInfoLabel = { useConnectivityInfoLabel }
                     zOrder = { 0 }
                     zoomEnabled = { true } />