Merge pull request eugenp#5774 from eugenp/BAEL-2331-v2

lor6 · web-flow · commit 469cd9ec2593 · 2018-11-25T20:21:30.000+02:00
add data order methods
diff --git a/persistence-modules/spring-data-jpa/src/main/java/com/baeldung/passenger/PassengerRepository.java b/persistence-modules/spring-data-jpa/src/main/java/com/baeldung/passenger/PassengerRepository.java
@@ -1,5 +1,6 @@
 package com.baeldung.passenger;
 
+import org.springframework.data.domain.Sort;
 import org.springframework.data.jpa.repository.JpaRepository;
 
 import java.util.List;
@@ -9,4 +10,8 @@ interface PassengerRepository extends JpaRepository<Passenger, Long>, CustomPass
     Passenger findFirstByOrderBySeatNumberAsc();
 
     List<Passenger> findByOrderBySeatNumberAsc();
+    
+    List<Passenger> findByLastNameOrderBySeatNumberAsc(String lastName);
+    
+    List<Passenger> findByLastName(String lastName, Sort sort);
 }
diff --git a/persistence-modules/spring-data-jpa/src/test/java/com/baeldung/passenger/PassengerRepositoryIntegrationTest.java b/persistence-modules/spring-data-jpa/src/test/java/com/baeldung/passenger/PassengerRepositoryIntegrationTest.java
@@ -70,7 +70,7 @@ public void givenSeveralPassengersWhenFindPageSortedByThenThePassengerInTheFirst
     }
 
     @Test
-    public void givenSeveralPassengersWhenOrderedBySeatNumberAscThenThePassengersReturnedInCorrectOrder() {
+    public void givenPassengers_whenOrderedBySeatNumberAsc_thenCorrectOrder() {
         Passenger fred = Passenger.from("Fred", "Bloggs", 22);
         Passenger ricki = Passenger.from("Ricki", "Bobbie", 36);
         Passenger jill = Passenger.from("Jill", "Smith", 50);
@@ -83,7 +83,7 @@ public void givenSeveralPassengersWhenOrderedBySeatNumberAscThenThePassengersRet
     }
 
     @Test
-    public void givenSeveralPassengersWhenFindAllWithSortBySeatNumberAscThenReturnPassengersInCorrectOrder() {
+    public void givenPassengers_whenFindAllWithSortBySeatNumberAsc_thenCorrectOrder() {
         Passenger fred = Passenger.from("Fred", "Bloggs", 22);
         Passenger ricki = Passenger.from("Ricki", "Bobbie", 36);
         Passenger jill = Passenger.from("Jill", "Smith", 50);
@@ -94,4 +94,5 @@ public void givenSeveralPassengersWhenFindAllWithSortBySeatNumberAscThenReturnPa
 
         assertThat(passengers, contains(fred, ricki, jill, siya, eve));
     }
+    
 }

Original file line number	Diff line number	Diff line change
`@@ -70,7 +70,7 @@ public void givenSeveralPassengersWhenFindPageSortedByThenThePassengerInTheFirst`
`70`	`70`	`}`
`71`	`71`
`72`	`72`	`@Test`
`73`		`- public void givenSeveralPassengersWhenOrderedBySeatNumberAscThenThePassengersReturnedInCorrectOrder() {`
	`73`	`+ public void givenPassengers_whenOrderedBySeatNumberAsc_thenCorrectOrder() {`
`74`	`74`	`Passenger fred = Passenger.from("Fred", "Bloggs", 22);`
`75`	`75`	`Passenger ricki = Passenger.from("Ricki", "Bobbie", 36);`
`76`	`76`	`Passenger jill = Passenger.from("Jill", "Smith", 50);`
`@@ -83,7 +83,7 @@ public void givenSeveralPassengersWhenOrderedBySeatNumberAscThenThePassengersRet`
`83`	`83`	`}`
`84`	`84`
`85`	`85`	`@Test`
`86`		`- public void givenSeveralPassengersWhenFindAllWithSortBySeatNumberAscThenReturnPassengersInCorrectOrder() {`
	`86`	`+ public void givenPassengers_whenFindAllWithSortBySeatNumberAsc_thenCorrectOrder() {`
`87`	`87`	`Passenger fred = Passenger.from("Fred", "Bloggs", 22);`
`88`	`88`	`Passenger ricki = Passenger.from("Ricki", "Bobbie", 36);`
`89`	`89`	`Passenger jill = Passenger.from("Jill", "Smith", 50);`
`@@ -94,4 +94,5 @@ public void givenSeveralPassengersWhenFindAllWithSortBySeatNumberAscThenReturnPa`
`94`	`94`
`95`	`95`	`assertThat(passengers, contains(fred, ricki, jill, siya, eve));`
`96`	`96`	`}`
	`97`	`+`
`97`	`98`	`}`