Compare commits
10 Commits
article-3-
...
main
| Author | SHA1 | Date | |
|---|---|---|---|
| 879a06929a | |||
| 51d876d1a7 | |||
| 382e31cc37 | |||
| 0eaac42e01 | |||
| 9d42a42d26 | |||
| 68b40c903b | |||
| f68ce9b41a | |||
| 34cba41805 | |||
| b3db1a66e7 | |||
| 626b053df5 |
29
README.md
29
README.md
@@ -1,16 +1,35 @@
|
||||
# sdjpa4-demo
|
||||
|
||||
Runnable companion project for [Spring Data JPA 3 to 4 Migration Guide](https://ankurm.com/spring-data-jpa-3-to-4-migration-guide-2/) on ankurm.com.
|
||||
Runnable companion project for the Spring Data JPA 3 to 4 migration series on ankurm.com:
|
||||
|
||||
Four entities, two repositories, a `@NullMarked` package, seven passing behavior tests, and an A-G demo runner that prints every labelled console output shown in the article. Boots H2 in memory, so there's nothing to install beyond a JDK.
|
||||
- [Spring Data JPA 3 to 4 Migration Guide](https://ankurm.com/spring-data-jpa-3-to-4-migration-guide/) (overview, start here)
|
||||
- [Migration Baseline](https://ankurm.com/spring-data-jpa-3-to-4-migration-guide-2/)
|
||||
- [Query Engine Rewrite: Criteria to JPQL](https://ankurm.com/spring-data-jpa-4-derived-queries-criteria-to-jpql/)
|
||||
- [Advanced Migration: Specifications, AOT, EntityManager](https://ankurm.com/spring-data-jpa-4-specifications-aot-entitymanager/)
|
||||
|
||||
Built and verified on Eclipse Temurin JDK 21.0.11, Spring Boot 4.0.6, Spring Data JPA 4.0.6, Hibernate ORM 7.2, H2 in-memory.
|
||||
Four entities, three repositories, a `@NullMarked` package, 20 passing behavior tests, and an A-L demo runner that prints every labelled console output shown in the articles. Boots H2 in memory, so there's nothing to install beyond a JDK.
|
||||
|
||||
Built and verified on Eclipse Temurin JDK 21, Spring Boot 4.0.6, Spring Data JPA 4.0.6, Hibernate ORM 7.2, H2 in-memory.
|
||||
|
||||
## Run it
|
||||
|
||||
```
|
||||
git clone https://ankurm.com/git.app/asmhatre/sdjpa4-demo.git
|
||||
cd sdjpa4-demo
|
||||
./mvnw spring-boot:run # prints the labelled A-G demonstrations
|
||||
./mvnw test # runs the 7 behavior tests (all green)
|
||||
./mvnw spring-boot:run # prints the labelled A-L demonstrations
|
||||
./mvnw test # runs the 20 behavior tests (all green)
|
||||
```
|
||||
|
||||
## Tags
|
||||
|
||||
- `article-1-baseline`, `article-2-query-engine`, `article-3-advanced` - the exact code each article quotes, frozen at publish time.
|
||||
- `corner-scenarios` (and `main`) - the enriched, current state described below. Some method signatures have moved on from the article-tagged snapshots (e.g. `Book`'s `price` is now an embedded `Money` value object, not a bare `BigDecimal`), so check out the matching article tag if you want the code to line up exactly with what's quoted in a given post.
|
||||
|
||||
## What's covered beyond the three articles (corner-scenario enrichment)
|
||||
|
||||
- **Refined Specification API** (`AuthorSpecifications.java`): `PredicateSpecification` reused across a read and a bulk delete, an explicit `DeleteSpecification` (`CriteriaDelete`-backed), and an `UpdateSpecification` (`CriteriaUpdate`-backed bulk update composed from an `UpdateOperation` + a `where(...)` predicate).
|
||||
- **`JpaSort.unsafe(...)` with a `CASE` expression** - a real `ORDER BY case when country = 'US' then 0 else 1 end` sort combined with a plain derived query.
|
||||
- **`Money`, an `@Embeddable` record value object** on `Book.price`, with derived queries that traverse the embedded path (`findByPriceAmountGreaterThanEqual`, `findByPriceAmount`).
|
||||
- **A genuine corner case, found by actually running it**: a derived-query class-based (record) projection resolves constructor-parameter names against *direct* entity properties only. `BookSummary(String title, BigDecimal amount)` does **not** resolve `amount` against the nested `price.amount` path via a plain `findBy...` derived method - it throws `PropertyReferenceException: No property 'amount' found for type 'Book'`. The fix is an explicit `@Query` constructor expression (`select new ...BookSummary(b.title, b.price.amount) from Book b where ...`), which does work for nested/embedded paths. See `BookRepository.findByPriceAmountLessThanEqual`.
|
||||
|
||||
All of the above is exercised by both the demo runner (`DemoRunner.java`, sections H-L) and dedicated tests in `MigrationBehaviorTests.java`.
|
||||
|
||||
295
mvnw
vendored
Normal file
295
mvnw
vendored
Normal file
@@ -0,0 +1,295 @@
|
||||
#!/bin/sh
|
||||
# ----------------------------------------------------------------------------
|
||||
# Licensed to the Apache Software Foundation (ASF) under one
|
||||
# or more contributor license agreements. See the NOTICE file
|
||||
# distributed with this work for additional information
|
||||
# regarding copyright ownership. The ASF licenses this file
|
||||
# to you 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
|
||||
#
|
||||
# http://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.
|
||||
# ----------------------------------------------------------------------------
|
||||
|
||||
# ----------------------------------------------------------------------------
|
||||
# Apache Maven Wrapper startup batch script, version 3.3.4
|
||||
#
|
||||
# Optional ENV vars
|
||||
# -----------------
|
||||
# JAVA_HOME - location of a JDK home dir, required when download maven via java source
|
||||
# MVNW_REPOURL - repo url base for downloading maven distribution
|
||||
# MVNW_USERNAME/MVNW_PASSWORD - user and password for downloading maven
|
||||
# MVNW_VERBOSE - true: enable verbose log; debug: trace the mvnw script; others: silence the output
|
||||
# ----------------------------------------------------------------------------
|
||||
|
||||
set -euf
|
||||
[ "${MVNW_VERBOSE-}" != debug ] || set -x
|
||||
|
||||
# OS specific support.
|
||||
native_path() { printf %s\\n "$1"; }
|
||||
case "$(uname)" in
|
||||
CYGWIN* | MINGW*)
|
||||
[ -z "${JAVA_HOME-}" ] || JAVA_HOME="$(cygpath --unix "$JAVA_HOME")"
|
||||
native_path() { cygpath --path --windows "$1"; }
|
||||
;;
|
||||
esac
|
||||
|
||||
# set JAVACMD and JAVACCMD
|
||||
set_java_home() {
|
||||
# For Cygwin and MinGW, ensure paths are in Unix format before anything is touched
|
||||
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"
|
||||
JAVACCMD="$JAVA_HOME/jre/sh/javac"
|
||||
else
|
||||
JAVACMD="$JAVA_HOME/bin/java"
|
||||
JAVACCMD="$JAVA_HOME/bin/javac"
|
||||
|
||||
if [ ! -x "$JAVACMD" ] || [ ! -x "$JAVACCMD" ]; then
|
||||
echo "The JAVA_HOME environment variable is not defined correctly, so mvnw cannot run." >&2
|
||||
echo "JAVA_HOME is set to \"$JAVA_HOME\", but \"\$JAVA_HOME/bin/java\" or \"\$JAVA_HOME/bin/javac\" does not exist." >&2
|
||||
return 1
|
||||
fi
|
||||
fi
|
||||
else
|
||||
JAVACMD="$(
|
||||
'set' +e
|
||||
'unset' -f command 2>/dev/null
|
||||
'command' -v java
|
||||
)" || :
|
||||
JAVACCMD="$(
|
||||
'set' +e
|
||||
'unset' -f command 2>/dev/null
|
||||
'command' -v javac
|
||||
)" || :
|
||||
|
||||
if [ ! -x "${JAVACMD-}" ] || [ ! -x "${JAVACCMD-}" ]; then
|
||||
echo "The java/javac command does not exist in PATH nor is JAVA_HOME set, so mvnw cannot run." >&2
|
||||
return 1
|
||||
fi
|
||||
fi
|
||||
}
|
||||
|
||||
# hash string like Java String::hashCode
|
||||
hash_string() {
|
||||
str="${1:-}" h=0
|
||||
while [ -n "$str" ]; do
|
||||
char="${str%"${str#?}"}"
|
||||
h=$(((h * 31 + $(LC_CTYPE=C printf %d "'$char")) % 4294967296))
|
||||
str="${str#?}"
|
||||
done
|
||||
printf %x\\n $h
|
||||
}
|
||||
|
||||
verbose() { :; }
|
||||
[ "${MVNW_VERBOSE-}" != true ] || verbose() { printf %s\\n "${1-}"; }
|
||||
|
||||
die() {
|
||||
printf %s\\n "$1" >&2
|
||||
exit 1
|
||||
}
|
||||
|
||||
trim() {
|
||||
# MWRAPPER-139:
|
||||
# Trims trailing and leading whitespace, carriage returns, tabs, and linefeeds.
|
||||
# Needed for removing poorly interpreted newline sequences when running in more
|
||||
# exotic environments such as mingw bash on Windows.
|
||||
printf "%s" "${1}" | tr -d '[:space:]'
|
||||
}
|
||||
|
||||
scriptDir="$(dirname "$0")"
|
||||
scriptName="$(basename "$0")"
|
||||
|
||||
# parse distributionUrl and optional distributionSha256Sum, requires .mvn/wrapper/maven-wrapper.properties
|
||||
while IFS="=" read -r key value; do
|
||||
case "${key-}" in
|
||||
distributionUrl) distributionUrl=$(trim "${value-}") ;;
|
||||
distributionSha256Sum) distributionSha256Sum=$(trim "${value-}") ;;
|
||||
esac
|
||||
done <"$scriptDir/.mvn/wrapper/maven-wrapper.properties"
|
||||
[ -n "${distributionUrl-}" ] || die "cannot read distributionUrl property in $scriptDir/.mvn/wrapper/maven-wrapper.properties"
|
||||
|
||||
case "${distributionUrl##*/}" in
|
||||
maven-mvnd-*bin.*)
|
||||
MVN_CMD=mvnd.sh _MVNW_REPO_PATTERN=/maven/mvnd/
|
||||
case "${PROCESSOR_ARCHITECTURE-}${PROCESSOR_ARCHITEW6432-}:$(uname -a)" in
|
||||
*AMD64:CYGWIN* | *AMD64:MINGW*) distributionPlatform=windows-amd64 ;;
|
||||
:Darwin*x86_64) distributionPlatform=darwin-amd64 ;;
|
||||
:Darwin*arm64) distributionPlatform=darwin-aarch64 ;;
|
||||
:Linux*x86_64*) distributionPlatform=linux-amd64 ;;
|
||||
*)
|
||||
echo "Cannot detect native platform for mvnd on $(uname)-$(uname -m), use pure java version" >&2
|
||||
distributionPlatform=linux-amd64
|
||||
;;
|
||||
esac
|
||||
distributionUrl="${distributionUrl%-bin.*}-$distributionPlatform.zip"
|
||||
;;
|
||||
maven-mvnd-*) MVN_CMD=mvnd.sh _MVNW_REPO_PATTERN=/maven/mvnd/ ;;
|
||||
*) MVN_CMD="mvn${scriptName#mvnw}" _MVNW_REPO_PATTERN=/org/apache/maven/ ;;
|
||||
esac
|
||||
|
||||
# apply MVNW_REPOURL and calculate MAVEN_HOME
|
||||
# maven home pattern: ~/.m2/wrapper/dists/{apache-maven-<version>,maven-mvnd-<version>-<platform>}/<hash>
|
||||
[ -z "${MVNW_REPOURL-}" ] || distributionUrl="$MVNW_REPOURL$_MVNW_REPO_PATTERN${distributionUrl#*"$_MVNW_REPO_PATTERN"}"
|
||||
distributionUrlName="${distributionUrl##*/}"
|
||||
distributionUrlNameMain="${distributionUrlName%.*}"
|
||||
distributionUrlNameMain="${distributionUrlNameMain%-bin}"
|
||||
MAVEN_USER_HOME="${MAVEN_USER_HOME:-${HOME}/.m2}"
|
||||
MAVEN_HOME="${MAVEN_USER_HOME}/wrapper/dists/${distributionUrlNameMain-}/$(hash_string "$distributionUrl")"
|
||||
|
||||
exec_maven() {
|
||||
unset MVNW_VERBOSE MVNW_USERNAME MVNW_PASSWORD MVNW_REPOURL || :
|
||||
exec "$MAVEN_HOME/bin/$MVN_CMD" "$@" || die "cannot exec $MAVEN_HOME/bin/$MVN_CMD"
|
||||
}
|
||||
|
||||
if [ -d "$MAVEN_HOME" ]; then
|
||||
verbose "found existing MAVEN_HOME at $MAVEN_HOME"
|
||||
exec_maven "$@"
|
||||
fi
|
||||
|
||||
case "${distributionUrl-}" in
|
||||
*?-bin.zip | *?maven-mvnd-?*-?*.zip) ;;
|
||||
*) die "distributionUrl is not valid, must match *-bin.zip or maven-mvnd-*.zip, but found '${distributionUrl-}'" ;;
|
||||
esac
|
||||
|
||||
# prepare tmp dir
|
||||
if TMP_DOWNLOAD_DIR="$(mktemp -d)" && [ -d "$TMP_DOWNLOAD_DIR" ]; then
|
||||
clean() { rm -rf -- "$TMP_DOWNLOAD_DIR"; }
|
||||
trap clean HUP INT TERM EXIT
|
||||
else
|
||||
die "cannot create temp dir"
|
||||
fi
|
||||
|
||||
mkdir -p -- "${MAVEN_HOME%/*}"
|
||||
|
||||
# Download and Install Apache Maven
|
||||
verbose "Couldn't find MAVEN_HOME, downloading and installing it ..."
|
||||
verbose "Downloading from: $distributionUrl"
|
||||
verbose "Downloading to: $TMP_DOWNLOAD_DIR/$distributionUrlName"
|
||||
|
||||
# select .zip or .tar.gz
|
||||
if ! command -v unzip >/dev/null; then
|
||||
distributionUrl="${distributionUrl%.zip}.tar.gz"
|
||||
distributionUrlName="${distributionUrl##*/}"
|
||||
fi
|
||||
|
||||
# verbose opt
|
||||
__MVNW_QUIET_WGET=--quiet __MVNW_QUIET_CURL=--silent __MVNW_QUIET_UNZIP=-q __MVNW_QUIET_TAR=''
|
||||
[ "${MVNW_VERBOSE-}" != true ] || __MVNW_QUIET_WGET='' __MVNW_QUIET_CURL='' __MVNW_QUIET_UNZIP='' __MVNW_QUIET_TAR=v
|
||||
|
||||
# normalize http auth
|
||||
case "${MVNW_PASSWORD:+has-password}" in
|
||||
'') MVNW_USERNAME='' MVNW_PASSWORD='' ;;
|
||||
has-password) [ -n "${MVNW_USERNAME-}" ] || MVNW_USERNAME='' MVNW_PASSWORD='' ;;
|
||||
esac
|
||||
|
||||
if [ -z "${MVNW_USERNAME-}" ] && command -v wget >/dev/null; then
|
||||
verbose "Found wget ... using wget"
|
||||
wget ${__MVNW_QUIET_WGET:+"$__MVNW_QUIET_WGET"} "$distributionUrl" -O "$TMP_DOWNLOAD_DIR/$distributionUrlName" || die "wget: Failed to fetch $distributionUrl"
|
||||
elif [ -z "${MVNW_USERNAME-}" ] && command -v curl >/dev/null; then
|
||||
verbose "Found curl ... using curl"
|
||||
curl ${__MVNW_QUIET_CURL:+"$__MVNW_QUIET_CURL"} -f -L -o "$TMP_DOWNLOAD_DIR/$distributionUrlName" "$distributionUrl" || die "curl: Failed to fetch $distributionUrl"
|
||||
elif set_java_home; then
|
||||
verbose "Falling back to use Java to download"
|
||||
javaSource="$TMP_DOWNLOAD_DIR/Downloader.java"
|
||||
targetZip="$TMP_DOWNLOAD_DIR/$distributionUrlName"
|
||||
cat >"$javaSource" <<-END
|
||||
public class Downloader extends java.net.Authenticator
|
||||
{
|
||||
protected java.net.PasswordAuthentication getPasswordAuthentication()
|
||||
{
|
||||
return new java.net.PasswordAuthentication( System.getenv( "MVNW_USERNAME" ), System.getenv( "MVNW_PASSWORD" ).toCharArray() );
|
||||
}
|
||||
public static void main( String[] args ) throws Exception
|
||||
{
|
||||
setDefault( new Downloader() );
|
||||
java.nio.file.Files.copy( java.net.URI.create( args[0] ).toURL().openStream(), java.nio.file.Paths.get( args[1] ).toAbsolutePath().normalize() );
|
||||
}
|
||||
}
|
||||
END
|
||||
# For Cygwin/MinGW, switch paths to Windows format before running javac and java
|
||||
verbose " - Compiling Downloader.java ..."
|
||||
"$(native_path "$JAVACCMD")" "$(native_path "$javaSource")" || die "Failed to compile Downloader.java"
|
||||
verbose " - Running Downloader.java ..."
|
||||
"$(native_path "$JAVACMD")" -cp "$(native_path "$TMP_DOWNLOAD_DIR")" Downloader "$distributionUrl" "$(native_path "$targetZip")"
|
||||
fi
|
||||
|
||||
# If specified, validate the SHA-256 sum of the Maven distribution zip file
|
||||
if [ -n "${distributionSha256Sum-}" ]; then
|
||||
distributionSha256Result=false
|
||||
if [ "$MVN_CMD" = mvnd.sh ]; then
|
||||
echo "Checksum validation is not supported for maven-mvnd." >&2
|
||||
echo "Please disable validation by removing 'distributionSha256Sum' from your maven-wrapper.properties." >&2
|
||||
exit 1
|
||||
elif command -v sha256sum >/dev/null; then
|
||||
if echo "$distributionSha256Sum $TMP_DOWNLOAD_DIR/$distributionUrlName" | sha256sum -c - >/dev/null 2>&1; then
|
||||
distributionSha256Result=true
|
||||
fi
|
||||
elif command -v shasum >/dev/null; then
|
||||
if echo "$distributionSha256Sum $TMP_DOWNLOAD_DIR/$distributionUrlName" | shasum -a 256 -c >/dev/null 2>&1; then
|
||||
distributionSha256Result=true
|
||||
fi
|
||||
else
|
||||
echo "Checksum validation was requested but neither 'sha256sum' or 'shasum' are available." >&2
|
||||
echo "Please install either command, or disable validation by removing 'distributionSha256Sum' from your maven-wrapper.properties." >&2
|
||||
exit 1
|
||||
fi
|
||||
if [ $distributionSha256Result = false ]; then
|
||||
echo "Error: Failed to validate Maven distribution SHA-256, your Maven distribution might be compromised." >&2
|
||||
echo "If you updated your Maven version, you need to update the specified distributionSha256Sum property." >&2
|
||||
exit 1
|
||||
fi
|
||||
fi
|
||||
|
||||
# unzip and move
|
||||
if command -v unzip >/dev/null; then
|
||||
unzip ${__MVNW_QUIET_UNZIP:+"$__MVNW_QUIET_UNZIP"} "$TMP_DOWNLOAD_DIR/$distributionUrlName" -d "$TMP_DOWNLOAD_DIR" || die "failed to unzip"
|
||||
else
|
||||
tar xzf${__MVNW_QUIET_TAR:+"$__MVNW_QUIET_TAR"} "$TMP_DOWNLOAD_DIR/$distributionUrlName" -C "$TMP_DOWNLOAD_DIR" || die "failed to untar"
|
||||
fi
|
||||
|
||||
# Find the actual extracted directory name (handles snapshots where filename != directory name)
|
||||
actualDistributionDir=""
|
||||
|
||||
# First try the expected directory name (for regular distributions)
|
||||
if [ -d "$TMP_DOWNLOAD_DIR/$distributionUrlNameMain" ]; then
|
||||
if [ -f "$TMP_DOWNLOAD_DIR/$distributionUrlNameMain/bin/$MVN_CMD" ]; then
|
||||
actualDistributionDir="$distributionUrlNameMain"
|
||||
fi
|
||||
fi
|
||||
|
||||
# If not found, search for any directory with the Maven executable (for snapshots)
|
||||
if [ -z "$actualDistributionDir" ]; then
|
||||
# enable globbing to iterate over items
|
||||
set +f
|
||||
for dir in "$TMP_DOWNLOAD_DIR"/*; do
|
||||
if [ -d "$dir" ]; then
|
||||
if [ -f "$dir/bin/$MVN_CMD" ]; then
|
||||
actualDistributionDir="$(basename "$dir")"
|
||||
break
|
||||
fi
|
||||
fi
|
||||
done
|
||||
set -f
|
||||
fi
|
||||
|
||||
if [ -z "$actualDistributionDir" ]; then
|
||||
verbose "Contents of $TMP_DOWNLOAD_DIR:"
|
||||
verbose "$(ls -la "$TMP_DOWNLOAD_DIR")"
|
||||
die "Could not find Maven distribution directory in extracted archive"
|
||||
fi
|
||||
|
||||
verbose "Found extracted Maven distribution directory: $actualDistributionDir"
|
||||
printf %s\\n "$distributionUrl" >"$TMP_DOWNLOAD_DIR/$actualDistributionDir/mvnw.url"
|
||||
mv -- "$TMP_DOWNLOAD_DIR/$actualDistributionDir" "$MAVEN_HOME" || [ -d "$MAVEN_HOME" ] || die "fail to move MAVEN_HOME"
|
||||
|
||||
clean || :
|
||||
exec_maven "$@"
|
||||
189
mvnw.cmd
vendored
Normal file
189
mvnw.cmd
vendored
Normal file
@@ -0,0 +1,189 @@
|
||||
<# : batch portion
|
||||
@REM ----------------------------------------------------------------------------
|
||||
@REM Licensed to the Apache Software Foundation (ASF) under one
|
||||
@REM or more contributor license agreements. See the NOTICE file
|
||||
@REM distributed with this work for additional information
|
||||
@REM regarding copyright ownership. The ASF licenses this file
|
||||
@REM to you under the Apache License, Version 2.0 (the
|
||||
@REM "License"); you may not use this file except in compliance
|
||||
@REM with the License. You may obtain a copy of the License at
|
||||
@REM
|
||||
@REM http://www.apache.org/licenses/LICENSE-2.0
|
||||
@REM
|
||||
@REM Unless required by applicable law or agreed to in writing,
|
||||
@REM software distributed under the License is distributed on an
|
||||
@REM "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
|
||||
@REM KIND, either express or implied. See the License for the
|
||||
@REM specific language governing permissions and limitations
|
||||
@REM under the License.
|
||||
@REM ----------------------------------------------------------------------------
|
||||
|
||||
@REM ----------------------------------------------------------------------------
|
||||
@REM Apache Maven Wrapper startup batch script, version 3.3.4
|
||||
@REM
|
||||
@REM Optional ENV vars
|
||||
@REM MVNW_REPOURL - repo url base for downloading maven distribution
|
||||
@REM MVNW_USERNAME/MVNW_PASSWORD - user and password for downloading maven
|
||||
@REM MVNW_VERBOSE - true: enable verbose log; others: silence the output
|
||||
@REM ----------------------------------------------------------------------------
|
||||
|
||||
@IF "%__MVNW_ARG0_NAME__%"=="" (SET __MVNW_ARG0_NAME__=%~nx0)
|
||||
@SET __MVNW_CMD__=
|
||||
@SET __MVNW_ERROR__=
|
||||
@SET __MVNW_PSMODULEP_SAVE=%PSModulePath%
|
||||
@SET PSModulePath=
|
||||
@FOR /F "usebackq tokens=1* delims==" %%A IN (`powershell -noprofile "& {$scriptDir='%~dp0'; $script='%__MVNW_ARG0_NAME__%'; icm -ScriptBlock ([Scriptblock]::Create((Get-Content -Raw '%~f0'))) -NoNewScope}"`) DO @(
|
||||
IF "%%A"=="MVN_CMD" (set __MVNW_CMD__=%%B) ELSE IF "%%B"=="" (echo %%A) ELSE (echo %%A=%%B)
|
||||
)
|
||||
@SET PSModulePath=%__MVNW_PSMODULEP_SAVE%
|
||||
@SET __MVNW_PSMODULEP_SAVE=
|
||||
@SET __MVNW_ARG0_NAME__=
|
||||
@SET MVNW_USERNAME=
|
||||
@SET MVNW_PASSWORD=
|
||||
@IF NOT "%__MVNW_CMD__%"=="" ("%__MVNW_CMD__%" %*)
|
||||
@echo Cannot start maven from wrapper >&2 && exit /b 1
|
||||
@GOTO :EOF
|
||||
: end batch / begin powershell #>
|
||||
|
||||
$ErrorActionPreference = "Stop"
|
||||
if ($env:MVNW_VERBOSE -eq "true") {
|
||||
$VerbosePreference = "Continue"
|
||||
}
|
||||
|
||||
# calculate distributionUrl, requires .mvn/wrapper/maven-wrapper.properties
|
||||
$distributionUrl = (Get-Content -Raw "$scriptDir/.mvn/wrapper/maven-wrapper.properties" | ConvertFrom-StringData).distributionUrl
|
||||
if (!$distributionUrl) {
|
||||
Write-Error "cannot read distributionUrl property in $scriptDir/.mvn/wrapper/maven-wrapper.properties"
|
||||
}
|
||||
|
||||
switch -wildcard -casesensitive ( $($distributionUrl -replace '^.*/','') ) {
|
||||
"maven-mvnd-*" {
|
||||
$USE_MVND = $true
|
||||
$distributionUrl = $distributionUrl -replace '-bin\.[^.]*$',"-windows-amd64.zip"
|
||||
$MVN_CMD = "mvnd.cmd"
|
||||
break
|
||||
}
|
||||
default {
|
||||
$USE_MVND = $false
|
||||
$MVN_CMD = $script -replace '^mvnw','mvn'
|
||||
break
|
||||
}
|
||||
}
|
||||
|
||||
# apply MVNW_REPOURL and calculate MAVEN_HOME
|
||||
# maven home pattern: ~/.m2/wrapper/dists/{apache-maven-<version>,maven-mvnd-<version>-<platform>}/<hash>
|
||||
if ($env:MVNW_REPOURL) {
|
||||
$MVNW_REPO_PATTERN = if ($USE_MVND -eq $False) { "/org/apache/maven/" } else { "/maven/mvnd/" }
|
||||
$distributionUrl = "$env:MVNW_REPOURL$MVNW_REPO_PATTERN$($distributionUrl -replace "^.*$MVNW_REPO_PATTERN",'')"
|
||||
}
|
||||
$distributionUrlName = $distributionUrl -replace '^.*/',''
|
||||
$distributionUrlNameMain = $distributionUrlName -replace '\.[^.]*$','' -replace '-bin$',''
|
||||
|
||||
$MAVEN_M2_PATH = "$HOME/.m2"
|
||||
if ($env:MAVEN_USER_HOME) {
|
||||
$MAVEN_M2_PATH = "$env:MAVEN_USER_HOME"
|
||||
}
|
||||
|
||||
if (-not (Test-Path -Path $MAVEN_M2_PATH)) {
|
||||
New-Item -Path $MAVEN_M2_PATH -ItemType Directory | Out-Null
|
||||
}
|
||||
|
||||
$MAVEN_WRAPPER_DISTS = $null
|
||||
if ((Get-Item $MAVEN_M2_PATH).Target[0] -eq $null) {
|
||||
$MAVEN_WRAPPER_DISTS = "$MAVEN_M2_PATH/wrapper/dists"
|
||||
} else {
|
||||
$MAVEN_WRAPPER_DISTS = (Get-Item $MAVEN_M2_PATH).Target[0] + "/wrapper/dists"
|
||||
}
|
||||
|
||||
$MAVEN_HOME_PARENT = "$MAVEN_WRAPPER_DISTS/$distributionUrlNameMain"
|
||||
$MAVEN_HOME_NAME = ([System.Security.Cryptography.SHA256]::Create().ComputeHash([byte[]][char[]]$distributionUrl) | ForEach-Object {$_.ToString("x2")}) -join ''
|
||||
$MAVEN_HOME = "$MAVEN_HOME_PARENT/$MAVEN_HOME_NAME"
|
||||
|
||||
if (Test-Path -Path "$MAVEN_HOME" -PathType Container) {
|
||||
Write-Verbose "found existing MAVEN_HOME at $MAVEN_HOME"
|
||||
Write-Output "MVN_CMD=$MAVEN_HOME/bin/$MVN_CMD"
|
||||
exit $?
|
||||
}
|
||||
|
||||
if (! $distributionUrlNameMain -or ($distributionUrlName -eq $distributionUrlNameMain)) {
|
||||
Write-Error "distributionUrl is not valid, must end with *-bin.zip, but found $distributionUrl"
|
||||
}
|
||||
|
||||
# prepare tmp dir
|
||||
$TMP_DOWNLOAD_DIR_HOLDER = New-TemporaryFile
|
||||
$TMP_DOWNLOAD_DIR = New-Item -Itemtype Directory -Path "$TMP_DOWNLOAD_DIR_HOLDER.dir"
|
||||
$TMP_DOWNLOAD_DIR_HOLDER.Delete() | Out-Null
|
||||
trap {
|
||||
if ($TMP_DOWNLOAD_DIR.Exists) {
|
||||
try { Remove-Item $TMP_DOWNLOAD_DIR -Recurse -Force | Out-Null }
|
||||
catch { Write-Warning "Cannot remove $TMP_DOWNLOAD_DIR" }
|
||||
}
|
||||
}
|
||||
|
||||
New-Item -Itemtype Directory -Path "$MAVEN_HOME_PARENT" -Force | Out-Null
|
||||
|
||||
# Download and Install Apache Maven
|
||||
Write-Verbose "Couldn't find MAVEN_HOME, downloading and installing it ..."
|
||||
Write-Verbose "Downloading from: $distributionUrl"
|
||||
Write-Verbose "Downloading to: $TMP_DOWNLOAD_DIR/$distributionUrlName"
|
||||
|
||||
$webclient = New-Object System.Net.WebClient
|
||||
if ($env:MVNW_USERNAME -and $env:MVNW_PASSWORD) {
|
||||
$webclient.Credentials = New-Object System.Net.NetworkCredential($env:MVNW_USERNAME, $env:MVNW_PASSWORD)
|
||||
}
|
||||
[Net.ServicePointManager]::SecurityProtocol = [Net.SecurityProtocolType]::Tls12
|
||||
$webclient.DownloadFile($distributionUrl, "$TMP_DOWNLOAD_DIR/$distributionUrlName") | Out-Null
|
||||
|
||||
# If specified, validate the SHA-256 sum of the Maven distribution zip file
|
||||
$distributionSha256Sum = (Get-Content -Raw "$scriptDir/.mvn/wrapper/maven-wrapper.properties" | ConvertFrom-StringData).distributionSha256Sum
|
||||
if ($distributionSha256Sum) {
|
||||
if ($USE_MVND) {
|
||||
Write-Error "Checksum validation is not supported for maven-mvnd. `nPlease disable validation by removing 'distributionSha256Sum' from your maven-wrapper.properties."
|
||||
}
|
||||
Import-Module $PSHOME\Modules\Microsoft.PowerShell.Utility -Function Get-FileHash
|
||||
if ((Get-FileHash "$TMP_DOWNLOAD_DIR/$distributionUrlName" -Algorithm SHA256).Hash.ToLower() -ne $distributionSha256Sum) {
|
||||
Write-Error "Error: Failed to validate Maven distribution SHA-256, your Maven distribution might be compromised. If you updated your Maven version, you need to update the specified distributionSha256Sum property."
|
||||
}
|
||||
}
|
||||
|
||||
# unzip and move
|
||||
Expand-Archive "$TMP_DOWNLOAD_DIR/$distributionUrlName" -DestinationPath "$TMP_DOWNLOAD_DIR" | Out-Null
|
||||
|
||||
# Find the actual extracted directory name (handles snapshots where filename != directory name)
|
||||
$actualDistributionDir = ""
|
||||
|
||||
# First try the expected directory name (for regular distributions)
|
||||
$expectedPath = Join-Path "$TMP_DOWNLOAD_DIR" "$distributionUrlNameMain"
|
||||
$expectedMvnPath = Join-Path "$expectedPath" "bin/$MVN_CMD"
|
||||
if ((Test-Path -Path $expectedPath -PathType Container) -and (Test-Path -Path $expectedMvnPath -PathType Leaf)) {
|
||||
$actualDistributionDir = $distributionUrlNameMain
|
||||
}
|
||||
|
||||
# If not found, search for any directory with the Maven executable (for snapshots)
|
||||
if (!$actualDistributionDir) {
|
||||
Get-ChildItem -Path "$TMP_DOWNLOAD_DIR" -Directory | ForEach-Object {
|
||||
$testPath = Join-Path $_.FullName "bin/$MVN_CMD"
|
||||
if (Test-Path -Path $testPath -PathType Leaf) {
|
||||
$actualDistributionDir = $_.Name
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
if (!$actualDistributionDir) {
|
||||
Write-Error "Could not find Maven distribution directory in extracted archive"
|
||||
}
|
||||
|
||||
Write-Verbose "Found extracted Maven distribution directory: $actualDistributionDir"
|
||||
Rename-Item -Path "$TMP_DOWNLOAD_DIR/$actualDistributionDir" -NewName $MAVEN_HOME_NAME | Out-Null
|
||||
try {
|
||||
Move-Item -Path "$TMP_DOWNLOAD_DIR/$MAVEN_HOME_NAME" -Destination $MAVEN_HOME_PARENT | Out-Null
|
||||
} catch {
|
||||
if (! (Test-Path -Path "$MAVEN_HOME" -PathType Container)) {
|
||||
Write-Error "fail to move MAVEN_HOME"
|
||||
}
|
||||
} finally {
|
||||
try { Remove-Item $TMP_DOWNLOAD_DIR -Recurse -Force | Out-Null }
|
||||
catch { Write-Warning "Cannot remove $TMP_DOWNLOAD_DIR" }
|
||||
}
|
||||
|
||||
Write-Output "MVN_CMD=$MAVEN_HOME/bin/$MVN_CMD"
|
||||
@@ -2,12 +2,15 @@ package com.ankurm.sdjpa4demo;
|
||||
|
||||
import com.ankurm.sdjpa4demo.domain.Author;
|
||||
import com.ankurm.sdjpa4demo.domain.Book;
|
||||
import com.ankurm.sdjpa4demo.domain.BookSummary;
|
||||
import com.ankurm.sdjpa4demo.domain.Money;
|
||||
import com.ankurm.sdjpa4demo.repo.AuthorRepository;
|
||||
import com.ankurm.sdjpa4demo.repo.BookRepository;
|
||||
import org.springframework.boot.CommandLineRunner;
|
||||
import org.springframework.dao.EmptyResultDataAccessException;
|
||||
import org.springframework.dao.IncorrectResultSizeDataAccessException;
|
||||
import org.springframework.data.domain.Sort;
|
||||
import org.springframework.data.jpa.domain.JpaSort;
|
||||
import org.springframework.context.annotation.Profile;
|
||||
import org.springframework.stereotype.Component;
|
||||
import org.springframework.transaction.annotation.Transactional;
|
||||
@@ -16,6 +19,10 @@ import java.math.BigDecimal;
|
||||
import java.util.List;
|
||||
import java.util.Optional;
|
||||
|
||||
import static com.ankurm.sdjpa4demo.repo.AuthorSpecifications.byCountryDelete;
|
||||
import static com.ankurm.sdjpa4demo.repo.AuthorSpecifications.hasCountry;
|
||||
import static com.ankurm.sdjpa4demo.repo.AuthorSpecifications.relabelCountry;
|
||||
|
||||
@Component
|
||||
@Profile("!test")
|
||||
public class DemoRunner implements CommandLineRunner {
|
||||
@@ -40,10 +47,12 @@ public class DemoRunner implements CommandLineRunner {
|
||||
Author anon = authors.save(new Author("Anonymous", null)); // null country
|
||||
authors.save(new Author("Martin Fowler", "UK"));
|
||||
authors.save(new Author("Kent Beck", "DE")); // bookless: safe to delete in section G
|
||||
Author wall1 = authors.save(new Author("Grady Booch", "US"));
|
||||
Author wall2 = authors.save(new Author("Robert Martin", "US"));
|
||||
|
||||
books.save(new Book("Design Patterns", new BigDecimal("42.00"), gof));
|
||||
books.save(new Book("Effective Java", new BigDecimal("45.00"), bloch));
|
||||
books.save(new Book("Refactoring", new BigDecimal("45.00"), anon)); // duplicate price 45.00
|
||||
books.save(new Book("Design Patterns", new Money(new BigDecimal("42.00"), "USD"), gof));
|
||||
books.save(new Book("Effective Java", new Money(new BigDecimal("45.00"), "USD"), bloch));
|
||||
books.save(new Book("Refactoring", new Money(new BigDecimal("45.00"), "USD"), anon)); // duplicate price 45.00
|
||||
|
||||
// (A) Optional vs non-null vs @Nullable single-result semantics
|
||||
section("A. Null-handling of single-result query methods");
|
||||
@@ -92,13 +101,13 @@ public class DemoRunner implements CommandLineRunner {
|
||||
System.out.println("badgeFor(gof) -> " + authors.badgeFor(gof.getId()));
|
||||
System.out.println("badgeFor(anon) -> " + authors.badgeFor(anon.getId()));
|
||||
|
||||
// (F) Single-result that matches multiple rows -> non-unique edge case
|
||||
section("F. Non-unique single result edge case");
|
||||
// (F) Single-result that matches multiple rows -> non-unique edge case (embedded-path version)
|
||||
section("F. Non-unique single result edge case (embedded value object path)");
|
||||
try {
|
||||
Book b = books.findByPrice(new BigDecimal("45.00"));
|
||||
System.out.println("findByPrice(45.00) -> " + b);
|
||||
Book b = books.findByPriceAmount(new BigDecimal("45.00"));
|
||||
System.out.println("findByPriceAmount(45.00) -> " + b);
|
||||
} catch (IncorrectResultSizeDataAccessException ex) {
|
||||
System.out.println("findByPrice(45.00) matches 2 rows -> throws "
|
||||
System.out.println("findByPriceAmount(45.00) matches 2 rows -> throws "
|
||||
+ ex.getClass().getSimpleName());
|
||||
}
|
||||
|
||||
@@ -107,6 +116,44 @@ public class DemoRunner implements CommandLineRunner {
|
||||
long deleted = authors.deleteByCountry("DE");
|
||||
System.out.println("deleteByCountry('DE') removed -> " + deleted + " row(s)");
|
||||
|
||||
// (H) PredicateSpecification: the SAME instance reused for a read and a bulk delete
|
||||
section("H. PredicateSpecification reused across read and delete");
|
||||
List<Author> usAuthors = authors.findAll(hasCountry("US"));
|
||||
System.out.println("findAll(hasCountry(\"US\")) -> "
|
||||
+ usAuthors.stream().map(Author::getName).toList());
|
||||
|
||||
// (I) DeleteSpecification: explicit CriteriaDelete-typed bulk delete
|
||||
section("I. DeleteSpecification bulk delete");
|
||||
long removedGB = authors.delete(byCountryDelete("GB")); // no GB rows on purpose -> 0
|
||||
System.out.println("delete(byCountryDelete(\"GB\")) removed -> " + removedGB + " row(s)");
|
||||
long removedViaPredicate = authors.delete(hasCountry("US").and((root, cb) ->
|
||||
cb.equal(root.get("name"), "Grady Booch")));
|
||||
System.out.println("delete(hasCountry(\"US\").and(name=\"Grady Booch\")) removed -> "
|
||||
+ removedViaPredicate + " row(s)");
|
||||
|
||||
// (J) UpdateSpecification: CriteriaUpdate-backed bulk update
|
||||
section("J. UpdateSpecification bulk update");
|
||||
long updated = authors.update(relabelCountry("UK", "GB"));
|
||||
System.out.println("update(relabelCountry(\"UK\" -> \"GB\")) updated -> " + updated + " row(s)");
|
||||
System.out.println("findAll(hasCountry(\"GB\")) -> "
|
||||
+ authors.findAll(hasCountry("GB")).stream().map(Author::getName).toList());
|
||||
|
||||
// (K) JpaSort.unsafe(...) with a CASE expression, combined with a plain derived query
|
||||
section("K. JpaSort.unsafe with a CASE expression");
|
||||
Sort usFirst = JpaSort.unsafe(Sort.Direction.ASC,
|
||||
"CASE WHEN country = 'US' THEN 0 ELSE 1 END");
|
||||
List<Author> caseOrdered = authors.findByNameStartingWith("", usFirst);
|
||||
System.out.println("JpaSort.unsafe(CASE WHEN country='US' ...) -> "
|
||||
+ caseOrdered.stream().map(a -> a.getName() + "(" + a.getCountry() + ")").toList());
|
||||
|
||||
// (L) Value-object corner scenarios: embedded-path traversal + record projection
|
||||
section("L. Money value object: embedded-path query and record projection");
|
||||
List<Book> pricey = books.findByPriceAmountGreaterThanEqual(new BigDecimal("45.00"));
|
||||
System.out.println("findByPriceAmountGreaterThanEqual(45.00) -> "
|
||||
+ pricey.stream().map(Book::getTitle).toList());
|
||||
List<BookSummary> summaries = books.findByPriceAmountLessThanEqual(new BigDecimal("42.00"));
|
||||
summaries.forEach(s -> System.out.println("BookSummary projection -> " + s));
|
||||
|
||||
section("DONE");
|
||||
}
|
||||
}
|
||||
|
||||
@@ -1,7 +1,6 @@
|
||||
package com.ankurm.sdjpa4demo.domain;
|
||||
|
||||
import jakarta.persistence.*;
|
||||
import java.math.BigDecimal;
|
||||
|
||||
@Entity
|
||||
public class Book {
|
||||
@@ -12,14 +11,16 @@ public class Book {
|
||||
|
||||
private String title;
|
||||
|
||||
private BigDecimal price;
|
||||
// Money's fields become columns on the book table: amount -> price_amount, currency -> price_currency.
|
||||
@Embedded
|
||||
private Money price;
|
||||
|
||||
@ManyToOne(fetch = FetchType.LAZY)
|
||||
private Author author;
|
||||
|
||||
protected Book() { }
|
||||
|
||||
public Book(String title, BigDecimal price, Author author) {
|
||||
public Book(String title, Money price, Author author) {
|
||||
this.title = title;
|
||||
this.price = price;
|
||||
this.author = author;
|
||||
@@ -27,7 +28,7 @@ public class Book {
|
||||
|
||||
public Long getId() { return id; }
|
||||
public String getTitle() { return title; }
|
||||
public BigDecimal getPrice() { return price; }
|
||||
public Money getPrice() { return price; }
|
||||
public Author getAuthor() { return author; }
|
||||
|
||||
@Override
|
||||
|
||||
@@ -0,0 +1,7 @@
|
||||
package com.ankurm.sdjpa4demo.domain;
|
||||
|
||||
import java.math.BigDecimal;
|
||||
|
||||
// Projection / DTO value object - only what the caller needs. Spring Data populates
|
||||
// this via constructor-matching (a class-based projection), no @Query required.
|
||||
public record BookSummary(String title, BigDecimal amount) {}
|
||||
10
src/main/java/com/ankurm/sdjpa4demo/domain/Money.java
Normal file
10
src/main/java/com/ankurm/sdjpa4demo/domain/Money.java
Normal file
@@ -0,0 +1,10 @@
|
||||
package com.ankurm.sdjpa4demo.domain;
|
||||
|
||||
import jakarta.persistence.Embeddable;
|
||||
|
||||
import java.math.BigDecimal;
|
||||
|
||||
// A value object: no identity, just its two values. Records make ideal @Embeddable
|
||||
// types in Spring Data JPA 4 / Hibernate 7 - immutable, and equals()/hashCode() come free.
|
||||
@Embeddable
|
||||
public record Money(BigDecimal amount, String currency) {}
|
||||
@@ -0,0 +1,36 @@
|
||||
package com.ankurm.sdjpa4demo.repo;
|
||||
|
||||
import com.ankurm.sdjpa4demo.domain.Author;
|
||||
import org.springframework.data.jpa.domain.DeleteSpecification;
|
||||
import org.springframework.data.jpa.domain.PredicateSpecification;
|
||||
import org.springframework.data.jpa.domain.UpdateSpecification;
|
||||
|
||||
/**
|
||||
* Reusable Specification-family helpers for Author, exercising the refined Specification API
|
||||
* introduced in Spring Data JPA 4.0: {@link PredicateSpecification} (context-agnostic, reusable
|
||||
* across select/update/delete), {@link DeleteSpecification} (CriteriaDelete-backed bulk delete),
|
||||
* and {@link UpdateSpecification} (CriteriaUpdate-backed bulk update).
|
||||
*/
|
||||
public final class AuthorSpecifications {
|
||||
|
||||
private AuthorSpecifications() { }
|
||||
|
||||
// PredicateSpecification: a bare Predicate, usable regardless of whether the surrounding
|
||||
// query is a select, update, or delete - this exact instance is reused for both below.
|
||||
public static PredicateSpecification<Author> hasCountry(String country) {
|
||||
return (root, cb) -> cb.equal(root.get("country"), country);
|
||||
}
|
||||
|
||||
// DeleteSpecification: explicit CriteriaDelete-typed specification for a bulk delete.
|
||||
public static DeleteSpecification<Author> byCountryDelete(String country) {
|
||||
return (root, delete, cb) -> cb.equal(root.get("country"), country);
|
||||
}
|
||||
|
||||
// UpdateSpecification: composes an UpdateOperation (the SET clause, over CriteriaUpdate)
|
||||
// with a PredicateSpecification (the WHERE clause) via UpdateSpecification.update(...).where(...).
|
||||
public static UpdateSpecification<Author> relabelCountry(String from, String to) {
|
||||
return UpdateSpecification
|
||||
.<Author>update((root, update, cb) -> update.set(root.get("country"), to))
|
||||
.where(hasCountry(from));
|
||||
}
|
||||
}
|
||||
@@ -1,16 +1,29 @@
|
||||
package com.ankurm.sdjpa4demo.repo;
|
||||
|
||||
import com.ankurm.sdjpa4demo.domain.Book;
|
||||
import com.ankurm.sdjpa4demo.domain.BookSummary;
|
||||
import org.springframework.data.jpa.repository.JpaRepository;
|
||||
import org.springframework.data.jpa.repository.Query;
|
||||
import org.springframework.data.repository.query.Param;
|
||||
|
||||
import java.math.BigDecimal;
|
||||
import java.util.List;
|
||||
|
||||
public interface BookRepository extends JpaRepository<Book, Long> {
|
||||
|
||||
List<Book> findByPriceGreaterThanEqual(BigDecimal price);
|
||||
// Embedded-value-object path traversal: price -> amount. Resolves to: where b.price.amount >= :min
|
||||
List<Book> findByPriceAmountGreaterThanEqual(BigDecimal min);
|
||||
|
||||
// Single-result derived query that can match MORE than one row on bad data:
|
||||
// Single-result derived query over an embedded path that can match MORE than one row on bad data:
|
||||
// demonstrates IncorrectResultSizeDataAccessException (the "non-unique" edge case).
|
||||
Book findByPrice(BigDecimal price);
|
||||
Book findByPriceAmount(BigDecimal price);
|
||||
|
||||
// CORNER CASE: a derived-query class-based (record) projection matches constructor-parameter names
|
||||
// against DIRECT entity properties only. BookSummary's "amount" component does not resolve against
|
||||
// the nested b.price.amount path this way - PropertyReferenceException: No property 'amount' found
|
||||
// for type 'Book' (verified by actually running the derived-method form before falling back to this).
|
||||
// A @Query constructor expression works for nested/embedded paths; the derived-method-only form does not.
|
||||
@Query("select new com.ankurm.sdjpa4demo.domain.BookSummary(b.title, b.price.amount) "
|
||||
+ "from Book b where b.price.amount <= :max")
|
||||
List<BookSummary> findByPriceAmountLessThanEqual(@Param("max") BigDecimal max);
|
||||
}
|
||||
|
||||
@@ -2,6 +2,8 @@ package com.ankurm.sdjpa4demo;
|
||||
|
||||
import com.ankurm.sdjpa4demo.domain.Author;
|
||||
import com.ankurm.sdjpa4demo.domain.Book;
|
||||
import com.ankurm.sdjpa4demo.domain.BookSummary;
|
||||
import com.ankurm.sdjpa4demo.domain.Money;
|
||||
import com.ankurm.sdjpa4demo.repo.AuthorRepository;
|
||||
import com.ankurm.sdjpa4demo.repo.BookRepository;
|
||||
import org.junit.jupiter.api.Test;
|
||||
@@ -11,11 +13,15 @@ import org.springframework.test.context.ActiveProfiles;
|
||||
import org.springframework.dao.EmptyResultDataAccessException;
|
||||
import org.springframework.dao.IncorrectResultSizeDataAccessException;
|
||||
import org.springframework.data.domain.Sort;
|
||||
import org.springframework.data.jpa.domain.JpaSort;
|
||||
import org.springframework.transaction.annotation.Transactional;
|
||||
|
||||
import java.math.BigDecimal;
|
||||
import java.util.List;
|
||||
|
||||
import static com.ankurm.sdjpa4demo.repo.AuthorSpecifications.byCountryDelete;
|
||||
import static com.ankurm.sdjpa4demo.repo.AuthorSpecifications.hasCountry;
|
||||
import static com.ankurm.sdjpa4demo.repo.AuthorSpecifications.relabelCountry;
|
||||
import static org.junit.jupiter.api.Assertions.*;
|
||||
|
||||
@SpringBootTest
|
||||
@@ -29,8 +35,8 @@ class MigrationBehaviorTests {
|
||||
private void seed() {
|
||||
Author a1 = authors.save(new Author("Joshua Bloch", "US"));
|
||||
authors.save(new Author("Anonymous", null));
|
||||
books.save(new Book("Effective Java", new BigDecimal("45.00"), a1));
|
||||
books.save(new Book("Refactoring", new BigDecimal("45.00"), a1));
|
||||
books.save(new Book("Effective Java", new Money(new BigDecimal("45.00"), "USD"), a1));
|
||||
books.save(new Book("Refactoring", new Money(new BigDecimal("45.00"), "USD"), a1));
|
||||
}
|
||||
|
||||
@Test
|
||||
@@ -70,7 +76,7 @@ class MigrationBehaviorTests {
|
||||
void nonUniqueSingleResultThrows() {
|
||||
seed();
|
||||
assertThrows(IncorrectResultSizeDataAccessException.class,
|
||||
() -> books.findByPrice(new BigDecimal("45.00")));
|
||||
() -> books.findByPriceAmount(new BigDecimal("45.00")));
|
||||
}
|
||||
|
||||
@Test
|
||||
@@ -78,4 +84,74 @@ class MigrationBehaviorTests {
|
||||
Author a = authors.save(new Author("Erich Gamma", "CH"));
|
||||
assertEquals("Erich_Gamma:CH", authors.badgeFor(a.getId()));
|
||||
}
|
||||
|
||||
// --- Corner scenarios: refined Specification API (DeleteSpecification / UpdateSpecification / PredicateSpecification) ---
|
||||
|
||||
@Test
|
||||
void predicateSpecificationReusedAcrossReadAndDelete() {
|
||||
authors.save(new Author("Grady Booch", "US"));
|
||||
authors.save(new Author("Robert Martin", "US"));
|
||||
authors.save(new Author("Martin Fowler", "UK"));
|
||||
|
||||
assertEquals(2, authors.findAll(hasCountry("US")).size());
|
||||
|
||||
long removed = authors.delete(hasCountry("US"));
|
||||
assertEquals(2, removed);
|
||||
assertTrue(authors.findAll(hasCountry("US")).isEmpty());
|
||||
}
|
||||
|
||||
@Test
|
||||
void deleteSpecificationBulkDelete() {
|
||||
authors.save(new Author("Kent Beck", "DE"));
|
||||
authors.save(new Author("Erich Gamma", "CH"));
|
||||
|
||||
long removed = authors.delete(byCountryDelete("DE"));
|
||||
|
||||
assertEquals(1, removed);
|
||||
assertTrue(authors.findByCountry("DE") == null);
|
||||
}
|
||||
|
||||
@Test
|
||||
void updateSpecificationBulkUpdate() {
|
||||
authors.save(new Author("Martin Fowler", "UK"));
|
||||
authors.save(new Author("Sam Newman", "UK"));
|
||||
|
||||
long updated = authors.update(relabelCountry("UK", "GB"));
|
||||
|
||||
assertEquals(2, updated);
|
||||
assertEquals(2, authors.findAll(hasCountry("GB")).size());
|
||||
assertTrue(authors.findAll(hasCountry("UK")).isEmpty());
|
||||
}
|
||||
|
||||
@Test
|
||||
void jpaSortUnsafeCaseExpressionOrdersResults() {
|
||||
authors.save(new Author("Alpha Author", "DE"));
|
||||
authors.save(new Author("Beta Author", "US"));
|
||||
|
||||
Sort usFirst = JpaSort.unsafe(Sort.Direction.ASC, "CASE WHEN country = 'US' THEN 0 ELSE 1 END");
|
||||
List<Author> ordered = authors.findByNameStartingWith("", usFirst);
|
||||
|
||||
assertEquals("US", ordered.get(0).getCountry());
|
||||
}
|
||||
|
||||
// --- Corner scenarios: Money embeddable value object ---
|
||||
|
||||
@Test
|
||||
void embeddedValueObjectPathTraversalWorks() {
|
||||
seed();
|
||||
List<Book> found = books.findByPriceAmountGreaterThanEqual(new BigDecimal("45.00"));
|
||||
assertEquals(2, found.size());
|
||||
}
|
||||
|
||||
@Test
|
||||
void recordProjectionPopulatesFromEmbeddedPath() {
|
||||
Author a = authors.save(new Author("Kathy Sierra", "US"));
|
||||
books.save(new Book("Head First Java", new Money(new BigDecimal("39.99"), "USD"), a));
|
||||
|
||||
List<BookSummary> summaries = books.findByPriceAmountLessThanEqual(new BigDecimal("40.00"));
|
||||
|
||||
assertEquals(1, summaries.size());
|
||||
assertEquals("Head First Java", summaries.get(0).title());
|
||||
assertEquals(new BigDecimal("39.99"), summaries.get(0).amount());
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user