Skip to content

Commit

Permalink
Extract message stashing and unstashing into a class. (#924)
Browse files Browse the repository at this point in the history
* Extract message stashing and unstashing into a class, add unit tests.

Signed-off-by: dblock <dblock@amazon.com>

* Added test pipeline for gathering parallel stage messages.

Signed-off-by: dblock <dblock@amazon.com>

* Added a test script that replicates OpenSearch Jenkinsfile but with echo instead of sh.

Signed-off-by: dblock <dblock@amazon.com>

* Update developer guide with Groovy code prerequisites.

Signed-off-by: dblock <dblock@amazon.com>

* Fill out testMessages unit test.

Signed-off-by: dblock <dblock@amazon.com>

* Simplify gradle defaults.

Signed-off-by: dblock <dblock@amazon.com>

* Reorganized tests and added placeholders.

Signed-off-by: dblock <dblock@amazon.com>
  • Loading branch information
dblock committed Nov 11, 2021
1 parent 734081f commit 5ed8176
Show file tree
Hide file tree
Showing 23 changed files with 986 additions and 75 deletions.
16 changes: 16 additions & 0 deletions .github/workflows/groovy-tests.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
name: groovy-tests

on: [push, pull_request]

jobs:
test:
runs-on: ubuntu-latest
steps:
- name: Setup Java
uses: actions/setup-java@v1
with:
java-version: 11
- uses: actions/checkout@v2
- name: Run Tests
run: |
./gradlew test --info
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
name: tests
name: python-tests

on: [push, pull_request]

Expand Down
4 changes: 3 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -12,10 +12,12 @@ coverage.xml

.gradle/
.settings/
/build/
out.txt

/builds/
/dist/
/test-results/

/.vscode/
/.vscode/

8 changes: 8 additions & 0 deletions .groovylintrc.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
{
"extends": "recommended",
"rules": {
"CompileStatic": {
"enabled": false
}
}
}
17 changes: 15 additions & 2 deletions DEVELOPER_GUIDE.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@
- [Pipenv](#pipenv)
- [NVM and Node](#nvm-and-node)
- [Yarn](#yarn)
- [Java](#java)
- [Install Dependencies](#install-dependencies)
- [Run Tests](#run-tests)
- [Build OpenSearch](#build-opensearch)
Expand Down Expand Up @@ -76,6 +77,10 @@ nvm install v10.24.1
npm install -g yarn
```

#### Java

This project recommends Java 11 for Jenkins jobs CI. This means you must have a JDK 11 installed with the environment variable `JAVA_HOME` referencing the path to Java home for your JDK installation, e.g. `JAVA_HOME=/usr/lib/jvm/jdk-11`. Download Java 11 from [here](https://adoptium.net/releases.html?variant=openjdk11).

### Install Dependencies

Install dependencies.
Expand All @@ -90,13 +95,21 @@ Alternatively, run a command inside the virtualenv with pipenv run.

### Run Tests

This project uses [pytest](https://docs.pytest.org/en/6.x/) to ensure code quality. See [tests](tests).
This project uses [pytest](https://docs.pytest.org/en/6.x/) to ensure Python code quality, and [JUnit](https://junit.org/) for Groovy code. See [tests](tests).

```
$ pipenv run pytest
2 passed in 02s
```

```
$ ./gradlew test
> Task :test
BUILD SUCCESSFUL in 7s
3 actionable tasks: 1 executed, 2 up-to-date
```

### Build OpenSearch

Try running `./build.sh`. It should complete and show usage.
Expand All @@ -115,7 +128,7 @@ build.sh: error: the following arguments are required: manifest

### Code Linting

This project uses a [pre-commit hook](https://pre-commit.com/) for linting python code.
This project uses a [pre-commit hook](https://pre-commit.com/) for linting Python code.

```
$ pipenv run pre-commit install
Expand Down
69 changes: 69 additions & 0 deletions build.gradle
Original file line number Diff line number Diff line change
@@ -0,0 +1,69 @@
/*
* SPDX-License-Identifier: Apache-2.0
*
* The OpenSearch Contributors require contributions made to
* this file be licensed under the Apache-2.0 license or a
* compatible open source license.
*/

/*
* This build file was generated by the Gradle 'init' task.
*
* This generated file contains a commented-out sample Java project to get you started.
* For more details take a look at the Java Quickstart chapter in the Gradle
* user guide available at https://docs.gradle.org/3.5/userguide/tutorial_java_projects.html
*/

plugins {
id 'com.mkobit.jenkins.pipelines.shared-library' version '0.10.1'
id 'jacoco'
id 'java'
id 'groovy'
}

repositories {
maven { url 'http://bits.netbeans.org/maven2/' }
maven { url 'https://repo.jenkins-ci.org/releases/' }
jcenter()
maven { url 'https://mvnrepository.com/artifact/' }
mavenLocal()
}

dependencies {
compile group: 'org.assertj', name: 'assertj-core', version: '3.4.1'
compile group: 'com.lesfurets', name:'jenkins-pipeline-unit', version: '1.7'
compile group: 'com.cloudbees', name: 'groovy-cps', version: '1.12'
}

sourceSets {
main {
groovy {
srcDirs = ['src/jenkins']
}
}

test {
groovy {
srcDirs = ['tests/jenkins']
}
}

jobs {
groovy {
srcDirs 'src/jenkins/jobs'
compileClasspath += main.compileClasspath
}

compileClasspath += sourceSets.main.output
runtimeClasspath += sourceSets.main.output
}
}

sharedLibrary {
coreVersion = '2.176.2'
testHarnessVersion = '2.54'
pluginDependencies {
workflowCpsGlobalLibraryPluginVersion = '2.16'
dependency('org.jenkins-ci.plugins', 'pipeline-input-step', '2.8')
}
}
13 changes: 13 additions & 0 deletions gradle.properties
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
#
# SPDX-License-Identifier: Apache-2.0
#
# The OpenSearch Contributors require contributions made to
# this file be licensed under the Apache-2.0 license or a
# compatible open source license.
#
# Modifications Copyright OpenSearch Contributors. See
# GitHub history for details.
#

org.gradle.warning.mode=none
org.gradle.parallel=true
Binary file added gradle/wrapper/gradle-wrapper.jar
Binary file not shown.
18 changes: 18 additions & 0 deletions gradle/wrapper/gradle-wrapper.properties
Original file line number Diff line number Diff line change
@@ -0,0 +1,18 @@

#
# SPDX-License-Identifier: Apache-2.0
#
# The OpenSearch Contributors require contributions made to
# this file be licensed under the Apache-2.0 license or a
# compatible open source license.
#
# Modifications Copyright OpenSearch Contributors. See
# GitHub history for details.
#

distributionBase=GRADLE_USER_HOME
distributionPath=wrapper/dists
distributionUrl=https\://services.gradle.org/distributions/gradle-6.6.1-all.zip
zipStoreBase=GRADLE_USER_HOME
zipStorePath=wrapper/dists
distributionSha256Sum=11657af6356b7587bfb37287b5992e94a9686d5c8a0a1b60b87b9928a2decde5
185 changes: 185 additions & 0 deletions gradlew
Original file line number Diff line number Diff line change
@@ -0,0 +1,185 @@
#!/usr/bin/env sh

#
# Copyright 2015 the original author or authors.
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
# https://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
#

##############################################################################
##
## Gradle start up script for UN*X
##
##############################################################################

# Attempt to set APP_HOME
# Resolve links: $0 may be a link
PRG="$0"
# Need this for relative symlinks.
while [ -h "$PRG" ] ; do
ls=`ls -ld "$PRG"`
link=`expr "$ls" : '.*-> \(.*\)$'`
if expr "$link" : '/.*' > /dev/null; then
PRG="$link"
else
PRG=`dirname "$PRG"`"/$link"
fi
done
SAVED="`pwd`"
cd "`dirname \"$PRG\"`/" >/dev/null
APP_HOME="`pwd -P`"
cd "$SAVED" >/dev/null

APP_NAME="Gradle"
APP_BASE_NAME=`basename "$0"`

# Add default JVM options here. You can also use JAVA_OPTS and GRADLE_OPTS to pass JVM options to this script.
DEFAULT_JVM_OPTS='"-Xmx64m" "-Xms64m"'

# Use the maximum available, or set MAX_FD != -1 to use that value.
MAX_FD="maximum"

warn () {
echo "$*"
}

die () {
echo
echo "$*"
echo
exit 1
}

# OS specific support (must be 'true' or 'false').
cygwin=false
msys=false
darwin=false
nonstop=false
case "`uname`" in
CYGWIN* )
cygwin=true
;;
Darwin* )
darwin=true
;;
MINGW* )
msys=true
;;
NONSTOP* )
nonstop=true
;;
esac

CLASSPATH=$APP_HOME/gradle/wrapper/gradle-wrapper.jar


# Determine the Java command to use to start the JVM.
if [ -n "$JAVA_HOME" ] ; then
if [ -x "$JAVA_HOME/jre/sh/java" ] ; then
# IBM's JDK on AIX uses strange locations for the executables
JAVACMD="$JAVA_HOME/jre/sh/java"
else
JAVACMD="$JAVA_HOME/bin/java"
fi
if [ ! -x "$JAVACMD" ] ; then
die "ERROR: JAVA_HOME is set to an invalid directory: $JAVA_HOME
Please set the JAVA_HOME variable in your environment to match the
location of your Java installation."
fi
else
JAVACMD="java"
which java >/dev/null 2>&1 || die "ERROR: JAVA_HOME is not set and no 'java' command could be found in your PATH.
Please set the JAVA_HOME variable in your environment to match the
location of your Java installation."
fi

# Increase the maximum file descriptors if we can.
if [ "$cygwin" = "false" -a "$darwin" = "false" -a "$nonstop" = "false" ] ; then
MAX_FD_LIMIT=`ulimit -H -n`
if [ $? -eq 0 ] ; then
if [ "$MAX_FD" = "maximum" -o "$MAX_FD" = "max" ] ; then
MAX_FD="$MAX_FD_LIMIT"
fi
ulimit -n $MAX_FD
if [ $? -ne 0 ] ; then
warn "Could not set maximum file descriptor limit: $MAX_FD"
fi
else
warn "Could not query maximum file descriptor limit: $MAX_FD_LIMIT"
fi
fi

# For Darwin, add options to specify how the application appears in the dock
if $darwin; then
GRADLE_OPTS="$GRADLE_OPTS \"-Xdock:name=$APP_NAME\" \"-Xdock:icon=$APP_HOME/media/gradle.icns\""
fi

# For Cygwin or MSYS, switch paths to Windows format before running java
if [ "$cygwin" = "true" -o "$msys" = "true" ] ; then
APP_HOME=`cygpath --path --mixed "$APP_HOME"`
CLASSPATH=`cygpath --path --mixed "$CLASSPATH"`

JAVACMD=`cygpath --unix "$JAVACMD"`

# We build the pattern for arguments to be converted via cygpath
ROOTDIRSRAW=`find -L / -maxdepth 1 -mindepth 1 -type d 2>/dev/null`
SEP=""
for dir in $ROOTDIRSRAW ; do
ROOTDIRS="$ROOTDIRS$SEP$dir"
SEP="|"
done
OURCYGPATTERN="(^($ROOTDIRS))"
# Add a user-defined pattern to the cygpath arguments
if [ "$GRADLE_CYGPATTERN" != "" ] ; then
OURCYGPATTERN="$OURCYGPATTERN|($GRADLE_CYGPATTERN)"
fi
# Now convert the arguments - kludge to limit ourselves to /bin/sh
i=0
for arg in "$@" ; do
CHECK=`echo "$arg"|egrep -c "$OURCYGPATTERN" -`
CHECK2=`echo "$arg"|egrep -c "^-"` ### Determine if an option

if [ $CHECK -ne 0 ] && [ $CHECK2 -eq 0 ] ; then ### Added a condition
eval `echo args$i`=`cygpath --path --ignore --mixed "$arg"`
else
eval `echo args$i`="\"$arg\""
fi
i=`expr $i + 1`
done
case $i in
0) set -- ;;
1) set -- "$args0" ;;
2) set -- "$args0" "$args1" ;;
3) set -- "$args0" "$args1" "$args2" ;;
4) set -- "$args0" "$args1" "$args2" "$args3" ;;
5) set -- "$args0" "$args1" "$args2" "$args3" "$args4" ;;
6) set -- "$args0" "$args1" "$args2" "$args3" "$args4" "$args5" ;;
7) set -- "$args0" "$args1" "$args2" "$args3" "$args4" "$args5" "$args6" ;;
8) set -- "$args0" "$args1" "$args2" "$args3" "$args4" "$args5" "$args6" "$args7" ;;
9) set -- "$args0" "$args1" "$args2" "$args3" "$args4" "$args5" "$args6" "$args7" "$args8" ;;
esac
fi

# Escape application args
save () {
for i do printf %s\\n "$i" | sed "s/'/'\\\\''/g;1s/^/'/;\$s/\$/' \\\\/" ; done
echo " "
}
APP_ARGS=`save "$@"`

# Collect all arguments for the java command, following the shell quoting and substitution rules
eval set -- $DEFAULT_JVM_OPTS $JAVA_OPTS $GRADLE_OPTS "\"-Dorg.gradle.appname=$APP_BASE_NAME\"" -classpath "\"$CLASSPATH\"" org.gradle.wrapper.GradleWrapperMain "$APP_ARGS"

exec "$JAVACMD" "$@"
Loading

0 comments on commit 5ed8176

Please sign in to comment.