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 cases for activity feed backend #1976

Merged
merged 17 commits into from
Mar 21, 2024
Merged

test cases for activity feed backend #1976

merged 17 commits into from
Mar 21, 2024

Conversation

palakgupta2712
Copy link
Contributor

@palakgupta2712 palakgupta2712 commented Mar 18, 2024

Date: 19/04/2024

Developer Name: Palak Gupta


Issue Ticket Number

closes Real-Dev-Squad/website-dashboard#700
Test cases for #1970

Description

Documentation Updated?

  • Yes
  • No

Under Feature Flag

  • Yes
  • No

Database Changes

  • Yes
  • No

Breaking Changes

  • Yes
  • No

Development Tested?

  • Yes
  • No

Screenshots

Screenshot 1

Test Coverage

Integration tests:-

image

----------------------------------------|---------|----------|---------|---------|-----------------------------

File % Stmts % Branch % Funcs % Lines Uncovered Line #s
/controllers
logs.js 83.33 90 100 82.35 20-21,57-63,73-74
/models
logs.js 79.06 75.38 84.21 78.15 28-29,55,58-69,100-101,120-122,128-129,149-150,155-156,177,186-187,189-190,236-237

Unit tests:-

image

----------------------------------------|---------|----------|---------|---------|-----------------------------

File % Stmts % Branch % Funcs % Lines Uncovered Line #s
/models
logs.js 82.17 64.61 100 84.03 28-29,55,85-93,100-101,128-129,155-156,236-237

Note:-
Will be creating a issue ticket to improve upon the test coverage for logs.js. Didn't cover the existing functionality of logs completely in the tests.

Related Tickets

Additional Notes

@palakgupta2712 palakgupta2712 changed the base branch from develop to issue-700-backend March 18, 2024 20:11
@palakgupta2712 palakgupta2712 self-assigned this Mar 18, 2024
Base automatically changed from issue-700-backend to develop March 20, 2024 18:03
Copy link
Member

@Achintya-Chatterjee Achintya-Chatterjee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@palakgupta2712 first resolve merge conflicts

test/fixtures/logs/tasks.js Show resolved Hide resolved
test/integration/logs.test.js Show resolved Hide resolved
test/integration/logs.test.js Outdated Show resolved Hide resolved
test/integration/logs.test.js Outdated Show resolved Hide resolved
test/integration/logs.test.js Show resolved Hide resolved
test/integration/logs.test.js Show resolved Hide resolved
test/integration/logs.test.js Outdated Show resolved Hide resolved
test/integration/logs.test.js Outdated Show resolved Hide resolved
test/integration/logs.test.js Outdated Show resolved Hide resolved
test/unit/models/logs.test.js Outdated Show resolved Hide resolved
@iamitprakash iamitprakash merged commit 124a627 into develop Mar 21, 2024
3 checks passed
@iamitprakash iamitprakash deleted the issue-700-tests branch March 21, 2024 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create a twitter like view for status update and other activities.
5 participants