Merge pull request particle-iot#42 from AntonPuko/flowClean

decorators flow annotations

Author: jlkalberer

src/lib/decorators/allowUpload.js

@@ -1,14 +1,20 @@
-import type { Decorator, HttpVerb } from './types';
+// @flow
 
+import type { Decorator, Descriptor } from './types';
+import type Controller from '../controllers/Controller';
+
+/* eslint-disable no-param-reassign */
 export default (
   fileName: string,
   maxCount: number = 0,
-): Decorator => (target, name, descriptor): Object => {
-  const allowedUploads = target[name].allowedUploads || [];
-  allowedUploads.push({
-    maxCount: maxCount ? maxCount : undefined,
-    name: fileName,
-  });
-  target[name].allowedUploads = allowedUploads;
-  return descriptor;
-};
+): Decorator<Controller> =>
+  (target: Controller, name: string, descriptor: Descriptor): Descriptor => {
+    const allowedUploads = target[name].allowedUploads || [];
+    allowedUploads.push({
+      maxCount,
+      name: fileName,
+    });
+
+    target[name].allowedUploads = allowedUploads;
+    return descriptor;
+  };

src/lib/decorators/anonymous.js

@@ -1,10 +1,11 @@
 // @flow
 
-import type { Decorator } from './types';
+import type { Decorator, Descriptor } from './types';
+import type Controller from '../controllers/Controller';
 
-export default (): Decorator =>
-  (target, name, descriptor): Object => {
-    // eslint-disable-next-line no-param-reassign
+/* eslint-disable no-param-reassign */
+export default (): Decorator<Controller> =>
+  (target: Controller, name: string, descriptor: Descriptor): Descriptor => {
     target[name].anonymous = true;
     return descriptor;
   };

src/lib/decorators/httpVerb.js

@@ -1,8 +1,11 @@
-import type { Decorator, HttpVerb } from './types';
+// @flow
 
-export default (
-  httpVerb: HttpVerb,
-): Decorator => (target, name, descriptor): Object => {
-  target[name].httpVerb = httpVerb;
-  return descriptor;
-};
+import type { Decorator, Descriptor } from './types';
+import type Controller from '../controllers/Controller';
+
+/* eslint-disable no-param-reassign */
+export default (httpVerb: HttpVerb): Decorator<Controller> =>
+  (target: Controller, name: string, descriptor: Descriptor): Descriptor => {
+    target[name].httpVerb = httpVerb;
+    return descriptor;
+  };

src/lib/decorators/route.js

@@ -1,6 +1,11 @@
-export default (
-  route: string,
-): Decorator => (target, name, descriptor): Object => {
-  target[name].route = route;
-  return descriptor;
-};
+// @flow
+
+import type { Decorator, Descriptor } from './types';
+import type Controller from '../controllers/Controller';
+
+/* eslint-disable no-param-reassign */
+export default (route: string): Decorator<Controller> =>
+  (target: Controller, name: string, descriptor: Descriptor): Descriptor => {
+    target[name].route = route;
+    return descriptor;
+  };

src/lib/decorators/types.js

@@ -1,10 +1,17 @@
 // @flow
 
-export type Decorator = (
-  target: Object,
+export type Decorator<TType> = (
+  target: TType,
   name: string,
-  descriptor: Object,
-) => Object;
+  descriptor: Descriptor,
+) => Descriptor;
+
+export type Descriptor = {
+    configurable: boolean,
+    enumerable: boolean,
+    value: Function,
+    writeable: boolean,
+};
 
 export type HttpVerb =
   'checkout' |