Add internet connection indicator

thyeggman · thyeggman · commit 56dbe2e136ed · 2023-02-27T17:31:08.000-08:00
diff --git a/src/treeViews/currentBranch.ts b/src/treeViews/currentBranch.ts
@@ -5,24 +5,13 @@ import {CurrentBranchRepoNode} from "./current-branch/currentBranchRepoNode";
 
 import {log, logDebug} from "../log";
 import {RunStore} from "../store/store";
-import {NoRunForBranchNode} from "./current-branch/noRunForBranchNode";
 import {AttemptNode} from "./shared/attemptNode";
-import {NoWorkflowJobsNode} from "./shared/noWorkflowJobsNode";
 import {PreviousAttemptsNode} from "./shared/previousAttemptsNode";
 import {WorkflowJobNode} from "./shared/workflowJobNode";
 import {WorkflowRunNode} from "./shared/workflowRunNode";
 import {WorkflowRunTreeDataProvider} from "./workflowRunTreeDataProvider";
-import {WorkflowStepNode} from "./workflows/workflowStepNode";
-
-type CurrentBranchTreeNode =
-  | CurrentBranchRepoNode
-  | WorkflowRunNode
-  | PreviousAttemptsNode
-  | AttemptNode
-  | WorkflowJobNode
-  | NoWorkflowJobsNode
-  | WorkflowStepNode
-  | NoRunForBranchNode;
+import {NoInternetConnectivityNode} from "./shared/noInternetConnectivityNode";
+import {CurrentBranchTreeNode} from "./settings/types";
 
 export class CurrentBranchTreeProvider
   extends WorkflowRunTreeDataProvider
@@ -51,7 +40,7 @@ export class CurrentBranchTreeProvider
     if (!element) {
       const gitHubContext = await getGitHubContext();
       if (!gitHubContext) {
-        return [];
+        return [new NoInternetConnectivityNode()];
       }
 
       if (gitHubContext.repos.length === 1) {
diff --git a/src/treeViews/settings.ts b/src/treeViews/settings.ts
@@ -14,6 +14,7 @@ import {EnvironmentSecretsNode} from "./settings/environmentSecretsNode";
 import {EnvironmentVariablesNode} from "./settings/environmentVariablesNode";
 import {OrgVariablesNode} from "./settings/orgVariablesNode";
 import {OrgSecretsNode} from "./settings/orgSecretsNode";
+import { NoInternetConnectivityNode } from "./shared/noInternetConnectivityNode";
 
 export class SettingsTreeProvider implements vscode.TreeDataProvider<SettingsExplorerNode> {
   private _onDidChangeTreeData = new vscode.EventEmitter<SettingsExplorerNode | null>();
@@ -30,7 +31,7 @@ export class SettingsTreeProvider implements vscode.TreeDataProvider<SettingsExp
   async getChildren(element?: SettingsExplorerNode | undefined): Promise<SettingsExplorerNode[]> {
     const gitHubContext = await getGitHubContext();
     if (!gitHubContext) {
-      return [];
+      return [new NoInternetConnectivityNode()];
     }
 
     if (!element) {
diff --git a/src/treeViews/settings/types.ts b/src/treeViews/settings/types.ts
@@ -11,6 +11,18 @@ import {RepoVariablesNode} from "./repoVariablesNode";
 import {OrgVariablesNode} from "./orgVariablesNode";
 import {OrgSecretsNode} from "./orgSecretsNode";
 import {RepoSecretsNode} from "./repoSecretsNode";
+import {NoGitHubRepositoryNode} from "../shared/noGitHubRepositoryNode";
+import {NoWorkflowJobsNode} from "../shared/noWorkflowJobsNode";
+import {WorkflowStepNode} from "../workflows/workflowStepNode";
+import {AttemptNode} from "../shared/attemptNode";
+import {AuthenticationNode} from "../shared/authenticationNode";
+import {PreviousAttemptsNode} from "../shared/previousAttemptsNode";
+import {WorkflowJobNode} from "../shared/workflowJobNode";
+import {WorkflowRunNode} from "../shared/workflowRunNode";
+import {WorkflowNode} from "../workflows/workflowNode";
+import {NoInternetConnectivityNode} from "../shared/noInternetConnectivityNode";
+import {NoRunForBranchNode} from "../current-branch/noRunForBranchNode";
+import {CurrentBranchRepoNode} from "../current-branch/currentBranchRepoNode";
 
 export type SettingsExplorerNode =
   | SecretsNode
@@ -25,4 +37,28 @@ export type SettingsExplorerNode =
   | RepoVariablesNode
   | VariableNode
   | VariablesNode
-  | EmptyNode;
+  | EmptyNode
+  | NoInternetConnectivityNode;
+
+export type WorkflowsTreeNode =
+  | AuthenticationNode
+  | NoGitHubRepositoryNode
+  | WorkflowNode
+  | WorkflowRunNode
+  | PreviousAttemptsNode
+  | AttemptNode
+  | WorkflowJobNode
+  | NoWorkflowJobsNode
+  | WorkflowStepNode
+  | NoInternetConnectivityNode;
+
+export type CurrentBranchTreeNode =
+  | CurrentBranchRepoNode
+  | WorkflowRunNode
+  | PreviousAttemptsNode
+  | AttemptNode
+  | WorkflowJobNode
+  | NoWorkflowJobsNode
+  | WorkflowStepNode
+  | NoRunForBranchNode
+  | NoInternetConnectivityNode;
diff --git a/src/treeViews/shared/noInternetConnectivityNode.ts b/src/treeViews/shared/noInternetConnectivityNode.ts
@@ -0,0 +1,11 @@
+import * as vscode from "vscode";
+
+/**
+ * When no github.com remote can be found in the current workspace.
+ */
+export class NoInternetConnectivityNode extends vscode.TreeItem {
+  constructor() {
+    super("No internet connection");
+    this.iconPath = new vscode.ThemeIcon("notebook-state-error");
+  }
+}
diff --git a/src/treeViews/workflows.ts b/src/treeViews/workflows.ts
@@ -1,32 +1,19 @@
 import * as vscode from "vscode";
 
-import {log, logDebug, logError} from "../log";
-import {getWorkflowNodes, WorkflowsRepoNode} from "./workflows/workflowsRepoNode";
-
 import {getGitHubContext} from "../git/repository";
+import {log, logDebug, logError} from "../log";
 import {RunStore} from "../store/store";
 import {AttemptNode} from "./shared/attemptNode";
 import {AuthenticationNode} from "./shared/authenticationNode";
 import {ErrorNode} from "./shared/errorNode";
-import {NoGitHubRepositoryNode} from "./shared/noGitHubRepositoryNode";
-import {NoWorkflowJobsNode} from "./shared/noWorkflowJobsNode";
 import {PreviousAttemptsNode} from "./shared/previousAttemptsNode";
 import {WorkflowJobNode} from "./shared/workflowJobNode";
 import {WorkflowRunNode} from "./shared/workflowRunNode";
 import {WorkflowRunTreeDataProvider} from "./workflowRunTreeDataProvider";
 import {WorkflowNode} from "./workflows/workflowNode";
-import {WorkflowStepNode} from "./workflows/workflowStepNode";
-
-type WorkflowsTreeNode =
-  | AuthenticationNode
-  | NoGitHubRepositoryNode
-  | WorkflowNode
-  | WorkflowRunNode
-  | PreviousAttemptsNode
-  | AttemptNode
-  | WorkflowJobNode
-  | NoWorkflowJobsNode
-  | WorkflowStepNode;
+import {getWorkflowNodes, WorkflowsRepoNode} from "./workflows/workflowsRepoNode";
+import {WorkflowsTreeNode} from "./settings/types"
+import { NoInternetConnectivityNode } from "./shared/noInternetConnectivityNode";
 
 export class WorkflowsTreeProvider
   extends WorkflowRunTreeDataProvider
@@ -59,8 +46,8 @@ export class WorkflowsTreeProvider
       try {
         const gitHubContext = await getGitHubContext();
         if (!gitHubContext) {
-          logDebug("could not get github context");
-          return [];
+          logDebug("could not get github context for workflows");
+          return [new NoInternetConnectivityNode()];
         }
 
         if (gitHubContext.repos.length > 0) {
