fix: use correct redis default port (#433)

IBM-Swift · Jan 26, 2018 · 623f206 · 623f206
1 parent b3e1870
commit 623f206
Show file tree

Hide file tree

Showing 4 changed files with 36 additions and 34 deletions.
diff --git a/lib/helpers.js b/lib/helpers.js
@@ -407,7 +407,7 @@ function sanitizeCredentialsAndFillInDefaults (serviceType, service) {
         port: service.port || defaults.port
       }
     case 'redis':
-      var defaultRedisURI = { protocol: 'redis', auth: ':', hostname: 'localhost', port: 6397, slashes: true }
+      var defaultRedisURI = { protocol: 'redis', auth: ':', hostname: 'localhost', port: 6379, slashes: true }
       if (service.host) defaultRedisURI.hostname = service.host
       if (service.port) defaultRedisURI.port = service.port
       if (service.password) defaultRedisURI.auth = `:${service.password}`

diff --git a/package-lock.json b/package-lock.json
diff --git a/test/integration/app/prompted_nobuild.js b/test/integration/app/prompted_nobuild.js
@@ -722,7 +722,7 @@ describe('Integration tests (prompt no build) for swiftserver:app', function ()
 
         commonTest.itCreatedServiceConfigFiles()
         commonTest.redis.itCreatedServiceFilesWithExpectedContent('myRedisService', {
-          uri: 'redis://:@localhost:6397'
+          uri: 'redis://:@localhost:6379'
         })
       })
 

diff --git a/test/unit/helpers.js b/test/unit/helpers.js
@@ -90,7 +90,7 @@ describe('Unit tests for helpers', function () {
             label: 'compose-for-redis',
             plan: 'Standard'
           },
-          uri: 'redis://:@localhost:6397'
+          uri: 'redis://:@localhost:6379'
         },
         objectStorage: {
           serviceInfo: {
@@ -326,14 +326,14 @@ describe('Unit tests for helpers', function () {
           var service = helpers.sanitizeServiceAndFillInDefaults(
             'redis', { serviceInfo: {name: 'my-service'}, password: 'my-password' }
           )
-          assert.equal(service.uri, 'redis://:my-password@localhost:6397')
+          assert.equal(service.uri, 'redis://:my-password@localhost:6379')
         })
 
         it('uri from host (merge with default)', function () {
           var service = helpers.sanitizeServiceAndFillInDefaults(
             'redis', { serviceInfo: {name: 'my-service'}, host: 'my-host' }
           )
-          assert.equal(service.uri, 'redis://:@my-host:6397')
+          assert.equal(service.uri, 'redis://:@my-host:6379')
         })
 
         it('uri from port (merge with default)', function () {