ki/routes.py

@@ -12,9 +12,6 @@ from ki.handlers import update_profile as update_profile_handler
 from ki.models import ContactType, Language, Skill, Token, User
 from app import app
 
-content_type_svg = "image/svg+xml"
-content_type_png = "image/png"
-
 
 def token_auth(func):
     @wraps(func)
@@ -74,22 +71,22 @@ def handle_icon_request(model, id, path):
     icon_svg_path = icon_base_path + ".svg"
 
     if os.path.exists(icon_svg_path):
-        return send_file(icon_svg_path, mimetype=content_type_svg)
+        return send_file(icon_svg_path, mimetype="image/svg")
 
     icon_png_path = icon_base_path + ".png"
 
     if os.path.exists(icon_png_path):
-        return send_file(icon_png_path, mimetype=content_type_png)
+        return send_file(icon_png_path, mimetype="image/png")
 
     unknown_svg_path = path + "unknown.svg"
 
     if os.path.exists(unknown_svg_path):
-        return send_file(unknown_svg_path, mimetype=content_type_svg)
+        return send_file(unknown_svg_path, mimetype="image/svg")
 
     unknown_png_path = path + "unknown.png"
 
     if os.path.exists(unknown_png_path):
-        return send_file(unknown_png_path, mimetype=content_type_png)
+        return send_file(unknown_png_path, mimetype="image/png")
 
     return make_response({"error": "icon not found"}, 404)
 

ki/test/test_languages_endpoint.py

@@ -1,38 +0,0 @@
-# SPDX-FileCopyrightText: WTF Kooperative eG <https://wtf-eg.de/>
-#
-# SPDX-License-Identifier: AGPL-3.0-or-later
-
-import unittest
-
-from ki.test.ApiTest import ApiTest
-
-
-class TestLanguagesEndpoint(ApiTest):
-    def test_skills_options(self):
-        response = self.client.options("/languages")
-        self.assertEqual(response.status_code, 200)
-        self.assertIn("Access-Control-Allow-Origin", response.headers)
-        self.assertEqual(response.headers["Access-Control-Allow-Origin"], "*")
-
-    def test_search_languages_fr(self):
-        token = self.login("peter", "geheim")["token"]
-
-        response = self.client.get("/languages?search=fr", headers={"Authorization": "Bearer " + token})
-        self.assertEqual(response.status_code, 200)
-        self.assertEqual({"languages": [{
-            "id": "fr",
-            "name": "Französisch",
-            "icon_url": "/languages/fr/icon"
-        }]}, response.json)
-        self.assertIn("Access-Control-Allow-Origin", response.headers)
-        self.assertEqual(response.headers["Access-Control-Allow-Origin"], "*")
-
-    def test_get_fr_icon(self):
-        response = self.client.get("/languages/fr/icon")
-        self.assertEqual(response.status_code, 200)
-        self.assertIn("Content-Type", response.headers)
-        self.assertEqual(response.headers["Content-Type"], "image/png")
-
-
-if __name__ == "main":
-    unittest.main()

ki/test/test_skills_endpoint.py

@@ -38,18 +38,6 @@ class TestSkillsEndpoint(ApiTest):
         self.assertIn("Access-Control-Allow-Origin", response.headers)
         self.assertEqual(response.headers["Access-Control-Allow-Origin"], "*")
 
-    def test_get_php_skill_icon(self):
-        response = self.client.get("/skills/1/icon")
-        self.assertEqual(response.status_code, 200)
-        self.assertIn("Content-Type", response.headers)
-        self.assertEqual(response.headers["Content-Type"], "image/svg+xml; charset=utf-8")
-
-    def test_get_fallback_skill_icon(self):
-        response = self.client.get("/skills/2/icon")
-        self.assertEqual(response.status_code, 200)
-        self.assertIn("Content-Type", response.headers)
-        self.assertEqual(response.headers["Content-Type"], "image/svg+xml; charset=utf-8")
-
 
 if __name__ == "main":
     unittest.main()