merge master

Author: ricardo-devis-agullo

src/cli/commands.ts

@@ -133,6 +133,11 @@ export default {
           boolean: true,
           description: 'Skip packaging step',
           default: false
+        },
+        registries: {
+          array: true,
+          description:
+            'List of registries to publish to. This setting will take precedence over oc.json file'
         }
       },
       description: 'Publish a component',

src/cli/facade/publish.ts

@@ -28,6 +28,7 @@ const publish = ({
       skipPackage?: boolean;
       username?: string;
       password?: string;
+      registries?: string[];
     }): Promise<void> => {
       const componentPath = opts.componentPath;
       const skipPackage = opts.skipPackage;
@@ -160,7 +161,7 @@ const publish = ({
       };
 
       try {
-        const registryLocations = await registry.get();
+        const registryLocations = opts.registries || (await registry.get());
 
         if (!skipPackage) {
           await handleDependencies({

test/unit/cli-facade-publish.js

@@ -20,7 +20,7 @@ describe('cli : facade : publish', () => {
 
   const execute = function (
     cb,
-    { creds = {}, skipPackage = false, fs = {} } = {}
+    { creds = {}, skipPackage = false, fs = {}, registries } = {}
   ) {
     logSpy.err = sinon.stub();
     logSpy.log = sinon.stub();
@@ -47,7 +47,8 @@ describe('cli : facade : publish', () => {
         componentPath: 'test/fixtures/components/hello-world/',
         username: creds.username,
         password: creds.password,
-        skipPackage
+        skipPackage,
+        registries
       },
       () => {
         cb();
@@ -101,6 +102,20 @@ describe('cli : facade : publish', () => {
         });
       });
 
+      it('should take precedence over the registries set through oc.json', done => {
+        sinon.stub(registry, 'putComponent').resolves('ok');
+        execute(
+          () => {
+            registry.putComponent.restore();
+
+            expect(logSpy.ok.args[0][0]).to.include('http://www.newapi.com');
+            expect(logSpy.ok.args[1][0]).to.include('http://www.newapi2.com');
+            done();
+          },
+          { registries: ['http://www.newapi.com', 'http://www.newapi2.com'] }
+        );
+      });
+
       describe('when packaging', () => {
         describe('when a component is not valid', () => {
           beforeEach(done => {