Uploaded image for project: 'Ivy'
  1. Ivy
  2. IVY-435

LatestRevisionStrategy.sort() doesn't sort as specified

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 2.0.0-alpha-1
    • Core
    • None

    Description

      According to the LatestStrategy.sort() javadoc, the sort method should sort the artifacts from latest to oldest.
      However, the LatestRevisionStrategy.sort() method does the opposite!

      The following junit test shows this (add to LatestRevisionStrategyTest.java):

          public void testSort() {
              ArtifactInfo[] revs = toMockAI(new String[] {
                      "0.2a", 
                      "0.2_b", 
                      "0.2rc1", 
                      "0.2-final", 
                      "1.0-dev1", 
                      "1.0-dev2", 
                      "1.0-alpha1", 
                      "1.0-alpha2", 
                      "1.0-beta1", 
                      "1.0-beta2", 
                      "1.0-gamma",
                      "1.0-rc1",
                      "1.0-rc2",
                      "1.0", 
                      "1.0.1", 
                      "2.0" 
                      });
              
              List shuffled = new ArrayList(Arrays.asList(revs)); 
              ArtifactInfo[] shuffledRevs = (ArtifactInfo[]) shuffled.toArray(new ArtifactInfo[revs.length]);
              
              LatestRevisionStrategy latestRevisionStrategy = new LatestRevisionStrategy();
              List sorted = latestRevisionStrategy.sort(shuffledRevs);
          	assertEquals(Arrays.asList(revs), sorted);
          }
      

      The question is: should we change the javadoc or the LatestRevisionStrategy.sort method?

      Attachments

        Activity

          People

            maartenc Maarten Coene
            maartenc Maarten Coene
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: