-
Bug
-
Resolution: Fixed
-
Minor
-
Powered by SuggestiMate
Snippet generator for xUnit produces some pretty ugly code. Similar to a number of other plugins, xUnit should be updated to use the "@Symbol" annotation to make the output more human readable. See JENKINS-37388 for one example.
[JENKINS-37611] Add "@Symbol" annotation to xunit and parameters
Actually this will be most likely be fixed JENKINS-27395 (https://github.com/jenkinsci/xunit-plugin/pull/49)
Code changed in jenkins
User: Nikolas Falco
Path:
pom.xml
src/main/java/org/jenkinsci/plugins/xunit/XUnitBuilder.java
src/main/java/org/jenkinsci/plugins/xunit/XUnitPublisher.java
src/main/java/org/jenkinsci/plugins/xunit/threshold/FailedThresholdDescriptor.java
src/main/java/org/jenkinsci/plugins/xunit/threshold/SkippedThresholdDescriptor.java
src/main/java/org/jenkinsci/plugins/xunit/types/AUnitJunitHudsonTestType.java
src/main/java/org/jenkinsci/plugins/xunit/types/BoostTestJunitHudsonTestType.java
src/main/java/org/jenkinsci/plugins/xunit/types/CTestType.java
src/main/java/org/jenkinsci/plugins/xunit/types/CUnitJunitHudsonTestType.java
src/main/java/org/jenkinsci/plugins/xunit/types/CheckType.java
src/main/java/org/jenkinsci/plugins/xunit/types/CppTestJunitHudsonTestType.java
src/main/java/org/jenkinsci/plugins/xunit/types/CppUnitJunitHudsonTestType.java
src/main/java/org/jenkinsci/plugins/xunit/types/CustomType.java
src/main/java/org/jenkinsci/plugins/xunit/types/EmbUnitType.java
src/main/java/org/jenkinsci/plugins/xunit/types/FPCUnitJunitHudsonTestType.java
src/main/java/org/jenkinsci/plugins/xunit/types/GTesterJunitHudsonTestType.java
src/main/java/org/jenkinsci/plugins/xunit/types/GoogleTestType.java
src/main/java/org/jenkinsci/plugins/xunit/types/JUnitType.java
src/main/java/org/jenkinsci/plugins/xunit/types/MSTestJunitHudsonTestType.java
src/main/java/org/jenkinsci/plugins/xunit/types/MbUnitType.java
src/main/java/org/jenkinsci/plugins/xunit/types/NUnit3TestType.java
src/main/java/org/jenkinsci/plugins/xunit/types/NUnitJunitHudsonTestType.java
src/main/java/org/jenkinsci/plugins/xunit/types/PHPUnitJunitHudsonTestType.java
src/main/java/org/jenkinsci/plugins/xunit/types/QTestLibInputMetric.java
src/main/java/org/jenkinsci/plugins/xunit/types/QTestLibType.java
src/main/java/org/jenkinsci/plugins/xunit/types/UnitTestJunitHudsonTestType.java
src/main/java/org/jenkinsci/plugins/xunit/types/ValgrindJunitHudsonTestType.java
src/main/java/org/jenkinsci/plugins/xunit/types/XUnitDotNetTestType.java
src/test/java/org/jenkinsci/plugins/xunit/XUnitWorkflowTest.java
http://jenkins-ci.org/commit/xunit-plugin/f8af22b430b869497c792d8edaafd31ae9a84950
Log:
JENKINS-37611 Add "@Symbol" annotation to xunit and parameters
Add @Symbol annotation to publisher, thresholds and test type to make pipeline readable. Add pipeline unit test that works with Jenkins 1.x
Update parent pom to latest 3.9
Code changed in jenkins
User: Nikolas Falco
Path:
src/main/java/org/jenkinsci/plugins/xunit/AliasInitializer.java
src/main/java/org/jenkinsci/plugins/xunit/XUnitBuilder.java
src/main/java/org/jenkinsci/plugins/xunit/XUnitPublisher.java
src/main/java/org/jenkinsci/plugins/xunit/XUnitUtil.java
src/main/java/org/jenkinsci/plugins/xunit/threshold/FailedThreshold.java
src/main/java/org/jenkinsci/plugins/xunit/threshold/SkippedThreshold.java
src/main/java/org/jenkinsci/plugins/xunit/threshold/XUnitThreshold.java
src/main/java/org/jenkinsci/plugins/xunit/threshold/XUnitThresholdDescriptor.java
src/main/resources/org/jenkinsci/plugins/xunit/XUnitBuilder/config.jelly
src/main/resources/org/jenkinsci/plugins/xunit/XUnitPublisher/config.jelly
src/main/resources/org/jenkinsci/plugins/xunit/threshold/Messages.properties
src/main/resources/util/hetero-list-readonly.jelly
src/main/resources/util/threshold.jelly
src/test/java/org/jenkinsci/plugins/xunit/XUnitSerialisationTest.java
src/test/java/org/jenkinsci/plugins/xunit/XUnitWorkflowTest.java
src/test/resources/org/jenkinsci/plugins/xunit/XUnitSerialisationTest/builder_1_103/jobs/foo/config.xml
src/test/resources/org/jenkinsci/plugins/xunit/XUnitSerialisationTest/publisher_1_103/jobs/foo/config.xml
src/test/resources/org/jenkinsci/plugins/xunit/config.xml
http://jenkins-ci.org/commit/xunit-plugin/46caa54a002cb93c218c4bb843b8d9ef1e7516a9
Log:
JENKINS-37611 Add "@Symbol" annotation to xunit and parameters
Move threshold parameters from required to optional. Remove also unreachable checks of % character in case of percent threshold mode. Add form validation for all threshold values.
Add unit test to guarantee serialisation works properly with plugin version before 1.103.
Manage negative value of time margin and fix NPE when a build step or recorder has no tools defined.
Compare: https://github.com/jenkinsci/xunit-plugin/compare/7ea630590a18...46caa54a002c
*NOTE:* This service been marked for deprecation: https://developer.github.com/changes/2018-04-25-github-services-deprecation/
Functionality will be removed from GitHub.com on January 31st, 2019.
Code changed in jenkins
User: Nikolas Falco
Path:
pom.xml
src/main/java/org/jenkinsci/plugins/xunit/XUnitBuilder.java
src/main/java/org/jenkinsci/plugins/xunit/XUnitPublisher.java
src/main/java/org/jenkinsci/plugins/xunit/threshold/FailedThresholdDescriptor.java
src/main/java/org/jenkinsci/plugins/xunit/threshold/SkippedThresholdDescriptor.java
src/main/java/org/jenkinsci/plugins/xunit/types/AUnitJunitHudsonTestType.java
src/main/java/org/jenkinsci/plugins/xunit/types/BoostTestJunitHudsonTestType.java
src/main/java/org/jenkinsci/plugins/xunit/types/CTestType.java
src/main/java/org/jenkinsci/plugins/xunit/types/CUnitJunitHudsonTestType.java
src/main/java/org/jenkinsci/plugins/xunit/types/CheckType.java
src/main/java/org/jenkinsci/plugins/xunit/types/CppTestJunitHudsonTestType.java
src/main/java/org/jenkinsci/plugins/xunit/types/CppUnitJunitHudsonTestType.java
src/main/java/org/jenkinsci/plugins/xunit/types/CustomType.java
src/main/java/org/jenkinsci/plugins/xunit/types/EmbUnitType.java
src/main/java/org/jenkinsci/plugins/xunit/types/FPCUnitJunitHudsonTestType.java
src/main/java/org/jenkinsci/plugins/xunit/types/GTesterJunitHudsonTestType.java
src/main/java/org/jenkinsci/plugins/xunit/types/GoogleTestType.java
src/main/java/org/jenkinsci/plugins/xunit/types/JUnitType.java
src/main/java/org/jenkinsci/plugins/xunit/types/MSTestJunitHudsonTestType.java
src/main/java/org/jenkinsci/plugins/xunit/types/MbUnitType.java
src/main/java/org/jenkinsci/plugins/xunit/types/NUnit3TestType.java
src/main/java/org/jenkinsci/plugins/xunit/types/NUnitJunitHudsonTestType.java
src/main/java/org/jenkinsci/plugins/xunit/types/PHPUnitJunitHudsonTestType.java
src/main/java/org/jenkinsci/plugins/xunit/types/QTestLibInputMetric.java
src/main/java/org/jenkinsci/plugins/xunit/types/QTestLibType.java
src/main/java/org/jenkinsci/plugins/xunit/types/UnitTestJunitHudsonTestType.java
src/main/java/org/jenkinsci/plugins/xunit/types/ValgrindJunitHudsonTestType.java
src/main/java/org/jenkinsci/plugins/xunit/types/XUnitDotNetTestType.java
src/test/java/org/jenkinsci/plugins/xunit/XUnitWorkflowTest.java
http://jenkins-ci.org/commit/xunit-plugin/e52107f6e3b88373f02c23b9c98ba1c263f5d85f
Log:
JENKINS-37611 Add "@Symbol" annotation to xunit and parameters
Add @Symbol annotation to publisher, thresholds and test type to make pipeline readable. Add pipeline unit test that works with Jenkins 1.x
Update parent pom to latest 3.9
Code changed in jenkins
User: Nikolas Falco
Path:
src/main/java/org/jenkinsci/plugins/xunit/AliasInitializer.java
src/main/java/org/jenkinsci/plugins/xunit/XUnitBuilder.java
src/main/java/org/jenkinsci/plugins/xunit/XUnitPublisher.java
src/main/java/org/jenkinsci/plugins/xunit/XUnitUtil.java
src/main/java/org/jenkinsci/plugins/xunit/threshold/FailedThreshold.java
src/main/java/org/jenkinsci/plugins/xunit/threshold/SkippedThreshold.java
src/main/java/org/jenkinsci/plugins/xunit/threshold/XUnitThreshold.java
src/main/java/org/jenkinsci/plugins/xunit/threshold/XUnitThresholdDescriptor.java
src/main/resources/org/jenkinsci/plugins/xunit/XUnitBuilder/config.jelly
src/main/resources/org/jenkinsci/plugins/xunit/XUnitPublisher/config.jelly
src/main/resources/org/jenkinsci/plugins/xunit/threshold/Messages.properties
src/main/resources/util/hetero-list-readonly.jelly
src/main/resources/util/threshold.jelly
src/test/java/org/jenkinsci/plugins/xunit/XUnitSerialisationTest.java
src/test/java/org/jenkinsci/plugins/xunit/XUnitWorkflowTest.java
src/test/resources/org/jenkinsci/plugins/xunit/XUnitSerialisationTest/builder_1_103/jobs/foo/config.xml
src/test/resources/org/jenkinsci/plugins/xunit/XUnitSerialisationTest/publisher_1_103/jobs/foo/config.xml
src/test/resources/org/jenkinsci/plugins/xunit/config.xml
http://jenkins-ci.org/commit/xunit-plugin/845615f222e19c949bcf7ab7f09126e4f33c7c3e
Log:
JENKINS-37611 Add "@Symbol" annotation to xunit and parameters
Move threshold parameters from required to optional. Remove also unreachable checks of % character in case of percent threshold mode. Add form validation for all threshold values.
Add unit test to guarantee serialisation works properly with plugin version before 1.103.
Manage negative value of time margin and fix NPE when a build step or recorder has no tools defined.
Compare: https://github.com/jenkinsci/xunit-plugin/compare/fadb7994c960...845615f222e1
*NOTE:* This service been marked for deprecation: https://developer.github.com/changes/2018-04-25-github-services-deprecation/
Functionality will be removed from GitHub.com on January 31st, 2019.
Created a PR: https://github.com/jenkinsci/xunit-plugin/pull/53