* Functional test for malformed waiver file
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
* Raise error for malformed yaml content and exit
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
* Updates functional test for malformed yaml waiver file and for empty waiver file
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
* Raises error in case of missing required parameters in waiver file
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
* Update functional test for missing parameters, extra parameters or column without headers in waiver file
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
* Fix linting
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
* Fix warning and error messages
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
* Updates nil and false check for yaml data and adds additional empty check.
Co-authored-by: Sathish Babu <80091550+sathish-progress@users.noreply.github.com>
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
* Adds more generic message as this yaml reader is now getting used by other functionalities like waiver file
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
* Fixed test description to reflect correct use case
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
* Fix validate headers was not validating the required parametes for all the data fields as it was not called inside the loop where we are iterating over the data and fetching the headers.
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
* Updates the test files for the use case to missing parameters and extra parameters
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
* Updates code to better handle errors and warnings related to missing required parameters and extra parameters in waivers file in all format i.e (yaml, json and csv).
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
* Updated functional test to capture the updated error and warning messages for waiver file validation
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
* Fix linting
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
* Deleted fixture file which is not required
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
* Refactor: Renamed method validate_headers to reflect whats it's doing and instead of return data in array it will now return the data in hash
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
---------
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
Co-authored-by: Sathish Babu <80091550+sathish-progress@users.noreply.github.com>
* EULA: Update latest EULA - 31AUG2023 in html-erb
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* Remove html tags in eula
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* CHORE: Fix known indentation
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* CHORE: Fix indentation - 2
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* CHORE: Break lines to maintain formatting
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
---------
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* Revert change to CLI page in b59e2cb
Signed-off-by: Ian Maddaus <ian.maddaus@progress.com>
* Reformat options so they are a description list
Signed-off-by: Ian Maddaus <ian.maddaus@progress.com>
* Add license content from #6583
Signed-off-by: Ian Maddaus <ian.maddaus@progress.com>
---------
Signed-off-by: Ian Maddaus <ian.maddaus@progress.com>
* Functional Test to verify git fetcher able to work in a directory where git is not initialized or does not contain the .git directory
Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io>
* Fix: If git branch, tag or reference information is not given it should clone the repo and checkout the default HEAD
Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io>
* Remove the logic to find default branch in case of branch, tag or ref is not provided as it's not required logically as we will be cloning the and checkout the default HEAD
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
* This is a temporary change to make verify test working on this PR. Revert once the work is done for the git fetcher fix
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
* Unit Test: Fix unit test
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
* Verifies the git url before creating cache_key, if given relative path does not exist make sure it deletes the cache directory created to copy the cloned repo. It raises error if any git command fails
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
* Reverted verify git repo url call from cache_key as cache_key and shifted that to resolve_ref which is already raising the error in case of git ls remote command fails
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
* Unit test to cover the changes done for resolve_ref when branch, tag or ref is not given and
unit test for invalid git repo
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
---------
Signed-off-by: Vasu1105 <vasundhara.jagdale@chef.io>
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
* Try explicit list of gems to pull from artifactory to avoid issue on CI
Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com>
* Fix linting
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
---------
Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com>
Signed-off-by: Vasu1105 <vasundhara.jagdale@progress.com>
Co-authored-by: Vasu1105 <vasundhara.jagdale@progress.com>
* CONFIG: Keeping the verify pipeline alive
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
* Expand the message returned to be more explnatory
Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com>
* Remove unused old file, was leading to confusion
Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com>
---------
Signed-off-by: Sonu Saha <sonu.saha@progress.com>
Signed-off-by: Clinton Wolfe <clintoncwolfe@gmail.com>
Co-authored-by: Clinton Wolfe <clintoncwolfe@gmail.com>
