Branch: refs/heads/master
  Home:   https://github.com/jenkinsci/jenkins
  Commit: 6cefcf17a8d92b43b738ec6cd5a2cf298cc1156e
      
https://github.com/jenkinsci/jenkins/commit/6cefcf17a8d92b43b738ec6cd5a2cf298cc1156e
  Author: Baptiste Mathus <bmat...@gmail.com>
  Date:   2018-02-02 (Fri, 02 Feb 2018)

  Changed paths:
    M core/src/test/java/hudson/util/AtomicFileWriterTest.java

  Log Message:
  -----------
  [JENKINS-48407] Re-enable AtomicFileWriterTest#checkPermissionsRespectUmask() 
(#3275)

* [JENKINS-48407] Re-enable test

The previous test assumed permissions would always be the same,
when they actually depend on umask settings.

This change creates a file *not* using the temporary API, gets its
permissions then compares it to the ones obtained using
AtomicFileWriter.

Note: we now only check the given permissions, not the "non-given".

* Use assertThat(..., equalTo()) instead of a manual loop

* Remove unused imports

* Use TemporaryFolder instead of manual temporary dir creation


-- 
You received this message because you are subscribed to the Google Groups 
"Jenkins Commits" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to jenkinsci-commits+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to