Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

parse for minor release number #36

Open
wants to merge 10 commits into
base: develop
Choose a base branch
from
Open

parse for minor release number #36

Changes from all commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
ee5e69a
parse for minor release number
jdub233 Aug 4, 2017
6527d0f
remove unhelpful grep
jdub233 Aug 4, 2017
12289cf
check minor version
jdub233 Aug 7, 2017
92450cf
fix regex for tests download
jdub233 Aug 7, 2017
14c6565
handle point zero case for tests
jdub233 Aug 7, 2017
9e113d7
fix spacing
jdub233 Aug 7, 2017
ebbbcd4
Merge branch 'develop' into latest-minor-release
desrosj Aug 7, 2017
2261408
add missing local declaration
jdub233 Aug 8, 2017
aa375ac
use latest_version instead of wp_version
jdub233 Aug 8, 2017
44f10ef
Merge remote-tracking branch 'origin/latest-minor-release' into lates…
jdub233 Aug 8, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
27 changes: 25 additions & 2 deletions bin/install-wp-tests.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,15 @@ download() {
fi
}

if [[ $WP_VERSION =~ [0-9]+\.[0-9]+(\.[0-9]+)? ]]; then
WP_TESTS_TAG="tags/$WP_VERSION"
if [[ $WP_VERSION =~ ^[0-9]+\.[0-9]+$ ]]; then
WP_TESTS_TAG="branches/$WP_VERSION"
elif [[ $WP_VERSION =~ [0-9]+\.[0-9]+\.[0-9]+ ]]; then
if [[ $WP_VERSION =~ [0-9]+\.[0-9]+\.[0] ]]; then
Copy link

@atla5 atla5 Oct 26, 2018 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this code is duplicated below in install_wp(). it looks like all this section should be doing is deciding whether the WP_TESTS_TAG should use branches/ or tags/.

I would change this code section to the following and put it right at the top. it'll eliminate the need to over-complicate the rest of the logic in the other two functions

  # shorten `1.2.0` to `1.2` 
  if [[ $WP_VERSION =~ [0-9]+\.[0-9]+\.[0] ]]; then
    $WP_VERSION  = $WP_VERSION%??;
  fi

Copy link

@atla5 atla5 Oct 26, 2018 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

since the WP_TESTS_TAG variable is only used in the install_test_suite() function, you could also move this section into there, as opposed to leaving it in the main body.

# version x.x.0 means the first release of the major version, so strip off the .0 and download version x.x
WP_TESTS_TAG="branches/${WP_VERSION%??}"
else
WP_TESTS_TAG="tags/$WP_VERSION"
fi
elif [[ $WP_VERSION == 'nightly' || $WP_VERSION == 'trunk' ]]; then
WP_TESTS_TAG="trunk"
else
Expand Down Expand Up @@ -57,6 +64,22 @@ install_wp() {
else
if [ $WP_VERSION == 'latest' ]; then
local ARCHIVE_NAME='latest'
elif [[ $WP_VERSION =~ [0-9]+\.[0-9]+ ]]; then
# https serves multiple offers, whereas http serves single.
download https://api.wordpress.org/core/version-check/1.7/ /tmp/wp-latest.json
if [[ $WP_VERSION =~ [0-9]+\.[0-9]+\.[0] ]]; then
# version x.x.0 means the first release of the major version, so strip off the .0 and download version x.x
LATEST_VERSION=${WP_VERSION%??}
else
# otherwise, scan the releases and get the most up to date minor version of the major release
local VERSION_ESCAPED=`echo $WP_VERSION | sed 's/\./\\\\./g'`
LATEST_VERSION=$(grep -o '"version":"'$VERSION_ESCAPED'[^"]*' /tmp/wp-latest.json | sed 's/"version":"//' | head -1)
fi
if [[ -z "$LATEST_VERSION" ]]; then
local ARCHIVE_NAME="wordpress-$WP_VERSION"
else
local ARCHIVE_NAME="wordpress-$LATEST_VERSION"
fi
else
local ARCHIVE_NAME="wordpress-$WP_VERSION"
fi
Expand Down