From 2639909f90a4e756b6d2a24b49126006aca8b8fd Mon Sep 17 00:00:00 2001
From: Hans-Christoph Steiner <hans@eds.org>
Date: Wed, 25 May 2022 10:09:52 +0200
Subject: [PATCH] add test cases for parsing smartcardoptions config field

---
 tests/common.TestCase | 94 +++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 94 insertions(+)

diff --git a/tests/common.TestCase b/tests/common.TestCase
index e77d7b34..955de457 100755
--- a/tests/common.TestCase
+++ b/tests/common.TestCase
@@ -54,6 +54,8 @@ class CommonTest(unittest.TestCase):
             os.makedirs(self.tmpdir)
         os.chdir(self.basedir)
         fdroidserver.common.config = None
+        fdroidserver.common.options = mock.Mock()
+        fdroidserver.common.options.verbose = False
         self.path = os.environ['PATH']
         self.android_home = os.environ.get('ANDROID_HOME')
 
@@ -2438,6 +2440,98 @@ class CommonTest(unittest.TestCase):
                 self.assertTrue(os.path.exists(f), f + ' was created')
                 self.assertFalse(is_repo_file(f), f + ' not repo file')
 
+    def test_get_smartcardoptions_list(self):
+        os.chdir(self.tmpdir)
+        with open('config.yml', 'w') as fp:
+            fp.write(
+                textwrap.dedent(
+                    """
+                    smartcardoptions:
+                      - -storetype
+                      -  PKCS11
+                      - -providerName
+                      -  SunPKCS11-OpenSC
+                      - -providerClass
+                      -  sun.security.pkcs11.SunPKCS11
+                      - -providerArg
+                      -  opensc-fdroid.cfg
+                    """
+                )
+            )
+        config = fdroidserver.common.read_config()
+        fdroidserver.common.config = config
+        self.assertTrue(isinstance(config['smartcardoptions'], list))
+        self.assertEqual(
+            [
+                '-storetype',
+                'PKCS11',
+                '-providerName',
+                'SunPKCS11-OpenSC',
+                '-providerClass',
+                'sun.security.pkcs11.SunPKCS11',
+                '-providerArg',
+                'opensc-fdroid.cfg',
+            ],
+            config['smartcardoptions'],
+        )
+
+    def test_get_smartcardoptions_spaces(self):
+        os.chdir(self.tmpdir)
+        with open('config.yml', 'w') as fp:
+            fp.write(
+                textwrap.dedent(
+                    """smartcardoptions: |
+                         -storetype      PKCS11
+                         -providerClass  sun.security.pkcs11.SunPKCS11
+                         -providerArg    /etc/pkcs11_java.cfg
+
+                    """
+                )
+            )
+        config = fdroidserver.common.read_config()
+        fdroidserver.common.config = config
+        self.assertTrue(isinstance(config['smartcardoptions'], list))
+        self.assertEqual(
+            [
+                '-storetype',
+                'PKCS11',
+                '-providerClass',
+                'sun.security.pkcs11.SunPKCS11',
+                '-providerArg',
+                '/etc/pkcs11_java.cfg',
+            ],
+            config['smartcardoptions'],
+        )
+
+    def test_get_smartcardoptions_config_py(self):
+        os.chdir(self.tmpdir)
+        with open('config.py', 'w') as fp:
+            fp.write(
+                textwrap.dedent(
+                    """
+                    smartcardoptions = '''
+                    \t-storetype\tPKCS11
+                    \t-providerClass\tsun.security.pkcs11.SunPKCS11
+                    \t-providerArg\t/etc/pkcs11_java.cfg
+
+                     '''
+                    """
+                )
+            )
+        config = fdroidserver.common.read_config()
+        fdroidserver.common.config = config
+        self.assertEqual(
+            [
+                '-storetype',
+                'PKCS11',
+                '-providerClass',
+                'sun.security.pkcs11.SunPKCS11',
+                '-providerArg',
+                '/etc/pkcs11_java.cfg',
+            ],
+            config['smartcardoptions'],
+        )
+
 
 if __name__ == "__main__":
     os.chdir(os.path.dirname(__file__))