fix(last-release): normalized registry url
This commit is contained in:
		
							parent
							
								
									caf09c9be0
								
							
						
					
					
						commit
						af15fb4c61
					
				| @ -65,7 +65,7 @@ npmconf.load({}, (err, conf) => { | |||||||
|           token: env.NPM_TOKEN |           token: env.NPM_TOKEN | ||||||
|         }, |         }, | ||||||
|         loglevel: log.level, |         loglevel: log.level, | ||||||
|         registry |         registry: registry + (registry[registry.length - 1] !== '/' ? '/' : '') | ||||||
|       }, |       }, | ||||||
|       plugins, |       plugins, | ||||||
|       (err, release) => { |       (err, release) => { | ||||||
|  | |||||||
| @ -5,7 +5,7 @@ module.exports = function (pkg, npmConfig, cb) { | |||||||
|   npmlog.level = npmConfig.loglevel || 'error' |   npmlog.level = npmConfig.loglevel || 'error' | ||||||
|   const client = new RegClient({log: npmlog}) |   const client = new RegClient({log: npmlog}) | ||||||
| 
 | 
 | ||||||
|   client.get(`${npmConfig.registry}/${pkg.name}`, { |   client.get(`${npmConfig.registry}${pkg.name}`, { | ||||||
|     auth: npmConfig.auth |     auth: npmConfig.auth | ||||||
|   }, (err, data) => { |   }, (err, data) => { | ||||||
|     if (err && err.statusCode === 404) return cb(null, {}) |     if (err && err.statusCode === 404) return cb(null, {}) | ||||||
|  | |||||||
| @ -4,7 +4,7 @@ require('../mocks/registry') | |||||||
| const lastRelease = require('../../dist/lib/last-release') | const lastRelease = require('../../dist/lib/last-release') | ||||||
| 
 | 
 | ||||||
| const npmConfig = { | const npmConfig = { | ||||||
|   registry: 'http://registry.npmjs.org' |   registry: 'http://registry.npmjs.org/' | ||||||
| } | } | ||||||
| 
 | 
 | ||||||
| test('last release from registry', (t) => { | test('last release from registry', (t) => { | ||||||
|  | |||||||
| @ -18,7 +18,7 @@ test('full pre run', (t) => { | |||||||
|     pre({ |     pre({ | ||||||
|       name: 'available' |       name: 'available' | ||||||
|     }, { |     }, { | ||||||
|       registry: 'http://registry.npmjs.org' |       registry: 'http://registry.npmjs.org/' | ||||||
|     }, |     }, | ||||||
|     plugins, |     plugins, | ||||||
|     (err, release) => { |     (err, release) => { | ||||||
| @ -34,7 +34,7 @@ test('full pre run', (t) => { | |||||||
|     pre({ |     pre({ | ||||||
|       name: 'unavailable' |       name: 'unavailable' | ||||||
|     }, { |     }, { | ||||||
|       registry: 'http://registry.npmjs.org' |       registry: 'http://registry.npmjs.org/' | ||||||
|     }, |     }, | ||||||
|     plugins, |     plugins, | ||||||
|     (err, release) => { |     (err, release) => { | ||||||
|  | |||||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user