6 files changed, 47 insertions, 43 deletions
diff --git a/gui/src/renderer/components/Settings.tsx b/gui/src/renderer/components/Settings.tsx
index 8f68a1c462..34fb3eb0c1 100644
--- a/gui/src/renderer/components/Settings.tsx
+++ b/gui/src/renderer/components/Settings.tsx
@@ -26,8 +26,10 @@ export default function Support() {
 
   const loginState = useSelector((state) => state.account.status);
   const connectedToDaemon = useSelector((state) => state.userInterface.connectedToDaemon);
+  const isMacOs13OrNewer = useSelector((state) => state.userInterface.isMacOs13OrNewer);
 
   const showSubSettings = loginState.type === 'ok' && connectedToDaemon;
+  const showSplitTunneling = window.env.platform !== 'darwin' || isMacOs13OrNewer;
 
   return (
     <BackAction action={history.pop}>
@@ -59,7 +61,7 @@ export default function Support() {
                         <VpnSettingsButton />
                       </Cell.Group>
 
-                      {(window.env.platform === 'linux' || window.env.platform === 'win32') && (
+                      {showSplitTunneling && (
                         <Cell.Group>
                           <SplitTunnelingButton />
                         </Cell.Group>
diff --git a/gui/src/renderer/components/SplitTunnelingSettings.tsx b/gui/src/renderer/components/SplitTunnelingSettings.tsx
index cb4de70deb..2cee823b3e 100644
--- a/gui/src/renderer/components/SplitTunnelingSettings.tsx
+++ b/gui/src/renderer/components/SplitTunnelingSettings.tsx
@@ -6,7 +6,7 @@ import { colors, strings } from '../../config.json';
 import {
   IApplication,
   ILinuxSplitTunnelingApplication,
-  IWindowsApplication,
+  ISplitTunnelingApplication,
 } from '../../shared/application-types';
 import { messages } from '../../shared/gettext';
 import { useAppContext } from '../context';
@@ -91,10 +91,8 @@ function PlatformSpecificSplitTunnelingSettings(props: IPlatformSplitTunnelingSe
   switch (window.env.platform) {
     case 'linux':
       return <LinuxSplitTunnelingSettings {...props} />;
-    case 'win32':
-      return <WindowsSplitTunnelingSettings {...props} />;
     default:
-      throw new Error(`Split tunneling not implemented on ${window.env.platform}`);
+      return <SplitTunnelingSettings {...props} />;
   }
 }
 
@@ -300,12 +298,12 @@ function LinuxApplicationRow(props: ILinuxApplicationRowProps) {
   );
 }
 
-export function WindowsSplitTunnelingSettings(props: IPlatformSplitTunnelingSettingsProps) {
+export function SplitTunnelingSettings(props: IPlatformSplitTunnelingSettingsProps) {
   const {
     addSplitTunnelingApplication,
     removeSplitTunnelingApplication,
     forgetManuallyAddedSplitTunnelingApplication,
-    getWindowsSplitTunnelingApplications,
+    getSplitTunnelingApplications,
     setSplitTunnelingState,
   } = useAppContext();
   const splitTunnelingEnabled = useSelector((state: IReduxState) => state.settings.splitTunneling);
@@ -314,13 +312,13 @@ export function WindowsSplitTunnelingSettings(props: IPlatformSplitTunnelingSett
   );
 
   const [searchTerm, setSearchTerm] = useState('');
-  const [applications, setApplications] = useState<IWindowsApplication[]>();
+  const [applications, setApplications] = useState<ISplitTunnelingApplication[]>();
   useAsyncEffect(async () => {
-    const { fromCache, applications } = await getWindowsSplitTunnelingApplications();
+    const { fromCache, applications } = await getSplitTunnelingApplications();
     setApplications(applications);
 
     if (fromCache) {
-      const { applications } = await getWindowsSplitTunnelingApplications(true);
+      const { applications } = await getSplitTunnelingApplications(true);
       setApplications(applications);
     }
   }, []);
@@ -345,7 +343,7 @@ export function WindowsSplitTunnelingSettings(props: IPlatformSplitTunnelingSett
   }, [applications, splitTunnelingApplications, searchTerm]);
 
   const addApplication = useCallback(
-    async (application: IWindowsApplication | string) => {
+    async (application: ISplitTunnelingApplication | string) => {
       if (!splitTunnelingEnabled) {
         await setSplitTunnelingState(true);
       }
@@ -354,26 +352,26 @@ export function WindowsSplitTunnelingSettings(props: IPlatformSplitTunnelingSett
     [addSplitTunnelingApplication, splitTunnelingEnabled, setSplitTunnelingState],
   );
 
-  const addApplicationAndUpdate = useCallback(
-    async (application: IWindowsApplication | string) => {
+  const addBrowsedForApplication = useCallback(
+    async (application: string) => {
       await addApplication(application);
-      const { applications } = await getWindowsSplitTunnelingApplications();
+      const { applications } = await getSplitTunnelingApplications();
       setApplications(applications);
     },
-    [addApplication, getWindowsSplitTunnelingApplications],
+    [addApplication, getSplitTunnelingApplications],
   );
 
   const forgetManuallyAddedApplicationAndUpdate = useCallback(
-    async (application: IWindowsApplication) => {
+    async (application: ISplitTunnelingApplication) => {
       await forgetManuallyAddedSplitTunnelingApplication(application);
-      const { applications } = await getWindowsSplitTunnelingApplications();
+      const { applications } = await getSplitTunnelingApplications();
       setApplications(applications);
     },
-    [forgetManuallyAddedSplitTunnelingApplication, getWindowsSplitTunnelingApplications],
+    [forgetManuallyAddedSplitTunnelingApplication, getSplitTunnelingApplications],
   );
 
   const removeApplication = useCallback(
-    async (application: IWindowsApplication) => {
+    async (application: ISplitTunnelingApplication) => {
       if (!splitTunnelingEnabled) {
         await setSplitTunnelingState(true);
       }
@@ -385,8 +383,8 @@ export function WindowsSplitTunnelingSettings(props: IPlatformSplitTunnelingSett
   const filePickerCallback = useFilePicker(
     messages.pgettext('split-tunneling-view', 'Add'),
     props.setBrowsing,
-    addApplicationAndUpdate,
-    { name: 'Executables', extensions: ['exe', 'lnk'] },
+    addBrowsedForApplication,
+    getFilePickerOptionsForPlatform(),
   );
 
   const addWithFilePicker = useCallback(async () => {
@@ -395,21 +393,17 @@ export function WindowsSplitTunnelingSettings(props: IPlatformSplitTunnelingSett
   }, [filePickerCallback, props.scrollToTop]);
 
   const excludedRowRenderer = useCallback(
-    (application: IWindowsApplication) => (
-      <WindowsApplicationRow application={application} onRemove={removeApplication} />
+    (application: ISplitTunnelingApplication) => (
+      <ApplicationRow application={application} onRemove={removeApplication} />
     ),
     [removeApplication],
   );
 
   const includedRowRenderer = useCallback(
-    (application: IWindowsApplication) => {
+    (application: ISplitTunnelingApplication) => {
       const onForget = application.deletable ? forgetManuallyAddedApplicationAndUpdate : undefined;
       return (
-        <WindowsApplicationRow
-          application={application}
-          onAdd={addApplication}
-          onDelete={onForget}
-        />
+        <ApplicationRow application={application} onAdd={addApplication} onDelete={onForget} />
       );
     },
     [addApplication, forgetManuallyAddedApplicationAndUpdate],
@@ -452,6 +446,7 @@ export function WindowsSplitTunnelingSettings(props: IPlatformSplitTunnelingSett
       <Accordion expanded={showSplitSection}>
         <Cell.Section sectionTitle={excludedTitle}>
           <ApplicationList
+            data-testid="split-applications"
             applications={filteredSplitApplications}
             rowRenderer={excludedRowRenderer}
           />
@@ -461,6 +456,7 @@ export function WindowsSplitTunnelingSettings(props: IPlatformSplitTunnelingSett
       <Accordion expanded={showNonSplitSection}>
         <Cell.Section sectionTitle={allTitle}>
           <ApplicationList
+            data-testid="non-split-applications"
             applications={filteredNonSplitApplications}
             rowRenderer={includedRowRenderer}
           />
@@ -490,6 +486,7 @@ export function WindowsSplitTunnelingSettings(props: IPlatformSplitTunnelingSett
 interface IApplicationListProps<T extends IApplication> {
   applications: T[] | undefined;
   rowRenderer: (application: T) => React.ReactElement;
+  'data-testid'?: string;
 }
 
 function ApplicationList<T extends IApplication>(props: IApplicationListProps<T>) {
@@ -501,8 +498,11 @@ function ApplicationList<T extends IApplication>(props: IApplicationListProps<T>
     );
   } else {
     return (
-      <StyledListContainer>
-        <List items={props.applications} getKey={applicationGetKey}>
+      <StyledListContainer data-testid={props['data-testid']}>
+        <List
+          data-testid={props['data-testid']}
+          items={props.applications.sort((a, b) => a.name.localeCompare(b.name))}
+          getKey={applicationGetKey}>
           {props.rowRenderer}
         </List>
       </StyledListContainer>
@@ -514,14 +514,14 @@ function applicationGetKey<T extends IApplication>(application: T): string {
   return application.absolutepath;
 }
 
-interface IWindowsApplicationRowProps {
-  application: IWindowsApplication;
-  onAdd?: (application: IWindowsApplication) => void;
-  onRemove?: (application: IWindowsApplication) => void;
-  onDelete?: (application: IWindowsApplication) => void;
+interface IApplicationRowProps {
+  application: ISplitTunnelingApplication;
+  onAdd?: (application: ISplitTunnelingApplication) => void;
+  onRemove?: (application: ISplitTunnelingApplication) => void;
+  onDelete?: (application: ISplitTunnelingApplication) => void;
 }
 
-function WindowsApplicationRow(props: IWindowsApplicationRowProps) {
+function ApplicationRow(props: IApplicationRowProps) {
   const onAdd = useCallback(() => {
     props.onAdd?.(props.application);
   }, [props.onAdd, props.application]);
@@ -576,3 +576,11 @@ function WindowsApplicationRow(props: IWindowsApplicationRowProps) {
 function includesSearchTerm(application: IApplication, searchTerm: string) {
   return application.name.toLowerCase().includes(searchTerm.toLowerCase());
 }
+
+function getFilePickerOptionsForPlatform():
+  | { name: string; extensions: Array<string> }
+  | undefined {
+  return window.env.platform === 'win32'
+    ? { name: 'Executables', extensions: ['exe', 'lnk'] }
+    : undefined;
+}
diff --git a/gui/src/renderer/components/Switch.tsx b/gui/src/renderer/components/Switch.tsx
index f87bf2a52b..595bc422a2 100644
--- a/gui/src/renderer/components/Switch.tsx
+++ b/gui/src/renderer/components/Switch.tsx
@@ -5,9 +5,7 @@ import { colors } from '../../config.json';
 
 interface IProps {
   id?: string;
-  // eslint-disable-next-line @typescript-eslint/naming-convention
   'aria-labelledby'?: string;
-  // eslint-disable-next-line @typescript-eslint/naming-convention
   'aria-describedby'?: string;
   isOn: boolean;
   onChange?: (isOn: boolean) => void;
diff --git a/gui/src/renderer/components/TransitionContainer.tsx b/gui/src/renderer/components/TransitionContainer.tsx
index 9773c64512..3e7007e3c8 100644
--- a/gui/src/renderer/components/TransitionContainer.tsx
+++ b/gui/src/renderer/components/TransitionContainer.tsx
@@ -48,7 +48,6 @@ interface StyledTransitionContentProps {
 
 export const StyledTransitionContent = styled.div.attrs<
   StyledTransitionContentProps,
-  // eslint-disable-next-line @typescript-eslint/naming-convention
   { 'data-testid': string }
 >({
   'data-testid': 'transition-content',
diff --git a/gui/src/renderer/components/cell/Selector.tsx b/gui/src/renderer/components/cell/Selector.tsx
index 5e5d685a59..6bfeeb3887 100644
--- a/gui/src/renderer/components/cell/Selector.tsx
+++ b/gui/src/renderer/components/cell/Selector.tsx
@@ -16,7 +16,6 @@ export interface SelectorItem<T> {
   label: string;
   value: T;
   disabled?: boolean;
-  // eslint-disable-next-line @typescript-eslint/naming-convention
   'data-testid'?: string;
 }
 
@@ -136,7 +135,6 @@ interface SelectorCellProps<T> {
   onSelect: (value: T) => void;
   children: React.ReactNode | Array<React.ReactNode>;
   forwardedRef?: React.Ref<HTMLButtonElement>;
-  // eslint-disable-next-line @typescript-eslint/naming-convention
   'data-testid'?: string;
 }
 
diff --git a/gui/src/renderer/components/cell/SettingsSelect.tsx b/gui/src/renderer/components/cell/SettingsSelect.tsx
index 4286cda1e2..f0d918ded6 100644
--- a/gui/src/renderer/components/cell/SettingsSelect.tsx
+++ b/gui/src/renderer/components/cell/SettingsSelect.tsx
@@ -87,7 +87,6 @@ interface SettingsSelectProps<T extends string> {
   items: Array<SettingsSelectItem<T>>;
   onUpdate: (value: T) => void;
   direction?: 'down' | 'up';
-  // eslint-disable-next-line @typescript-eslint/naming-convention
   'data-testid'?: string;
 }