Merge branch 'master' into emit

Author: danhsiung

__tests__/$send.js

@@ -1,10 +1,15 @@
 // purposefully test COMPILED platform.js! (just as lambda_maker would)
-const { $sendConfigRuntimeTypeChecker } = require("../dist")
+const { $sendConfigRuntimeTypeChecker, HTTP_METHODS } = require("../dist")
 
 function randString() {
   return ""+Math.random()
 }
 
+function randHttpMethod() {
+  const idx = Math.floor(Math.random() * HTTP_METHODS.length)
+  return HTTP_METHODS[idx]
+}
+
 const emptyConfig = {}
 
 // XXX would be nice to generate a lot of these tests automatically...
@@ -65,7 +70,7 @@ describe("$send.http", () => {
   let config
   beforeEach(() => {
     config = {
-      method: randString(),
+      method: randHttpMethod(),
       url: randString(),
       auth: {
         password: randString(),

dist/index.d.ts

@@ -20,8 +20,9 @@ export declare const SendConfigEmit: t.IntersectionC<[t.ExactC<t.TypeC<{
     event: t.ObjectC;
 }>]>;
 export declare type SendConfigEmit = t.TypeOf<typeof SendConfigEmit>;
+export declare const HTTP_METHODS: string[];
 export declare const SendConfigHTTP: t.IntersectionC<[t.ExactC<t.TypeC<{
-    method: t.StringC;
+    method: t.KeyofC<{}>;
     url: t.StringC;
 }>>, t.PartialC<{
     auth: t.ExactC<t.TypeC<{
@@ -82,7 +83,7 @@ export declare const sendTypeMap: {
         event: t.ObjectC;
     }>]>;
     http: t.IntersectionC<[t.ExactC<t.TypeC<{
-        method: t.StringC;
+        method: t.KeyofC<{}>;
         url: t.StringC;
     }>>, t.PartialC<{
         auth: t.ExactC<t.TypeC<{

dist/index.js

@@ -27,8 +27,25 @@ const SendConfigHTTPAuth = t.strict({
     password: t.string,
     username: t.string,
 });
+// https://developer.mozilla.org/en-US/docs/Web/HTTP/Methods
+exports.HTTP_METHODS = [
+    "GET",
+    "HEAD",
+    "POST",
+    "PUT",
+    "DELETE",
+    "CONNECT",
+    "OPTIONS",
+    "TRACE",
+    "PATCH",
+];
+// HTTP method must be uppercase (for kotlin in coordinator -- i voted to make it case insensitive, but w.e for now)
+const SendConfigHTTPMethod = t.keyof(exports.HTTP_METHODS.reduce((acc, v) => {
+    acc[v] = null;
+    return acc;
+}, {}));
 const SendConfigHTTP_required = t.strict({
-    method: t.string,
+    method: SendConfigHTTPMethod,
     url: t.string,
 });
 const SendConfigHTTP_optional = t.partial({

lib/index.ts

@@ -35,8 +35,26 @@ const SendConfigHTTPAuth = t.strict({
     username: t.string,
 });
 type SendConfigHTTPAuth = t.TypeOf<typeof SendConfigHTTPAuth>;
+// https://developer.mozilla.org/en-US/docs/Web/HTTP/Methods
+export const HTTP_METHODS = [
+    "GET",
+    "HEAD",
+    "POST",
+    "PUT",
+    "DELETE",
+    "CONNECT",
+    "OPTIONS",
+    "TRACE",
+    "PATCH",
+]
+// HTTP method must be uppercase (for kotlin in coordinator -- i voted to make it case insensitive, but w.e for now)
+const SendConfigHTTPMethod = t.keyof(HTTP_METHODS.reduce((acc, v) => {
+    acc[v] = null
+    return acc
+}, {}))
+type SendConfigHTTPMethod = t.TypeOf<typeof SendConfigHTTPMethod>;
 const SendConfigHTTP_required = t.strict({
-    method: t.string, // XXX proper enum for methods here?
+    method: SendConfigHTTPMethod,
     url: t.string,
 })
 const SendConfigHTTP_optional = t.partial({