Add linting steps and improve testing steps

Author: code-asher

.drone.yml

@@ -6,22 +6,29 @@ platform:
   arch: amd64
 
 steps:
-- name: submodules
-  image: alpine/git
+- name: cache:restore
+  image: codercom/nbin:centos
   commands:
-    - git submodule update --init
+    - yum install -y libxkbfile-devel libsecret-devel
+    - . /opt/rh/devtoolset-6/enable
+    - ./scripts/cacher.sh
 
-- name: cache:restore
+- name: lint
   image: node:12
   commands:
-    - ./scripts/cacher.sh
+    - yarn lint
+
+- name: test
+  image: node:12
+  commands:
+    - yarn test
 
 - name: build
   image: codercom/nbin:centos
   commands:
     - yum install -y libxkbfile-devel libsecret-devel
     - . /opt/rh/devtoolset-6/enable
-    - timeout 50m ./scripts/ci.bash || echo 'Timed out or failed; continuing so we can preserve cache for the next run'
+    - timeout 50m ./scripts/ci.bash || echo 'Timed out or failed; continuing to preserve cache for the next run'
 
 - name: cache:package
   image: node:12
@@ -40,10 +47,10 @@ steps:
   when:
     event: push
 
-- name: test
+- name: test:build
   image: node:12
   commands:
-    - yarn test
+    - yarn test:build
 
 - name: publish:github
   image: plugins/github-release
@@ -90,21 +97,28 @@ platform:
   arch: amd64
 
 steps:
-- name: submodules
-  image: alpine/git
-  commands:
-    - git submodule update --init
-
 - name: cache:restore
   image: node:12-alpine
   commands:
+    - apk add libxkbfile-dev libsecret-dev build-base git bash python
     - ./scripts/cacher.sh
 
+- name: lint
+  image: node:12-alpine
+  commands:
+    - yarn lint
+
+- name: test
+  image: node:12-alpine
+  commands:
+    - apk add openssl
+    - yarn test
+
 - name: build
   image: node:12-alpine
   commands:
     - apk add libxkbfile-dev libsecret-dev build-base git bash python
-    - timeout 50m ./scripts/ci.bash || echo 'Timed out or failed; continuing so we can preserve cache for the next run'
+    - timeout 50m ./scripts/ci.bash || echo 'Timed out or failed; continuing to preserve cache for the next run'
 
 - name: cache:package
   image: node:12-alpine
@@ -123,10 +137,10 @@ steps:
   when:
     event: push
 
-- name: test
+- name: test:build
   image: node:12-alpine
   commands:
-    - yarn test
+    - yarn test:build
 
 - name: publish:github
   image: plugins/github-release
@@ -159,22 +173,27 @@ platform:
   arch: arm64
 
 steps:
-- name: submodules
-  image: alpine
-  commands:
-    - apk add git
-    - git submodule update --init
-
 - name: cache:restore
   image: node:12
   commands:
+    - apt update && apt install -y build-essential git libsecret-1-dev libx11-dev libxkbfile-dev
     - ./scripts/cacher.sh
 
+- name: lint
+  image: node:12
+  commands:
+    - yarn lint
+
+- name: test
+  image: node:12
+  commands:
+    - yarn test
+
 - name: build
   image: node:12
   commands:
     - apt update && apt install -y build-essential git libsecret-1-dev libx11-dev libxkbfile-dev
-    - timeout 50m ./scripts/ci.bash || echo 'Timed out or failed; continuing so we can preserve cache for the next run'
+    - timeout 50m ./scripts/ci.bash || echo 'Timed out or failed; continuing to preserve cache for the next run'
 
 - name: cache:package
   image: node:12
@@ -193,10 +212,10 @@ steps:
   when:
     event: push
 
-- name: test
+- name: test:build
   image: node:12
   commands:
-    - yarn test
+    - yarn test:build
 
 - name: publish:github
   image: plugins/github-release
@@ -243,22 +262,28 @@ platform:
   arch: arm64
 
 steps:
-- name: submodules
-  image: alpine
-  commands:
-    - apk add git
-    - git submodule update --init
-
 - name: cache:restore
   image: node:12-alpine
   commands:
+    - apk add libxkbfile-dev libsecret-dev build-base git bash python
     - ./scripts/cacher.sh
 
+- name: lint
+  image: node:12-alpine
+  commands:
+    - yarn lint
+
+- name: test
+  image: node:12-alpine
+  commands:
+    - apk add openssl
+    - yarn test
+
 - name: build
   image: node:12-alpine
   commands:
     - apk add libxkbfile-dev libsecret-dev build-base git bash python
-    - timeout 50m ./scripts/ci.bash || echo 'Timed out or failed; continuing so we can preserve cache for the next run'
+    - timeout 50m ./scripts/ci.bash || echo 'Timed out or failed; continuing to preserve cache for the next run'
 
 - name: cache:package
   image: node:12-alpine
@@ -277,10 +302,10 @@ steps:
   when:
     event: push
 
-- name: test
+- name: test:build
   image: node:12-alpine
   commands:
-    - yarn test
+    - yarn test:build
 
 - name: publish:github
   image: plugins/github-release
@@ -313,22 +338,27 @@ platform:
   arch: arm
 
 steps:
-- name: submodules
-  image: alpine
-  commands:
-    - apk add git
-    - git submodule update --init
-
 - name: cache:restore
   image: node:12
   commands:
+    - apt update && apt install -y build-essential git libsecret-1-dev libx11-dev libxkbfile-dev
     - ./scripts/cacher.sh
 
+- name: lint
+  image: node:12
+  commands:
+    - yarn lint
+
+- name: test
+  image: node:12
+  commands:
+    - yarn test
+
 - name: build
   image: node:12
   commands:
     - apt update && apt install -y build-essential git libsecret-1-dev libx11-dev libxkbfile-dev
-    - timeout 50m ./scripts/ci.bash || echo 'Timed out or failed; continuing so we can preserve cache for the next run'
+    - timeout 50m ./scripts/ci.bash || echo 'Timed out or failed; continuing to preserve cache for the next run'
 
 - name: cache:package
   image: node:12
@@ -347,11 +377,11 @@ steps:
   when:
     event: push
 
-- name: test
+- name: test:build
   image: node:12
   failure: ignore
   commands:
-    - yarn test
+    - yarn test:build
 
 # - name: publish:github
 #   image: plugins/github-release
@@ -388,22 +418,28 @@ platform:
   arch: arm
 
 steps:
-- name: submodules
-  image: alpine
-  commands:
-    - apk add git
-    - git submodule update --init
-
 - name: cache:restore
   image: node:12-alpine
   commands:
+    - apk add libxkbfile-dev libsecret-dev build-base git bash python
     - ./scripts/cacher.sh
 
+- name: lint
+  image: node:12-alpine
+  commands:
+    - yarn lint
+
+- name: test
+  image: node:12-alpine
+  commands:
+    - apk add openssl
+    - yarn test
+
 - name: build
   image: node:12-alpine
   commands:
     - apk add libxkbfile-dev libsecret-dev build-base git bash python
-    - timeout 50m ./scripts/ci.bash || echo 'Timed out or failed; continuing so we can preserve cache for the next run'
+    - timeout 50m ./scripts/ci.bash || echo 'Timed out or failed; continuing to preserve cache for the next run'
 
 - name: cache:package
   image: node:12-alpine
@@ -422,11 +458,11 @@ steps:
   when:
     event: push
 
-- name: test
+- name: test:build
   image: node:12-alpine
   failure: ignore
   commands:
-    - yarn test
+    - yarn test:build
 
 # - name: publish:github
 #   image: plugins/github-release

package.json

@@ -9,7 +9,8 @@
     "patch:generate": "cd ./lib/vscode && git diff HEAD > ../../scripts/vscode.patch",
     "patch:apply": "cd ./lib/vscode && git apply ../../scripts/vscode.patch",
     "test": "mocha -r ts-node/register ./test/*.test.ts",
-    "lint:js": "eslint {src,test,scripts} --ext .ts,.tsx",
+    "test:build": "./scripts/test.sh",
+    "lint:js": "eslint src test scripts --ext .ts,.tsx",
     "lint:css": "stylelint 'src/**/*.css'",
     "lint": "./scripts/lint.sh",
     "watch": "yarn runner watch",

scripts/cacher.sh

@@ -46,7 +46,18 @@ main() {
 
   # The action is determined by the name of the step.
   case $DRONE_STEP_NAME in
-    *restore*) restore "$branch" "$DRONE_REPO_BRANCH" ;;
+    *restore*)
+      # Sub-modules must be pulled first since extracting the cache directories
+      # will prevent git from cloning into them.
+      git submodule update --init
+
+      restore "$branch" "$DRONE_REPO_BRANCH"
+
+      # Now make sure the pulled Node modules are up to date.
+      YARN_CACHE_FOLDER="$(pwd)/yarn-cache"
+      export YARN_CACHE_FOLDER
+      yarn
+    ;;
     *rebuild*|*package*) package "$branch" ;;
     *) exit 1 ;;
   esac

scripts/ci.bash

@@ -3,6 +3,7 @@
 
 set -euo pipefail
 
+# This script assumes that yarn has already ran.
 function main() {
   cd "$(dirname "${0}")/.."
 
@@ -21,10 +22,6 @@ function main() {
     export PACKAGE="true"
   fi
 
-  if [[ -z ${SKIP_YARN:-} ]] ; then
-    yarn
-  fi
-
   yarn build
   yarn binary
   if [[ -n ${PACKAGE:-} ]] ; then

scripts/test.sh

@@ -0,0 +1,19 @@
+#!/usr/bin/env sh
+# test.sh -- Simple build test.
+
+set -eu
+
+main() {
+  cd "$(dirname "$0")/.."
+
+  # The main goal here is to ensure that the build fully completed and the
+  # result looks usable.
+  version=$(node ./build/out/node/entry.js --version --json)
+  echo "Got '$version' for the version"
+  case $version in
+    "{ codeServer":*) exit 0 ;;
+    *) exit 1 ;;
+  esac
+}
+
+main "$@"

src/node/entry.ts

@@ -49,7 +49,7 @@ const main = async (args: Args = {}): Promise<void> => {
     new VscodeHttpProvider([], { base: "/vscode-embed", auth, password })
   )
 
-  ipcMain.onDispose(() => httpServer.dispose())
+  ipcMain().onDispose(() => httpServer.dispose())
 
   const serverAddress = await httpServer.listen()
   logger.info(`Server listening on ${serverAddress}`)
@@ -84,4 +84,18 @@ const main = async (args: Args = {}): Promise<void> => {
   }
 }
 
-wrap(main)
+// TODO: Implement CLI parser.
+if (process.argv.includes("--version")) {
+  const version = require("../../package.json").version
+  if (process.argv.includes("--json")) {
+    console.log({
+      codeServer: version,
+      vscode: require("../../lib/vscode/package.json").version,
+    })
+  } else {
+    console.log(version)
+  }
+  process.exit(0)
+} else {
+  wrap(main)
+}