Fixed mime type for known extensions (#2393)

* on create backup do no longer call processExit internally

- because this leads to unwanted success-like code running in parallel

* refactor file name

* use writeJson

* do not prettify the json, as we need to save space

* fixed mimetype

* fix test ;-)

Author: foxriver76

packages/controller/test/lib/testFiles.ts

@@ -218,6 +218,31 @@ export function register(it: Mocha.TestFunction, expect: Chai.ExpectStatic, cont
         expect(file).to.be.equal(dataText);
     });
 
+    it(testName + 'writeFile with known extension should be inferred', async () => {
+        const objId = `vis.0`;
+        /** no extension but string content should lead to plain text */
+        const fileName = 'testFile.json';
+        const dataJson = { quote: "Toto, I've a feeling we're not in Kansas anymore." };
+        const content = JSON.stringify(dataJson);
+        // create an object of type file first
+        await context.adapter.setForeignObjectAsync(objId, {
+            type: 'meta',
+            common: {
+                name: 'Files and more',
+                type: 'meta.user'
+            },
+            native: {}
+        });
+
+        // now we write a file state
+        await context.adapter.writeFileAsync(objId, fileName, content);
+
+        const { file, mimeType } = await context.adapter.readFileAsync(objId, fileName);
+
+        expect(mimeType).to.be.equal('application/json');
+        expect(file).to.be.equal(content);
+    });
+
     it(testName + 'deleteFile', async () => {
         const objId = `vis.0`;
         const fileName = 'testFile.bin';

packages/controller/test/lib/testObjects.ts

@@ -414,6 +414,7 @@ export function register(it: Mocha.TestFunction, expect: Chai.ExpectStatic, cont
         expect(exists).to.be.true;
     });
 
+    // TODO: here are also a few file tests, move them to testFiles.ts instead
     it(testName + 'should create and read file', done => {
         const objects = context.objects;
         objects.setObject(testId, { type: 'meta', native: {} } as ioBroker.SettableMetaObject, err => {

packages/db-objects-redis/src/lib/objects/objectsInRedisClient.ts

@@ -1058,7 +1058,7 @@ export class ObjectsInRedisClient {
 
         const isTextData = typeof data === 'string';
 
-        const { mimeType, isBinary } = utils.getMimeType(ext[0], isTextData);
+        const { mimeType, isBinary } = utils.getMimeType(ext, isTextData);
 
         const metaID = this.getFileId(id, name, true);
         if (!this.client) {