Skip to content
This repository has been archived by the owner on Feb 12, 2022. It is now read-only.

Added flag to fail execution when no test classes exist for specified manifest and/or regex only if it's set to true #80

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lukaszkostrzewa
Copy link

Currently, when no test class is found for specified manifest and/or regex, then the execution fails. I've added a flag -require.at.least.one.test which doesn't fail the execution in such a scenario unless it's set to true (which is a default value).

Motivation: our case is that we are running tests while validating Pull Requests in our CI/CD pipeline and we are running only these tests which have changed or which source classes have changed, e.g. when MyClassApex has changed, then we are running test MyClassApexTest. We are simply appending 'Test' to the class name (by convention) but not all classes have tests and in such a situation, we don't want to fail the build.

Feel free to merge, maybe someone will have a similar situation to ours :)

@salesforce-cla
Copy link

salesforce-cla bot commented Dec 7, 2019

Thanks for the contribution! Unfortunately we can't verify the commit author(s): Kostrzewa, Lukasz <l***@s***.com>. One possible solution is to add that email to your GitHub account. Alternatively you can change your commits to another email and force push the change. After getting your commits associated with your GitHub account, sign the Salesforce.com Contributor License Agreement and this Pull Request will be revalidated.

…sses exist for specified manifest and/or regex only if it's set to true
@lukaszkostrzewa
Copy link
Author

Expected feature release number in range of 9 to 14, but got: 7

How about adding dist: trusty to .travis.yml?

https://travis-ci.community/t/error-installing-oraclejdk8-expected-feature-release-number-in-range-of-9-to-14-but-got-8/3766/4

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant