Comments (3)
please do not be crazy with over testing.
If that is required , it is already sign that tool need to be written in java and have complete set of validations.
We did this tool on groovy only to be a bit more user friendly (not a shell) and keep simple (script like).
In all my experience with groovy, any investment in complicated groovy will result in high cost of support. As you grow script it should become multi-file , so not a script any more so need to be compiled , so..... become the same as java.
As I told you in other issue, it is enough to keep in CI full run with recheck of expected output.
from contribution.
need to be written in java .... We did this tool on groovy only
I think you are confusing what tool I am talking about here.
patch-diff-report-tool is already in Java, not groovy. I am not talking about making tests for groovy scripts here.
from contribution.
yes, my bad.
tests for java diff tool is ok.
from contribution.
Related Issues (20)
- releasenotes-builder: Release notes for GitHub shows escape at-symbol HOT 4
- Extend config of VariableDeclarationUsageDistance with validateBetweenScopes HOT 1
- diff.groovy: NullPointerException HOT 1
- diff.groovy: Unable to find configuration file at location: false HOT 1
- IDEA Docker Image documentation
- Update diff.groovy to work with latest GROOVY version HOT 5
- Convert maven-checkstyle-plugin run to ant run HOT 1
- bump-cs-version missed eclipsecs-sevntu-plugin HOT 1
- releasenotes builder: release version validation should work on digits HOT 6
- deprecate RSS and mail list publishers HOT 1
- releasenotes-builder: Exception on unknown Issue HOT 12
- diff.groovy: Reports downloading messages
- releasenotes-builder: raise error in case releaseNumber "10.0.1."
- releasenotes-builder: no help or usage print by default HOT 2
- patch-diff-report-tool: deprecated attributes HOT 1
- dtd dfiles on puppycrawl.com are not accessible HOT 2
- Update codenarc to latest version HOT 4
- diff.groovy: add multi thread support HOT 2
- Update checkstyle/jdk-11-groovy-git-mvn to use latest version of maven, git, and groovy HOT 12
- diff.groovy: Not all spans are closed and leak to other projects HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from contribution.