Address feedback from Pierre

- Actually remove unnecessary license optional chaining (whoops)
- Have to update engines overview test with a mock license to make up for it

x-pack/plugins/enterprise_search/public/applications/app_search/components/engine_overview/engine_overview.test.tsx

@@ -28,7 +28,7 @@ describe('EngineOverview', () => {
       const wrapper: Cheerio = render(
         <I18nProvider>
           <KibanaContext.Provider value={{ http: {} }}>
-            <LicenseContext.Provider value={{ license: {} }}>
+            <LicenseContext.Provider value={{ license: { isActive: false, type: 'trial' } }}>
               <EngineOverview />
             </LicenseContext.Provider>
           </KibanaContext.Provider>
@@ -158,7 +158,10 @@ describe('EngineOverview', () => {
     // TBH, I don't fully understand why since Enzyme's mount is supposed to
     // have act() baked in - could be because of the wrapping context provider?
     await act(async () => {
-      wrapper = mountWithContext(<EngineOverview />, { http: httpMock, license });
+      wrapper = mountWithContext(<EngineOverview />, {
+        http: httpMock,
+        license: license || { isActive: false, type: 'trial' },
+      });
     });
     if (wrapper) {
       wrapper.update(); // This seems to be required for the DOM to actually update

x-pack/plugins/enterprise_search/public/applications/shared/licensing/license_checks.ts

@@ -7,5 +7,5 @@
 import { ILicense } from '../../../../../licensing/public';
 
 export const hasPlatinumLicense = (license: ILicense) => {
-  return license?.isActive && ['platinum', 'enterprise', 'trial'].includes(license?.type as string);
+  return license.isActive && ['platinum', 'enterprise', 'trial'].includes(license.type as string);
 };