Refactor warning messages

Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>

src/main/cluster.ts

@@ -741,6 +741,6 @@ export class Cluster implements ClusterModel, ClusterState {
   }
 
   isInLocalKubeconfig() {
-    return this.kubeConfigPath.includes(storedKubeConfigFolder());
+    return this.kubeConfigPath.startsWith(storedKubeConfigFolder());
   }
 }

src/renderer/components/delete-cluster-dialog/delete-cluster-dialog.tsx

@@ -151,27 +151,44 @@ export class DeleteClusterDialog extends React.Component {
     );
   }
 
-  renderWarning() {
+  getWarningMessage() {
-    const { cluster } = dialogState;
+    const { cluster, config } = dialogState;
-    const isInternalKubeconfig = cluster.isInLocalKubeconfig();
+    const contexts = config.contexts.filter(context => context.name !== cluster.contextName);
-    let warning: JSX.Element = null;
 
-    if (this.isCurrentContext()) {
+    if (!contexts.length) {
-      warning = <p data-testid="context-warning">
+      return (
-        This will remove active context in kubeconfig. Use drop down below to&nbsp;select a&nbsp;different one.
+        <p data-testid="no-more-contexts-warning">
-      </p>;
+          This will remove the last context in kubeconfig. There will be no active context.
-    } else if (!isInternalKubeconfig) {
+        </p>
-      warning = <p>The contents of kubeconfig file will be changed!</p>;
+      );
     }
 
-    if (!warning) {
+    if (this.isCurrentContext()) {
-      return null;
+      return (
+        <p data-testid="current-context-warning">
+          This will remove active context in kubeconfig. Use drop down below to&nbsp;select a&nbsp;different one.
+        </p>
+      );
+    }
+
+    if (cluster.isInLocalKubeconfig()) {
+      return (
+        <p data-testid="internal-kubeconfig-warning">
+          Are you sure you want to delete it? It can be re-added through the copy/paste mechanism.
+        </p>
+      );
     }
 
+    return (
+      <p data-testid="kubeconfig-change-warning">The contents of kubeconfig file will be changed!</p>
+    );
+  }
+
+  renderWarning() {
     return (
       <div className={styles.warning}>
         <Icon material="warning_amber" className={styles.warningIcon}/>
-        {warning}
+        {this.getWarningMessage()}
       </div>
     );
   }