Filter daily build versions.

Daily builds available through Chocolatey (e.g. 3.6.8.20200110) were not
filtered and considered as stables.

Versions are now matched against a regular expression which only accepts
versions like `x.y.z`. Then they are compared to the specifier.

Fix #9.

Author: damienflament

CHANGELOG.md

@@ -10,6 +10,9 @@ and this project adheres to [Semantic Versioning].
  - Travis CI shared configuration snippet intented to be sourced in user
    configuration (`dev.yml`).
 
+### Fixed
+ - Filtering of daily build (available through Chocolatey) (#9).
+
 ## [0.1.2] - 2020-01-13
 ### Fixed
  - Recommended way of sourcing the `travis-python.bash` script (#2).

spec/units/versions_spec.sh

@@ -44,13 +44,13 @@ Describe "__latest_matching_version()"
     End
 
     It "filters release candidates"
-        When call __latest_matching_version "3.7" "3.7.6" "3.7.6rc1" "3.7.6rc2"
+        When call __latest_matching_version "3.7" "3.7.6" "3.7.6rc2"
         The output should equal "3.7.6"
     End
 
-    It "filters sources versions"
-        When call __latest_matching_version "pypy-5.7" "pypy-5.7.1" "pypy-5.7.1-src"
-        The output should equal "pypy-5.7.1"
+    It "filters daily builds"
+        When call __latest_matching_version "3.7" "3.7.6" "3.7.6.20200125"
+        The output should equal "3.7.6"
     End
 
     Parameters

travis-python.bash

@@ -80,7 +80,7 @@ __windows_path() {
 
     if [[ $converted == \\* ]]; then
         # If it is an absolute path, convert the first component to a drive letter
-        drive_letter=$(tr '[:lower:]' '[:upper:]' <<<"${converted:1:1}" )
+        drive_letter=$(tr '[:lower:]' '[:upper:]' <<<"${converted:1:1}")
         converted="$drive_letter:${converted:2}"
     fi
 
@@ -96,7 +96,7 @@ __latest_matching_version() {
     # specifier can be a complete version (major.minor.patch) or omit one or
     # more leading components.
     #
-    # Only stable versions are considered ()
+    # Only stable versions are considered.
     #
     local -r specifier=${1:?the specifier must be specified}
     local -r specifier_pattern=${specifier//./"\."}
@@ -116,12 +116,15 @@ __latest_matching_version() {
     shopt -s extglob
 
     for version in "${versions[@]}"; do
-        if [[ $version =~ ^${specifier_pattern}(\.[[:digit:]]+)*$ ]]; then
+        if [[ $version =~ ^[[:digit:]]+(\.[[:digit:]]+){2}$ && \
+            $version =~ ^${specifier_pattern} ]]; then
             found_version="$version"
         fi
     done
 
-    [[ -n $found_version ]] || return
+    if [[ -z $found_version ]]; then
+        return 1
+    fi
 
     echo "$found_version"
 }