lint and test

Signed-off-by: Kerry Archibald <kerrya@element.io>

res/css/components/views/location/_ShareType.scss

@@ -1,3 +1,18 @@
+/*
+Copyright 2022 The Matrix.org Foundation C.I.C.
+
+Licensed under the Apache License, Version 2.0 (the "License");
+you may not use this file except in compliance with the License.
+You may obtain a copy of the License at
+
+    http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing, software
+distributed under the License is distributed on an "AS IS" BASIS,
+WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+See the License for the specific language governing permissions and
+limitations under the License.
+*/
 
 .mx_ShareType {
     flex: 1;
@@ -34,7 +49,7 @@
     justify-content: flex-start;
     padding: $spacing-8 $spacing-20;
     margin-top: $spacing-12;
-    
+
     color: $primary-content;
     border: 1px solid $quinary-content;
     border-radius: 8px;

src/components/views/location/ShareType.tsx

@@ -21,6 +21,7 @@ import { _t } from '../../../languageHandler';
 import { OwnProfileStore } from '../../../stores/OwnProfileStore';
 import { getUserNameColorClass } from '../../../utils/FormattingUtils';
 import BaseAvatar from '../avatars/BaseAvatar';
+import AccessibleButton from '../elements/AccessibleButton';
 import Heading from '../typography/Heading';
 
 const UserAvatar = () => {
@@ -55,7 +56,8 @@ export enum LocationShareType {
 type ShareTypeOptionProps = HTMLAttributes<HTMLButtonElement> & { label: string, shareType: LocationShareType };
 const ShareTypeOption: React.FC<ShareTypeOptionProps> = ({
     onClick, label, shareType, ...rest
-}) => <button
+}) => <AccessibleButton
+    element='button'
     className='mx_ShareType_option'
     onClick={onClick}
     // not yet implemented
@@ -66,7 +68,7 @@ const ShareTypeOption: React.FC<ShareTypeOptionProps> = ({
     { shareType === LocationShareType.Pin && <LocationPinIcon /> }
 
     { label }
-</button>;
+</AccessibleButton>;
 
 interface Props {
     setShareType: (shareType: LocationShareType) => void;

test/components/views/location/LocationShareMenu-test.tsx

@@ -29,6 +29,10 @@ import { MatrixClientPeg } from '../../../../src/MatrixClientPeg';
 import { LocationShareType } from '../../../../src/components/views/location/ShareType';
 import { findByTestId } from '../../../test-utils';
 
+jest.mock('../../../../src/components/views/messages/MLocationBody', () => ({
+    findMapStyleUrl: jest.fn().mockReturnValue('test'),
+}));
+
 jest.mock('../../../../src/settings/SettingsStore', () => ({
     getValue: jest.fn(),
     monitorSetting: jest.fn(),
@@ -47,6 +51,7 @@ describe('<LocationShareMenu />', () => {
     const userId = '@ernie:server.org';
     const mockClient = {
         on: jest.fn(),
+        removeListener: jest.fn(),
         getUserId: jest.fn().mockReturnValue(userId),
         getClientWellKnown: jest.fn().mockResolvedValue({
             map_style_url: 'maps.com',