Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test e2e #11

Closed
wants to merge 24 commits into from
Closed

test e2e #11

wants to merge 24 commits into from

Conversation

domoberzin
Copy link
Owner

Fixes #

Outline of Solution

mingyuanc and others added 24 commits March 25, 2024 20:48
Fix lint

Rerun test

Fix lint

Retry lint

[TEAMMATES#12048] Configure connection pool using hikari (TEAMMATES#12978)

* Configure hikari

* Remove spacing

* Lint

[TEAMMATES#12048] Update liquibase configuration (TEAMMATES#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]>

[TEAMMATES#12048] Migrate AccountRequestsLogicTest (TEAMMATES#12780)

* Migrate test cases for AccountRequestsLogic

* Remove test case

* Split test cases

[TEAMMATES#12048] Migrate AdminSearchPageE2ETest SQL (TEAMMATES#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]>

[TEAMMATES#12995] Create documentation for unit tests (TEAMMATES#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]>

[TEAMMATES#12048] Remove feedbackSession attributes @fetch annotation (TEAMMATES#12992)

* Remove feedbackSession @fetch annotation

[TEAMMATES#12048] create skeleton for sql LNP tests (TEAMMATES#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]>

[TEAMMATES#12048] Migrate FeedbackNumScaleQuestionE2ETest (TEAMMATES#12940)

* Migrate num scale e2e

* Fix team id

* Fix bugs

Add v9.0.0 tag to liquibase changelog (TEAMMATES#13005)

sort courses by id before comparison (TEAMMATES#13003)

Co-authored-by: Dominic Lim <[email protected]>

Test

Test2

Test 3

Test 4

Remove buggy components
Copy link

Hi @domoberzin, thank you for your interest in contributing to TEAMMATES!
However, your PR does not appear to follow our contribution guidelines:

  • Title must start with the issue number the PR is fixing in square brackets, e.g. [#<issue-number>]
  • Description must reference the issue number the PR is fixing, e.g. Fixes #<issue-number> (or Part of #<issue-number> if the PR does not address the issue fully)

Please address the above before we proceed to review your PR.

@domoberzin domoberzin closed this Apr 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants