Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Documentation: Improve Dspot diff documentation #768

Closed
nicolabertazzo opened this issue Apr 18, 2019 · 3 comments
Closed

Documentation: Improve Dspot diff documentation #768

nicolabertazzo opened this issue Apr 18, 2019 · 3 comments
Assignees
Labels
diff-test-selection This issue is related to the module DSpot - Diff Test Selection documentation improvement

Comments

@nicolabertazzo
Copy link
Contributor

Characteristics

  • Issue Type: feature
  • Reproducibility: always
  • Severity: feature
  • Tool/Service/Component: dispot-diff / documentation
  • Execution Environment:
  • Reporter: @nicolabertazzo , [email protected]

Description

Hi @danglotb,

Some properties used to execute Dspot diff are not documented.

This command line properties is not documented:

  • path-to-test-list-csv

This dspot.proerties properties is not documented:

  • configPath

The name of this dspot.properties is not clear:

  • folderPath

I suggest to add a description of this properties also in dspot-diff-test-selection (because it's mandatory when you want to use dispot-diff)

Relationships

Help on issue template

Preview to follow the link or open file .github/ISSUE_DOC.md

@nicolabertazzo nicolabertazzo added the diff-test-selection This issue is related to the module DSpot - Diff Test Selection label Apr 18, 2019
@danglotb
Copy link
Member

Hello @nicolabertazzo

configPath

Is no longer needed nor available.

I suggest to add a description of this properties also in dspot-diff-test-selection (because it's mandatory when you want to use dispot-diff)

It is not. The dspot-diff-test-selection does not need any properties file. It is if you want to run sequencially dspot and dspot-diff-test-selection.

This dspot.proerties properties is not documented:

Thank you, I added specific options of the maven plugin.

Please review #769

@nicolabertazzo
Copy link
Contributor Author

Tnx @danglotb ,

I found the maven property .

I'll test it and I'll update.

@nicolabertazzo
Copy link
Contributor Author

I tested maven properties with the SNAPSHOT version and it works.
I'll wait the stable version to integrate it on CI/CD in Jenkins.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
diff-test-selection This issue is related to the module DSpot - Diff Test Selection documentation improvement
Projects
None yet
Development

No branches or pull requests

2 participants