Remove v1 defaults (#2391)

GitOrigin-RevId: e9812c9d7785ac64500dcd3925a6b5be4cb04d7c

services/web/config/settings.defaults.coffee

@@ -15,10 +15,16 @@ httpAuthUsers[httpAuthUser] = httpAuthPass
 
 sessionSecret = process.env['SESSION_SECRET'] or "secret-please-change"
 
-v1Api =
+if process.env['V1_API_URL'] or process.env['V1_HOST']
-	url: process.env['V1_API_URL'] or "http://#{process.env['V1_HOST'] or 'localhost'}:5000"
+	v1Api =
-	user: process.env['V1_API_USER'] or 'overleaf'
+		url: process.env['V1_API_URL'] or "http://#{process.env['V1_HOST']}:5000"
-	pass: process.env['V1_API_PASSWORD'] or 'password'
+		user: process.env['V1_API_USER']
+		pass: process.env['V1_API_PASSWORD']
+else
+	v1Api =
+		url: undefined
+		user: undefined
+		pass: undefined
 
 module.exports = settings =
 

services/web/test/acceptance/config/settings.test.coffee

@@ -1,10 +1,16 @@
 v1Api =
-	url: "http://#{process.env['V1_HOST'] or 'localhost'}:5000"
+	url: "http://localhost:5000"
+	user: 'overleaf'
+	pass: 'password'
 
 module.exports =
 	enableSubscriptions: true
 
 	apis:
+		v1:
+			url: v1Api.url
+			user: v1Api.user
+			pass: v1Api.pass
 		recurly:
 			# Set up our own mock recurly server
 			url: 'http://localhost:6034'

services/web/test/unit/src/Analytics/AnalyticsControllerTests.js

@@ -19,6 +19,10 @@ describe('AnalyticsController', function() {
       getInstitutionLicences: sinon.stub().callsArgWith(4)
     }
 
+    this.Features = {
+      hasFeature: sinon.stub().returns(true)
+    }
+
     this.controller = SandboxedModule.require(modulePath, {
       globals: {
         console: console
@@ -28,6 +32,7 @@ describe('AnalyticsController', function() {
         '../Authentication/AuthenticationController': this
           .AuthenticationController,
         '../Institutions/InstitutionsAPI': this.InstitutionsAPI,
+        '../../infrastructure/Features': this.Features,
         'logger-sharelatex': {
           log() {}
         },