Skip to content

Commit

Permalink
Add America/Pangnirtung timezone to the ignore list (#93301)
Browse files Browse the repository at this point in the history
This was updated in tzdata 2022f by jdk 11.0.18, making it different to the joda time version
  • Loading branch information
thecoop committed Jan 27, 2023
1 parent 8d85607 commit d7961a1
Showing 1 changed file with 1 addition and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ public class DateUtilsTests extends ESTestCase {

// A temporary list of zones and JDKs, where Joda and the JDK timezone data are out of sync, until either Joda or the JDK
// are updated, see https://github.com/elastic/elasticsearch/issues/82356
private static final Set<String> IGNORE_IDS = new HashSet<>(Arrays.asList("Pacific/Niue"));
private static final Set<String> IGNORE_IDS = new HashSet<>(Arrays.asList("Pacific/Niue", "America/Pangnirtung"));

private static boolean maybeIgnore(String jodaId) {
if (IGNORE_IDS.contains(jodaId)) {
Expand All @@ -59,7 +59,6 @@ private static boolean maybeIgnore(String jodaId) {
return false;
}

@AwaitsFix(bugUrl = "https://github.com/elastic/elasticsearch/issues/93291")
public void testTimezoneIds() {
assertNull(DateUtils.dateTimeZoneToZoneId(null));
assertNull(DateUtils.zoneIdToDateTimeZone(null));
Expand Down

0 comments on commit d7961a1

Please sign in to comment.