Branch: refs/heads/master
  Home:   https://github.com/jenkinsci/pretested-integration-plugin
  Commit: a48f690943cb79034717e495706c6428c7e710e4
      
https://github.com/jenkinsci/pretested-integration-plugin/commit/a48f690943cb79034717e495706c6428c7e710e4
  Author: Bue Petersen <b...@praqma.net>
  Date:   2015-07-07 (Tue, 07 Jul 2015)

  Changed paths:
    M pom.xml
    M 
src/main/java/org/jenkinsci/plugins/pretestedintegration/scm/git/AccumulatedCommitStrategy.java
    M 
src/main/java/org/jenkinsci/plugins/pretestedintegration/scm/git/GitBridge.java
    R 
src/test/java/org/jenkinsci/plugins/pretestedintegration/integration/scm/git/CommitMessagesWithDoubleQuotes.java
    A 
src/test/java/org/jenkinsci/plugins/pretestedintegration/integration/scm/git/CommitMessagesWithSpecialChars.java
    M 
src/test/java/org/jenkinsci/plugins/pretestedintegration/integration/scm/git/CustomIntegrationBranch.java
    M 
src/test/java/org/jenkinsci/plugins/pretestedintegration/integration/scm/git/StaticGitRepositoryTestBase.java
    M 
src/test/java/org/jenkinsci/plugins/pretestedintegration/integration/scm/git/TestUtilsFactory.java
    A src/test/resources/JENKINS-28640.md
    A src/test/resources/JENKINS-28640.zip
    A src/test/resources/commitMessagesWithDoubleQuotesSingleQuotesMade.md
    A 
src/test/resources/commitMessagesWithDoubleQuotesSingleQuotesMade_linux-repo_description.log
    A src/test/resources/commitMessagesWithDoubleQuotesSingleQuotesMade_linux.sh
    A 
src/test/resources/commitMessagesWithDoubleQuotesSingleQuotesMade_linux.zip
    A 
src/test/resources/commitMessagesWithDoubleQuotesSingleQuotesMade_windows-repo_description.log
    A 
src/test/resources/commitMessagesWithDoubleQuotesSingleQuotesMade_windows.bat
    A 
src/test/resources/commitMessagesWithDoubleQuotesSingleQuotesMade_windows.zip

  Log Message:
  -----------
  JENKINS-28640: Quotation marks in commit msg:

Only related to the accumulated strategy.

Temporary fix for the bug with quotation marks in the commit message
leading to merge failure because of not properly escaped and formatted
command line arguments.

Fix is temporary replacing " (quotation marks) with ' (plings).

The squashed strategy is not affected, as the git command automatically
generated the commit message, where as the plugin creates a look-a-like
squash message, slightly modified, is using accumulated strategy.

Other improvements:

- fixed hard-coded dependency on the maven-hpi-plugin in the pom file,
  new version works now
- cleaned few tests from wrong comments, and not clearly formatted error
  messages if the tests failed
- renamed test suite for commit message prolems related to quotation
  marks to special chars test suite
- shared a useful method from one test suite through test utils class
  that can pretty print console output from the job to use for
verification in the tests.


  Commit: 349570612582f04199683850676cad203b4d8fb9
      
https://github.com/jenkinsci/pretested-integration-plugin/commit/349570612582f04199683850676cad203b4d8fb9
  Author: Bue Petersen <b...@praqma.net>
  Date:   2015-07-07 (Tue, 07 Jul 2015)

  Changed paths:
    R 
src/test/java/org/jenkinsci/plugins/pretestedintegration/integration/scm/git/CommitMessagesWithSpecialChars.java
    A 
src/test/java/org/jenkinsci/plugins/pretestedintegration/integration/scm/git/CommitMessagesWithSpecialCharsIT.java
    R 
src/test/java/org/jenkinsci/plugins/pretestedintegration/integration/scm/git/CustomIntegrationBranch.java
    A 
src/test/java/org/jenkinsci/plugins/pretestedintegration/integration/scm/git/CustomIntegrationBranchIT.java
    R 
src/test/java/org/jenkinsci/plugins/pretestedintegration/integration/scm/git/DoNotAllowMasterBranchAsReadyBranch.java
    A 
src/test/java/org/jenkinsci/plugins/pretestedintegration/integration/scm/git/DoNotAllowMasterBranchAsReadyBranchIT.java
    R 
src/test/java/org/jenkinsci/plugins/pretestedintegration/integration/scm/git/UseAuthorOfLastCommit.java
    A 
src/test/java/org/jenkinsci/plugins/pretestedintegration/integration/scm/git/UseAuthorOfLastCommitIT.java

  Log Message:
  -----------
  JENKINS-28640: Quotation marks - enabled tests:

Not all tests have automatically been executed before.

Fixing the filenames to include all tests.


  Commit: 9b4e052022bb618a4d85c7852a5ac76d5391d89e
      
https://github.com/jenkinsci/pretested-integration-plugin/commit/9b4e052022bb618a4d85c7852a5ac76d5391d89e
  Author: Bue Petersen <b...@praqma.net>
  Date:   2015-07-12 (Sun, 12 Jul 2015)

  Changed paths:
    M README.md
    M 
src/main/java/org/jenkinsci/plugins/pretestedintegration/scm/git/AccumulatedCommitStrategy.java
    M 
src/main/java/org/jenkinsci/plugins/pretestedintegration/scm/git/GetAllCommitsFromBranchCallback.java
    M 
src/main/java/org/jenkinsci/plugins/pretestedintegration/scm/git/SquashCommitStrategy.java
    M 
src/test/java/org/jenkinsci/plugins/pretestedintegration/integration/scm/git/BuildResultValidator.java
    M 
src/test/java/org/jenkinsci/plugins/pretestedintegration/integration/scm/git/CommitMessagesWithSpecialCharsIT.java
    M 
src/test/java/org/jenkinsci/plugins/pretestedintegration/integration/scm/git/CustomIntegrationBranchIT.java
    M 
src/test/java/org/jenkinsci/plugins/pretestedintegration/integration/scm/git/StaticGitRepositoryTestBase.java
    M 
src/test/java/org/jenkinsci/plugins/pretestedintegration/integration/scm/git/TestUtilsFactory.java

  Log Message:
  -----------
  JENKINS-29369 Fix locale dependent accumulated msg

The accumulated commit message is now using english locale when
formatting date and time. The commit message is generated by the plugin
and formatted to look like the squash commit message except for the
header line.

Tests have been improved to include detailed commit message asserts and
extend to include few more tests for squashed also.
Each test now verifies that the formatting of dates are correct for the
tests regarding special charecters and custom integration branch (those
were the newer tests we added so the follow a better design).

Other improvements and refactoring:

- removed un-used code in squash commit strategy
- removed debug print of squash commit strategy to build console
- add debug printing of exception during integration to the build
  console
- fixed minor bug in test utils regarding printing console which is null
- test base class for static git repos extended to print out cleaning
  repos and temp files


Compare: 
https://github.com/jenkinsci/pretested-integration-plugin/compare/cbd24a9b1b9a...9b4e052022bb

-- 
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