From c60491db091bf5172ebe1ee8dc51c22842796970 Mon Sep 17 00:00:00 2001
From: Takeshi Kimata <117462761+kimatata@users.noreply.github.com>
Date: Sun, 28 Jul 2024 00:20:24 +0900
Subject: [PATCH] fix: Typescript check (#21)
* fix: typescript check error
* fix: typescript check error
* fix: typescript check error
* fix: typescript check error
* fix: typescript check error
---
frontend/components/ProjectDialog.tsx | 22 +--
frontend/components/UserAvatar.tsx | 2 +-
frontend/config/selection.ts | 21 +--
frontend/messages/en.json | 26 ++--
frontend/messages/ja.json | 26 ++--
frontend/next.config.js | 7 -
frontend/src/app/[locale]/DropdownAccount.tsx | 8 +-
frontend/src/app/[locale]/Header.tsx | 3 +-
.../src/app/[locale]/HeaderNavbarMenu.tsx | 9 +-
frontend/src/app/[locale]/PaneMainTitle.tsx | 3 +-
.../src/app/[locale]/account/AccountPage.tsx | 9 +-
.../src/app/[locale]/account/authControl.ts | 7 +-
.../src/app/[locale]/account/authPage.tsx | 5 +-
frontend/src/app/[locale]/account/page.tsx | 7 +-
.../src/app/[locale]/account/signin/page.tsx | 6 +-
.../src/app/[locale]/account/signup/page.tsx | 6 +-
frontend/src/app/[locale]/admin/AdminPage.tsx | 5 +-
.../src/app/[locale]/admin/UsersTable.tsx | 12 +-
frontend/src/app/[locale]/admin/page.tsx | 6 +-
frontend/src/app/[locale]/layout.tsx | 5 +-
frontend/src/app/[locale]/page.tsx | 6 +-
.../app/[locale]/projects/ProjectsPage.tsx | 10 +-
.../app/[locale]/projects/ProjectsTable.tsx | 23 +--
.../[projectId]/folders/FoldersPane.tsx | 2 +-
.../folders/[folderId]/cases/CasesPane.tsx | 5 +-
.../[folderId]/cases/TestCaseTable.tsx | 23 +--
.../cases/[caseId]/CaseAttachmentsEditor.tsx | 2 +-
.../[folderId]/cases/[caseId]/CaseEditor.tsx | 147 +++++++++++-------
.../folders/[folderId]/cases/page.tsx | 3 +-
.../projects/[projectId]/home/ProjectHome.tsx | 25 ++-
.../home/TestPriorityDonutChart.tsx | 7 +-
.../home/TestProgressColumnChart.tsx | 5 +-
.../[projectId]/home/TestTypesDonutChart.tsx | 6 +-
.../projects/[projectId]/home/aggregate.ts | 39 +++--
.../[locale]/projects/[projectId]/layout.tsx | 3 +-
.../[projectId]/members/AddMemberDialog.tsx | 4 +-
.../[projectId]/members/CandidatesTable.tsx | 18 ++-
.../[projectId]/members/MembersPage.tsx | 32 ++--
.../[projectId]/members/MembersTable.tsx | 28 ++--
.../[projectId]/members/membersControl.ts | 6 +-
.../projects/[projectId]/runs/RunsPage.tsx | 3 +-
.../projects/[projectId]/runs/RunsTable.tsx | 17 +-
.../[projectId]/runs/[runId]/RunEditor.tsx | 14 +-
.../runs/[runId]/RunPregressDonutChart.tsx | 3 +-
.../projects/[projectId]/runs/page.tsx | 3 +-
.../projects/[projectId]/runs/runsControl.ts | 2 +
.../[projectId]/settings/SettingsPage.tsx | 7 +-
.../projects/[projectId]/settings/page.tsx | 27 +++-
frontend/src/app/[locale]/projects/page.tsx | 24 ++-
frontend/src/i18n.ts | 3 +-
frontend/types/base.ts | 21 +++
frontend/types/case.ts | 25 +--
frontend/types/chart.ts | 16 ++
frontend/types/locale.ts | 6 +
frontend/types/project.ts | 24 ++-
frontend/types/run.ts | 3 +
frontend/types/settings.ts | 3 -
frontend/types/toast.ts | 2 +-
frontend/types/user.ts | 7 +-
frontend/utils/TokenProvider.tsx | 8 +-
60 files changed, 496 insertions(+), 311 deletions(-)
create mode 100644 frontend/types/base.ts
create mode 100644 frontend/types/chart.ts
create mode 100644 frontend/types/locale.ts
diff --git a/frontend/components/ProjectDialog.tsx b/frontend/components/ProjectDialog.tsx
index d6e3dd2..15de41c 100644
--- a/frontend/components/ProjectDialog.tsx
+++ b/frontend/components/ProjectDialog.tsx
@@ -12,17 +12,17 @@ import {
ModalBody,
ModalFooter,
} from '@nextui-org/react';
-import { ProjectType, ProjectsMessages } from '@/types/project';
+import { ProjectType, ProjectDialogMessages } from '@/types/project';
type Props = {
isOpen: boolean;
editingProject: ProjectType | null;
onCancel: () => void;
onSubmit: (name: string, detail: string, isPublic: boolean) => void;
- messages: ProjectsMessages;
+ projectDialogMessages: ProjectDialogMessages;
};
-export default function ProjectDialog({ isOpen, editingProject, onCancel, onSubmit, messages }: Props) {
+export default function ProjectDialog({ isOpen, editingProject, onCancel, onSubmit, projectDialogMessages }: Props) {
const [projectName, setProjectName] = useState({
text: editingProject ? editingProject.name : '',
isInvalid: false,
@@ -83,7 +83,7 @@ export default function ProjectDialog({ isOpen, editingProject, onCancel, onSubm
setProjectName({
text: '',
isInvalid: true,
- errorMessage: messages.pleaseEnter,
+ errorMessage: projectDialogMessages.pleaseEnter,
});
return;
@@ -101,11 +101,11 @@ export default function ProjectDialog({ isOpen, editingProject, onCancel, onSubm
}}
>
- {messages.project}
+ {projectDialogMessages.project}
- {messages.public}
+ {projectDialogMessages.public}
- {messages.ifYouMakePublic}
+ {projectDialogMessages.ifYouMakePublic}
diff --git a/frontend/components/UserAvatar.tsx b/frontend/components/UserAvatar.tsx
index 6325b0c..efc2055 100644
--- a/frontend/components/UserAvatar.tsx
+++ b/frontend/components/UserAvatar.tsx
@@ -10,7 +10,7 @@ export default function PublicityChip({ context }: Props) {
return context.isSignedIn() ? (
diff --git a/frontend/config/selection.ts b/frontend/config/selection.ts
index 06c2b64..7005610 100644
--- a/frontend/config/selection.ts
+++ b/frontend/config/selection.ts
@@ -1,14 +1,15 @@
+import { AutomationStatusType, GlobalRoleType, MemberRoleType, TemplateType } from '@/types/base';
import { RunStatusType, TestRunCaseStatusType } from '@/types/status';
import { TestTypeType } from '@/types/testType';
import { PriorityType } from '@/types/priority';
+import { LocaleType } from '@/types/locale';
-const roles = [{ uid: 'administrator' }, { uid: 'user' }];
-
-const memberRoles = [{ uid: 'manager' }, { uid: 'developer' }, { uid: 'reporter' }];
+const roles: GlobalRoleType[] = [{ uid: 'administrator' }, { uid: 'user' }];
+const memberRoles: MemberRoleType[] = [{ uid: 'manager' }, { uid: 'developer' }, { uid: 'reporter' }];
const categoricalPalette = ['#fba91e', '#6ea56c', '#3ac6e1', '#feda2f', '#f15f47', '#244470', '#9c80bb', '#f595a6'];
-const locales = [
+const locales: LocaleType[] = [
{ code: 'en', name: 'English' },
{ code: 'ja', name: '日本語' },
];
@@ -59,14 +60,14 @@ const testTypes: TestTypeType[] = [
{ uid: 'manual', chartColor: categoricalPalette[4] },
];
-const automationStatus = [
- { name: 'Automated', uid: 'automated' },
- { name: 'Automation Not Required', uid: 'automation-not-required' },
- { name: 'Cannot Be Automated', uid: 'cannot-be-automated' },
- { name: 'Obsolete', uid: 'obsolete' },
+const automationStatus: AutomationStatusType[] = [
+ { uid: 'automated' },
+ { uid: 'automation-not-required' },
+ { uid: 'cannot-be-automated' },
+ { uid: 'obsolete' },
];
-const templates = [{ uid: 'text' }, { uid: 'step' }];
+const templates: TemplateType[] = [{ uid: 'text' }, { uid: 'step' }];
export {
roles,
diff --git a/frontend/messages/en.json b/frontend/messages/en.json
index 598c61b..2fdf064 100644
--- a/frontend/messages/en.json
+++ b/frontend/messages/en.json
@@ -35,6 +35,18 @@
"automated": "Automated",
"manual": "Manual"
},
+ "ProjectDialog": {
+ "project": "Project",
+ "project_name": "Project Name",
+ "project_detail": "Project Detail",
+ "public": "Public",
+ "private": "Private",
+ "if_you_make_public": "Making a project public makes it visible to users who are not project members.",
+ "close": "Close",
+ "create": "Create",
+ "update": "Update",
+ "please_enter": "Please enter project name"
+ },
"Index": {
"get_started": "Get Started",
"demo": "Demo",
@@ -114,21 +126,14 @@
},
"Projects": {
"projectList": "Project List",
- "project": "Project",
"new_project": "New Project",
"id": "ID",
"publicity": "Publicity",
+ "public": "Public",
+ "private": "Private",
"name": "Name",
"detail": "Detail",
"last_update": "Last update",
- "project_name": "Project Name",
- "project_detail": "Project Detail",
- "public": "Public",
- "private": "Private",
- "if_you_make_public": "Making a project public makes it visible to users who are not project members.",
- "close": "Close",
- "create": "Create",
- "please_enter": "Please enter project name",
"no_projects_found": "No projects found"
},
"Project": {
@@ -283,11 +288,8 @@
"project_detail": "Project Detail",
"project_owner": "Project Owner",
"edit_project": "Edit Project",
- "project": "Project",
"publicity": "Publicity",
"public": "Public",
- "private": "Private",
- "if_you_make_public": "Making a project public makes it visible to users who are not project members.",
"update": "Update",
"delete_project": "Delete Project",
"delete": "Delete",
diff --git a/frontend/messages/ja.json b/frontend/messages/ja.json
index 9f4567e..bb35092 100644
--- a/frontend/messages/ja.json
+++ b/frontend/messages/ja.json
@@ -36,6 +36,18 @@
"automated": "自動",
"manual": "手動"
},
+ "ProjectDialog": {
+ "project": "プロジェクト",
+ "project_name": "プロジェクト名",
+ "project_detail": "プロジェクト詳細",
+ "public": "パブリック",
+ "private": "プライベート",
+ "if_you_make_public": "プロジェクトをパブリックにすると、プロジェクトメンバーではないユーザーからも見えるようになります。",
+ "close": "閉じる",
+ "create": "作成",
+ "update": "更新",
+ "please_enter": "プロジェクト名を入力してください"
+ },
"Index": {
"get_started": "テスト管理を始める",
"demo": "デモ",
@@ -115,21 +127,14 @@
},
"Projects": {
"projectList": "プロジェクト一覧",
- "project": "プロジェクト",
"new_project": "新規プロジェクト",
"id": "ID",
"publicity": "公開",
+ "public": "パブリック",
+ "private": "プライベート",
"name": "名前",
"detail": "詳細",
"last_update": "最終更新",
- "project_name": "プロジェクト名",
- "project_detail": "プロジェクト詳細",
- "public": "パブリック",
- "private": "プライベート",
- "if_you_make_public": "プロジェクトをパブリックにすると、プロジェクトメンバーではないユーザーからも見えるようになります。",
- "close": "閉じる",
- "create": "作成",
- "please_enter": "プロジェクト名を入力してください",
"no_projects_found": "プロジェクトがありません"
},
"Project": {
@@ -284,12 +289,9 @@
"project_detail": "プロジェクト詳細",
"project_owner": "所有者",
"edit_project": "プロジェクトの編集",
- "project": "プロジェクト",
"publicity": "公開",
"public": "パブリック",
"private": "プライベート",
- "if_you_make_public": "プロジェクトをパブリックにすると、プロジェクトメンバーではないユーザーからも見えるようになります。",
- "update": "更新",
"delete_project": "プロジェクトの削除",
"delete": "削除",
"close": "閉じる",
diff --git a/frontend/next.config.js b/frontend/next.config.js
index 7ab7479..5097335 100644
--- a/frontend/next.config.js
+++ b/frontend/next.config.js
@@ -3,13 +3,6 @@ const withNextIntl = createNextIntlPlugin();
/** @type {import('next').NextConfig} */
const nextConfig = {
- typescript: {
- // !! WARN !!
- // Dangerously allow production builds to successfully complete even if
- // your project has type errors.
- // !! WARN !!
- ignoreBuildErrors: true,
- },
output: 'standalone',
};
diff --git a/frontend/src/app/[locale]/DropdownAccount.tsx b/frontend/src/app/[locale]/DropdownAccount.tsx
index a58b8b4..df4dcb1 100644
--- a/frontend/src/app/[locale]/DropdownAccount.tsx
+++ b/frontend/src/app/[locale]/DropdownAccount.tsx
@@ -18,7 +18,11 @@ export default function DropdownAccount({ messages, locale, onItemPress }: Props
const context = useContext(TokenContext);
const signOut = () => {
- context.setToken(null);
+ context.setToken({
+ access_token: '',
+ expires_at: 0,
+ user: null,
+ });
context.removeTokenFromLocalStorage();
router.push(`/`, { locale: locale });
};
@@ -74,7 +78,7 @@ export default function DropdownAccount({ messages, locale, onItemPress }: Props
startContent={}
endContent={}
>
- {context.isSignedIn() ? context.token.user.username : messages.signIn}
+ {context.isSignedIn() ? context.token!.user!.username : messages.signIn}
{context.isSignedIn() ? (
diff --git a/frontend/src/app/[locale]/Header.tsx b/frontend/src/app/[locale]/Header.tsx
index bfd4cb4..00a7c80 100644
--- a/frontend/src/app/[locale]/Header.tsx
+++ b/frontend/src/app/[locale]/Header.tsx
@@ -1,7 +1,8 @@
import { useTranslations } from 'next-intl';
import HeaderNavbarMenu from './HeaderNavbarMenu';
+import { LocaleCodeType } from '@/types/locale';
-export default function Header(params: { locale: string }) {
+export default function Header(params: { locale: LocaleCodeType }) {
const t = useTranslations('Header');
const messages = {
projects: t('projects'),
diff --git a/frontend/src/app/[locale]/HeaderNavbarMenu.tsx b/frontend/src/app/[locale]/HeaderNavbarMenu.tsx
index b7c1aa7..b6d8a8c 100644
--- a/frontend/src/app/[locale]/HeaderNavbarMenu.tsx
+++ b/frontend/src/app/[locale]/HeaderNavbarMenu.tsx
@@ -23,6 +23,7 @@ import { locales } from '@/config/selection';
import DropdownAccount from './DropdownAccount';
import DropdownLanguage from './DropdownLanguage';
import UserAvatar from '@/components/UserAvatar';
+import { LocaleCodeType } from '@/types/locale';
type NabbarMenuMessages = {
projects: string;
@@ -39,7 +40,7 @@ type NabbarMenuMessages = {
type Props = {
messages: NabbarMenuMessages;
- locale: string;
+ locale: LocaleCodeType;
};
export default function HeaderNavbarMenu({ messages, locale }: Props) {
@@ -202,7 +203,11 @@ export default function HeaderNavbarMenu({ messages, locale }: Props) {
title={messages.signOut}
startContent={}
onPress={() => {
- context.setToken(null);
+ context.setToken({
+ access_token: '',
+ expires_at: 0,
+ user: null,
+ });
context.removeTokenFromLocalStorage();
router.push(`/`, { locale: locale });
setIsMenuOpen(false);
diff --git a/frontend/src/app/[locale]/PaneMainTitle.tsx b/frontend/src/app/[locale]/PaneMainTitle.tsx
index 43036f8..fb90732 100644
--- a/frontend/src/app/[locale]/PaneMainTitle.tsx
+++ b/frontend/src/app/[locale]/PaneMainTitle.tsx
@@ -3,9 +3,10 @@ import { Button, Link as NextUiLink } from '@nextui-org/react';
import { MoveUpRight } from 'lucide-react';
import { Link } from '@/src/navigation';
import { useTranslations } from 'next-intl';
+import { LocaleCodeType } from '@/types/locale';
type Props = {
- locale: string;
+ locale: LocaleCodeType;
};
export default function MainTitle({ locale }: Props) {
diff --git a/frontend/src/app/[locale]/account/AccountPage.tsx b/frontend/src/app/[locale]/account/AccountPage.tsx
index 4094256..84bb19e 100644
--- a/frontend/src/app/[locale]/account/AccountPage.tsx
+++ b/frontend/src/app/[locale]/account/AccountPage.tsx
@@ -7,6 +7,7 @@ import Avatar from 'boring-avatars';
import { fetchMyProjects } from '@/utils/projectsControl';
import { ProjectType } from '@/types/project';
import PublicityChip from '@/components/PublicityChip';
+import { LocaleCodeType } from '@/types/locale';
type AccountPageMessages = {
yourProjects: string;
@@ -17,7 +18,7 @@ type AccountPageMessages = {
type Props = {
messages: AccountPageMessages;
- locale: string;
+ locale: LocaleCodeType;
};
export default function AccountPage({ messages, locale }: Props) {
@@ -50,13 +51,13 @@ export default function AccountPage({ messages, locale }: Props) {
-
{context.token.user.username}
-
{context.token.user.email}
+
{context.token!.user!.username}
+
{context.token!.user!.email}
diff --git a/frontend/src/app/[locale]/account/authControl.ts b/frontend/src/app/[locale]/account/authControl.ts
index 2e98572..530535a 100644
--- a/frontend/src/app/[locale]/account/authControl.ts
+++ b/frontend/src/app/[locale]/account/authControl.ts
@@ -77,9 +77,4 @@ function generateRandomEmail() {
return `${randomString}@example.com`;
}
-function getRandomAvatarPath() {
- const randomIndex = Math.floor(Math.random() * avatars.length);
- return avatars[randomIndex];
-}
-
-export { signUp, signIn, signInAsGuest, getRandomAvatarPath };
+export { signUp, signIn, signInAsGuest };
diff --git a/frontend/src/app/[locale]/account/authPage.tsx b/frontend/src/app/[locale]/account/authPage.tsx
index 1961890..6c8bcd2 100644
--- a/frontend/src/app/[locale]/account/authPage.tsx
+++ b/frontend/src/app/[locale]/account/authPage.tsx
@@ -11,12 +11,13 @@ import { isValidEmail, isValidPassword } from './validate';
import { TokenContext } from '@/utils/TokenProvider';
import { useRouter } from '@/src/navigation';
import Config from '@/config/config';
+import { LocaleCodeType } from '@/types/locale';
const isDemoSite = Config.isDemoSite;
type Props = {
- isSignup: Boolean;
+ isSignup: boolean;
messages: AuthMessages;
- locale: string;
+ locale: LocaleCodeType;
};
export default function AuthPage({ isSignup, messages, locale }: Props) {
diff --git a/frontend/src/app/[locale]/account/page.tsx b/frontend/src/app/[locale]/account/page.tsx
index 0f1f2a0..c046a90 100644
--- a/frontend/src/app/[locale]/account/page.tsx
+++ b/frontend/src/app/[locale]/account/page.tsx
@@ -1,6 +1,9 @@
import { useTranslations } from 'next-intl';
import AccountPage from './AccountPage';
-export default function Page(params: { locale: string }) {
+import { PageType } from '@/types/base';
+import { LocaleCodeType } from '@/types/locale';
+
+export default function Page({ params }: PageType) {
const t = useTranslations('Auth');
const messages = {
yourProjects: t('your_projects'),
@@ -9,5 +12,5 @@ export default function Page(params: { locale: string }) {
noProjectsFound: t('no_projects_found'),
};
- return ;
+ return ;
}
diff --git a/frontend/src/app/[locale]/account/signin/page.tsx b/frontend/src/app/[locale]/account/signin/page.tsx
index 10ba38a..c169a12 100644
--- a/frontend/src/app/[locale]/account/signin/page.tsx
+++ b/frontend/src/app/[locale]/account/signin/page.tsx
@@ -1,7 +1,9 @@
+import { PageType } from '@/types/base';
import AuthPage from '../authPage';
import { useTranslations } from 'next-intl';
+import { LocaleCodeType } from '@/types/locale';
-export default function Page(params: { locale: string }) {
+export default function Page({ params }: PageType) {
const t = useTranslations('Auth');
const messages = {
title: t('signin'),
@@ -24,7 +26,7 @@ export default function Page(params: { locale: string }) {
};
return (
<>
-
+
>
);
}
diff --git a/frontend/src/app/[locale]/account/signup/page.tsx b/frontend/src/app/[locale]/account/signup/page.tsx
index fc9d6cf..495247c 100644
--- a/frontend/src/app/[locale]/account/signup/page.tsx
+++ b/frontend/src/app/[locale]/account/signup/page.tsx
@@ -1,7 +1,9 @@
+import { PageType } from '@/types/base';
import AuthPage from '../authPage';
import { useTranslations } from 'next-intl';
+import { LocaleCodeType } from '@/types/locale';
-export default function Page(params: { locale: string }) {
+export default function Page({ params }: PageType) {
const t = useTranslations('Auth');
const messages = {
title: t('signup'),
@@ -24,7 +26,7 @@ export default function Page(params: { locale: string }) {
};
return (
<>
-
+
>
);
}
diff --git a/frontend/src/app/[locale]/admin/AdminPage.tsx b/frontend/src/app/[locale]/admin/AdminPage.tsx
index 1d57c5b..394ba90 100644
--- a/frontend/src/app/[locale]/admin/AdminPage.tsx
+++ b/frontend/src/app/[locale]/admin/AdminPage.tsx
@@ -5,11 +5,12 @@ import { UserType, AdminMessages } from '@/types/user';
import { TokenContext } from '@/utils/TokenProvider';
import UsersTable from './UsersTable';
import Config from '@/config/config';
+import { LocaleCodeType } from '@/types/locale';
const apiServer = Config.apiServer;
type Props = {
messages: AdminMessages;
- locale: string;
+ locale: LocaleCodeType;
};
async function fetchUsers(jwt: string) {
@@ -62,7 +63,7 @@ export default function AdminPage({ messages, locale }: Props) {
{messages.userManagement}
-
+
);
}
diff --git a/frontend/src/app/[locale]/admin/UsersTable.tsx b/frontend/src/app/[locale]/admin/UsersTable.tsx
index 997e7a2..7eea004 100644
--- a/frontend/src/app/[locale]/admin/UsersTable.tsx
+++ b/frontend/src/app/[locale]/admin/UsersTable.tsx
@@ -7,10 +7,9 @@ import Avatar from 'boring-avatars';
type Props = {
users: UserType[];
messages: AdminMessages;
- locale: string;
};
-export default function UsersTable({ users, messages, locale }: Props) {
+export default function UsersTable({ users, messages }: Props) {
const headerColumns = [
{ name: messages.avatar, uid: 'avatar', sortable: false },
{ name: messages.id, uid: 'id', sortable: true },
@@ -34,7 +33,7 @@ export default function UsersTable({ users, messages, locale }: Props) {
});
}, [sortDescriptor, users]);
- const renderCell = useCallback((user: UserType, columnKey: Key) => {
+ const renderCell = useCallback((user: UserType, columnKey: string) => {
const cellValue = user[columnKey as keyof UserType];
switch (columnKey) {
@@ -54,7 +53,8 @@ export default function UsersTable({ users, messages, locale }: Props) {
case 'name':
return cellValue;
case 'role':
- return {messages[roles[cellValue].uid]};
+ return {messages[roles[cellValue as number].uid]};
+
default:
return cellValue;
}
@@ -101,7 +101,9 @@ export default function UsersTable({ users, messages, locale }: Props) {
{(item) => (
- {(columnKey) => {renderCell(item, columnKey)}}
+
+ {(columnKey) => {renderCell(item, columnKey as string)}}
+
)}
diff --git a/frontend/src/app/[locale]/admin/page.tsx b/frontend/src/app/[locale]/admin/page.tsx
index 0197bfa..205b6fe 100644
--- a/frontend/src/app/[locale]/admin/page.tsx
+++ b/frontend/src/app/[locale]/admin/page.tsx
@@ -1,7 +1,9 @@
import { useTranslations } from 'next-intl';
import AdminPage from './AdminPage';
+import { PageType } from '@/types/base';
+import { LocaleCodeType } from '@/types/locale';
-export default function Page(params: { locale: string }) {
+export default function Page({ params }: PageType) {
const t = useTranslations('Admin');
const messages = {
admin: t('admin'),
@@ -18,7 +20,7 @@ export default function Page(params: { locale: string }) {
return (
);
}
diff --git a/frontend/src/app/[locale]/layout.tsx b/frontend/src/app/[locale]/layout.tsx
index 74d1992..a478ad9 100644
--- a/frontend/src/app/[locale]/layout.tsx
+++ b/frontend/src/app/[locale]/layout.tsx
@@ -5,8 +5,9 @@ import Header from './Header';
import clsx from 'clsx';
import { getTranslations } from 'next-intl/server';
import { useTranslations } from 'next-intl';
+import { LocaleCodeType } from '@/types/locale';
-export async function generateMetadata({ params: { locale } }) {
+export async function generateMetadata({ params: { locale } }: { params: { locale: LocaleCodeType } }) {
const t = await getTranslations({ locale, namespace: 'Header' });
return {
title: t('title'),
@@ -24,7 +25,7 @@ export default function RootLayout({
params: { locale },
}: {
children: React.ReactNode;
- params: { locale: string };
+ params: { locale: LocaleCodeType };
}) {
const t = useTranslations('Toast');
const toastMessages = {
diff --git a/frontend/src/app/[locale]/page.tsx b/frontend/src/app/[locale]/page.tsx
index fde6e26..6b35534 100644
--- a/frontend/src/app/[locale]/page.tsx
+++ b/frontend/src/app/[locale]/page.tsx
@@ -4,8 +4,10 @@ import { title, subtitle } from '@/components/primitives';
import PaneMainTitle from './PaneMainTitle';
import PaneMainFeatures from './PaneMainFeatures';
import DemoImage from './DemoImage';
+import { PageType } from '@/types/base';
+import { LocaleCodeType } from '@/types/locale';
-export default function Home(params: { locale: string }) {
+export default function Home({ params }: PageType) {
const t = useTranslations('Index');
const demoImages = [
@@ -35,7 +37,7 @@ export default function Home(params: { locale: string }) {
diff --git a/frontend/src/app/[locale]/projects/ProjectsPage.tsx b/frontend/src/app/[locale]/projects/ProjectsPage.tsx
index 6ebd48b..80424f9 100644
--- a/frontend/src/app/[locale]/projects/ProjectsPage.tsx
+++ b/frontend/src/app/[locale]/projects/ProjectsPage.tsx
@@ -3,17 +3,19 @@ import { useEffect, useState, useContext } from 'react';
import { Button } from '@nextui-org/react';
import { Plus } from 'lucide-react';
import { TokenContext } from '@/utils/TokenProvider';
-import { ProjectType, ProjectsMessages } from '@/types/project';
+import { ProjectDialogMessages, ProjectType, ProjectsMessages } from '@/types/project';
import ProjectsTable from './ProjectsTable';
import ProjectDialog from '@/components/ProjectDialog';
import { fetchProjects, createProject } from '@/utils/projectsControl';
+import { LocaleCodeType } from '@/types/locale';
export type Props = {
messages: ProjectsMessages;
- locale: string;
+ projectDialogMessages: ProjectDialogMessages;
+ locale: LocaleCodeType;
};
-export default function ProjectsPage({ messages, locale }: Props) {
+export default function ProjectsPage({ messages, projectDialogMessages, locale }: Props) {
const context = useContext(TokenContext);
const [projects, setProjects] = useState
([]);
@@ -73,7 +75,7 @@ export default function ProjectsPage({ messages, locale }: Props) {
editingProject={editingProject}
onCancel={closeDialog}
onSubmit={onSubmit}
- messages={messages}
+ projectDialogMessages={projectDialogMessages}
/>
);
diff --git a/frontend/src/app/[locale]/projects/ProjectsTable.tsx b/frontend/src/app/[locale]/projects/ProjectsTable.tsx
index d7ab4d0..1744358 100644
--- a/frontend/src/app/[locale]/projects/ProjectsTable.tsx
+++ b/frontend/src/app/[locale]/projects/ProjectsTable.tsx
@@ -1,14 +1,15 @@
-import { useState, useMemo, useCallback } from 'react';
+import { useState, useMemo, useCallback, ReactNode } from 'react';
import { Table, TableHeader, TableColumn, TableBody, TableRow, TableCell, SortDescriptor } from '@nextui-org/react';
import { Link, NextUiLinkClasses } from '@/src/navigation';
import { ProjectType, ProjectsMessages } from '@/types/project';
import dayjs from 'dayjs';
import PublicityChip from '@/components/PublicityChip';
+import { LocaleCodeType } from '@/types/locale';
type Props = {
projects: ProjectType[];
messages: ProjectsMessages;
- locale: string;
+ locale: LocaleCodeType;
};
export default function ProjectsTable({ projects, messages, locale }: Props) {
@@ -38,21 +39,23 @@ export default function ProjectsTable({ projects, messages, locale }: Props) {
return text.length > maxLength ? text.slice(0, maxLength) + '...' : text;
};
- const renderCell = useCallback((project: ProjectType, columnKey: Key) => {
+ const renderCell = useCallback((project: ProjectType, columnKey: string): ReactNode => {
const cellValue = project[columnKey as keyof ProjectType];
switch (columnKey) {
case 'id':
- return
{cellValue};
+ return
{cellValue as number};
case 'isPublic':
- return
;
+ return (
+
+ );
case 'name':
const maxLength = 30;
const truncatedDetail = truncateText(project.detail, maxLength);
return (
- {cellValue}
+ {cellValue as string}
{truncatedDetail}
@@ -60,9 +63,9 @@ export default function ProjectsTable({ projects, messages, locale }: Props) {
);
case 'updatedAt':
- return
{dayjs(cellValue).format('YYYY/MM/DD HH:mm')};
+ return
{dayjs(cellValue as number).format('YYYY/MM/DD HH:mm')};
default:
- return cellValue;
+ return cellValue as string;
}
}, []);
@@ -107,7 +110,9 @@ export default function ProjectsTable({ projects, messages, locale }: Props) {
{(item) => (
- {(columnKey) => {renderCell(item, columnKey)}}
+
+ {(columnKey) => {renderCell(item, columnKey as string)}}
+
)}
diff --git a/frontend/src/app/[locale]/projects/[projectId]/folders/FoldersPane.tsx b/frontend/src/app/[locale]/projects/[projectId]/folders/FoldersPane.tsx
index d215e97..2e8ff59 100644
--- a/frontend/src/app/[locale]/projects/[projectId]/folders/FoldersPane.tsx
+++ b/frontend/src/app/[locale]/projects/[projectId]/folders/FoldersPane.tsx
@@ -33,7 +33,7 @@ export default function FoldersPane({ projectId, messages, locale }: Props) {
return;
}
try {
- const folders: FolderType[] = await fetchFolders(context.token.access_token, projectId);
+ const folders: FolderType[] = await fetchFolders(context.token.access_token, Number(projectId));
setFolders(folders);
// no folder on project
diff --git a/frontend/src/app/[locale]/projects/[projectId]/folders/[folderId]/cases/CasesPane.tsx b/frontend/src/app/[locale]/projects/[projectId]/folders/[folderId]/cases/CasesPane.tsx
index b689d95..c0ea790 100644
--- a/frontend/src/app/[locale]/projects/[projectId]/folders/[folderId]/cases/CasesPane.tsx
+++ b/frontend/src/app/[locale]/projects/[projectId]/folders/[folderId]/cases/CasesPane.tsx
@@ -7,13 +7,14 @@ import { CaseType, CasesMessages } from '@/types/case';
import DeleteConfirmDialog from '@/components/DeleteConfirmDialog';
import CaseDialog from './CaseDialog';
import { PriorityMessages } from '@/types/priority';
+import { LocaleCodeType } from '@/types/locale';
type Props = {
projectId: string;
folderId: string;
messages: CasesMessages;
priorityMessages: PriorityMessages;
- locale: string;
+ locale: LocaleCodeType;
};
export default function CasesPane({ projectId, folderId, messages, priorityMessages, locale }: Props) {
@@ -67,7 +68,7 @@ export default function CasesPane({ projectId, folderId, messages, priorityMessa
const onConfirm = async () => {
if (deleteCaseIds.length > 0) {
- await deleteCases(context.token.access_token, deleteCaseIds, projectId);
+ await deleteCases(context.token.access_token, deleteCaseIds, Number(projectId));
setCases(cases.filter((entry) => !deleteCaseIds.includes(entry.id)));
closeDeleteConfirmDialog();
}
diff --git a/frontend/src/app/[locale]/projects/[projectId]/folders/[folderId]/cases/TestCaseTable.tsx b/frontend/src/app/[locale]/projects/[projectId]/folders/[folderId]/cases/TestCaseTable.tsx
index e4110a1..e040f25 100644
--- a/frontend/src/app/[locale]/projects/[projectId]/folders/[folderId]/cases/TestCaseTable.tsx
+++ b/frontend/src/app/[locale]/projects/[projectId]/folders/[folderId]/cases/TestCaseTable.tsx
@@ -1,4 +1,4 @@
-import { useState, useMemo, useCallback } from 'react';
+import { useState, useMemo, useCallback, ReactNode } from 'react';
import {
Table,
TableHeader,
@@ -15,10 +15,11 @@ import {
SortDescriptor,
} from '@nextui-org/react';
import { Link, NextUiLinkClasses } from '@/src/navigation';
-import { Plus, MoreVertical, Trash, Circle } from 'lucide-react';
+import { Plus, MoreVertical, Trash } from 'lucide-react';
import { CaseType, CasesMessages } from '@/types/case';
import { PriorityMessages } from '@/types/priority';
import TestCasePriority from '@/components/TestCasePriority';
+import { LocaleCodeType } from '@/types/locale';
type Props = {
projectId: string;
@@ -29,7 +30,7 @@ type Props = {
onDeleteCases: (caseIds: number[]) => void;
messages: CasesMessages;
priorityMessages: PriorityMessages;
- locale: string;
+ locale: LocaleCodeType;
};
export default function TestCaseTable({
@@ -73,12 +74,12 @@ export default function TestCaseTable({
onDeleteCase(deleteCaseId);
};
- const renderCell = useCallback((testCase: CaseType, columnKey: Key) => {
+ const renderCell = useCallback((testCase: CaseType, columnKey: string): ReactNode => {
const cellValue = testCase[columnKey as keyof CaseType];
switch (columnKey) {
case 'id':
- return
{cellValue};
+ return
{cellValue as number};
case 'title':
return (
);
case 'priority':
- return
;
+ return
;
case 'actions':
return (
@@ -113,7 +114,7 @@ export default function TestCaseTable({
);
default:
- return cellValue;
+ return cellValue as string;
}
}, []);
@@ -152,7 +153,7 @@ export default function TestCaseTable({
{messages.testCaseList}
- {(selectedKeys.size > 0 || selectedKeys === 'all') && (
+ {((selectedKeys !== 'all' && selectedKeys.size > 0) || selectedKeys === 'all') && (
}
size="sm"
@@ -200,7 +201,9 @@ export default function TestCaseTable({
{(item) => (
- {(columnKey) => {renderCell(item, columnKey)}}
+
+ {(columnKey) => {renderCell(item, columnKey as string)}}
+
)}
diff --git a/frontend/src/app/[locale]/projects/[projectId]/folders/[folderId]/cases/[caseId]/CaseAttachmentsEditor.tsx b/frontend/src/app/[locale]/projects/[projectId]/folders/[folderId]/cases/[caseId]/CaseAttachmentsEditor.tsx
index 0255db7..b6d85f0 100644
--- a/frontend/src/app/[locale]/projects/[projectId]/folders/[folderId]/cases/[caseId]/CaseAttachmentsEditor.tsx
+++ b/frontend/src/app/[locale]/projects/[projectId]/folders/[folderId]/cases/[caseId]/CaseAttachmentsEditor.tsx
@@ -9,7 +9,7 @@ type Props = {
attachments: AttachmentType[];
onAttachmentDownload: (attachmentId: number, downloadFileName: string) => void;
onAttachmentDelete: (attachmentId: number) => void;
- onFilesDrop: (event: DragEvent) => void;
+ onFilesDrop: (event: DragEvent
) => void;
onFilesInput: (event: ChangeEvent) => void;
messages: CaseMessages;
};
diff --git a/frontend/src/app/[locale]/projects/[projectId]/folders/[folderId]/cases/[caseId]/CaseEditor.tsx b/frontend/src/app/[locale]/projects/[projectId]/folders/[folderId]/cases/[caseId]/CaseEditor.tsx
index c956a9e..d78992e 100644
--- a/frontend/src/app/[locale]/projects/[projectId]/folders/[folderId]/cases/[caseId]/CaseEditor.tsx
+++ b/frontend/src/app/[locale]/projects/[projectId]/folders/[folderId]/cases/[caseId]/CaseEditor.tsx
@@ -1,5 +1,5 @@
'use client';
-import { useState, useEffect, useContext } from 'react';
+import { useState, useEffect, useContext, ChangeEvent, DragEvent } from 'react';
import { Input, Textarea, Select, SelectItem, Button, Divider, Tooltip } from '@nextui-org/react';
import { useRouter } from '@/src/navigation';
import { Save, Plus, ArrowLeft, Circle } from 'lucide-react';
@@ -134,13 +134,22 @@ export default function CaseEditor({
}
};
- const handleDrop = async (event: DragEvent) => {
+ const handleDrop = (event: DragEvent) => {
event.preventDefault();
- handleFetchCreateAttachments(Number(caseId), event.dataTransfer.files);
+ if (event.dataTransfer) {
+ const filesArray = Array.from(event.dataTransfer.files);
+ handleFetchCreateAttachments(Number(caseId), filesArray);
+ }
};
- const handleInput = (event: DragEvent) => {
- handleFetchCreateAttachments(Number(caseId), event.target.files);
+ const handleInput = (event: ChangeEvent) => {
+ if (event.target) {
+ const input = event.target as HTMLInputElement;
+ if (input.files) {
+ const filesArray = Array.from(input.files);
+ handleFetchCreateAttachments(Number(caseId), filesArray);
+ }
+ }
};
const handleFetchCreateAttachments = async (caseId: number, files: File[]) => {
@@ -149,28 +158,36 @@ export default function CaseEditor({
if (newAttachments) {
const newAttachmentsWithJoinTable = [];
newAttachments.forEach((attachment: AttachmentType) => {
- attachment.caseAttachments = { AttachmentId: attachment.id };
+ attachment.caseAttachments = {
+ createdAt: new Date(),
+ updatedAt: new Date(),
+ caseId: 0,
+ attachmentId: attachment.id,
+ };
newAttachmentsWithJoinTable.push(attachment);
});
const updatedAttachments = testCase.Attachments;
- updatedAttachments.push(...newAttachments);
+ if (updatedAttachments) {
+ updatedAttachments.push(...newAttachments);
- setTestCase({
- ...testCase,
- Attachments: updatedAttachments,
- });
+ setTestCase({
+ ...testCase,
+ Attachments: updatedAttachments,
+ });
+ }
}
};
const onAttachmentDelete = async (attachmentId: number) => {
await fetchDeleteAttachment(attachmentId);
+ if (testCase.Attachments) {
+ const filteredAttachments = testCase.Attachments.filter((attachment) => attachment.id !== attachmentId);
- const filteredAttachments = testCase.Attachments.filter((attachment) => attachment.id !== attachmentId);
-
- setTestCase({
- ...testCase,
- Attachments: filteredAttachments,
- });
+ setTestCase({
+ ...testCase,
+ Attachments: filteredAttachments,
+ });
+ }
};
useEffect(() => {
@@ -261,10 +278,12 @@ export default function CaseEditor({
size="sm"
variant="bordered"
selectedKeys={[priorities[testCase.priority].uid]}
- onSelectionChange={(e) => {
- const selectedUid = e.anchorKey;
- const index = priorities.findIndex((priority) => priority.uid === selectedUid);
- setTestCase({ ...testCase, priority: index });
+ onSelectionChange={(newSelection) => {
+ if (newSelection !== 'all' && newSelection.size !== 0) {
+ const selectedUid = Array.from(newSelection)[0];
+ const index = priorities.findIndex((priority) => priority.uid === selectedUid);
+ setTestCase({ ...testCase, priority: index });
+ }
}}
startContent={
@@ -285,10 +304,12 @@ export default function CaseEditor({
size="sm"
variant="bordered"
selectedKeys={[testTypes[testCase.type].uid]}
- onSelectionChange={(e) => {
- const selectedUid = e.anchorKey;
- const index = testTypes.findIndex((type) => type.uid === selectedUid);
- setTestCase({ ...testCase, type: index });
+ onSelectionChange={(newSelection) => {
+ if (newSelection !== 'all' && newSelection.size !== 0) {
+ const selectedUid = Array.from(newSelection)[0];
+ const index = testTypes.findIndex((type) => type.uid === selectedUid);
+ setTestCase({ ...testCase, type: index });
+ }
}}
label={messages.type}
className="mt-3 max-w-xs"
@@ -306,10 +327,12 @@ export default function CaseEditor({
size="sm"
variant="bordered"
selectedKeys={[templates[testCase.template].uid]}
- onSelectionChange={(e) => {
- const selectedUid = e.anchorKey;
- const index = templates.findIndex((template) => template.uid === selectedUid);
- setTestCase({ ...testCase, template: index });
+ onSelectionChange={(newSelection) => {
+ if (newSelection !== 'all' && newSelection.size !== 0) {
+ const selectedUid = Array.from(newSelection)[0];
+ const index = templates.findIndex((template) => template.uid === selectedUid);
+ setTestCase({ ...testCase, template: index });
+ }
}}
label={messages.template}
className="mt-3 max-w-xs"
@@ -365,41 +388,47 @@ export default function CaseEditor({
{messages.newStep}
-
{
- setTestCase({
- ...testCase,
- Steps: testCase.Steps.map((step) => {
- if (step.id === stepId) {
- return changeStep;
- } else {
- return step;
- }
- }),
- });
- }}
- onStepPlus={onPlusClick}
- onStepDelete={onDeleteClick}
- messages={messages}
- />
+ {testCase.Steps && (
+ {
+ if (testCase.Steps) {
+ setTestCase({
+ ...testCase,
+ Steps: testCase.Steps.map((step) => {
+ if (step.id === stepId) {
+ return changeStep;
+ } else {
+ return step;
+ }
+ }),
+ });
+ }
+ }}
+ onStepPlus={onPlusClick}
+ onStepDelete={onDeleteClick}
+ messages={messages}
+ />
+ )}
)}
{messages.attachments}
-
- fetchDownloadAttachment(attachmentId, downloadFileName)
- }
- onAttachmentDelete={onAttachmentDelete}
- onFilesDrop={handleDrop}
- onFilesInput={handleInput}
- messages={messages}
- />
+ {testCase.Attachments && (
+
+ fetchDownloadAttachment(attachmentId, downloadFileName)
+ }
+ onAttachmentDelete={onAttachmentDelete}
+ onFilesDrop={handleDrop}
+ onFilesInput={handleInput}
+ messages={messages}
+ />
+ )}
>
);
diff --git a/frontend/src/app/[locale]/projects/[projectId]/folders/[folderId]/cases/page.tsx b/frontend/src/app/[locale]/projects/[projectId]/folders/[folderId]/cases/page.tsx
index 1ce606c..9247666 100644
--- a/frontend/src/app/[locale]/projects/[projectId]/folders/[folderId]/cases/page.tsx
+++ b/frontend/src/app/[locale]/projects/[projectId]/folders/[folderId]/cases/page.tsx
@@ -1,6 +1,7 @@
import { PriorityMessages } from '@/types/priority';
import CasesPane from './CasesPane';
import { useTranslations } from 'next-intl';
+import { LocaleCodeType } from '@/types/locale';
export default function Page({ params }: { params: { projectId: string; folderId: string; locale: string } }) {
const t = useTranslations('Cases');
@@ -36,7 +37,7 @@ export default function Page({ params }: { params: { projectId: string; folderId
diff --git a/frontend/src/app/[locale]/projects/[projectId]/home/ProjectHome.tsx b/frontend/src/app/[locale]/projects/[projectId]/home/ProjectHome.tsx
index 4f98ff6..d157663 100644
--- a/frontend/src/app/[locale]/projects/[projectId]/home/ProjectHome.tsx
+++ b/frontend/src/app/[locale]/projects/[projectId]/home/ProjectHome.tsx
@@ -3,7 +3,6 @@ import { useState, useEffect, useContext } from 'react';
import { title, subtitle } from '@/components/primitives';
import { Card, CardBody, Chip, Divider } from '@nextui-org/react';
import { Folder, Clipboard, FlaskConical } from 'lucide-react';
-import { CaseTypeCountType, CasePriorityCountType } from '@/types/case';
import { ProgressSeriesType } from '@/types/run';
import { HomeMessages } from './page';
import { TokenContext } from '@/utils/TokenProvider';
@@ -16,6 +15,8 @@ import TestProgressBarChart from './TestProgressColumnChart';
import { TestRunCaseStatusMessages } from '@/types/status';
import { TestTypeMessages } from '@/types/testType';
import { PriorityMessages } from '@/types/priority';
+import { ProjectType } from '@/types/project';
+import { CasePriorityCountType, CaseTypeCountType } from '@/types/chart';
const apiServer = Config.apiServer;
@@ -59,19 +60,24 @@ export function ProjectHome({
}: Props) {
const context = useContext(TokenContext);
const { theme, setTheme } = useTheme();
- const [project, setProject] = useState({
+ const [project, setProject] = useState
({
+ id: 0,
name: '',
detail: '',
- Folders: [{ Cases: [] }],
- Runs: [{ RunCases: [] }],
+ isPublic: false,
+ userId: 0,
+ createdAt: '',
+ updatedAt: '',
+ Folders: [],
+ Runs: [],
});
const [folderNum, setFolderNum] = useState(0);
const [caseNum, setCaseNum] = useState(0);
const [runNum, setRunNum] = useState(0);
- const [typesCounts, setTypesCounts] = useState();
- const [priorityCounts, setPriorityCounts] = useState();
- const [progressCategories, setProgressCategories] = useState();
- const [progressSeries, setProgressSeries] = useState();
+ const [typesCounts, setTypesCounts] = useState([]);
+ const [priorityCounts, setPriorityCounts] = useState([]);
+ const [progressCategories, setProgressCategories] = useState([]);
+ const [progressSeries, setProgressSeries] = useState([]);
useEffect(() => {
async function fetchDataEffect() {
@@ -92,6 +98,9 @@ export function ProjectHome({
useEffect(() => {
async function aggregate() {
+ if (!project) {
+ return;
+ }
const { folderNum, runNum, caseNum } = aggregateBasicInfo(project);
setFolderNum(folderNum);
setRunNum(runNum);
diff --git a/frontend/src/app/[locale]/projects/[projectId]/home/TestPriorityDonutChart.tsx b/frontend/src/app/[locale]/projects/[projectId]/home/TestPriorityDonutChart.tsx
index 757647e..487e2de 100644
--- a/frontend/src/app/[locale]/projects/[projectId]/home/TestPriorityDonutChart.tsx
+++ b/frontend/src/app/[locale]/projects/[projectId]/home/TestPriorityDonutChart.tsx
@@ -2,18 +2,19 @@ import React from 'react';
import { useState, useEffect } from 'react';
import dynamic from 'next/dynamic';
import { priorities } from '@/config/selection';
-import { CasePriorityCountType } from '@/types/case';
+import { CasePriorityCountType } from '@/types/chart';
import { PriorityMessages } from '@/types/priority';
+import { ChartDataType } from '@/types/chart';
const Chart = dynamic(() => import('react-apexcharts'), { ssr: false });
type Props = {
priorityCounts: CasePriorityCountType[];
priorityMessages: PriorityMessages;
- theme: string;
+ theme: string | undefined;
};
export default function TestPriorityDonutChart({ priorityCounts, priorityMessages, theme }: Props) {
- const [chartData, setChartData] = useState({
+ const [chartData, setChartData] = useState({
series: [],
options: {
labels: [],
diff --git a/frontend/src/app/[locale]/projects/[projectId]/home/TestProgressColumnChart.tsx b/frontend/src/app/[locale]/projects/[projectId]/home/TestProgressColumnChart.tsx
index ff09202..0f3581c 100644
--- a/frontend/src/app/[locale]/projects/[projectId]/home/TestProgressColumnChart.tsx
+++ b/frontend/src/app/[locale]/projects/[projectId]/home/TestProgressColumnChart.tsx
@@ -3,16 +3,17 @@ import { useState, useEffect } from 'react';
import dynamic from 'next/dynamic';
import { ProgressSeriesType } from '@/types/run';
import { testRunCaseStatus } from '@/config/selection';
+import { ChartDataType } from '@/types/chart';
const Chart = dynamic(() => import('react-apexcharts'), { ssr: false });
type Props = {
progressSeries: ProgressSeriesType[];
progressCategories: string[];
- theme: string;
+ theme: string | undefined;
};
export default function TestProgressBarChart({ progressSeries, progressCategories, theme }: Props) {
- const [chartData, setChartData] = useState({
+ const [chartData, setChartData] = useState({
series: [],
options: {
labels: [],
diff --git a/frontend/src/app/[locale]/projects/[projectId]/home/TestTypesDonutChart.tsx b/frontend/src/app/[locale]/projects/[projectId]/home/TestTypesDonutChart.tsx
index 8a2486b..a324fc4 100644
--- a/frontend/src/app/[locale]/projects/[projectId]/home/TestTypesDonutChart.tsx
+++ b/frontend/src/app/[locale]/projects/[projectId]/home/TestTypesDonutChart.tsx
@@ -2,18 +2,18 @@ import React from 'react';
import { useState, useEffect } from 'react';
import dynamic from 'next/dynamic';
import { testTypes } from '@/config/selection';
-import { CaseTypeCountType } from '@/types/case';
import { TestTypeMessages } from '@/types/testType';
+import { CaseTypeCountType, ChartDataType } from '@/types/chart';
const Chart = dynamic(() => import('react-apexcharts'), { ssr: false });
type Props = {
typesCounts: CaseTypeCountType[];
testTypeMessages: TestTypeMessages;
- theme: string;
+ theme: string | undefined;
};
export default function TestTypesDonutChart({ typesCounts, testTypeMessages, theme }: Props) {
- const [chartData, setChartData] = useState({
+ const [chartData, setChartData] = useState({
series: [],
options: {
labels: [],
diff --git a/frontend/src/app/[locale]/projects/[projectId]/home/aggregate.ts b/frontend/src/app/[locale]/projects/[projectId]/home/aggregate.ts
index 41c7ffc..ba771a1 100644
--- a/frontend/src/app/[locale]/projects/[projectId]/home/aggregate.ts
+++ b/frontend/src/app/[locale]/projects/[projectId]/home/aggregate.ts
@@ -1,6 +1,7 @@
import { ProjectType } from '@/types/project';
import { testTypes, priorities, testRunCaseStatus } from '@/config/selection';
import { TestRunCaseStatusMessages } from '@/types/status';
+import { CasePriorityCountType, CaseTypeCountType } from '@/types/chart';
// aggregate folder, case, run mum
function aggregateBasicInfo(project: ProjectType) {
@@ -15,48 +16,58 @@ function aggregateBasicInfo(project: ProjectType) {
return { folderNum, runNum, caseNum };
}
-// aggregate test types of each case
-function aggregateTestType(project: ProjectType) {
- const typesCounts = {};
+function aggregateTestType(project: ProjectType): CaseTypeCountType[] {
+ // count how many test cases are for each type
+ const typesCounts: number[] = testTypes.map((entry) => {
+ return 0;
+ });
project.Folders.forEach((folder) => {
folder.Cases.forEach((testcase) => {
const type = testcase.type;
- typesCounts[type] = (typesCounts[type] || 0) + 1;
+ typesCounts[type]++;
});
});
- const result = [];
+ const result: CaseTypeCountType[] = [];
for (let type = 0; type <= testTypes.length; type++) {
- result.push({ type: type, count: typesCounts[type] || 0 });
+ result.push({ type: type, count: typesCounts[type] });
}
return result;
}
function aggregateTestPriority(project: ProjectType) {
- const priorityCounts = {};
+ // count how many test cases are for each priority
+ const priorityCounts: number[] = priorities.map((entry) => {
+ return 0;
+ });
project.Folders.forEach((folder) => {
folder.Cases.forEach((testcase) => {
const priority = testcase.priority;
- priorityCounts[priority] = (priorityCounts[priority] || 0) + 1;
+ priorityCounts[priority]++;
});
});
- const result = [];
+ const result: CasePriorityCountType[] = [];
for (let priority = 0; priority <= priorities.length; priority++) {
- result.push({ priority: priority, count: priorityCounts[priority] || 0 });
+ result.push({ priority: priority, count: priorityCounts[priority] });
}
return result;
}
function aggregateProgress(project: ProjectType, testRunCaseStatusMessages: TestRunCaseStatusMessages) {
- let series = testRunCaseStatus.map((status) => {
+ type ChartSeries = { name: string; data: number[] };
+ let series: ChartSeries[] = testRunCaseStatus.map((status) => {
return { name: testRunCaseStatusMessages[status.uid], data: [] };
});
- let categories = [];
+ let categories: string[] = [];
project.Runs.forEach((run) => {
+ if (!run.RunCases) {
+ return;
+ }
+
run.RunCases.forEach((runCase) => {
const createdAtDate = new Date(runCase.createdAt);
const dateString = createdAtDate.toISOString().slice(0, 10);
@@ -72,6 +83,10 @@ function aggregateProgress(project: ProjectType, testRunCaseStatusMessages: Test
});
project.Runs.forEach((run) => {
+ if (!run.RunCases) {
+ return;
+ }
+
run.RunCases.forEach((runCase) => {
const createdAtDate = new Date(runCase.createdAt);
const dateString = createdAtDate.toISOString().slice(0, 10);
diff --git a/frontend/src/app/[locale]/projects/[projectId]/layout.tsx b/frontend/src/app/[locale]/projects/[projectId]/layout.tsx
index 6c4a6a2..8a25f2e 100644
--- a/frontend/src/app/[locale]/projects/[projectId]/layout.tsx
+++ b/frontend/src/app/[locale]/projects/[projectId]/layout.tsx
@@ -1,3 +1,4 @@
+import { ProjectMessages } from '@/types/project';
import Sidebar from './Sidebar';
import { useTranslations } from 'next-intl';
@@ -9,7 +10,7 @@ export default function SidebarLayout({
params: { locale: string };
}) {
const t = useTranslations('Project');
- const messages = {
+ const messages: ProjectMessages = {
home: t('home'),
testCases: t('test_cases'),
testRuns: t('test_runs'),
diff --git a/frontend/src/app/[locale]/projects/[projectId]/members/AddMemberDialog.tsx b/frontend/src/app/[locale]/projects/[projectId]/members/AddMemberDialog.tsx
index 32722aa..6711e52 100644
--- a/frontend/src/app/[locale]/projects/[projectId]/members/AddMemberDialog.tsx
+++ b/frontend/src/app/[locale]/projects/[projectId]/members/AddMemberDialog.tsx
@@ -2,18 +2,18 @@
import React from 'react';
import { useState, useEffect, useContext } from 'react';
import { Button, Input, Modal, ModalContent, ModalHeader, ModalBody, ModalFooter } from '@nextui-org/react';
-import { SettingsMessages } from '@/types/settings';
import { TokenContext } from '@/utils/TokenProvider';
import { UserType } from '@/types/user';
import { searchUsers } from '@/utils/usersControl';
import CandidatesTable from './CandidatesTable';
+import { MembersMessages } from '@/types/member';
type Props = {
isOpen: boolean;
projectId: string;
onCancel: () => void;
onAddMember: (memberAdded: UserType) => void;
- messages: SettingsMessages;
+ messages: MembersMessages;
};
export default function AddMemberDialog({ isOpen, projectId, onCancel, onAddMember, messages }: Props) {
diff --git a/frontend/src/app/[locale]/projects/[projectId]/members/CandidatesTable.tsx b/frontend/src/app/[locale]/projects/[projectId]/members/CandidatesTable.tsx
index 5479e8f..f215aef 100644
--- a/frontend/src/app/[locale]/projects/[projectId]/members/CandidatesTable.tsx
+++ b/frontend/src/app/[locale]/projects/[projectId]/members/CandidatesTable.tsx
@@ -1,24 +1,24 @@
import { useMemo, useCallback } from 'react';
import { Button, Table, TableHeader, TableColumn, TableBody, TableRow, TableCell } from '@nextui-org/react';
import { UserType } from '@/types/user';
-import { SettingsMessages } from '@/types/settings';
import Avatar from 'boring-avatars';
+import { MembersMessages } from '@/types/member';
type Props = {
candidates: UserType[];
onAddPress: (userAdded: UserType) => void;
- messages: SettingsMessages;
+ messages: MembersMessages;
};
export default function MembersTable({ candidates, onAddPress, messages }: Props) {
const headerColumns = [
- { name: messages.avatar, uid: 'avatar' },
- { name: messages.email, uid: 'email' },
- { name: messages.username, uid: 'username' },
- { name: messages.add, uid: 'add' },
+ { name: messages.avatar, uid: 'avatar', sortable: false },
+ { name: messages.email, uid: 'email', sortable: false },
+ { name: messages.username, uid: 'username', sortable: false },
+ { name: messages.add, uid: 'add', sortable: false },
];
- const renderCell = useCallback((candidate: UserType, columnKey: Key) => {
+ const renderCell = useCallback((candidate: UserType, columnKey: string) => {
const cellValue = candidate[columnKey as keyof UserType];
switch (columnKey) {
@@ -81,7 +81,9 @@ export default function MembersTable({ candidates, onAddPress, messages }: Props
{(item) => (
- {(columnKey) => {renderCell(item, columnKey)}}
+
+ {(columnKey) => {renderCell(item, columnKey as string)}}
+
)}
diff --git a/frontend/src/app/[locale]/projects/[projectId]/members/MembersPage.tsx b/frontend/src/app/[locale]/projects/[projectId]/members/MembersPage.tsx
index 53f9eb5..2d30fca 100644
--- a/frontend/src/app/[locale]/projects/[projectId]/members/MembersPage.tsx
+++ b/frontend/src/app/[locale]/projects/[projectId]/members/MembersPage.tsx
@@ -40,11 +40,13 @@ export default function MembersPage({ projectId, messages, locale }: Props) {
}, [context]);
const handleAddMember = async (userAdded: UserType) => {
- const newMember = await addMember(context.token.access_token, userAdded.id, projectId);
- newMember.User = userAdded;
- const updateMembers = [...members];
- updateMembers.push(newMember);
- setMembers(updateMembers);
+ if (userAdded.id) {
+ const newMember = await addMember(context.token.access_token, userAdded.id, Number(projectId));
+ newMember.User = userAdded;
+ const updateMembers = [...members];
+ updateMembers.push(newMember);
+ setMembers(updateMembers);
+ }
setIsDialogOpen(false);
};
@@ -64,22 +66,24 @@ export default function MembersPage({ projectId, messages, locale }: Props) {
const onConfirm = async () => {
if (deleteMemberId) {
- await deleteMember(context.token.access_token, deleteMemberId, projectId);
+ await deleteMember(context.token.access_token, deleteMemberId, Number(projectId));
setMembers(members.filter((member) => member.User.id !== deleteMemberId));
closeDeleteConfirmDialog();
}
};
const handleChangeRole = async (userEdit: UserType, role: number) => {
- await updateMember(context.token.access_token, userEdit.id, projectId, role);
- setMembers((prevMembers) => {
- return prevMembers.map((member) => {
- if (member.User.id === userEdit.id) {
- return { ...member, role: role };
- }
- return member;
+ if (userEdit.id) {
+ await updateMember(context.token.access_token, userEdit.id, Number(projectId), role);
+ setMembers((prevMembers) => {
+ return prevMembers.map((member) => {
+ if (member.User.id === userEdit.id) {
+ return { ...member, role: role };
+ }
+ return member;
+ });
});
- });
+ }
};
return (
diff --git a/frontend/src/app/[locale]/projects/[projectId]/members/MembersTable.tsx b/frontend/src/app/[locale]/projects/[projectId]/members/MembersTable.tsx
index 3e1922d..f54afac 100644
--- a/frontend/src/app/[locale]/projects/[projectId]/members/MembersTable.tsx
+++ b/frontend/src/app/[locale]/projects/[projectId]/members/MembersTable.tsx
@@ -1,4 +1,4 @@
-import { useState, useMemo } from 'react';
+import { useState, useMemo, ReactNode } from 'react';
import {
Button,
Table,
@@ -15,16 +15,16 @@ import {
} from '@nextui-org/react';
import { ChevronDown } from 'lucide-react';
import { MemberType, UserType } from '@/types/user';
-import { SettingsMessages } from '@/types/settings';
import { memberRoles } from '@/config/selection';
import Avatar from 'boring-avatars';
+import { MembersMessages } from '@/types/member';
type Props = {
members: MemberType[];
isDisabled: boolean;
onChangeRole: (userEdit: UserType, role: number) => void;
onDeleteMember: (deletedUserId: number) => void;
- messages: SettingsMessages;
+ messages: MembersMessages;
};
export default function MembersTable({ members, isDisabled, onChangeRole, onDeleteMember, messages }: Props) {
@@ -42,17 +42,17 @@ export default function MembersTable({ members, isDisabled, onChangeRole, onDele
});
const sortedItems = useMemo(() => {
- return [...members].sort((a: UserType, b: UserType) => {
- const first = a[sortDescriptor.column as keyof UserType] as number;
- const second = b[sortDescriptor.column as keyof UserType] as number;
+ return [...members].sort((a: MemberType, b: MemberType) => {
+ const first = a[sortDescriptor.column as keyof MemberType] as number;
+ const second = b[sortDescriptor.column as keyof MemberType] as number;
const cmp = first < second ? -1 : first > second ? 1 : 0;
return sortDescriptor.direction === 'descending' ? -cmp : cmp;
});
}, [sortDescriptor, members]);
- const renderCell = (member: UserType, columnKey: Key) => {
- const cellValue = member[columnKey as keyof UserType];
+ const renderCell = (member: MemberType, columnKey: string) => {
+ const cellValue = member[columnKey as keyof MemberType];
switch (columnKey) {
case 'avatar':
@@ -73,7 +73,7 @@ export default function MembersTable({ members, isDisabled, onChangeRole, onDele
}>
- {messages[memberRoles[cellValue].uid]}
+ {messages[memberRoles[cellValue as number].uid]}
@@ -92,7 +92,11 @@ export default function MembersTable({ members, isDisabled, onChangeRole, onDele
isDisabled={isDisabled}
color="danger"
variant="light"
- onClick={() => onDeleteMember(member.User.id)}
+ onClick={() => {
+ if (member.User.id) {
+ onDeleteMember(member.User.id);
+ }
+ }}
>
{messages.deleteMember}
@@ -143,7 +147,9 @@ export default function MembersTable({ members, isDisabled, onChangeRole, onDele
{(item) => (
- {(columnKey) => {renderCell(item, columnKey)}}
+
+ {(columnKey) => {renderCell(item, columnKey as string) as ReactNode}}
+
)}
diff --git a/frontend/src/app/[locale]/projects/[projectId]/members/membersControl.ts b/frontend/src/app/[locale]/projects/[projectId]/members/membersControl.ts
index 017ffbf..c1f975f 100644
--- a/frontend/src/app/[locale]/projects/[projectId]/members/membersControl.ts
+++ b/frontend/src/app/[locale]/projects/[projectId]/members/membersControl.ts
@@ -24,7 +24,7 @@ async function fetchProjectMembers(jwt: string, projectId: string) {
}
}
-async function addMember(jwt: string, userId: string, projectId: string) {
+async function addMember(jwt: string, userId: number, projectId: number) {
const fetchOptions = {
method: 'POST',
headers: {
@@ -47,7 +47,7 @@ async function addMember(jwt: string, userId: string, projectId: string) {
}
}
-async function deleteMember(jwt: string, userId: string, projectId: string) {
+async function deleteMember(jwt: string, userId: number, projectId: number) {
const fetchOptions = {
method: 'DELETE',
headers: {
@@ -68,7 +68,7 @@ async function deleteMember(jwt: string, userId: string, projectId: string) {
}
}
-async function updateMember(jwt: string, userId: string, projectId: string, role: number) {
+async function updateMember(jwt: string, userId: number, projectId: number, role: number) {
const fetchOptions = {
method: 'PUT',
headers: {
diff --git a/frontend/src/app/[locale]/projects/[projectId]/runs/RunsPage.tsx b/frontend/src/app/[locale]/projects/[projectId]/runs/RunsPage.tsx
index 93a92e4..845d199 100644
--- a/frontend/src/app/[locale]/projects/[projectId]/runs/RunsPage.tsx
+++ b/frontend/src/app/[locale]/projects/[projectId]/runs/RunsPage.tsx
@@ -8,10 +8,11 @@ import { RunType, RunsMessages } from '@/types/run';
import RunDialog from './RunDialog';
import DeleteConfirmDialog from '@/components/DeleteConfirmDialog';
import { TokenContext } from '@/utils/TokenProvider';
+import { LocaleCodeType } from '@/types/locale';
type Props = {
projectId: string;
- locale: string;
+ locale: LocaleCodeType;
messages: RunsMessages;
};
diff --git a/frontend/src/app/[locale]/projects/[projectId]/runs/RunsTable.tsx b/frontend/src/app/[locale]/projects/[projectId]/runs/RunsTable.tsx
index 7266bdb..1645234 100644
--- a/frontend/src/app/[locale]/projects/[projectId]/runs/RunsTable.tsx
+++ b/frontend/src/app/[locale]/projects/[projectId]/runs/RunsTable.tsx
@@ -1,4 +1,4 @@
-import { useState, useEffect, useMemo } from 'react';
+import { useState, useEffect, useMemo, ReactNode } from 'react';
import {
Table,
TableHeader,
@@ -17,6 +17,7 @@ import { Link, NextUiLinkClasses } from '@/src/navigation';
import { MoreVertical } from 'lucide-react';
import { RunsMessages, RunType } from '@/types/run';
import dayjs from 'dayjs';
+import { LocaleCodeType } from '@/types/locale';
type Props = {
projectId: string;
@@ -24,7 +25,7 @@ type Props = {
runs: RunType[];
onDeleteRun: (runId: number) => void;
messages: RunsMessages;
- locale: string;
+ locale: LocaleCodeType;
};
export default function RunsTable({ projectId, isDisabled, runs, onDeleteRun, messages, locale }: Props) {
@@ -63,19 +64,19 @@ export default function RunsTable({ projectId, isDisabled, runs, onDeleteRun, me
return text.length > maxLength ? text.slice(0, maxLength) + '...' : text;
};
- const renderCell = (run: RunType, columnKey: Key) => {
+ const renderCell = (run: RunType, columnKey: string) => {
const cellValue = run[columnKey as keyof RunType];
switch (columnKey) {
case 'id':
- return {cellValue};
+ return {cellValue as number};
case 'name':
const maxLength = 30;
const truncatedDescription = truncateText(run.description, maxLength);
return (
- {cellValue}
+ {cellValue as string}
{truncatedDescription}
@@ -83,7 +84,7 @@ export default function RunsTable({ projectId, isDisabled, runs, onDeleteRun, me
);
case 'updatedAt':
- return
{dayjs(cellValue).format('YYYY/MM/DD HH:mm')};
+ return
{dayjs(cellValue as string).format('YYYY/MM/DD HH:mm')};
case 'actions':
return (
@@ -145,7 +146,9 @@ export default function RunsTable({ projectId, isDisabled, runs, onDeleteRun, me
{(item) => (
- {(columnKey) => {renderCell(item, columnKey)}}
+
+ {(columnKey) => {renderCell(item, columnKey as string) as ReactNode}}
+
)}
diff --git a/frontend/src/app/[locale]/projects/[projectId]/runs/[runId]/RunEditor.tsx b/frontend/src/app/[locale]/projects/[projectId]/runs/[runId]/RunEditor.tsx
index 098bc4f..84c082b 100644
--- a/frontend/src/app/[locale]/projects/[projectId]/runs/[runId]/RunEditor.tsx
+++ b/frontend/src/app/[locale]/projects/[projectId]/runs/[runId]/RunEditor.tsx
@@ -77,7 +77,7 @@ export default function RunEditor({
const { theme, setTheme } = useTheme();
const [testRun, setTestRun] = useState(defaultTestRun);
const [folders, setFolders] = useState([]);
- const [runStatusCounts, setRunStatusCounts] = useState();
+ const [runStatusCounts, setRunStatusCounts] = useState([]);
const [selectedKeys, setSelectedKeys] = useState(new Set([]));
const [selectedFolder, setSelectedFolder] = useState(null);
const [testCases, setTestCases] = useState([]);
@@ -256,10 +256,12 @@ export default function RunEditor({
size="sm"
variant="bordered"
selectedKeys={[testRunStatus[testRun.state].uid]}
- onSelectionChange={(e) => {
- const selectedUid = e.anchorKey;
- const index = testRunStatus.findIndex((template) => template.uid === selectedUid);
- setTestRun({ ...testRun, state: index });
+ onSelectionChange={(newSelection) => {
+ if (newSelection !== 'all' && newSelection.size !== 0) {
+ const selectedUid = Array.from(newSelection)[0];
+ const index = testRunStatus.findIndex((template) => template.uid === selectedUid);
+ setTestRun({ ...testRun, state: index });
+ }
}}
label={messages.status}
className="mt-3 max-w-xs"
@@ -278,7 +280,7 @@ export default function RunEditor({
{messages.selectTestCase}
- {(selectedKeys.size > 0 || selectedKeys === 'all') && (
+ {(selectedKeys === 'all' || selectedKeys.size > 0) && (