Script 'mail_helper' called by obssrc
Hello community,

here is the log from the commit of package gnome-maps for openSUSE:Factory 
checked in at 2022-11-01 13:40:48
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/gnome-maps (Old)
 and      /work/SRC/openSUSE:Factory/.gnome-maps.new.2275 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "gnome-maps"

Tue Nov  1 13:40:48 2022 rev:75 rq:1032392 version:43.1

Changes:
--------
--- /work/SRC/openSUSE:Factory/gnome-maps/gnome-maps.changes    2022-10-25 
11:18:52.313985639 +0200
+++ /work/SRC/openSUSE:Factory/.gnome-maps.new.2275/gnome-maps.changes  
2022-11-01 13:40:52.667356513 +0100
@@ -1,0 +2,6 @@
+Fri Oct 28 14:55:12 UTC 2022 - Michael Gorse <mgo...@suse.com>
+
+- Add gnome-maps-icu72.patch: fix comparisons in time tests
+  (glgo#GNOME/gnome-maps!261).
+
+-------------------------------------------------------------------

New:
----
  gnome-maps-icu72.patch

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ gnome-maps.spec ++++++
--- /var/tmp/diff_new_pack.JCXwNi/_old  2022-11-01 13:40:53.903363088 +0100
+++ /var/tmp/diff_new_pack.JCXwNi/_new  2022-11-01 13:40:53.911363131 +0100
@@ -26,6 +26,8 @@
 Source0:        
https://download.gnome.org/sources/gnome-maps/43/%{name}-%{version}.tar.xz
 # PATCH-FIX-UPSTREAM gnome-maps-fix-dependency.patch -- Fix upstream dodo when 
setting dependency for libshumate
 Patch0:         gnome-maps-fix-dependency.patch
+# PATCH-FIX-UPSTREAM gnome-maps-icu72.patch mgo...@suse.com -- fix comparisons 
in time tests.
+Patch1:         gnome-maps-icu72.patch
 
 BuildRequires:  appstream-glib
 BuildRequires:  desktop-file-utils

++++++ gnome-maps-icu72.patch ++++++
>From bec3d2f26de1b3a8c8b7e603f6d6a46c853426fa Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Pablo=20Correa=20G=C3=B3mez?= <ablocor...@hotmail.com>
Date: Fri, 28 Oct 2022 01:05:28 +0200
Subject: [PATCH 1/2] tests/timeTest: Fix tests for time formats with different
 kind of spaces

Time.format does not warranty the character used for spaces, neither
that it will not introduce preceding byte order marks. Solve it by
comparing inclusion of the data and that it finishes with AM or PM.

Closes #506
---
 tests/timeTest.js | 15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)

diff --git a/tests/timeTest.js b/tests/timeTest.js
index 96803f2f..d025e8e8 100644
--- a/tests/timeTest.js
+++ b/tests/timeTest.js
@@ -23,6 +23,11 @@ const JsUnit = imports.jsUnit;
 
 import * as Time from './time.js';
 
+function compare12HourTime(format, hoursMinutes, AMPM) {
+    JsUnit.assertTrue(format.includes(hoursMinutes));
+    JsUnit.assertTrue(format.endsWith(AMPM));
+}
+
 function formatTimeWithTZOffsetTest() {
     // mock to always use 24 hour format
     Time._setIs12HourFunction(() => { return false; });
@@ -35,8 +40,8 @@ function formatTimeWithTZOffsetTest() {
     // mock to always use 12 hour format
     Time._setIs12HourFunction(() => { return true; });
 
-    JsUnit.assertEquals('10:54 PM',
-                        Time.formatTimeWithTZOffset(1607982864000, 3600000));
+    compare12HourTime(Time.formatTimeWithTZOffset(1607982864000, 3600000),
+                      '10:54', 'PM');
 }
 
 function formatTimeFromHoursAndMinsTest() {
@@ -50,9 +55,9 @@ function formatTimeFromHoursAndMinsTest() {
     // mock to always use 12 hour format
     Time._setIs12HourFunction(() => { return true; });
 
-    JsUnit.assertEquals('12:34 PM', Time.formatTimeFromHoursAndMins(12, 34));
-    JsUnit.assertEquals('12:00 AM', Time.formatTimeFromHoursAndMins(24, 0));
-    JsUnit.assertEquals('12:01 PM', Time.formatTimeFromHoursAndMins(12, 1));
+    compare12HourTime(Time.formatTimeFromHoursAndMins(12, 34), '12:34', 'PM');
+    compare12HourTime(Time.formatTimeFromHoursAndMins(24, 0), '12:00', 'AM');
+    compare12HourTime(Time.formatTimeFromHoursAndMins(12, 1), '12:01', 'PM');
 }
 
 formatTimeWithTZOffsetTest();
-- 
2.38.0

Reply via email to