refactor: move test-plugin to integration suite
This seems more appropriate given this tests how a plugin might work within code-server.
This commit is contained in:
parent
a0944006e7
commit
bc02005dc0
@ -24,6 +24,13 @@ main() {
|
||||
path="$CODE_SERVER_PATH"
|
||||
fi
|
||||
|
||||
# TODO@jsjoeio - skip if already built
|
||||
# TODO@jsjoeio - move to integration test suite too
|
||||
echo "Building test plugin"
|
||||
pushd test/integration/test-plugin
|
||||
make -s out/index.js
|
||||
popd
|
||||
|
||||
echo "Running tests with code-server binary: '$path'"
|
||||
|
||||
if [[ ! -f $path ]]; then
|
||||
@ -33,7 +40,7 @@ main() {
|
||||
exit 1
|
||||
fi
|
||||
|
||||
CODE_SERVER_PATH="$path" CS_DISABLE_PLUGINS=true ./test/node_modules/.bin/jest "$@" --coverage=false --testRegex "./test/integration" --testPathIgnorePatterns "./test/integration/fixtures"
|
||||
CODE_SERVER_PATH="$path" CS_DISABLE_PLUGINS=true ./test/node_modules/.bin/jest "$@" --coverage=false --testRegex "./test/integration" --testPathIgnorePatterns "./test/integration/fixtures" --testPathIgnorePatterns "./test/integration/test-plugin"
|
||||
}
|
||||
|
||||
main "$@"
|
||||
|
@ -6,13 +6,6 @@ main() {
|
||||
|
||||
source ./ci/lib.sh
|
||||
|
||||
# TODO@jsjoeio - skip if already built
|
||||
# TODO@jsjoeio - move to integration test suite too
|
||||
echo "Building test plugin"
|
||||
pushd test/unit/node/test-plugin
|
||||
make -s out/index.js
|
||||
popd
|
||||
|
||||
# Our code imports from `out` in order to work during development but if you
|
||||
# have only built for production you will have not have this directory. In
|
||||
# that case symlink `out` to a production build directory.
|
||||
|
@ -2,11 +2,11 @@ import { logger } from "@coder/logger"
|
||||
import * as express from "express"
|
||||
import * as fs from "fs"
|
||||
import * as path from "path"
|
||||
import { HttpCode } from "../../../src/common/http"
|
||||
import { AuthType } from "../../../src/node/cli"
|
||||
import { codeServer, PluginAPI } from "../../../src/node/plugin"
|
||||
import * as apps from "../../../src/node/routes/apps"
|
||||
import * as httpserver from "../../utils/httpserver"
|
||||
import { HttpCode } from "../../src/common/http"
|
||||
import { AuthType } from "../../src/node/cli"
|
||||
import { codeServer, PluginAPI } from "../../src/node/plugin"
|
||||
import * as apps from "../../src/node/routes/apps"
|
||||
import * as httpserver from "../utils/httpserver"
|
||||
const fsp = fs.promises
|
||||
|
||||
// Jest overrides `require` so our usual override doesn't work.
|
Before Width: | Height: | Size: 3.0 KiB After Width: | Height: | Size: 3.0 KiB |
@ -1,22 +1,26 @@
|
||||
// TODO@jsjoeio - how do I fix this?
|
||||
// @ts-ignore - we know code-server exists
|
||||
import * as cs from "code-server"
|
||||
import * as fspath from "path"
|
||||
|
||||
type FixMeLater = any
|
||||
|
||||
export const plugin: cs.Plugin = {
|
||||
displayName: "Test Plugin",
|
||||
routerPath: "/test-plugin",
|
||||
homepageURL: "https://example.com",
|
||||
description: "Plugin used in code-server tests.",
|
||||
|
||||
init(config) {
|
||||
init(config: FixMeLater) {
|
||||
config.logger.debug("test-plugin loaded!")
|
||||
},
|
||||
|
||||
router() {
|
||||
const r = cs.express.Router()
|
||||
r.get("/test-app", (_, res) => {
|
||||
r.get("/test-app", (_: FixMeLater, res: FixMeLater) => {
|
||||
res.sendFile(fspath.resolve(__dirname, "../public/index.html"))
|
||||
})
|
||||
r.get("/goland/icon.svg", (_, res) => {
|
||||
r.get("/goland/icon.svg", (_: FixMeLater, res: FixMeLater) => {
|
||||
res.sendFile(fspath.resolve(__dirname, "../public/icon.svg"))
|
||||
})
|
||||
r.get("/error", () => {
|
||||
@ -27,8 +31,8 @@ export const plugin: cs.Plugin = {
|
||||
|
||||
wsRouter() {
|
||||
const wr = cs.WsRouter()
|
||||
wr.ws("/test-app", (req) => {
|
||||
cs.wss.handleUpgrade(req, req.ws, req.head, (ws) => {
|
||||
wr.ws("/test-app", (req: FixMeLater) => {
|
||||
cs.wss.handleUpgrade(req, req.ws, req.head, (ws: FixMeLater) => {
|
||||
req.ws.resume()
|
||||
ws.send("hello")
|
||||
})
|
@ -1,5 +1,4 @@
|
||||
{
|
||||
"extends": "../tsconfig.json",
|
||||
"include": ["./**/*.ts"],
|
||||
"exclude": ["./unit/node/test-plugin"]
|
||||
}
|
||||
|
Reference in New Issue
Block a user