test: add server_test.ts and fix server.ts

bridgecrew-perf7 · Aug 14, 2021 · 3574f03 · 3574f03
1 parent 4032d9f
commit 3574f03
Show file tree

Hide file tree

Showing 3 changed files with 65 additions and 8 deletions.
diff --git a/server.ts b/server.ts
@@ -12,27 +12,30 @@ import { renderHtml } from "./render_html.ts";
 import { ConfigObject } from "./types.ts";
 import { deleteItem, getItem, putItem } from "./dynamodb.ts";
 
-function applyHash(token: string) {
+export function applyHash(token: string) {
   return createHash("sha256").update(`${token}`).toString();
 }
 
-const NAME_REGEX = /[a-z][a-z0-9_-]{2,64}/;
-const TOKEN_REGEX = /[!-~]{8,128}/;
-function validateName(name: string) {
+const NAME_REGEX = /^[a-z][a-z0-9_-]{2,64}$/;
+const TOKEN_REGEX = /^[!-~]{8,128}$/;
+
+export function validateName(name: string) {
   return NAME_REGEX.test(name);
 }
-function validateToken(token: string) {
+
+export function validateToken(token: string) {
   return TOKEN_REGEX.test(token);
 }
-function validateConfig(config: string) {
+
+export function validateConfig(config: string) {
   try {
     const { list } = JSON.parse(config);
     return !!(list);
   } catch (_) {
     return false;
   }
 }
-function encodeConfig(rawConfig: string) {
+export function encodeConfig(rawConfig: string) {
   // use parse and stringify to minify json
   return encode(
     gzip(
@@ -42,7 +45,7 @@ function encodeConfig(rawConfig: string) {
     ),
   );
 }
-function decodeConfig(compressedConfig: string) {
+export function decodeConfig(compressedConfig: string) {
   return JSON.parse(
     new TextDecoder().decode(gunzip(decode(compressedConfig))),
   ) as ConfigObject;

diff --git a/server_test.ts b/server_test.ts
@@ -0,0 +1,52 @@
+import { assert, assertEquals } from "./deps.ts";
+Deno.env.set("AWS_ACCESS_KEY_ID", "dummy-id");
+Deno.env.set("AWS_SECRET_ACCESS_KEY", "dummy-key");
+import {
+  decodeConfig,
+  encodeConfig,
+  validateConfig,
+  validateName,
+  validateToken,
+} from "./server.ts";
+
+Deno.test("encode and decode", () => {
+  const config = {
+    description: "<description>",
+    twitter: "twitter",
+    list: {
+      id1: {
+        icon: "feather/github",
+        items: [{ icon: "feather/github" }],
+      },
+    },
+  };
+  const configStr = JSON.stringify(config);
+  assertEquals(decodeConfig(encodeConfig(configStr)), config);
+});
+
+Deno.test("validateConfig", () => {
+  assert(
+    validateConfig(
+      `{"list":{"id1":{"icon":"devicons/github","items":[{"text":"github","link":"https://github.com/"}]}}}`,
+    ),
+  );
+  assert(!validateConfig("{}"));
+  assert(!validateConfig(""));
+});
+
+Deno.test("validateName", () => {
+  assert(validateName("this-is-valid_123"));
+  assert(!validateName("invalid name"));
+  assert(!validateName(""));
+  assert(!validateName("o"));
+  assert(!validateName("_no"));
+  assert(!validateName("名前"));
+});
+
+Deno.test("validateToken", () => {
+  assert(validateToken("this-is-valid_123"));
+  assert(!validateToken("invalid token"));
+  assert(!validateToken(""));
+  assert(!validateToken("short"));
+  assert(!validateToken("秘密"));
+});
diff --git a/velociraptor.yml b/velociraptor.yml
@@ -59,6 +59,8 @@ scripts:
   test:
     desc: Runs the tests
     cmd: deno test --reload
+    allow:
+      - env
     gitHook: pre-push
 
   cov:
-Original file line number
+Diff line change
@@ Expand Up / @@ -59,6 +59,8 @@ scripts: @@
       test:
         desc: Runs the tests
         cmd: deno test --reload
+        allow:
+          - env
         gitHook: pre-push
       cov:
@@ Expand Down @@