Skip to content

Commit

Permalink
PRO-16923: ProvarDX runtests fails to handle special characters in Sc…
Browse files Browse the repository at this point in the history
…ratch Org Passwords properly (#25)

* PRO-16923: ProvarDX runtests fails to handle special characters in Scratch Org Passwords properly

* PRO-16923: Special characters were causing issues on different OS so encoded the password

* PRO-16897: ProvarDX: Runtests secrets flag override incorrectly documented

* PRO-16897: ProvarDX: Runtests secrets flag override incorrectly documented
  • Loading branch information
riteshnahar committed Aug 28, 2020
1 parent a660e66 commit 609db17
Show file tree
Hide file tree
Showing 8 changed files with 73 additions and 9 deletions.
1 change: 0 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -8,4 +8,3 @@
node_modules
/.circleci
/.sfdx
/.vscode
35 changes: 35 additions & 0 deletions .vscode/launch.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,35 @@
{
// Use IntelliSense to learn about possible attributes.
// Hover to view descriptions of existing attributes.
// For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
"version": "0.2.0",
"configurations": [
{
"type": "node",
"request": "attach",
"name": "Attach to Remote",
"address": "127.0.0.1",
"port": 9229
},
{
"name": "Unit Tests",
"type": "node",
"request": "launch",
"protocol": "inspector",
"program": "${workspaceRoot}/node_modules/.bin/_mocha",
"args": [
"--require",
"test/helpers/init.js",
"--require",
"ts-node/register",
"--require",
"source-map-support/register",
"--recursive",
"--reporter",
"spec",
"test/**/*.test.ts"
],
"cwd": "${workspaceRoot}"
}
]
}
15 changes: 15 additions & 0 deletions .vscode/provardx-copyright.code-snippets
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
{
"Insert Copyright Header": {
"prefix": "copyright",
"body": [
"/*",
" * Copyright (c) 2020 Make Positive Provar Ltd",
" * All rights reserved.",
" * Licensed under the BSD 3-Clause license.",
" * For full license text, see LICENSE.md file in the repo root or https://opensource.org/licenses/BSD-3-Clause",
" */",
"$0"
],
"description": "Insert the Provar copyright header"
}
}
17 changes: 17 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,23 @@ All notable changes to this project will be documented in this file.
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).

## [0.2.3] - 2020-08-25

### Fixed

- ProvarDX runtests, metadatacache command fails in case we are having special characters in Scratch Org password

### Changed

- ProvarDX trademark changes
- Updated help messages for runtests

### Added

- Added .vscode folder containing
- `launch.json`: debug configuration for vscode
- `provardx-copyright.code-snippets`: used to add copyright tag on source files

## [0.2.2] - 2020-08-04

### Changed
Expand Down
6 changes: 3 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# provartesting/provardx
# ProvarDX™

ProvarDX is a Salesforce CLI Plugin for existing Provar customer to allow them to execute Provar Test Cases from the command line, leveraging the Salesforce CLI and SalesforceDX applications. This provides an alternative mechanism for running test cases than running under ANT.
ProvarDX is a Salesforce CLI Plugin for existing Provar customer to allow them to execute Provar Test Cases from the command line, leveraging the Salesforce CLI and SalesforceDX applications. This provides an alternative mechanism for running test cases than running under ANT.
You must be a Provar customer with a valid paid license to write and maintain your test cases.

[![Version](https://img.shields.io/npm/v/@provartesting/provardx.svg)](https://npmjs.org/package/@provartesting/provardx)
Expand Down Expand Up @@ -126,7 +126,7 @@ OPTIONS
must be in JSON format and conform to the provardx-properties
JSON schema.
-s, --secrets=secrets Specify path to secrets file
-s, --secrets=secrets Specify a secrets encryption key for accessing the chosen project/environment.
--json format output as json
Expand Down
2 changes: 1 addition & 1 deletion messages/runtests.json
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,6 @@
"connectionOverridefile": "Connection file in the format provardx-connection-schema.json providing a mapping between the Provar project Connection names and the target users to be used.",
"cachePathFlagDescription": "Specify relative or full file path for where a metadata cache has already been downloaded using either a VCS extract or metadata ProvarDX command.",
"metadataLevelFlagDescription": "Specify permitted values reload (get all metadata - default) | refresh (only download changes). This overrides any settings made in the propertyfile.",
"secretsFlagDescription": "Specify path to secrets file",
"secretsFlagDescription": "Specify a secrets encryption key for accessing the chosen project/environment.",
"loglevelFlagDescription": "Specify the level of feedback provided during the compilation (see above) and execution."
}
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
{
"name": "@provartesting/provardx",
"description": "A plugin for the Salesforce CLI to run provar testcases",
"version": "0.2.2",
"version": "0.2.3",
"author": "Provar",
"bugs": "https://github.com/ProvarTesting/provardx/issues",
"dependencies": {
Expand Down
4 changes: 1 addition & 3 deletions src/utilities/ProvarDXUtility.ts
Original file line number Diff line number Diff line change
Expand Up @@ -169,9 +169,7 @@ export default class ProvarDXUtility {

private handleSpecialCharacters(password: string): string {
if (password) {
password = password.split('&').join('"&"');
password = password.split('|').join('"|"');
password = password.split('^').join('"^"');
password = encodeURIComponent(password);
}
return password;
}
Expand Down

0 comments on commit 609db17

Please sign in to comment.