Skip to content

Commit a3f0d1e

Browse files
committed
MLE-25002 Added order-by to JDBC test
Also fixed typo in the SQL query
1 parent 1a3d970 commit a3f0d1e

File tree

1 file changed

+8
-10
lines changed

1 file changed

+8
-10
lines changed

flux-cli/src/test/java/com/marklogic/flux/impl/importdata/ImportJdbcWithAggregatesTest.java

Lines changed: 8 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -72,9 +72,11 @@ void customerWithArrayOfRentalsAndArrayOfPayments() {
7272
r.rental_id, r.inventory_id,
7373
p.payment_id, p.amount
7474
from customer c
75-
inner join public.rental r on c.customer_id = r.customer_id
76-
inner join public.payment p on p.customer_id = p.customer_id
77-
where c.customer_id = 1 and r.rental_id < 1000 and p.payment_id < 17506""";
75+
inner join rental r on c.customer_id = r.customer_id
76+
inner join payment p on c.customer_id = p.customer_id
77+
where c.customer_id = 1 and r.rental_id < 1000 and p.payment_id < 19000
78+
order by p.amount
79+
""";
7880

7981
run(
8082
"import-jdbc",
@@ -95,13 +97,9 @@ void customerWithArrayOfRentalsAndArrayOfPayments() {
9597
assertEquals("Mary", doc.get("first_name").asText());
9698

9799
ArrayNode payments = (ArrayNode) doc.get("payments");
98-
assertEquals(3, payments.size(), "The query should have selected 3 related payments.");
99-
assertEquals(17503, payments.get(0).get("payment_id").asInt());
100-
assertEquals(7.99, payments.get(0).get("amount").asDouble());
101-
assertEquals(17504, payments.get(1).get("payment_id").asInt());
102-
assertEquals(1.99, payments.get(1).get("amount").asDouble());
103-
assertEquals(17505, payments.get(2).get("payment_id").asInt());
104-
assertEquals(7.99, payments.get(2).get("amount").asDouble());
100+
assertEquals(7, payments.size(), "The query should have selected 7 related payments.");
101+
assertEquals(0.99, payments.get(0).get("amount").asDouble());
102+
assertEquals(9.99, payments.get(6).get("amount").asDouble());
105103

106104
ArrayNode rentals = (ArrayNode) doc.get("rentals");
107105
assertEquals(2, rentals.size(), "The query should have selected 2 related rentals.");

0 commit comments

Comments
 (0)