codefresh 0.78.3 → 0.79.0

package/e2e/scenarios/environments.sh

@@ -0,0 +1 @@
+codefresh get environments

package/e2e/scenarios/helm-repo.sh

@@ -0,0 +1 @@
+codefresh get helm-repo

package/e2e/scenarios/images.sh

@@ -0,0 +1 @@
+codefresh get images

package/e2e/scenarios/pipeline-run.sh

@@ -0,0 +1,26 @@
+project=test-run
+pipeline="$project/test-run"
+pipeline_file="$SCRIPT_DIR/data/test-run.pip.yaml"
+
+existing_pipeline=`exists pipeline $pipeline`
+existing_project=`exists project $project`
+
+if [ -z "$existing_project" ]; then
+  echo "Project does not exist: $project"
+  echo "Creating project: $project"
+  codefresh create project $project
+else
+  echo "Project already exists: $project"
+fi
+
+if [ -z "$existing_pipeline" ]; then
+  echo "Pipeline does not exist: $pipeline"
+  echo "Creating pipeline: $pipeline"
+  codefresh create -f $pipeline_file
+else
+  echo "Pipeline already exists: $pipeline"
+fi
+
+codefresh run $pipeline
+codefresh delete pipeline $pipeline
+codefresh delete project $project

package/e2e/scenarios/pipelines.sh

@@ -0,0 +1,31 @@
+project='pipeline-crud'
+pipeline="$project/crud"
+pipeline_file="$SCRIPT_DIR/data/crud.pip.yaml"
+
+existing_project=`exists project $project`
+existing_pipeline=`exists pipeline $pipeline`
+
+if [ -z "$existing_project" ]; then
+  echo "Project does not exist: $project"
+  echo "Creating project: $project"
+  codefresh create project $project
+else
+  echo "Project already exists: $project"
+fi
+
+if [ -z "$existing_pipeline" ]; then
+  echo "Pipeline does not exist: $pipeline"
+  echo "Creating pipeline: $pipeline"
+  codefresh create -f $pipeline_file
+else
+  echo "Pipeline already exists: $pipeline"
+  codefresh delete pipeline $pipeline
+  codefresh create -f $pipeline_file
+fi
+
+codefresh get pipelines
+codefresh get pipeline $pipeline
+
+codefresh delete pipeline $pipeline
+codefresh delete project $project
+

package/e2e/scenarios/projects.sh

@@ -0,0 +1,17 @@
+project='project-crud'
+
+existing_project=`exists project $project`
+
+if [ -z "$existing_project" ]; then
+  echo "Project does not exist: $project"
+  echo "Creating project: $project"
+  codefresh create project $project
+else
+  echo "Project already exists: $project"
+  codefresh delete project $project
+  codefresh create project $project
+fi
+
+codefresh get projects
+codefresh get project $project
+codefresh delete project $project

package/e2e/scenarios/registry.sh

@@ -0,0 +1 @@
+codefresh get registry

package/e2e/scenarios/repository.sh

@@ -0,0 +1,2 @@
+codefresh get repository
+codefresh get repository -a

package/e2e/scenarios/runtime-environment.sh

@@ -0,0 +1 @@
+codefresh get runtime-environments

package/e2e/scenarios/step-types.sh

@@ -0,0 +1 @@
+codefresh get step-types

package/e2e/scenarios/system-runtime-environments.sh

@@ -0,0 +1 @@
+codefresh get system-runtime-environments

package/e2e/scenarios/teams.sh

@@ -0,0 +1 @@
+codefresh get teams

package/e2e/scenarios/tokens.sh

@@ -0,0 +1 @@
+codefresh get tokens

package/e2e/scenarios/triggers.sh

@@ -0,0 +1,3 @@
+codefresh get triggers
+codefresh get trigger-events
+codefresh get trigger-types

package/lib/interface/cli/commands/workflow/logs.cmd.js

@@ -20,6 +20,10 @@ const run = new Command({
                 describe: 'Continue showing build logs until it will finish',
                 type: 'boolean',
             })
+            .option('no-timestamps', {
+                describe: 'Don\'t show timestamps',
+                type: 'boolean',
+            })
             .example('codefresh logs ID', 'Get logs of build ID')
             .example('codefresh logs -f ID', 'Get all previous logs of build ID and attach to future logs');
 
@@ -28,8 +32,9 @@ const run = new Command({
     handler: async (argv) => {
         const workflowId = argv.id;
         const follow = argv.f;
+        const removeTimestamps = argv.noTimestamps
 
-        await sdk.logs.showWorkflowLogs(workflowId, follow);
+        await sdk.logs.showWorkflowLogs(workflowId, follow, removeTimestamps);
         // TODO fix this. its a workaround since something related to firebase in not properly closed
         process.exit(0);
     },

package/lib/interface/cli/completion/completion.spec.js

@@ -1,11 +1,14 @@
 const initCompletion = require('./index');
 const { handleOptions: mockHandleOptions, fileDir: mockFileDir } = require('./helpers');
-const mockPath = require('path');
 
-const completions = initCompletion();
+const mockCompletions = initCompletion();
+
+function mockCwd() {
+    return process.cwd();
+}
 
 function getCompletion(args) {
-    return new Promise(resolve => completions.getCompletion(args, resolve));
+    return new Promise(resolve => mockCompletions.getCompletion(args, resolve));
 }
 
 jest.mock('./tree', () => ({
@@ -67,11 +70,11 @@ jest.mock('./context/create.completion', () => { // eslint-disable-line
 
 jest.mock('fs', () => {
     const existsSync = (p) => {
-        if (p.startsWith(`${process.cwd()}/`)) {
-            p = p.replace(`${process.cwd()}/`, '');
+        if (p.startsWith(`${mockCwd()}/`)) {
+            p = p.replace(`${mockCwd()}/`, '');
         }
-        if (p.startsWith(process.cwd())) {
-            p = p.replace(process.cwd(), '');
+        if (p.startsWith(mockCwd())) {
+            p = p.replace(mockCwd(), '');
         }
         switch (p) {
             case '':
@@ -89,11 +92,11 @@ jest.mock('fs', () => {
     const lstatSync = (p) => {
         let isFile = false;
         let isDir = true;
-        if (p.startsWith(`${process.cwd()}/`)) {
-            p = p.replace(`${process.cwd()}/`, '');
+        if (p.startsWith(`${mockCwd()}/`)) {
+            p = p.replace(`${mockCwd()}/`, '');
         }
-        if (p.startsWith(process.cwd())) {
-            p = p.replace(process.cwd(), '');
+        if (p.startsWith(mockCwd())) {
+            p = p.replace(mockCwd(), '');
         }
         switch (p) {
             case 'some.yaml':
@@ -116,11 +119,11 @@ jest.mock('fs', () => {
     };
 
     const readdirSync = (p) => {
-        if (p.startsWith(`${process.cwd()}/`)) {
-            p = p.replace(`${process.cwd()}/`, '');
+        if (p.startsWith(`${mockCwd()}/`)) {
+            p = p.replace(`${mockCwd()}/`, '');
         }
-        if (p.startsWith(process.cwd())) {
-            p = p.replace(process.cwd(), '');
+        if (p.startsWith(mockCwd())) {
+            p = p.replace(mockCwd(), '');
         }
         switch (p) {
             case '':

package/lib/interface/cli/helpers/helpers.sdk.spec.js

@@ -67,15 +67,6 @@ describe('helpers using sdk', () => {
             expect(sdk.logs.showWorkflowLogs).toBeCalled();
             await verifyResponsesReturned([DEFAULT_RESPONSE]); // eslint-disable-line
         });
-        it('should not show logs if check version throw an exception', async () => {
-            const workflowId = 'id';
-            jest.spyOn(sdk.logs, 'showWorkflowLogs').mockImplementation();
-            versionChecker.checkVersion.mockImplementationOnce(() => {
-                throw new Error();
-            });
-            await followLogs(workflowId);
-            expect(sdk.logs.showWorkflowLogs).not.toBeCalled();
-        });
     });
 
     describe('auth', () => {

package/lib/interface/cli/helpers/logs.js

@@ -6,9 +6,8 @@ const CFError = require('cf-errors');
 const { sdk } = require('../../../logic');
 const Workflow = require('../../../logic/entities/Workflow');
 const Output = require('../../../output/Output');
-const { checkVersion } = require('./../../../../check-version');
-
-const nodeVersionToValidate = '<=10.x >=8.x';
+const { checkVersion } = require('../../../../check-version');
+const { engines } = require('../../../../package.json');
 
 const END_STATUSES = ['error', 'success', 'terminated'];
 
@@ -56,15 +55,6 @@ async function _fallbackLogs(workflowId, interval, retriesLeft) {
 }
 
 const followLogs = async (workflowId) => {
-    try {
-        checkVersion(nodeVersionToValidate);
-    } catch (error) {
-        const url = _.get(sdk, 'config.context.url', 'https://g.codefresh.io');
-        const buildLink = `${url}/build/${workflowId}`;
-        // eslint-disable-next-line max-len
-        console.log(`Your node version (${process.version}) is not compatible with supported node version (${nodeVersionToValidate}) for this command, click the link to see it in codefresh UI : ${buildLink}`);
-        return;
-    }
     try {
         await sdk.logs.showWorkflowLogs(workflowId, true);
     } catch (e) {

package/lib/logic/cli-config/Manager.js

@@ -82,7 +82,6 @@ class CliConfigManager {
         }, {});
     }
 
-
     static set(propertyName, value) {
         const properties = Model.findProperties(propertyName);
         _validate(properties, propertyName);
@@ -135,9 +134,9 @@ class CliConfigManager {
     static meta(propertyName) {
         const meta = Model.meta();
         return _.keys(meta)
-            .filter(key => !propertyName || !propertyName.length || key.includes(propertyName))
+            .filter((key) => !propertyName || !propertyName.length || key.includes(propertyName))
             .sort()
-            .map(key => Object.assign({ key }, _.get(meta, key)));
+            .map((key) => ({ key, ..._.get(meta, key) }));
     }
 
     static _preloadConfig() {
@@ -168,12 +167,12 @@ class CliConfigManager {
         try {
             Model.validate(CURRENT_CONFIG);
         } catch (e) {
-            const errorPaths = e.errors.map(er => er.dataPath.replace('.', ''));
+            const errorPaths = e.errors.map((er) => er.dataPath.replace('.', ''));
 
             console.warn('---------------');
             console.warn('!!! Some properties are invalid -- replacing with defaults:');
 
-            errorPaths.forEach(ep => {
+            errorPaths.forEach((ep) => {
                 console.warn(ep);
                 _.set(CURRENT_CONFIG, ep, undefined);
             });

package/lib/logic/cli-config/Model.js

@@ -11,12 +11,12 @@ let DEFAULTS;
 let PROPERTIES;
 let PROPERTIES_META;
 
-
 /**
  * Every schema object need to have default set to {} in order to init its properties with default value
  * */
 function _fillSchemaObjectsWithDefault(schema) {
     if (schema.type === 'object') {
+        // eslint-disable-next-line no-param-reassign
         schema.default = schema.default || {};
         if (schema.properties) {
             _.values(schema.properties).forEach((value) => {
@@ -38,13 +38,13 @@ function _compileSchema() {
 
 function _extractMeta(props) {
     return props.reduce((obj, prop) => {
+        // eslint-disable-next-line no-param-reassign
         obj[prop] = prop.split('.')
             .reduce((prev, pathPart) => prev.properties[pathPart], cliConfigSchema);
         return obj;
     }, {});
 }
 
-
 class Model {
     static default() {
         return _.cloneDeep(DEFAULTS);
@@ -55,13 +55,13 @@ class Model {
     }
 
     static findProperties(name) {
-        return PROPERTIES.filter(prop => prop.includes(name));
+        return PROPERTIES.filter((prop) => prop.includes(name));
     }
 
-
     static meta() {
         return _.cloneDeep(PROPERTIES_META);
     }
+
     /**
      * also fills with default values and coerces types
      * */
@@ -79,4 +79,3 @@ PROPERTIES = _.keys(flatten(DEFAULTS));
 PROPERTIES_META = _extractMeta(PROPERTIES);
 
 module.exports = Model;
-

package/lib/logic/cli-config/manager.spec.js

@@ -1,23 +1,22 @@
 const _ = require('lodash');
 const path = require('path');
 
+const fs = require('fs');
 const Manager = require('./Manager');
 const Model = require('./Model');
 
 const { CODEFRESH_PATH: mockCodefreshPath } = require('../../interface/cli/defaults');
+
 const mockDirPath = path.resolve(mockCodefreshPath, 'cli-config');
 const mockFilePath = path.resolve(mockDirPath, 'config.yaml');
 
-
 jest.mock('fs', () => { // eslint-disable-line
     const readFileSync = () => this.configFile;
-    const existsSync = (path) => {
-        return {
-            [mockCodefreshPath]: true,
-            [mockDirPath]: true,
-            [mockFilePath]: this.exists,
-        }[path];
-    };
+    const existsSync = (targetPath) => ({
+        [mockCodefreshPath]: true,
+        [mockDirPath]: true,
+        [mockFilePath]: this.exists,
+    }[targetPath]);
     const openSync = () => null;
     const writeSync = (file, config) => {
         this.configFile = config;
@@ -47,8 +46,8 @@ jest.mock('fs', () => { // eslint-disable-line
 });
 
 jest.mock('js-yaml', () => {
-    const safeLoad = d => d;
-    const safeDump = d => d;
+    const safeLoad = (d) => d;
+    const safeDump = (d) => d;
     return {
         safeLoad,
         safeDump,
@@ -67,7 +66,7 @@ jest.mock('./schema', () => { // eslint-disable-line
                     pretty: {
                         type: 'boolean',
                         default: false,
-                        description: 'Defines whether to show data in table view in pretty mode or not'
+                        description: 'Defines whether to show data in table view in pretty mode or not',
                     },
                     test: {
                         type: 'object',
@@ -92,8 +91,6 @@ jest.mock('./schema', () => { // eslint-disable-line
     };
 });
 
-const fs = require('fs');
-
 describe('CliConfigManager', () => {
     describe('config loading', () => {
         // this test case must be first due to node module cache

package/lib/logic/entities/Agent.js

@@ -1,6 +1,6 @@
 const _ = require('lodash');
-const Entity = require('./Entity');
 const chalk = require('chalk');
+const Entity = require('./Entity');
 
 class Agent extends Entity {
     constructor(data) {
@@ -42,11 +42,9 @@ class Agent extends Entity {
         }
     }
 
-
     static fromResponse(response) {
         return new Agent(_.pick(response, 'id', 'name', 'runtimes', 'status', 'lastReported'));
     }
 }
 
 module.exports = Agent;
-

package/lib/logic/entities/CodefreshRepo.js

@@ -10,7 +10,7 @@ class CodefreshRepo extends Entity {
     }
 
     static fromResponse(response) {
-        const data = Object.assign({}, response);
+        const data = { ...response };
         data.name_id = response.serviceName;
         data.git_context = response.provider;
         data.owner = response.owner.login;

package/lib/logic/entities/Entity.js

@@ -11,6 +11,7 @@ class Entity {
     toDefault() {
         return this.extractValues(this.defaultColumns);
     }
+
     // wide Columns means all the fields we want to show when using get with -o wide option (return more information then the defualt)
     toWide() {
         return this.extractValues(this.wideColumns);

package/lib/logic/entities/Environment.js

@@ -2,6 +2,7 @@ const Entity = require('./Entity');
 
 const _getServiceInfo = (environment) => {
     const services = [];
+    // eslint-disable-next-line no-plusplus
     for (let i = 0; i < environment.instances.length; i++) {
         const service = environment.instances[i];
         services.push({

package/lib/logic/entities/GitRepo.js

@@ -10,7 +10,7 @@ class GitRepo extends Entity {
     }
 
     static fromResponse(response) {
-        const data = Object.assign({}, response);
+        const data = { ...response };
         data.git_context = response.provider;
         data.owner = response.owner.login;
         data.repo_shortcut = `${data.owner}/${data.name}`;

package/lib/logic/entities/HelmRepo.js

@@ -1,6 +1,5 @@
-const moment = require('moment');
-const Entity = require('./Entity');
 const _ = require('lodash');
+const Entity = require('./Entity');
 
 class HelmRepo extends Entity {
     constructor(data) {

package/lib/logic/entities/Pipeline.js

@@ -1,5 +1,5 @@
-const Entity = require('./Entity');
 const _ = require('lodash');
+const Entity = require('./Entity');
 
 class Pipeline extends Entity {
     constructor(data) {

package/lib/logic/entities/RuntimeEnvironments.js

@@ -1,5 +1,5 @@
-const Entity = require('./Entity');
 const _ = require('lodash');
+const Entity = require('./Entity');
 
 class RuntimeEnvironments extends Entity {
     constructor(data) {

package/lib/logic/entities/StepType.js

@@ -1,5 +1,5 @@
-const Entity = require('./Entity');
 const _ = require('lodash');
+const Entity = require('./Entity');
 
 class StepType extends Entity {
     constructor(data) {
@@ -8,7 +8,7 @@ class StepType extends Entity {
         this.info = data;
         this.name = this.info.metadata.name;
         this.visibility = this.info.metadata.isPublic ? 'public' : 'private';
-        this.official = this.info.metadata.official ? "true": "false";
+        this.official = this.info.metadata.official ? 'true' : 'false';
         this.version = this.info.metadata.version;
         this.category = this.info.metadata.categories;
         this.stage = this.info.metadata.stage;

package/lib/logic/entities/StepTypeVersion.js

@@ -1,5 +1,4 @@
 const Entity = require('./Entity');
-const _ = require('lodash');
 
 class StepTypeVersion extends Entity {
     constructor(data) {

package/lib/logic/entities/Team.js

@@ -11,13 +11,13 @@ class Team extends Entity {
 
     toDefault() {
         const data = super.toDefault();
-        data.users = data.users.map(user => user.userName);
+        data.users = data.users.map((user) => user.userName);
         return data;
     }
 
     toWide() {
         const data = super.toWide();
-        data.users = data.users.map(user => user.userName);
+        data.users = data.users.map((user) => user.userName);
         return data;
     }
 

package/lib/logic/entities/Token.js

@@ -1,5 +1,5 @@
-const Entity = require('./Entity');
 const _ = require('lodash');
+const Entity = require('./Entity');
 
 class Token extends Entity {
     constructor(data) {
@@ -11,7 +11,7 @@ class Token extends Entity {
     }
 
     static fromResponse(response) {
-        const data = Object.assign({}, response);
+        const data = { ...response };
         data.id = data._id;
         data.token_prefix = response.tokenPrefix;
         data.scopes = _.isArray(response.scopes) ? response.scopes : ['any'];

package/lib/logic/entities/Workflow.js

@@ -1,5 +1,5 @@
-const Entity = require('./Entity');
 const moment = require('moment');
+const Entity = require('./Entity');
 
 class Workflow extends Entity {
     constructor(data) {

package/lib/logic/index.js

@@ -3,5 +3,3 @@ const sdk = require('./sdk');
 module.exports = {
     sdk,
 };
-
-