Clean up settings tests

services/filestore/test/unit/js/SettingsTests.js

@@ -1,33 +1,19 @@
-/* eslint-disable
-    camelcase,
-    no-unused-vars,
-*/
-// TODO: This file was created by bulk-decaffeinate.
-// Fix any style issues and re-enable lint.
-/*
- * decaffeinate suggestions:
- * DS102: Remove unnecessary code created because of implicit returns
- * Full docs: https://github.com/decaffeinate/decaffeinate/blob/master/docs/suggestions.md
- */
-const { assert } = require('chai')
-const sinon = require('sinon')
 const chai = require('chai')
-const should = chai.should()
 const { expect } = chai
-const modulePath = '../../../app/js/BucketController.js'
 
-describe('Settings', () =>
+describe('Settings', function() {
-  describe('s3', () =>
+  describe('s3', function() {
-    it('should use JSONified env var if present', function(done) {
+    it('should use JSONified env var if present', function() {
-      const s3_settings = {
+      const s3Settings = {
         bucket1: {
           auth_key: 'bucket1_key',
           auth_secret: 'bucket1_secret'
         }
       }
-      process.env.S3_BUCKET_CREDENTIALS = JSON.stringify(s3_settings)
+      process.env.S3_BUCKET_CREDENTIALS = JSON.stringify(s3Settings)
 
       const settings = require('settings-sharelatex')
-      expect(settings.filestore.s3BucketCreds).to.deep.equal(s3_settings)
+      expect(settings.filestore.s3BucketCreds).to.deep.equal(s3Settings)
-      return done()
+    })
-    })))
+  })
+})