From 139a28e0ea063120a6adc4880f93f9c4d14bbdd4 Mon Sep 17 00:00:00 2001
From: Anmol Sethi <hi@nhooyr.io>
Date: Tue, 3 Nov 2020 21:14:19 -0500
Subject: [PATCH] plugin.ts: Describe private counterpart functions

Addresses Will's comments.
---
 src/node/plugin.ts            | 12 ++++++++++++
 src/node/routes/apps.ts       |  3 +++
 test/plugin.test.ts           |  6 +++---
 test/test-plugin/src/index.ts |  4 ++--
 4 files changed, 20 insertions(+), 5 deletions(-)

diff --git a/src/node/plugin.ts b/src/node/plugin.ts
index 8d5e552b..ce424770 100644
--- a/src/node/plugin.ts
+++ b/src/node/plugin.ts
@@ -110,6 +110,13 @@ export class PluginAPI {
     await this._loadPlugins(path.join(__dirname, "../../plugins"))
   }
 
+  /**
+   * _loadPlugins is the counterpart to loadPlugins.
+   *
+   * It differs in that it loads all plugins in a single
+   * directory whereas loadPlugins uses all available directories
+   * as documented.
+   */
   private async _loadPlugins(dir: string): Promise<void> {
     try {
       const entries = await fsp.readdir(dir, { withFileTypes: true })
@@ -149,6 +156,11 @@ export class PluginAPI {
     }
   }
 
+  /**
+   * _loadPlugin is the counterpart to loadPlugin and actually
+   * loads the plugin now that we know there is no duplicate
+   * and that the package.json has been read.
+   */
   private _loadPlugin(dir: string, packageJSON: PackageJSON): Plugin {
     dir = path.resolve(dir)
 
diff --git a/src/node/routes/apps.ts b/src/node/routes/apps.ts
index 970bd3cb..c678f2fe 100644
--- a/src/node/routes/apps.ts
+++ b/src/node/routes/apps.ts
@@ -1,6 +1,9 @@
 import * as express from "express"
 import { PluginAPI } from "../plugin"
 
+/**
+ * Implements the /api/applications endpoint
+ */
 export function router(papi: PluginAPI): express.Router {
   const router = express.Router()
 
diff --git a/test/plugin.test.ts b/test/plugin.test.ts
index 1e63fa8d..8c419139 100644
--- a/test/plugin.test.ts
+++ b/test/plugin.test.ts
@@ -17,10 +17,10 @@ describe("plugin", () => {
     assert.deepEqual(
       [
         {
-          name: "test app",
+          name: "Test App",
           version: "4.0.0",
 
-          description: "my description",
+          description: "This app does XYZ.",
           iconPath: "/icon.svg",
 
           plugin: {
@@ -28,8 +28,8 @@ describe("plugin", () => {
             version: "1.0.0",
             modulePath: path.join(__dirname, "test-plugin"),
 
-            description: "Plugin used in code-server tests.",
             displayName: "Test Plugin",
+            description: "Plugin used in code-server tests.",
             path: "/test-plugin",
           },
         },
diff --git a/test/test-plugin/src/index.ts b/test/test-plugin/src/index.ts
index 6435592b..f9f316b4 100644
--- a/test/test-plugin/src/index.ts
+++ b/test/test-plugin/src/index.ts
@@ -21,11 +21,11 @@ export function router(): express.Router {
 export function applications(): pluginapi.Application[] {
   return [
     {
-      name: "test app",
+      name: "Test App",
       version: "4.0.0",
       iconPath: "/icon.svg",
 
-      description: "my description",
+      description: "This app does XYZ.",
     },
   ]
 }