From f343ab272da117eda15a5c55ada9804cb3ee9a56 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E5=BC=A0=E5=AD=A6=E5=B3=B0?= Date: Fri, 24 Jun 2022 16:03:00 +0800 Subject: [PATCH] =?UTF-8?q?fix=EF=BC=9Acase=20label=20and=20suites=20bug?= =?UTF-8?q?=20fix?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- services/case_service.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/services/case_service.py b/services/case_service.py index dfafbf8..3dcf4a3 100644 --- a/services/case_service.py +++ b/services/case_service.py @@ -57,13 +57,13 @@ async def create_case(data, owner): if 'test_type' in data: data.update(dict({'type': data['test_type']})) if 'labels' in data: - data.update({'labels': ','.join(data['labels'])}) + data.update({'labels': data['labels']}) if not await TestSuite.query_obj_one(TestSuite.name == data['suite_name']): return ERROR_UN_EXISTED_TS, False try: result = await Case().save(data) if 'labels' in data: - await insert_label_map([{'label_name': label, 'case_id': result.id} for label in data['labels']]) + await insert_label_map([{'label_name': label, 'case_id': result.id} for label in data['labels'].split(',')]) return result.to_dict(), True except sqlalchemy.exc.IntegrityError: return ERROR_DUPLICATED_NAME, False @@ -594,7 +594,7 @@ async def rename_test_suite(data, user): ts.name = data['name'] try: await ts.update() - return ts.to_dict(), False + return ts.to_dict(), True except sqlalchemy.exc.IntegrityError: return ERROR_DUPLICATED_NAME, False @@ -644,7 +644,7 @@ async def rename_labels(data, user): label.name = data['name'] try: await label.update() - return label.to_dict(), False + return label.to_dict(), True except sqlalchemy.exc.IntegrityError: return ERROR_DUPLICATED_NAME, False @@ -664,5 +664,5 @@ async def remove_labels(ids, user): diff_list = list(set(ids).difference(set(label_ids))) if len(diff_list) > 0: return f'无权删除标签{",".join(diff_list)}', True - await CaseLabel.batch_delete(TestSuite.id.in_(ids)) + await CaseLabel.batch_delete(CaseLabel.id.in_(ids)) return None, True -- Gitee