Merge pull request #23 from hapinessjs/next

Next

package.json

@@ -1,6 +1,6 @@
 {
     "name": "@hapiness/mingo",
-    "version": "2.1.1",
+    "version": "2.1.2",
     "description": "Manage files using minio as file storage and mongodb for metadata",
     "main": "commonjs/index.js",
     "types": "index.d.ts",

src/module/managers/files.manager.ts

@@ -137,7 +137,17 @@ export class FilesManager {
         filename: string, projection?: string | string[], options?: { [key: string]: any }
     ): Observable<MingoFileInterface> {
         const projectionStr = projection && projection instanceof Array ? projection.join(' ') : projection as string;
-        return this.fileRepository.findFileByFilename(filename, this.bucketService.getName(), projectionStr, options);
+        return Observable.of(filename)
+            .flatMap(fname => !!fname
+                ? this.fileRepository
+                    .findFileByFilename(
+                        filename,
+                        this.bucketService.getName(),
+                        projectionStr,
+                        options
+                    )
+                : Observable.throw(Biim.badRequest(`No filename provided`))
+            );
     }
 
     /**

test/functional/mingo.module.test.ts

@@ -60,6 +60,8 @@ export class MingoModuleFunctionalTest {
 
                 fb().create(fs.createReadStream('./package.json'), 'package.json', 'application/json', null)
                     .do(_ => Object.assign(fileProperties, { id: _.id, created_at: _.created_at, updated_at: _.updated_at }))
+                    // dunno why but etag given by stat function is no more a md5 ?!
+                    .do(_ => fileProperties.md5 = _.md5)
                     .do(_ => unit
                         .object(_)
                         .is(fileProperties)