Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-16942

hudson.util.SecretRewriterTest test fail on Windows 7

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • core

      Running hudson.util.SecretRewriterTest
      Scanning C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\a\foo.xml
      Copied C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\a\foo.xml to C:\Users\wang\workspace\jenkins\core\target\hudson6550280505320084052tmp\a\foo.xml as a backup
      Rewritten C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\a\foo.xml
      Copied C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\a\symlink\foo.xml to C:\Users\wang\workspace\jenkins\core\target\hudson6550280505320084052tmp\a\symlink\foo.xml as a backup
      Rewritten C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\a\symlink\foo.xml
      Copied C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\b\foo.xml to C:\Users\wang\workspace\jenkins\core\target\hudson6550280505320084052tmp\b\foo.xml as a backup
      Rewritten C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\b\foo.xml
      Copied C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\c\d\e\foo.xml to C:\Users\wang\workspace\jenkins\core\target\hudson6550280505320084052tmp\c\d\e\foo.xml as a backup
      Rewritten C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\c\d\e\foo.xml
      Copied C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\c\d\foo.xml to C:\Users\wang\workspace\jenkins\core\target\hudson6550280505320084052tmp\c\d\foo.xml as a backup
      Rewritten C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\c\d\foo.xml
      Copied C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\c\foo.xml to C:\Users\wang\workspace\jenkins\core\target\hudson6550280505320084052tmp\c\foo.xml as a backup
      Rewritten C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\c\foo.xml
      Scanning C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\c\symlink\b\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\foo.xml
      Scanning C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\c\symlink\c\symlink\c\foo.xml
      Scanning C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\c\symlink\c\symlink\c\symlink\c\symlink\b\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\foo.xml
      Scanning C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\b\symlink\symlink\symlink\symlink\symlink\foo.xml
      Scanning C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\b\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\foo.xml
      Scanning C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\b\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\foo.xml
      Scanning C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\b\symlink\symlink\symlink\symlink\symlink\foo.xml
      Scanning C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\b\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\foo.xml
      Scanning C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\b\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\foo.xml
      Scanning C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\b\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\foo.xml
      Scanning C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\b\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\foo.xml
      Scanning C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\b\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\foo.xml
      Scanning C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\b\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\foo.xml
      Scanning C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\a\foo.xml
      Scanning C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\b\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\foo.xml
      Scanning C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\b\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\foo.xml
      Scanning C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\b\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\foo.xml
      Scanning C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\b\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\foo.xml
      Scanning C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\b\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\symlink\foo.xml
      Scanning C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\b\symlink\symlink\symlink\foo.xml
      Scanning C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\b\symlink\symlink\symlink\symlink\symlink\foo.xml
      Scanning C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\b\symlink\symlink\symlink\symlink\symlink\foo.xml
      Scanning C:\Users\wang\workspace\jenkins\core\target\hudson8568238377048846297tmp\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\symlink\c\foo.xml
      Tests run: 2, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 464.97 sec <<< FAILURE!
      singleFileRewrite(hudson.util.SecretRewriterTest) Time elapsed: 0.918 sec <<< FAILURE!
      Assertion failed:

      assert after.trim()==f.text.trim()

               
              <abc>
              <foo>VJJT58qPAwRt7BYvOI/AVTZZCtwtcCf7MXhMEdmO0dM=</foo>
              </abc>
            <abc>
            <foo>VJJT58qPAwRt7BYvOI/AVTZZCtwtcCf7MXhMEdmO0dM=</foo>
            </abc>
          C:\Users\wang\workspace\jenkins\core\target\test5450512927023768251xml
        false
      <abc>
      <foo>VJJT58qPAwRt7BYvOI/AVTZZCtwtcCf7MXhMEdmO0dM=</foo>
      </abc>
      <abc>
      <foo>VJJT58qPAwRt7BYvOI/AVTZZCtwtcCf7MXhMEdmO0dM=</foo>
      </abc>

      at org.codehaus.groovy.runtime.InvokerHelper.assertFailed(InvokerHelper.java:386)
      at org.codehaus.groovy.runtime.ScriptBytecodeAdapter.assertFailed(ScriptBytecodeAdapter.java:658)
      at hudson.util.SecretRewriterTest.roundtrip(SecretRewriterTest.groovy:55)
      at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
      at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
      at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
      at java.lang.reflect.Method.invoke(Method.java:601)
      at org.codehaus.groovy.runtime.callsite.PogoMetaMethodSite$PogoCachedMethodSiteNoUnwrap.invoke(PogoMetaMethodSite.java:247)
      at org.codehaus.groovy.runtime.callsite.PogoMetaMethodSite.callCurrent(PogoMetaMethodSite.java:52)
      at org.codehaus.groovy.runtime.callsite.CallSiteArray.defaultCallCurrent(CallSiteArray.java:46)
      at org.codehaus.groovy.runtime.callsite.AbstractCallSite.callCurrent(AbstractCallSite.java:133)
      at org.codehaus.groovy.runtime.callsite.AbstractCallSite.callCurrent(AbstractCallSite.java:145)
      at hudson.util.SecretRewriterTest.singleFileRewrite(SecretRewriterTest.groovy:46)
      at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
      at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
      at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
      at java.lang.reflect.Method.invoke(Method.java:601)
      at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:45)
      at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:15)
      at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:42)
      at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:20)
      at org.junit.rules.ExternalResource$1.evaluate(ExternalResource.java:46)
      at org.junit.rules.ExternalResource$1.evaluate(ExternalResource.java:46)
      at org.junit.rules.RunRules.evaluate(RunRules.java:18)
      at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:263)
      at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:68)
      at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:47)
      at org.junit.runners.ParentRunner$3.run(ParentRunner.java:231)
      at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:60)
      at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:229)
      at org.junit.runners.ParentRunner.access$000(ParentRunner.java:50)
      at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:222)
      at org.junit.runners.ParentRunner.run(ParentRunner.java:300)
      at org.apache.maven.surefire.junit4.JUnit4Provider.execute(JUnit4Provider.java:252)
      at org.apache.maven.surefire.junit4.JUnit4Provider.executeTestSet(JUnit4Provider.java:141)
      at org.apache.maven.surefire.junit4.JUnit4Provider.invoke(JUnit4Provider.java:112)
      at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
      at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
      at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
      at java.lang.reflect.Method.invoke(Method.java:601)
      at org.apache.maven.surefire.util.ReflectionUtils.invokeMethodWithArray(ReflectionUtils.java:189)
      at org.apache.maven.surefire.booter.ProviderFactory$ProviderProxy.invoke(ProviderFactory.java:165)
      at org.apache.maven.surefire.booter.ProviderFactory.invokeProvider(ProviderFactory.java:85)
      at org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess(ForkedBooter.java:115)
      at org.apache.maven.surefire.booter.ForkedBooter.main(ForkedBooter.java:75)

          [JENKINS-16942] hudson.util.SecretRewriterTest test fail on Windows 7

          kutzi added a comment -

          Pull requests are welcome.

          kutzi added a comment - Pull requests are welcome.

          Andrew Mann added a comment -

          Pull request: https://github.com/jenkinsci/jenkins/pull/745

          Note that SecretRewriterTest still fails on the recursion test unless run with administrator privileges (Windows 8), and hudson.util.ProcessTreeTest errors trying to read process environment variables with any execution privilege level (Windows 8)

          Andrew Mann added a comment - Pull request: https://github.com/jenkinsci/jenkins/pull/745 Note that SecretRewriterTest still fails on the recursion test unless run with administrator privileges (Windows 8), and hudson.util.ProcessTreeTest errors trying to read process environment variables with any execution privilege level (Windows 8)

          Jesse Glick added a comment -

          This was merged but I guess the JIRA commented bot is broken?

          Jesse Glick added a comment - This was merged but I guess the JIRA commented bot is broken?

          Code changed in jenkins
          User: Andrew Mann
          Path:
          core/src/test/java/hudson/util/SecretRewriterTest.groovy
          http://jenkins-ci.org/commit/jenkins/899f5aa41769c6b6265c95bd63517b35e760f1fa
          Log:
          Fix for JENKINS-16942

          https://issues.jenkins-ci.org/browse/JENKINS-16942

          PrintWriter.println() uses the OS specific line separator. When
          SecretRewriter rewrites the secret file, the lines are converted to the
          format of the running OS. The test case does not account for this.
          Since the test case is coded expecting line endings of \n, this fix
          converts \r\n to \n before comparing.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Andrew Mann Path: core/src/test/java/hudson/util/SecretRewriterTest.groovy http://jenkins-ci.org/commit/jenkins/899f5aa41769c6b6265c95bd63517b35e760f1fa Log: Fix for JENKINS-16942 https://issues.jenkins-ci.org/browse/JENKINS-16942 PrintWriter.println() uses the OS specific line separator. When SecretRewriter rewrites the secret file, the lines are converted to the format of the running OS. The test case does not account for this. Since the test case is coded expecting line endings of \n, this fix converts \r\n to \n before comparing.

          Code changed in jenkins
          User: Andrew Mann
          Path:
          changelog.html
          cli/pom.xml
          core/pom.xml
          core/src/main/grammar/crontab.g
          core/src/main/java/hudson/model/AbstractCIBase.java
          core/src/main/java/hudson/model/Computer.java
          core/src/main/java/hudson/model/Queue.java
          core/src/main/java/hudson/scheduler/BaseParser.java
          core/src/main/java/hudson/scheduler/CronTab.java
          core/src/main/java/hudson/security/ACL.java
          core/src/main/java/hudson/security/HttpSessionContextIntegrationFilter2.java
          core/src/main/java/hudson/security/InvalidatableUserDetails.java
          core/src/main/java/hudson/security/NotSerilizableSecurityContext.java
          core/src/main/java/hudson/slaves/SlaveComputer.java
          core/src/main/java/jenkins/model/Jenkins.java
          core/src/main/java/jenkins/security/NonSerializableSecurityContext.java
          core/src/main/resources/hudson/model/AbstractItem/help-concurrentBuild_zh_TW.html
          core/src/main/resources/hudson/model/Messages_zh_TW.properties
          core/src/main/resources/hudson/scheduler/Messages.properties
          core/src/main/resources/hudson/scheduler/Messages_zh_TW.properties
          core/src/main/resources/hudson/security/Messages_zh_TW.properties
          core/src/main/resources/hudson/triggers/TimerTrigger/help-spec.html
          core/src/main/resources/lib/hudson/executors.jelly
          core/src/test/java/hudson/scheduler/CronTabTest.java
          debian/debian/changelog
          maven-plugin/pom.xml
          plugins/pom.xml
          pom.xml
          test/pom.xml
          test/src/test/groovy/hudson/matrix/MatrixProjectTest.groovy
          test/src/test/java/hudson/model/QueueTest.java
          ui-samples-plugin/pom.xml
          war/pom.xml
          http://jenkins-ci.org/commit/jenkins/00b28d40e283619344c9980440bb367f5d24aba7
          Log:
          Merge branch 'master' into JENKINS-16942

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Andrew Mann Path: changelog.html cli/pom.xml core/pom.xml core/src/main/grammar/crontab.g core/src/main/java/hudson/model/AbstractCIBase.java core/src/main/java/hudson/model/Computer.java core/src/main/java/hudson/model/Queue.java core/src/main/java/hudson/scheduler/BaseParser.java core/src/main/java/hudson/scheduler/CronTab.java core/src/main/java/hudson/security/ACL.java core/src/main/java/hudson/security/HttpSessionContextIntegrationFilter2.java core/src/main/java/hudson/security/InvalidatableUserDetails.java core/src/main/java/hudson/security/NotSerilizableSecurityContext.java core/src/main/java/hudson/slaves/SlaveComputer.java core/src/main/java/jenkins/model/Jenkins.java core/src/main/java/jenkins/security/NonSerializableSecurityContext.java core/src/main/resources/hudson/model/AbstractItem/help-concurrentBuild_zh_TW.html core/src/main/resources/hudson/model/Messages_zh_TW.properties core/src/main/resources/hudson/scheduler/Messages.properties core/src/main/resources/hudson/scheduler/Messages_zh_TW.properties core/src/main/resources/hudson/security/Messages_zh_TW.properties core/src/main/resources/hudson/triggers/TimerTrigger/help-spec.html core/src/main/resources/lib/hudson/executors.jelly core/src/test/java/hudson/scheduler/CronTabTest.java debian/debian/changelog maven-plugin/pom.xml plugins/pom.xml pom.xml test/pom.xml test/src/test/groovy/hudson/matrix/MatrixProjectTest.groovy test/src/test/java/hudson/model/QueueTest.java ui-samples-plugin/pom.xml war/pom.xml http://jenkins-ci.org/commit/jenkins/00b28d40e283619344c9980440bb367f5d24aba7 Log: Merge branch 'master' into JENKINS-16942

          Code changed in jenkins
          User: Jesse Glick
          Path:
          core/src/test/java/hudson/util/SecretRewriterTest.groovy
          http://jenkins-ci.org/commit/jenkins/e0a445c20ef03dbd85a94ab023e328669c99f742
          Log:
          Merge pull request #745 from rhadman/JENKINS-16942

          [FIXED JENKINS-16942] SecretRewriterTest failure on Windows 7

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: core/src/test/java/hudson/util/SecretRewriterTest.groovy http://jenkins-ci.org/commit/jenkins/e0a445c20ef03dbd85a94ab023e328669c99f742 Log: Merge pull request #745 from rhadman/ JENKINS-16942 [FIXED JENKINS-16942] SecretRewriterTest failure on Windows 7

            Unassigned Unassigned
            lymanva limin wang
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: