Hello community,

here is the log from the commit of package python-mailman-hyperkitty for 
openSUSE:Leap:15.2 checked in at 2020-03-16 12:20:40
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Leap:15.2/python-mailman-hyperkitty (Old)
 and      /work/SRC/openSUSE:Leap:15.2/.python-mailman-hyperkitty.new.3160 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "python-mailman-hyperkitty"

Mon Mar 16 12:20:40 2020 rev:1 rq:766854 version:1.1.0

Changes:
--------
New Changes file:

--- /dev/null   2020-03-10 18:28:06.918142398 +0100
+++ 
/work/SRC/openSUSE:Leap:15.2/.python-mailman-hyperkitty.new.3160/python-mailman-hyperkitty.changes
  2020-03-16 12:20:41.455704919 +0100
@@ -0,0 +1,28 @@
+-------------------------------------------------------------------
+Mon Dec  2 10:47:15 UTC 2019 - pgaj...@suse.com
+
+- call spec-cleaner
+
+-------------------------------------------------------------------
+Wed Nov 27 14:29:11 UTC 2019 - pgaj...@suse.com
+
+- ship mailman-hyperkitty.cfg, in doc for now
+
+-------------------------------------------------------------------
+Thu Nov 14 17:58:56 UTC 2019 - pgaj...@suse.com
+
+- fix testsuite
+- added patches
+  
https://gitlab.com/mailman/mailman-hyperkitty/commit/84e05811fb71aa105fd85fd14399bff813ed744d
+  + python-mailman-hyperkitty-reflect-changes-in-mailman-core.patch
+
+-------------------------------------------------------------------
+Thu Nov 14 12:36:41 UTC 2019 - Tomáš Chvátal <tchva...@suse.com>
+
+- Use nose2 to run the tests as the package can't use pytest
+
+-------------------------------------------------------------------
+Wed Nov 13 15:01:47 UTC 2019 - pgaj...@suse.com
+
+- initial version 1.1.0 [SLE-7686]
+

New:
----
  mailman-hyperkitty-1.1.0.tar.gz
  python-mailman-hyperkitty-reflect-changes-in-mailman-core.patch
  python-mailman-hyperkitty.changes
  python-mailman-hyperkitty.spec

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

Other differences:
------------------
++++++ python-mailman-hyperkitty.spec ++++++
#
# spec file for package python-mailman-hyperkitty
#
# Copyright (c) 2019 SUSE LINUX GmbH, Nuernberg, Germany.
#
# All modifications and additions to the file contributed by third parties
# remain the property of their copyright owners, unless otherwise agreed
# upon. The license for this file, and modifications and additions to the
# file, is the same license as for the pristine package itself (unless the
# license for the pristine package is not an Open Source License, in which
# case the license is the MIT License). An "Open Source License" is a
# license that conforms to the Open Source Definition (Version 1.9)
# published by the Open Source Initiative.

# Please submit bugfixes or comments via https://bugs.opensuse.org/
#


%{?!python_module:%define python_module() python-%{**} python3-%{**}}
%define skip_python2 1
Name:           python-mailman-hyperkitty
Version:        1.1.0
Release:        0
Summary:        Mailman archiver plugin for HyperKitty
License:        GPL-3.0-only
URL:            https://gitlab.com/mailman/mailman-hyperkitty/
Source:         
https://files.pythonhosted.org/packages/source/m/mailman-hyperkitty/mailman-hyperkitty-%{version}.tar.gz
# 
https://gitlab.com/mailman/mailman-hyperkitty/commit/84e05811fb71aa105fd85fd14399bff813ed744d
Patch0:         python-mailman-hyperkitty-reflect-changes-in-mailman-core.patch
BuildRequires:  %{python_module setuptools}
BuildRequires:  fdupes
BuildRequires:  python-rpm-macros
Requires:       python-mailman
Requires:       python-requests
Requires:       python-setuptools
Requires:       python-zope.interface
BuildArch:      noarch
# SECTION test requirements
BuildRequires:  %{python_module mailman}
BuildRequires:  %{python_module mock}
BuildRequires:  %{python_module nose2}
BuildRequires:  %{python_module requests}
BuildRequires:  %{python_module setuptools}
BuildRequires:  %{python_module zope.interface}
# /SECTION
%python_subpackages

%description
Mailman archiver plugin for HyperKitty

%prep
%setup -q -n mailman-hyperkitty-%{version}
%patch0 -p1

%build
%python_build

%install
%python_install
%python_expand %fdupes %{buildroot}%{$python_sitelib}

%check
%python_expand PYTHONPATH=%{buildroot}%{$python_sitelib} $python -m nose2 -v

%files %{python_files}
%doc README.rst mailman-hyperkitty.cfg
%license LICENSE.txt
%{python_sitelib}/*

%changelog
++++++ python-mailman-hyperkitty-reflect-changes-in-mailman-core.patch ++++++
diff --git a/mailman_hyperkitty/tests/test_archiver.py 
b/mailman_hyperkitty/tests/test_archiver.py
index 
3d8896b6bb80eb3fe0b3aaa78c1eb3ac67076c5d..8430e3e3c0a943021707a2a34b4c700eb0a9a053
 100644
--- a/mailman_hyperkitty/tests/test_archiver.py
+++ b/mailman_hyperkitty/tests/test_archiver.py
@@ -102,10 +102,10 @@ class ArchiverTestCase(TestCase):
         self.requests_patcher = patch("mailman_hyperkitty.requests")
         self.requests = self.requests_patcher.start()
         self.fake_response = None
-        self.requests.get.side_effect = \
-            lambda url, *a, **kw: self.fake_response
-        self.requests.post.side_effect = \
-            lambda url, *a, **kw: self.fake_response
+        self.requests.get.side_effect = (
+            lambda url, *a, **kw: self.fake_response)
+        self.requests.post.side_effect = (
+            lambda url, *a, **kw: self.fake_response)
 
     def tearDown(self):
         self.requests_patcher.stop()
@@ -345,19 +345,14 @@ class ArchiverTestCase(TestCase):
             self.archiver._switchboard.queue_directory)), 1)
         self.assertEqual(len(self.archiver._switchboard.files), 1)
 
-    def test_archive_message_unserializable(self):
+    def test_archive_message_unserializable_raises_no_errors(self):
+        self.fake_response = FakeResponse(200, {"url": "dummy"})
         msg = self._get_msg()
         msg["content-type"] = 'text/plain; charset="UTF-8"'
         msg.set_payload(b"this contains encoded unicode \xc3\xa9 \xc3\xa0")
-        # If you try to serialize this message to text, it will cause a:
-        # KeyError: 'content-transfer-encoding'
+        # If you try to serialize this message to text, Mailman should be able
+        # to prevent KeyError by ignoring the bad-characters.
         with patch("mailman_hyperkitty.logger") as logger:
             self.archiver.archive_message(self.mlist, msg)
-        # Check error log
-        self.assertEqual(logger.error.call_count, 1)
-        self.assertTrue(isinstance(
-            logger.error.call_args_list[0][0][2], KeyError))
-        # Check that the message is not stored in the spool.
-        self.assertEqual(len(os.listdir(
-            self.archiver._switchboard.queue_directory)), 0)
-        self.assertEqual(len(self.archiver._switchboard.files), 0)
+        # Check no errors in the log.
+        self.assertEqual(logger.error.call_count, 0)

Reply via email to