You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@olingo.apache.org by ra...@apache.org on 2019/11/26 06:12:59 UTC
[olingo-odata4] branch OLINGO-1408 updated: [OLINGO-1408] Fix tests
This is an automated email from the ASF dual-hosted git repository.
ramyav pushed a commit to branch OLINGO-1408
in repository https://gitbox.apache.org/repos/asf/olingo-odata4.git
The following commit(s) were added to refs/heads/OLINGO-1408 by this push:
new 636fdde [OLINGO-1408] Fix tests
636fdde is described below
commit 636fdde13dda35d9decae1768d9846f04bb355d9
Author: ramya vasanth <ra...@sap.com>
AuthorDate: Tue Nov 26 11:42:50 2019 +0530
[OLINGO-1408] Fix tests
---
.../olingo/server/tecsvc/data/DataCreator.java | 30 +++++++++++-----------
.../json/ODataJsonDeserializerEntityTest.java | 4 +--
.../serializer/json/ODataJsonSerializerTest.java | 3 ++-
3 files changed, 19 insertions(+), 18 deletions(-)
diff --git a/lib/server-tecsvc/src/main/java/org/apache/olingo/server/tecsvc/data/DataCreator.java b/lib/server-tecsvc/src/main/java/org/apache/olingo/server/tecsvc/data/DataCreator.java
index 38964fe..7007b89 100644
--- a/lib/server-tecsvc/src/main/java/org/apache/olingo/server/tecsvc/data/DataCreator.java
+++ b/lib/server-tecsvc/src/main/java/org/apache/olingo/server/tecsvc/data/DataCreator.java
@@ -22,9 +22,15 @@ import java.math.BigDecimal;
import java.net.URI;
import java.nio.charset.Charset;
import java.sql.Timestamp;
+import java.time.LocalDate;
+import java.time.LocalDateTime;
+import java.time.LocalTime;
+import java.time.ZoneId;
+import java.time.ZonedDateTime;
import java.util.ArrayList;
import java.util.Arrays;
import java.util.Calendar;
+import java.util.GregorianCalendar;
import java.util.HashMap;
import java.util.List;
import java.util.Map;
@@ -2111,11 +2117,8 @@ public class DataCreator {
private static Calendar getDateTime(final int year, final int month, final int day,
final int hour, final int minute, final int second) {
// Date/Time values are serialized with a timezone offset, so we choose a predictable timezone.
- Calendar dateTime = Calendar.getInstance(TimeZone.getTimeZone("GMT"));
- dateTime.clear();
- dateTime.set(year, month - 1, day, hour, minute, second);
- dateTime.set(Calendar.MILLISECOND, 0);
- return dateTime;
+ ZonedDateTime zdt = ZonedDateTime.of(year, month, day, hour, minute, second, 0, ZoneId.of("Z"));
+ return GregorianCalendar.from(zdt);
}
private static int getDuration(final int days, final int hours, final int minutes, final int seconds) {
@@ -2127,20 +2130,17 @@ public class DataCreator {
private static Calendar getDate(final int year, final int month, final int day) {
// Date values are always in the local timezone.
- Calendar date = Calendar.getInstance();
- date.clear();
- date.set(year, month - 1, day, 0, 0, 0);
- date.set(Calendar.MILLISECOND, 0);
- return date;
+ LocalDate date = LocalDate.of(year, month, day);
+ ZonedDateTime zdt = LocalDateTime.of(date, LocalTime.MIDNIGHT).atZone(ZoneId.systemDefault());
+ return GregorianCalendar.from(zdt);
}
private static Calendar getTime(final int hour, final int minute, final int second) {
// Time values are always in the local timezone.
- Calendar time = Calendar.getInstance();
- time.clear();
- time.set(1970, Calendar.JANUARY, 1, hour, minute, second);
- time.set(Calendar.MILLISECOND, 0);
- return time;
+ LocalTime time = LocalTime.of(hour, minute, second);
+ LocalDate EPOCH = LocalDate.ofEpochDay(0l);
+ ZonedDateTime zdt = LocalDateTime.of(EPOCH, time).atZone(ZoneId.systemDefault());
+ return GregorianCalendar.from(zdt);
}
private static Timestamp getTimestamp(final int year, final int month, final int day,
diff --git a/lib/server-test/src/test/java/org/apache/olingo/server/core/deserializer/json/ODataJsonDeserializerEntityTest.java b/lib/server-test/src/test/java/org/apache/olingo/server/core/deserializer/json/ODataJsonDeserializerEntityTest.java
index 39d6d1d..8961a1b 100644
--- a/lib/server-test/src/test/java/org/apache/olingo/server/core/deserializer/json/ODataJsonDeserializerEntityTest.java
+++ b/lib/server-test/src/test/java/org/apache/olingo/server/core/deserializer/json/ODataJsonDeserializerEntityTest.java
@@ -1154,9 +1154,9 @@ public class ODataJsonDeserializerEntityTest extends AbstractODataDeserializerTe
assertEquals(2, properties.size());
assertNotNull(entity.getProperty("PropertyDate").getValue());
- assertEquals(Date.class, entity.getProperty("PropertyDate").getValue().getClass());
+ assertEquals(java.sql.Date.class, entity.getProperty("PropertyDate").getValue().getClass());
assertNotNull(entity.getProperty("PropertyDateTimeOffset").getValue());
- assertEquals(Date.class, entity.getProperty("PropertyDateTimeOffset").getValue().getClass());
+ assertEquals(java.sql.Timestamp.class, entity.getProperty("PropertyDateTimeOffset").getValue().getClass());
}
// ---------------------------------- Negative Tests -----------------------------------------------------------
diff --git a/lib/server-test/src/test/java/org/apache/olingo/server/core/serializer/json/ODataJsonSerializerTest.java b/lib/server-test/src/test/java/org/apache/olingo/server/core/serializer/json/ODataJsonSerializerTest.java
index d196330..5230e60 100644
--- a/lib/server-test/src/test/java/org/apache/olingo/server/core/serializer/json/ODataJsonSerializerTest.java
+++ b/lib/server-test/src/test/java/org/apache/olingo/server/core/serializer/json/ODataJsonSerializerTest.java
@@ -152,7 +152,8 @@ public class ODataJsonSerializerTest {
final EdmEntitySet edmEntitySet = entityContainer.getEntitySet("ESAllPrim");
final Entity entity = data.readAll(edmEntitySet).getEntities().get(0);
entity.getProperty("PropertyDate").setValue(ValueType.PRIMITIVE, LocalDate.parse("2012-12-03"));
- entity.getProperty("PropertyDateTimeOffset").setValue(ValueType.PRIMITIVE, Instant.parse("2012-12-03T07:16:23Z"));
+ entity.getProperty("PropertyDateTimeOffset")
+ .setValue(ValueType.PRIMITIVE, Instant.parse("2012-12-03T07:16:23Z"));
entity.getProperty("PropertyTimeOfDay").setValue(ValueType.PRIMITIVE, LocalTime.parse("03:26:05"));
InputStream result = serializer.entity(metadata, edmEntitySet.getEntityType(), entity,
EntitySerializerOptions.with()