fix: fix review issues

lib/gui/event-source.ts

@@ -8,19 +8,21 @@ export class EventSource {
         this._connections = [];
     }
 
+    private _write(connection: Response, event: string, data?: unknown): void {
+        connection.write('event: ' + event + '\n');
+        connection.write('data: ' + stringify(data) + '\n');
+        connection.write('\n\n');
+    }
+
     addConnection(connection: Response): void {
         this._connections.push(connection);
 
-        connection.write('event: ' + ClientEvents.CONNECTED + '\n');
-        connection.write('data: 1\n');
-        connection.write('\n\n');
+        this._write(connection, ClientEvents.CONNECTED, 1);
     }
 
     emit(event: string, data?: unknown): void {
-        this._connections.forEach(function(connection) {
-            connection.write('event: ' + event + '\n');
-            connection.write('data: ' + stringify(data) + '\n');
-            connection.write('\n\n');
+        this._connections.forEach((connection) => {
+            this._write(connection, event, data);
         });
     }
 }

lib/static/modules/actions/gui-server-connection.ts

@@ -3,7 +3,6 @@ import actionNames from '@/static/modules/action-names';
 
 type SetGuiServerConnectionStatusAction = Action<typeof actionNames.SET_GUI_SERVER_CONNECTION_STATUS, {
     isConnected: boolean;
-    wasDisconnected?: boolean;
 }>;
 export const setGuiServerConnectionStatus = (payload: SetGuiServerConnectionStatusAction['payload']): SetGuiServerConnectionStatusAction =>
     ({type: actionNames.SET_GUI_SERVER_CONNECTION_STATUS, payload});

lib/static/modules/default-state.ts

@@ -128,8 +128,7 @@ export default Object.assign({config: configDefaults}, {
             currentDirection: SortDirection.Asc
         },
         guiServerConnection: {
-            isConnected: false,
-            wasDisconnected: false
+            isConnected: false
         }
     },
     ui: {

lib/static/modules/reducers/gui-server-connection.ts

@@ -9,8 +9,7 @@ export default (state: State, action: SomeAction): State => {
             return applyStateUpdate(state, {
                 app: {
                     guiServerConnection: {
-                        isConnected: action.payload.isConnected,
-                        wasDisconnected: action.payload.wasDisconnected
+                        isConnected: action.payload.isConnected
                     }
                 }
             });

lib/static/new-ui/app/gui.tsx

@@ -26,7 +26,7 @@ function Gui(): ReactNode {
             store.dispatch({type: actionNames.UPDATE_LOADING_TITLE, payload: 'Lost connection to Testplane UI server. Trying to reconnect'});
             store.dispatch({type: actionNames.UPDATE_LOADING_VISIBILITY, payload: true});
 
-            store.dispatch(setGuiServerConnectionStatus({isConnected: false, wasDisconnected: true}));
+            store.dispatch(setGuiServerConnectionStatus({isConnected: false}));
         };
 
         eventSource.addEventListener(ClientEvents.BEGIN_SUITE, (e) => {

lib/static/new-ui/components/InfoPanel/index.tsx

@@ -26,7 +26,7 @@ export function InfoPanel(): ReactNode {
 
     sections.push(<PanelSection
         title={'Testplane UI v' + version}
-        description={<span>To get the most out of Testplane UI, try to keep it updated to the latest version. Check out fresh releases <a href={'https://github.com/gemini-testing/html-reporter/releases'}>on GitHub</a>.</span>}
+        description={<span>To get the most out of Testplane UI, try to keep it updated to the latest version. Check out fresh <a href={'https://github.com/gemini-testing/html-reporter/releases'}>releases on GitHub</a>.</span>}
     />);
 
     const timestamp = useSelector(state => state.timestamp);
@@ -56,11 +56,13 @@ export function InfoPanel(): ReactNode {
         </div>
     </PanelSection>);
 
+    const lastSection = sections.pop();
+
     return <AsidePanel title={'Info'} className={styles.infoPanel}>
-        {sections.slice(0, -1).map((section, index) => <React.Fragment key={index}>
+        {sections.map((section, index) => <React.Fragment key={index}>
             {section}
             <Divider orientation={'horizontal'} className={styles.divider}/>
         </React.Fragment>)}
-        {sections[sections.length - 1]}
+        {lastSection}
     </AsidePanel>;
 }

lib/static/new-ui/types/store.ts

@@ -269,7 +269,6 @@ export interface State {
         };
         guiServerConnection: {
             isConnected: boolean;
-            wasDisconnected: boolean | undefined;
         };
     };
     ui: {