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

dturn jest tests #154

Closed
7 of 9 tasks
ddfridley opened this issue Jul 12, 2024 · 0 comments · Fixed by #187
Closed
7 of 9 tasks

dturn jest tests #154

ddfridley opened this issue Jul 12, 2024 · 0 comments · Fixed by #187
Assignees

Comments

@ddfridley
Copy link
Contributor

ddfridley commented Jul 12, 2024

Create jest tests for the dturn algorithm. These should be in app/dturn/tests

It's okay to create multiple files, or add tests to an existing test file, depending on what makes sense.
It's okay for multiple people to assign themselves to this task to work on different tests, in different files, but please coordinate on which tests you are working on so there is no redundancy.

It's great to generate other tests, this list can keep growing.

If there are questions about these, or something doesn't make sense please ask. That's why we are doing t
Where it says statements here, it's statements that are being added to the discussion, some tests will have to make and keep statement objects in order to do fulfill some of the tests

  • requires getDiscussionStatus #155 to use to test the state of a discussion after operations are preformed
  • Can initialize an empty discussion
  • Can add 1 statement to a discussion
  • Can not get statements from a discussion that doesn't have enough statements
  • Can add 10 statements to a discussion, with group size of 10
  • Can add 10 statements to a discussion and then get 9 (group_size-1) statements from the discussion
  • Can put groupings of 2 statements in one group, and 3 statements in the other
  • Can rank 2 statements as most important
  • more tests go here
@Chenxuan-Liu Chenxuan-Liu self-assigned this Jul 22, 2024
@Chenxuan-Liu Chenxuan-Liu moved this from Todo to In Progress in EnCiv Cross Repo Project Management Jul 22, 2024
@ddfridley ddfridley moved this from In Progress to Ready for Review in EnCiv Cross Repo Project Management Aug 7, 2024
@ddfridley ddfridley linked a pull request Aug 12, 2024 that will close this issue
@ddfridley ddfridley moved this from Ready for Review to In Progress in EnCiv Cross Repo Project Management Aug 20, 2024
@ddfridley ddfridley moved this from Ready for Review to In Progress in EnCiv Cross Repo Project Management Aug 27, 2024
@github-project-automation github-project-automation bot moved this from Ready for Review to Done in EnCiv Cross Repo Project Management Sep 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging a pull request may close this issue.

2 participants