From eeb01bf6e849a5b8835bb5cebff521450320558b Mon Sep 17 00:00:00 2001
From: srsh <rabenstein@srsh.io>
Date: Tue, 19 Mar 2024 20:14:48 +0100
Subject: [PATCH] fix linter issues

---
 ki/test/ApiTest.py | 45 +++++++++++++++++++++------------------------
 1 file changed, 21 insertions(+), 24 deletions(-)

diff --git a/ki/test/ApiTest.py b/ki/test/ApiTest.py
index 4a820f1..f82611c 100644
--- a/ki/test/ApiTest.py
+++ b/ki/test/ApiTest.py
@@ -1,6 +1,6 @@
-# SPDX-FileCopyrightText: WTF Kooperative eG <https://wtf-eg.de/>
-#
-# SPDX-License-Identifier: AGPL-3.0-or-later
+#  SPDX-FileCopyrightText: WTF Kooperative eG <https://wtf-eg.de/>
+# 
+#  SPDX-License-Identifier: AGPL-3.0-or-later
 
 from alembic import command
 import json
@@ -10,7 +10,7 @@ from app import app, db, migrate
 from ki.actions import seed
 from ki.models import Skill
 
-from sqlalchemy import select
+# from sqlalchemy import select
 
 
 class ApiTest(unittest.TestCase):
@@ -27,28 +27,25 @@ class ApiTest(unittest.TestCase):
 
         config = migrate.get_config()
        
-        with app.app_context():   
+        with app.app_context():
             command.upgrade(config, "head")
             seed(True)
 
-            #statement = select(Skill).order_by(Skill.id.desc())
-            #print(statement)
-            #skill_obj = db.session.scalars(statement).all()
-            #print(skill_obj)
-            #statement = select(Skill.id)
-            #print(statement)
-          
-            #max_skill = db.session.Skill().order_by(Skill.id.desc()).first()
-            #max_skill = Skill.query.order_by(Skill.id.desc()).first() # TODO: problematic
- 
-            with db.session.no_autoflush: # only works on first test run
-                max_skill = db.session.query(Skill).order_by(Skill.id.desc()).first() # TODO: also problematic,
-  
-            #skills = db.session.execute(db.select(Skill)).scalars()
-            #print(max_skill)
-            #max_skill = db.session.execute(db.select(Skill)
-            ##                   .order_by(Skill.id.desc())
-             #                  ).scalar_one()
+            # statement = select(Skill).order_by(Skill.id.desc())
+            # print(statement)
+            # skill_obj = db.session.scalars(statement).all()
+            # print(skill_obj)
+            # statement = select(Skill.id)
+            # print(statement)
+            # max_skill = db.session.Skill().order_by(Skill.id.desc()).first()
+            # max_skill = Skill.query.order_by(Skill.id.desc()).first() #  TODO: problematic
+            with db.session.no_autoflush: #  only works on first test run
+                max_skill = db.session.query(Skill).order_by(Skill.id.desc()).first() #  TODO: also problematic,
+            # skills = db.session.execute(db.select(Skill)).scalars()
+            # print(max_skill)
+            # max_skill = db.session.execute(db.select(Skill)
+            #                   .order_by(Skill.id.desc())
+            #                   ).scalar_one()
             print(max_skill)
             print("max_skill done")
         self.max_skill_id = max_skill.id
@@ -60,7 +57,7 @@ class ApiTest(unittest.TestCase):
             db.engine.dispose()
 
     def login(self, username, password):
-        #with app.app_context():
+        # with app.app_context():
         login_data = {"username": username, "password": password}
         login_response = self.client.post("/users/login", data=json.dumps(login_data), content_type="application/json")