Improve test code coverage of core components #12588

cedricongjh · 2023-09-13T09:34:23Z

Description of feature/enhancement
The components in src/web/app/components are used throughout TEAMMATES, but some of them have poor test coverage. Let's improve the test coverage in these components:

Steps to contribute

Choose component(s) to work on in the table below, and comment on this issue to indicate that you'd like to contribute tests for the component(s)
Create a PR with tests for the component, check the codecov link to ensure that the functionality not covered are indeed tested
Submit the PR, link it to this issue by starting the description with "Part of Improve test code coverage of core components #12588"

Components:

Component	Assignee	Codecov	PR	Done
AddingQuestionPanelComponent	-	link	-
CommentEditFormComponent	@HarryLu33	link	#12604	✅
CommentRowComponent	@ThomasGreen123	link	#12612	✅
CommentTableComponent	-	link	-
CommentTableModalComponent	@kenneySiu	link	#12609	✅
CommentVisibilityControlNamePipe, CommentVisibilityTypeDescriptionPipe, CommentVisibilityTypeNamePipe, CommentVisibilityTypesJointNamePipe	@cedricongjh	link1, link2, link3, link4	#12589	✅
CopyCourseModalComponent	@cedricongjh	link	#12590	✅
CopySessionModalComponent	@Astrid641	link	#12616	✅
CourseEditFormComponent	@lunoos	link	#12597	✅
DatepickerComponent	@dlimyy	link	#12592	✅
DatePickerFormatter	@dlimyy	link	#12593	✅
ErrorReportComponent	@u7488099	link	#12610	✅
ExtensionConfirmModalComponent	@MatheuMa	link	#12622	✅
FeedbackPathPanelComponent	@JYH11	link	#12613	✅
LogsHistogramComponent	-	link	-
LogsTableComponent	-	link	-
LogsDetails	-	link	-
LogsLine	-	link	-
QuestionEditForm	@abaduy123	link	-
QuestionResponsePanelComponent	@leyguan	link	#12867	✅
GqrRqgViewResponsesComponent	@RohithSurya	link	-
GroupedResponsesComponent	-	link	-
GrqRgqViewResponsesComponent	-	link	-
PerQuestionViewResponsesComponent	@WayyGood	link	-
SingleStatisticsComponent	@hberehan	link	-
InstructorResponsesViewBase	@justinsegawa	link	-
QuestionSubmissionFormComponent	@kaioenzo	link	-
RecipientTypeNamePipe	@ThomasGreen123	link	#12626	✅
QuestionConstraint	@cedricongjh	link	#12747	✅
QuestionEditAnswerForm	@skylar-mo	link	-
QuestionEditDetailsForm	@rushithapenikalapati	link	-
RichTextEditorComponent	-	link	-
SessionEditForm	@kenneySiu	link	#12627	✅
SessionsTable	-	link	-
StudentList	@cedricongjh	link	#12854	✅
ToastComponent	@techMedMau	link	#12916	✅
ViewResultsPanelComponent	@techMedMau	link	#12918	✅
VisibilityMessages	-	link	-
VisiblityPanel	@Andy-W-Developer	link	#12868	✅

The text was updated successfully, but these errors were encountered:

dlimyy · 2023-09-20T15:41:01Z

Hi, could I work on the unit tests for DatepickerComponent and DatePickerFormatter? Thanks!

cedricongjh · 2023-09-20T17:17:59Z

Hi, could I work on the unit tests for DatepickerComponent and DatePickerFormatter? Thanks!

yup sure, go ahead! thank you :D

blejkowski · 2023-09-21T13:02:53Z

May I work on the tests for the AddingQuestionPanelComponent?

cedricongjh · 2023-09-21T14:16:33Z

May I work on the tests for the AddingQuestionPanelComponent?

yes, thank you!

lunoos · 2023-09-22T04:08:56Z

Hi, i could work on test cases for CourseEditFormComponent.

* Add unit tests to datepickerformatter * Update wording for comment regarding date --------- Co-authored-by: Jason Qiu <[email protected]>

HarryLu33 · 2023-10-03T00:47:26Z

Hi, may I work on test cases for CommentEditFormComponent?

ThomasGreen123 · 2023-10-04T00:49:46Z

Hi @cedricongjh , I'm new to OSS and would like to work on this as my first issue. I want to work on test cases for CommentRowComponent. May I take it?

kenneySiu · 2023-10-04T01:34:00Z

Hi is it okay if i can work on test cases for CommentTableModalComponent?

JYH11 · 2023-10-04T03:18:17Z

Hi, is it fine to work on test cases for FeedbackPathPanelComponent? Thanks!

Astrid641 · 2023-10-05T05:24:28Z

Hi, may I work on the CopySessionModalComponent? Thanks!

MatheuMa · 2023-10-12T06:58:32Z

Hey, can I work on ExtensionConfirmModalComponent?

u7632387 Workshop10 (This line is just a reference for workshop, please ignore)

Miranda-lrx · 2023-10-12T10:22:46Z

Hi, I'd like to work on adding tests for the CommentTableComponent. Thank you.
My id is U7763307, Workshop 10

u7488099 · 2023-10-13T06:16:44Z

Hi, can I work on ErrorReportComponent? Thanks!

cedricongjh · 2023-10-14T14:05:15Z

Hi, I'd like to work on adding tests for the AddingQuestionPanelComponent. Thank you. My id is U7763307, Workshop 10

hey @Miranda-lrx, i think @blejkowski (do let me know if you still are) is currently working on this, do pick another component to work on, and i'll assign it to you!

u7505884 · 2023-10-15T06:06:16Z

Hello, may I work on the tests for the CommentTableComponent?

Miranda-lrx · 2023-10-15T06:13:51Z

Hi, I'd like to work on adding tests for the AddingQuestionPanelComponent. Thank you. My id is U7763307, Workshop 10

hey @Miranda-lrx, i think @blejkowski (do let me know if you still are) is currently working on this, do pick another component to work on, and i'll assign it to you!

I'd like to work on adding tests for the CommentTableComponent. Thank you.
My id is U7763307, Workshop 10

Miranda-lrx · 2023-10-15T06:19:03Z

Thank you, then may I choose the CommentTableModalComponent? Sorry I didn’t notice there were someone already selected the AddingQuestion task. I am willing to help with the issue! Josiiie ***@***.***>于2023年10月15日周日下午5:06写道：

…

Hello, may I work on the tests for the CommentTableComponent? — Reply to this email directly, view it on GitHub <#12588 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/BBSYQZ2NKEFSC7FFKBFLOYTX7N4OLANCNFSM6AAAAAA4WD25AE> . You are receiving this because you were mentioned.Message ID: ***@***.***>

ziqing26 · 2024-04-05T03:18:36Z

@kaioenzo @RohithSurya Hi feel free to submit a PR!

WayyGood · 2024-04-05T19:20:58Z

Hi! Got started on the coverage. Is there any special permission i need in order to submit a PR?

rushithapenikalapati · 2024-04-08T16:53:16Z

hi @rushithapenikalapati, @skylar-mo and @WayyGood, do go ahead and submit a PR!

When I push my changes upstream I'm getting an error that says "authentication failed"

@fetch

* Update chrome driver download link in e2e-testing.md (#12924) * [#12048] Add SQL configuration into build.properties and build-dev.properties (#12917) * Add production config * Remove forgotten host and password * Fix lint --------- Co-authored-by: Zhang Ziqing <[email protected]> * [#12048] Add SQL description for postgres config (#12931) * Add production config * Remove forgotten host and password * Fix lint * Address changes, include production_user * Linting * [#12588] Improve test code coverage of core components - ToastComponent (#12916) * add test cases * add test case for isTemplate() --------- Co-authored-by: Cedric Ong <[email protected]> Co-authored-by: Dominic Lim <[email protected]> * [#12588] Add unit tests to question edit answer form (#12935) * add unit tests to constsum-options-question-edit-answer-form * add unit tests to constsum-options-question-edit-answer-form --------- Co-authored-by: Zhang Ziqing <[email protected]> * add delay to task queuer for indexing account request (#12936) Co-authored-by: Nicolas <[email protected]> * Make account req data migration script rerunnable (#12932) * [#12048] Relax read notif verification for migration verification script (#12937) * Fix account requests with wrong field during seed * Relax account attributes verification * Fix lint errors * Fix order of account request variables * [#12920] Create script to migrate noSQL test data to SQL schema format (#12922) * Add classes to migrate test json data * Add toposort script * Add function to remove foreign key data * Cleanup * WIP * Simplify keys for students and instructors * Fix lint issues * Output SQL JSON in same folder as JSON * Change output file name * Fix bug: wrong jsonkey used * Fix lint error * Make section and team name unique * Set read notification key to be unique * Delete python file * [#12588] Improve test code coverage of core components - ViewResultsPanelComponent (#12918) * add test cases to ViewResultsPanelComponent * fix lint errors --------- Co-authored-by: Dominic Lim <[email protected]> Co-authored-by: Zhang Ziqing <[email protected]> * fix resetAccountAction (#12934) Co-authored-by: Zhang Ziqing <[email protected]> * [#12048] Migrate Feedback Rank Option E2E test (#12902) * Initial commit * Fix lint * Follow convention and add test * Change file path * Fix requested changes * Fixed testcases * Fix lint * Add deepcopy * Fixed e2e test --------- Co-authored-by: Wei Qing <[email protected]> Co-authored-by: Cedric Ong <[email protected]> * [#12048] Migrate FeedbackMcqQuestionE2ETest (#12820) * Migrate MCQ E2E * Fix lint * Fix lint * Update xml --------- Co-authored-by: Cedric Ong <[email protected]> * [#12048] Remove unnecessary loading of datastore entities in InstructorNotificationsPageE2ETest (#12911) * migrate instructor notif e2e --------- Co-authored-by: Cedric Ong <[email protected]> * [#12048] Migrate InstructorCourseDetailsPageE2ETest (#12908) * Add teammates.e2e.cases.sql.InstructorCourseDetailsPageE2ETest * Remove data properly to prevent clashes * Add SQL data bundle * Verify loaded details * Use email address when getting a student row * Check student links * Verify the sending of invites * Verify the reminding of all students to join * Remove SQL data properly to prevent clashes * Verify the downloading of the student list * Implement helper methods for Student * Add BaseTestCaseWithSqlDatabaseAccess::verifyAbsentInDatabase * Add to testng-e2e-sql.xml * Verify the deleting of students * Verify the deleting of all the students * Fix lint * Remove duplicate equality check for students * [#12588] add unit tests for question submission form (#12897) Co-authored-by: Zhang Ziqing <[email protected]> * Update developers.json (#12958) * Merge pull request #12960 from TEAMMATES/master (#12961) * [#12048] Fix account request indexing (#12967) * Add isTransactionNeeded method to Action * Remove delay from taskqueuer * Change CreateAccountRequest to handle own transactions * configure agroal connection pool (#12971) * [#12048] Configure connection pool using hikari (#12978) * Configure hikari * Remove spacing * Lint * [#12048] Update liquibase configuration (#12930) * Update gradle config * Update liquibase config for v9 * Turn off table generate for prod * Update of changelog file * Add configuration for generating changelog * Add schema migration docs --------- Co-authored-by: FergusMok <[email protected]> * [#12048] Migrate AccountRequestsLogicTest (#12780) * Migrate test cases for AccountRequestsLogic * Remove test case * Split test cases * [#12048] Migrate AdminSearchPageE2ETest SQL (#12811) * test e2e changes * fix: reduce e2e test json file size * fix student key * fix course key * fix instructor keys * fix filepath * fix e2e test * remove extra data from bundle * Add correct removal logic to avoid constraint violation * Fix e2e tests and lint fix reset google id test fix e2e tests fix e2e tests fix tests remove double click fix unknown symbol add toast check change toast verification message remove toast check * fix: add null check * move admin search page e2e test to sql cases * Rename AdminSearchPageE2ETest_SQLEntities.json to AdminSearchPageE2ETest_SqlEntities.json * fix failing test * fix: remove extra null check * fix: add test to e2e sql xml file * fix function call * remove unnecessary changes * create new file for sql entities * revert unnecessary changes * remove trailing whitespace * add teardown for account requests --------- Co-authored-by: Cedric Ong <[email protected]> * [#12995] Create documentation for unit tests (#12996) * Create documentation for unit tests * Update docs/unit-testing.md Co-authored-by: Zhang Ziqing <[email protected]> * Update docs/unit-testing.md Co-authored-by: Zhang Ziqing <[email protected]> --------- Co-authored-by: Zhang Ziqing <[email protected]> * [#12048] Remove feedbackSession attributes @fetch annotation (#12992) * Remove feedbackSession @fetch annotation * [#12048] create skeleton for sql LNP tests (#12994) * create skelton for sql LNP tests * allow lnp test to access sql storage and ensure sql lnp tests are independant of each other --------- Co-authored-by: Zhang Ziqing <[email protected]> * [#12048] Migrate FeedbackNumScaleQuestionE2ETest (#12940) * Migrate num scale e2e * Fix team id * Fix bugs * Add v9.0.0 tag to liquibase changelog (#13005) * sort courses by id before comparison (#13003) Co-authored-by: Dominic Lim <[email protected]> --------- Co-authored-by: Nada Ayesh <[email protected]> Co-authored-by: FergusMok <[email protected]> Co-authored-by: Maureen Chang <[email protected]> Co-authored-by: Cedric Ong <[email protected]> Co-authored-by: Dominic Lim <[email protected]> Co-authored-by: Nicolas <[email protected]> Co-authored-by: Ching Ming Yuan <[email protected]> Co-authored-by: Wei Qing <[email protected]> Co-authored-by: DS <[email protected]> Co-authored-by: Jay Aljelo Ting <[email protected]> Co-authored-by: Xenos F <[email protected]> Co-authored-by: domoberzin <[email protected]> Co-authored-by: Marques Tye Jia Jun <[email protected]>

@fetch

* Enable CI on account request form branch * [#11878] Remove AccountRequest unique constraint (#12899) * Remove AccountRequest unique constraint * Remove EntityAlreadyExistsException from the throws clause * Remove unused import of EntityAlreadyExistsException * Fix failing checks * Remove EntityAlreadyExistsException in dependents * Remove assertion that is now incorrect * Remove mysterious trailing whitespaces that appeared out of nowhere * Remove parts in E2E test that are no longer relevant * Remove unused import * Improve clarity of test case Co-authored-by: EuniceSim142 <[email protected]> --------- Co-authored-by: EuniceSim142 <[email protected]> * [#11878] Add status and comments to AccountRequest (#12898) * Add AccountRequestStatus * Add AccountRequest status attribute * Add status to AccountRequest constructor * Add AccountRequest comments attribute * Add comments to AccountRequest constructor * Wrap lines * Remove mysterious unnecessary imports that appeared out of nowhere * Use non-null placeholder * Use literal placeholder * [#11878] Add new account request alert email for admins (#12926) * Add admin alert email * Add email type * Add subject * Add email content * Indicate that action is needed in the email subject * [#11878] Add GetAllPendingAccountRequests API (#12927) * add endpoint * remove 'all' in class and method names * fix checkstyle * add it test * fix checkstyle * fix checkstyle * fix failing test * update endpoint url * update it tests * fix linting * update param name * update request param condition * [#11878] Modify CreateAccountRequestAction (#12913) * Add AccountCreateRequest instructorComments attribute * Add new AccountRequestData attributes * Remove check for registered instructor * Remove sending of registration email * Use AccountCreateRequest comments * Change output of CreateAccountRequestAction to AccountRequestData * Add CreateAccountRequestActionIT * Test execute with null arguments * Test execute with valid requests * Test execute on invalid arguments * Allow anybody to create an account request * Fix architecture test * Fix test * Update tests to verify search indexing * [#11878] Upgrade instructor request form UI (#12929) * Add confirmation prompt * Remove old form iframe * Improve declaration view spacing * Edit page heading phrasing for clarity * Create request form * Add validation messages * Fix form validation * Set up form submission confirmation * Create submission acknowledgement view * Fix URL checking regex * Fix initial state * Display placeholder when optional field is empty * Fix code style * Edit comment for clarity * Fix institution and country combination Co-authored-by: Jay Aljelo Ting <[email protected]> * Fix naming * Remove hard line break * Add explanatory comment for regex * Remove newline * Add newlines at end of file * Clear styles file * Re-add styles file * Include test * Add test cases for requestSubmissionEvent * Improve test case readability * Edit test case name for clarity * Add snapshot tests * Revert "Add snapshot tests" This reverts commit ec7395d. * Fix lint errors * Rename methods to be clearer * Disable submit button when not ready to submit --------- Co-authored-by: Jay Aljelo Ting <[email protected]> * [#11878] Update Admin Home Page UI for ARF (#12933) * create component for account request table * cherry pick admin home page changes * remove testing code * fix lint and css issues * fix admin home page snaps * update admin home snaps * remove edit approve and reject components * modify css * delete edit and reject modal components * revert spec file changes * integrate new types * fix lint * use enum for status * fix lint * fix css lint * fix lint * fix lint * use enum and remove infinite scroll * remove approve account request code * remove extra div * fix url * modify comments * revert extra formatting * remove plural form and use date pipe * fix naming * fix spec file and update institute formatting * fix lint * combine institute and country columns * [#11878] Admin Search UI Update for ARF (#12945) * update admin search page to use acc req component * fix selector for e2e test * fix spec files and imports * update e2e selector * fix column numbers * [#11878] Add methods to get an account request by ID (#12953) * Add facade logic method to get an account request by ID * Add storage method to get an account request by ID * Add logic method to get an account request by ID * [#11878] Add snapshot tests for instructor request form UI (#12942) * Add snapshot tests * Change double quotes to single quotes * [#11878] Foundation for getting by ID in account request endpoints (#12957) * Add account request ID query parameter * Add account request ID to related front-end types * Get account requests by ID in storage update method (#12955) * Get an account request by ID in SQL injection tests (#12956) * [#11878] Create instructor request acknowledgement email (#12944) * Create instructor request acknowledgement email * Add tests for acknowledgement email * Fix test cases * Fix comments in expected email * Use config support email value in email template * Fix email recipient * Fix test expected emails * Remove trailing space * Use placeholder for support email * Sanitize acknowledgement email * Set acknowledgement email to bcc support * [#11878] Merge master into account-request-form (#12972) * Update chrome driver download link in e2e-testing.md (#12924) * [#12048] Add SQL configuration into build.properties and build-dev.properties (#12917) * Add production config * Remove forgotten host and password * Fix lint --------- Co-authored-by: Zhang Ziqing <[email protected]> * [#12048] Add SQL description for postgres config (#12931) * Add production config * Remove forgotten host and password * Fix lint * Address changes, include production_user * Linting * [#12588] Improve test code coverage of core components - ToastComponent (#12916) * add test cases * add test case for isTemplate() --------- Co-authored-by: Cedric Ong <[email protected]> Co-authored-by: Dominic Lim <[email protected]> * [#12588] Add unit tests to question edit answer form (#12935) * add unit tests to constsum-options-question-edit-answer-form * add unit tests to constsum-options-question-edit-answer-form --------- Co-authored-by: Zhang Ziqing <[email protected]> * add delay to task queuer for indexing account request (#12936) Co-authored-by: Nicolas <[email protected]> * Make account req data migration script rerunnable (#12932) * [#12048] Relax read notif verification for migration verification script (#12937) * Fix account requests with wrong field during seed * Relax account attributes verification * Fix lint errors * Fix order of account request variables * [#12920] Create script to migrate noSQL test data to SQL schema format (#12922) * Add classes to migrate test json data * Add toposort script * Add function to remove foreign key data * Cleanup * WIP * Simplify keys for students and instructors * Fix lint issues * Output SQL JSON in same folder as JSON * Change output file name * Fix bug: wrong jsonkey used * Fix lint error * Make section and team name unique * Set read notification key to be unique * Delete python file * [#12588] Improve test code coverage of core components - ViewResultsPanelComponent (#12918) * add test cases to ViewResultsPanelComponent * fix lint errors --------- Co-authored-by: Dominic Lim <[email protected]> Co-authored-by: Zhang Ziqing <[email protected]> * fix resetAccountAction (#12934) Co-authored-by: Zhang Ziqing <[email protected]> * [#12048] Migrate Feedback Rank Option E2E test (#12902) * Initial commit * Fix lint * Follow convention and add test * Change file path * Fix requested changes * Fixed testcases * Fix lint * Add deepcopy * Fixed e2e test --------- Co-authored-by: Wei Qing <[email protected]> Co-authored-by: Cedric Ong <[email protected]> * [#12048] Migrate FeedbackMcqQuestionE2ETest (#12820) * Migrate MCQ E2E * Fix lint * Fix lint * Update xml --------- Co-authored-by: Cedric Ong <[email protected]> * [#12048] Remove unnecessary loading of datastore entities in InstructorNotificationsPageE2ETest (#12911) * migrate instructor notif e2e --------- Co-authored-by: Cedric Ong <[email protected]> * [#12048] Migrate InstructorCourseDetailsPageE2ETest (#12908) * Add teammates.e2e.cases.sql.InstructorCourseDetailsPageE2ETest * Remove data properly to prevent clashes * Add SQL data bundle * Verify loaded details * Use email address when getting a student row * Check student links * Verify the sending of invites * Verify the reminding of all students to join * Remove SQL data properly to prevent clashes * Verify the downloading of the student list * Implement helper methods for Student * Add BaseTestCaseWithSqlDatabaseAccess::verifyAbsentInDatabase * Add to testng-e2e-sql.xml * Verify the deleting of students * Verify the deleting of all the students * Fix lint * Remove duplicate equality check for students * [#12588] add unit tests for question submission form (#12897) Co-authored-by: Zhang Ziqing <[email protected]> * Update developers.json (#12958) * Merge pull request #12960 from TEAMMATES/master (#12961) * [#12048] Fix account request indexing (#12967) * Add isTransactionNeeded method to Action * Remove delay from taskqueuer * Change CreateAccountRequest to handle own transactions * configure agroal connection pool (#12971) * Fix comment style for merge * Remove unnecessary check for account request --------- Co-authored-by: Nada Ayesh <[email protected]> Co-authored-by: FergusMok <[email protected]> Co-authored-by: Maureen Chang <[email protected]> Co-authored-by: Cedric Ong <[email protected]> Co-authored-by: Dominic Lim <[email protected]> Co-authored-by: Nicolas <[email protected]> Co-authored-by: Ching Ming Yuan <[email protected]> Co-authored-by: Wei Qing <[email protected]> Co-authored-by: DS <[email protected]> Co-authored-by: Jay Aljelo Ting <[email protected]> * [#11878] Change institute length limit (#12974) * Change institute name max length to 128 * Edit test case for new length limit * [#11878] Update SearchAccountRequests endpoint (#12950) * update search document and create-core script * update it * modify relevant classes to use id instead of (email, institute) * remove duplicate method * fix it tests * fix failing tests * remove unnecessary comment * [#11878] Integrate instructor request form with API (#12943) * Integrate instructor request form FE with API * Remove redundant statement * Move URL regex const to backend const file * Fix import path * Move URL regex to FieldValidator * Add validators to match backend fields * Add error message box * Change submit button display when loading * Combine final action into subscribe * Add max length validators for institution and country * Fix lint errors * Add test cases to test submission * Add specific error messages for form validation * Remove home page URL field * Fix lint errors * Remove url regex from test * Update snap * Clean up test code * Remove comment about home page URL * Change canSubmit check to getter * Fix form submit button not re-enabling on error * Add name pattern validator to front-end * Fix snapshot * [#11878] Create Update Account Request Action (#12982) * create update action and IT * update javadocs * update tests * add more tests * simplify logic * remove unused string * fix test * allow null comments * add more tests * use EntityNotFoundException * cleanup after create account requests test * remove unncessary check * [#11878] Fix Account Request Update Search Indexing (#12984) * update account request indexing * add methods to test access control * refactoring for transactions * [#11878] Add Edit and Approve Account Requests functionality (#12975) * add edit and approve functionality * remove rejection code * fix snap * integrate endpoint * disable approve button for approved requests * use comments instead of comment * use searchString instead of searchQuery * fix snap * [#11878] Add AccountRequest Rejection email generator. (#12987) * add rejection-email template and email generator method * add javadoc for email generator method * add test * fix test method names * fix test method name 2 * fix lint * add bcc for rejection email * [#11878] Create reject account request endpoint (#12985) * Create account request rejection endpoint * Add validation * Add check for already rejected request when sending email * Add integration test cases * Set request method to post * Fix lint errors * Update tests list * Update validation check * Add test for validation * Fix lint errors * Fix validation comparison * Fix error message test * Add email sending * Update test cases * Refactor reason check code for clarity * Remove unused modal (#12998) * use transactions for reject account request action (#13001) * [#11878] Create Rejection Modal for Account Requests (#12989) * Create rejection modal * fix lint and tests * fix placeholders and lint * remove title * integrate api * check undefined title and body * fix trailing white spaces * fix whitespace * change error message * re-add account request table on home page * replace support email * [#11878] Update DeleteAccountRequest to reference by ID (#12997) * Update to delete by id * fix lint * fix lint * fix frontend lint * [#11878] Update ResetAccountRequest to reference by ID (#13002) * Update reset to reference by id * fix comments * [#11878] Add Error Message for Approving Existing Account (#13004) * add error message for duplicate account request * add tests * [#11878] Get account request by uuid (#13007) * change GetAccountRequestAction to get by id * fix tests * remove unncessary todo * [#11878] Handle Duplicate Approved Account Requests (#13009) * [#11878] Merge master into feature (#13011) * Update chrome driver download link in e2e-testing.md (#12924) * [#12048] Add SQL configuration into build.properties and build-dev.properties (#12917) * Add production config * Remove forgotten host and password * Fix lint --------- Co-authored-by: Zhang Ziqing <[email protected]> * [#12048] Add SQL description for postgres config (#12931) * Add production config * Remove forgotten host and password * Fix lint * Address changes, include production_user * Linting * [#12588] Improve test code coverage of core components - ToastComponent (#12916) * add test cases * add test case for isTemplate() --------- Co-authored-by: Cedric Ong <[email protected]> Co-authored-by: Dominic Lim <[email protected]> * [#12588] Add unit tests to question edit answer form (#12935) * add unit tests to constsum-options-question-edit-answer-form * add unit tests to constsum-options-question-edit-answer-form --------- Co-authored-by: Zhang Ziqing <[email protected]> * add delay to task queuer for indexing account request (#12936) Co-authored-by: Nicolas <[email protected]> * Make account req data migration script rerunnable (#12932) * [#12048] Relax read notif verification for migration verification script (#12937) * Fix account requests with wrong field during seed * Relax account attributes verification * Fix lint errors * Fix order of account request variables * [#12920] Create script to migrate noSQL test data to SQL schema format (#12922) * Add classes to migrate test json data * Add toposort script * Add function to remove foreign key data * Cleanup * WIP * Simplify keys for students and instructors * Fix lint issues * Output SQL JSON in same folder as JSON * Change output file name * Fix bug: wrong jsonkey used * Fix lint error * Make section and team name unique * Set read notification key to be unique * Delete python file * [#12588] Improve test code coverage of core components - ViewResultsPanelComponent (#12918) * add test cases to ViewResultsPanelComponent * fix lint errors --------- Co-authored-by: Dominic Lim <[email protected]> Co-authored-by: Zhang Ziqing <[email protected]> * fix resetAccountAction (#12934) Co-authored-by: Zhang Ziqing <[email protected]> * [#12048] Migrate Feedback Rank Option E2E test (#12902) * Initial commit * Fix lint * Follow convention and add test * Change file path * Fix requested changes * Fixed testcases * Fix lint * Add deepcopy * Fixed e2e test --------- Co-authored-by: Wei Qing <[email protected]> Co-authored-by: Cedric Ong <[email protected]> * [#12048] Migrate FeedbackMcqQuestionE2ETest (#12820) * Migrate MCQ E2E * Fix lint * Fix lint * Update xml --------- Co-authored-by: Cedric Ong <[email protected]> * [#12048] Remove unnecessary loading of datastore entities in InstructorNotificationsPageE2ETest (#12911) * migrate instructor notif e2e --------- Co-authored-by: Cedric Ong <[email protected]> * [#12048] Migrate InstructorCourseDetailsPageE2ETest (#12908) * Add teammates.e2e.cases.sql.InstructorCourseDetailsPageE2ETest * Remove data properly to prevent clashes * Add SQL data bundle * Verify loaded details * Use email address when getting a student row * Check student links * Verify the sending of invites * Verify the reminding of all students to join * Remove SQL data properly to prevent clashes * Verify the downloading of the student list * Implement helper methods for Student * Add BaseTestCaseWithSqlDatabaseAccess::verifyAbsentInDatabase * Add to testng-e2e-sql.xml * Verify the deleting of students * Verify the deleting of all the students * Fix lint * Remove duplicate equality check for students * [#12588] add unit tests for question submission form (#12897) Co-authored-by: Zhang Ziqing <[email protected]> * Update developers.json (#12958) * Merge pull request #12960 from TEAMMATES/master (#12961) * [#12048] Fix account request indexing (#12967) * Add isTransactionNeeded method to Action * Remove delay from taskqueuer * Change CreateAccountRequest to handle own transactions * configure agroal connection pool (#12971) * [#12048] Configure connection pool using hikari (#12978) * Configure hikari * Remove spacing * Lint * [#12048] Update liquibase configuration (#12930) * Update gradle config * Update liquibase config for v9 * Turn off table generate for prod * Update of changelog file * Add configuration for generating changelog * Add schema migration docs --------- Co-authored-by: FergusMok <[email protected]> * [#12048] Migrate AccountRequestsLogicTest (#12780) * Migrate test cases for AccountRequestsLogic * Remove test case * Split test cases * [#12048] Migrate AdminSearchPageE2ETest SQL (#12811) * test e2e changes * fix: reduce e2e test json file size * fix student key * fix course key * fix instructor keys * fix filepath * fix e2e test * remove extra data from bundle * Add correct removal logic to avoid constraint violation * Fix e2e tests and lint fix reset google id test fix e2e tests fix e2e tests fix tests remove double click fix unknown symbol add toast check change toast verification message remove toast check * fix: add null check * move admin search page e2e test to sql cases * Rename AdminSearchPageE2ETest_SQLEntities.json to AdminSearchPageE2ETest_SqlEntities.json * fix failing test * fix: remove extra null check * fix: add test to e2e sql xml file * fix function call * remove unnecessary changes * create new file for sql entities * revert unnecessary changes * remove trailing whitespace * add teardown for account requests --------- Co-authored-by: Cedric Ong <[email protected]> * [#12995] Create documentation for unit tests (#12996) * Create documentation for unit tests * Update docs/unit-testing.md Co-authored-by: Zhang Ziqing <[email protected]> * Update docs/unit-testing.md Co-authored-by: Zhang Ziqing <[email protected]> --------- Co-authored-by: Zhang Ziqing <[email protected]> * [#12048] Remove feedbackSession attributes @fetch annotation (#12992) * Remove feedbackSession @fetch annotation * [#12048] create skeleton for sql LNP tests (#12994) * create skelton for sql LNP tests * allow lnp test to access sql storage and ensure sql lnp tests are independant of each other --------- Co-authored-by: Zhang Ziqing <[email protected]> * [#12048] Migrate FeedbackNumScaleQuestionE2ETest (#12940) * Migrate num scale e2e * Fix team id * Fix bugs * Add v9.0.0 tag to liquibase changelog (#13005) * sort courses by id before comparison (#13003) Co-authored-by: Dominic Lim <[email protected]> --------- Co-authored-by: Nada Ayesh <[email protected]> Co-authored-by: FergusMok <[email protected]> Co-authored-by: Maureen Chang <[email protected]> Co-authored-by: Cedric Ong <[email protected]> Co-authored-by: Dominic Lim <[email protected]> Co-authored-by: Nicolas <[email protected]> Co-authored-by: Ching Ming Yuan <[email protected]> Co-authored-by: Wei Qing <[email protected]> Co-authored-by: DS <[email protected]> Co-authored-by: Jay Aljelo Ting <[email protected]> Co-authored-by: Xenos F <[email protected]> Co-authored-by: domoberzin <[email protected]> Co-authored-by: Marques Tye Jia Jun <[email protected]> * [#11878] Add tests for Account Request Table (#12977) * add component tests for account request table * modify tests * remove old tests * remove comment * remove unnecessary code * add tests * update disabled criteria * remove extra builders and update snaps * [#11878] Reference account requests by ID in tests (#13017) * Reference by ID in GetCourseJoinStatusActionIT * Reference by ID in AccountRequestsDbIT * Reference by ID in AccountRequestsLogicIT * Reference by ID in CreateAccountActionIT * Reference by ID in BaseTestCaseWithSqlDatabaseAccess * Remove now irrelevant reference by email address and institute * [#11878] Fix Approval Email Bug (#13027) * [#11878] Fix reject email content (#13029) * [#11878] Add Toasts (#13028) * add toasts * lint * [#11878] Remove mention of home page URL from confirmation email (#13030) * fix highlighting and null statuses (#13031) * [#11878] Fix Overlapping Tooltip (#13026) * fix tooltip * re-add removed lines * update snap * update snaps * lint * suppress warning * lint * [#11878] Check if account request is not created by admin before sending email (#13032) * Add check if admin when creating account request * Trim down test case * [#11878] Add sort by created_at for getAllPendingRequests (#13038) * feat: add order-by * fix: fix failing test case * fix: add missing created-at field * fix: test case comment * [#11878] Request Page E2E (#13015) * Add E2E skeleton * Fix test and lint * Add verifyEmailSent * fix fe tests * Remove method from logic and db (#13044) * [#11878] Migrate AccountRequestsLogic unit tests (#13043) * Add get typical account request method * Migrate AccountRequestsLogicTest * Remove test for get by email address and institute --------- Co-authored-by: Jay Aljelo Ting <[email protected]> * [#11878] Add Admin E2E Tests (#13020) * add admin e2e tests * remove exception catching and update snaps * fix snaps * add comment verification * fix test * [#11878] Add SQLI tests (#13047) * Revert "Enable CI on account request form branch" (#13049) This reverts commit 186a97a. --------- Co-authored-by: Jay Ting <[email protected]> Co-authored-by: EuniceSim142 <[email protected]> Co-authored-by: Xenos F <[email protected]> Co-authored-by: domoberzin <[email protected]> Co-authored-by: Nada Ayesh <[email protected]> Co-authored-by: FergusMok <[email protected]> Co-authored-by: Maureen Chang <[email protected]> Co-authored-by: Cedric Ong <[email protected]> Co-authored-by: Dominic Lim <[email protected]> Co-authored-by: Nicolas <[email protected]> Co-authored-by: Ching Ming Yuan <[email protected]> Co-authored-by: Wei Qing <[email protected]> Co-authored-by: DS <[email protected]> Co-authored-by: Marques Tye Jia Jun <[email protected]>

justinsegawa · 2024-04-22T21:01:45Z

Hello, can I work on InstructorResponsesViewBase component for a class assignment?

cedricongjh · 2024-04-23T05:15:35Z

hi @justinsegawa, go ahead!

hberehan · 2024-04-25T02:56:28Z

Hello, may I start working on the SingleStatisticsComponent for a class assignment due the April 26th?

cedricongjh · 2024-04-26T02:01:03Z

hi @hberehan, go ahead

itstrueitstrueitsrealitsreal · 2024-06-18T03:27:34Z

Hi, @cedricongjh, can I work on the tests for AddingQuestionPanelComponent?

…13087) * test coverage improvements for InstructorResponsesViewBase * Lint fixes * Update per-question-view-responses.component.spec.ts * Fix for component tests --------- Co-authored-by: Wei Qing <[email protected]> Co-authored-by: domoberzin <[email protected]>

Selena-WZ · 2024-10-07T05:40:07Z

Hi, may I work on the VisibilityMessages?

Adeeth101 · 2024-10-08T05:13:11Z

@cedricongjh I am a student at the Australian National University working with a 4 person team. We are interested in working on open source projects like this one. We are all new to this repository and would love to contribute to this issue. I see that there are certain test cases that need coverage.

Can we be assigned to:
AddingQuestionPanelComponent
CommentTableComponent

Let me know if these are available. If not please advise on which tests still need coverage. Thank you.

SamGreenwood04 · 2024-10-08T12:05:15Z

Hi @cedricongjh, I am keen on working on the "LogsTableComponent" component. If it is still available, may I please be assigned to it? Thanks!

100% coverage on RichTextEditorComponent and improves coverage on SessionsTable

100% coverage on RichTextEditorComponent and 100% coverage in sessions-table.component.spec.ts

rafgiann · 2024-10-31T20:45:01Z

hello @cedricongjh can i work on the tests for LogsLine? thanks!

vilioupa · 2024-10-31T20:48:04Z

Hi @cedricongjh , can i work on the tests for the LogsHistogramComponent? Thanks!

tourkaelisa · 2024-10-31T20:50:04Z

Hi @cedricongjh i am intrested in working on "GroupedResponsesComponent" , can I? Thanks!!

sofiaapap · 2024-10-31T21:19:45Z

Hello @cedricongjh I would like to contribute on "RichTextEditorComponent" if that's possible. Thank you!

cedricongjh added help wanted Moderate difficulty, small localized change; suitable for novice contributors a-Testing Testing-related traits such as efficiency, robustness, coverage labels Sep 13, 2023

github-actions bot mentioned this issue Sep 13, 2023

[teammates/TEAMMATES] Issues Digest damithc/digest-test#3

Open

This was referenced Sep 20, 2023

[#12588] Add unit tests to comment-visibility-setting-pipe #12589

Merged

[#12588] Add unit tests to copy course modal component #12590

Merged

This was referenced Sep 25, 2023

[#12588] Add unit tests to datepicker component #12592

Merged

[#12588] Add unit tests to DatePickerFormatter #12593

Merged

jasonqiu212 added a commit that referenced this issue Sep 26, 2023

[#12588] Add unit tests to DatePickerFormatter (#12593)

70be153

* Add unit tests to datepickerformatter * Update wording for comment regarding date --------- Co-authored-by: Jason Qiu <[email protected]>

lunoos added a commit to lunoos/teammates that referenced this issue Oct 1, 2023

[TEAMMATES#12588] Added unit tests to CourseEditForm component

f3e9dae

lunoos added a commit to lunoos/teammates that referenced this issue Oct 1, 2023

[TEAMMATES#12588] Added unit tests to CourseEditForm component

9063ae3

lunoos mentioned this issue Oct 1, 2023

[#12588] Added unit tests to CourseEditForm component. #12597

Merged

HarryLu33 mentioned this issue Oct 8, 2023

error TS2307: Cannot find module '../../../types/api-output' or its corresponding type declarations. #12602

Closed

HarryLu33 mentioned this issue Oct 12, 2023

[#12588] Add unit test for every method in CommentEditFormComponent #12604

Merged

rushithapenikalapati mentioned this issue Apr 8, 2024

[#12588] Improve test code coverage of QuestionEditDetailsForm #12999

Closed

skylar-mo mentioned this issue Apr 8, 2024

[#12588] Improve test code coverage of core components #13000

Closed

WayyGood mentioned this issue Apr 17, 2024

[#12588] Improve test code coverage of core components #13050

Closed

justinsegawa mentioned this issue Apr 24, 2024

[#12588] Test coverage improvements for InstructorResponsesViewBase #13087

Merged

skylar-mo mentioned this issue Apr 24, 2024

Code Coverage Issue: struggling to pass all tests #13096

Closed

SamGreenwood04 mentioned this issue Oct 18, 2024

[#12588] Added unit tests to LogsTable component #13181

Open

damithc unpinned this issue Oct 20, 2024

Surge747 added a commit to Surge747/teammates that referenced this issue Oct 20, 2024

Part of Improve test code coverage of core components TEAMMATES#12588

fdf818c

100% coverage on RichTextEditorComponent and improves coverage on SessionsTable

Surge747 mentioned this issue Oct 20, 2024

[#12588] Part of Improve test code coverage of core components #13189

Open

Surge747 added a commit to Surge747/teammates that referenced this issue Oct 23, 2024

Part of Improve test code coverage of core components TEAMMATES#12588

c249c81

100% coverage on RichTextEditorComponent and 100% coverage in sessions-table.component.spec.ts

Surge747 added a commit to Surge747/teammates that referenced this issue Oct 23, 2024

Part of Improve test code coverage of core components TEAMMATES#12588

9c7235c

100% coverage on RichTextEditorComponent and 100% coverage in sessions-table.component.spec.ts

tyler-cyl mentioned this issue Oct 24, 2024

[#12588] Added unit tests to Visibility-Capability component #13190

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Improve test code coverage of core components #12588

Improve test code coverage of core components #12588

cedricongjh commented Sep 13, 2023 •

edited

Loading

dlimyy commented Sep 20, 2023

cedricongjh commented Sep 20, 2023

blejkowski commented Sep 21, 2023

cedricongjh commented Sep 21, 2023

lunoos commented Sep 22, 2023

HarryLu33 commented Oct 3, 2023

ThomasGreen123 commented Oct 4, 2023

kenneySiu commented Oct 4, 2023

JYH11 commented Oct 4, 2023

Astrid641 commented Oct 5, 2023

MatheuMa commented Oct 12, 2023

Miranda-lrx commented Oct 12, 2023 •

edited

Loading

u7488099 commented Oct 13, 2023

cedricongjh commented Oct 14, 2023 •

edited

Loading

u7505884 commented Oct 15, 2023

Miranda-lrx commented Oct 15, 2023

Miranda-lrx commented Oct 15, 2023 via email

ziqing26 commented Apr 5, 2024

WayyGood commented Apr 5, 2024

rushithapenikalapati commented Apr 8, 2024

justinsegawa commented Apr 22, 2024

cedricongjh commented Apr 23, 2024

hberehan commented Apr 25, 2024 •

edited

Loading

cedricongjh commented Apr 26, 2024

itstrueitstrueitsrealitsreal commented Jun 18, 2024

Selena-WZ commented Oct 7, 2024

Adeeth101 commented Oct 8, 2024

SamGreenwood04 commented Oct 8, 2024

rafgiann commented Oct 31, 2024

vilioupa commented Oct 31, 2024

tourkaelisa commented Oct 31, 2024

sofiaapap commented Oct 31, 2024

Improve test code coverage of core components #12588

Improve test code coverage of core components #12588

Comments

cedricongjh commented Sep 13, 2023 • edited Loading

dlimyy commented Sep 20, 2023

cedricongjh commented Sep 20, 2023

blejkowski commented Sep 21, 2023

cedricongjh commented Sep 21, 2023

lunoos commented Sep 22, 2023

HarryLu33 commented Oct 3, 2023

ThomasGreen123 commented Oct 4, 2023

kenneySiu commented Oct 4, 2023

JYH11 commented Oct 4, 2023

Astrid641 commented Oct 5, 2023

MatheuMa commented Oct 12, 2023

Miranda-lrx commented Oct 12, 2023 • edited Loading

u7488099 commented Oct 13, 2023

cedricongjh commented Oct 14, 2023 • edited Loading

u7505884 commented Oct 15, 2023

Miranda-lrx commented Oct 15, 2023

Miranda-lrx commented Oct 15, 2023 via email

ziqing26 commented Apr 5, 2024

WayyGood commented Apr 5, 2024

rushithapenikalapati commented Apr 8, 2024

justinsegawa commented Apr 22, 2024

cedricongjh commented Apr 23, 2024

hberehan commented Apr 25, 2024 • edited Loading

cedricongjh commented Apr 26, 2024

itstrueitstrueitsrealitsreal commented Jun 18, 2024

Selena-WZ commented Oct 7, 2024

Adeeth101 commented Oct 8, 2024

SamGreenwood04 commented Oct 8, 2024

rafgiann commented Oct 31, 2024

vilioupa commented Oct 31, 2024

tourkaelisa commented Oct 31, 2024

sofiaapap commented Oct 31, 2024

cedricongjh commented Sep 13, 2023 •

edited

Loading

Miranda-lrx commented Oct 12, 2023 •

edited

Loading

cedricongjh commented Oct 14, 2023 •

edited

Loading

hberehan commented Apr 25, 2024 •

edited

Loading