tests for src/lib/resolvers/Subscription/messageSentToGroupChat.ts

[KrutikaBhatt]

What kind of change does this PR introduce?

Issue Number: #889

Fixes #889

Description:
Written tests for messageSentToGroupChat subscription by creating a mock test GroupChat Message.

Did you add tests for your changes? : Yes

Snapshots/Videos:
The coverage obtained is 100%

Other information
The subscription uses withFilter which has a filter function parameter that returns true if the user in the GroupChat is currentUser. The code block is the same just converted into a separate function for testing and used in the subscription

Have you read the contributing guide? Yes

[Cioppolo14]

@KrutikaBhatt Please refer to the PR_GUIDELINES.md document in the root folder of the repo. It tells you how to automatically close your issue when the PR is merged.

[KrutikaBhatt]

@KrutikaBhatt Please refer to the PR_GUIDELINES.md document in the root folder of the repo. It tells you how to automatically close your issue when the PR is merged.

Yes, sure! Will follow

[noman2002]

Why package-lock.json is removed ? @KrutikaBhatt

[KrutikaBhatt]

No, it shouldn't be removed; it might be edited by mistake. I will make the changes

[codecov]

Codecov Report

Merging #1095 (5681888) into develop (83f22af) will increase coverage by 0.31%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop    #1095      +/-   ##
===========================================
+ Coverage    94.70%   95.01%   +0.31%     
===========================================
  Files          165      165              
  Lines         9889     9891       +2     
  Branches       802      805       +3     
===========================================
+ Hits          9365     9398      +33     
+ Misses         523      492      -31     
  Partials         1        1              

Impacted Files	Coverage Δ
...c/resolvers/Subscription/messageSentToGroupChat.ts	100.00% <100.00%> (+100.00%)	⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[KrutikaBhatt]

The only branch not covered -

This change was introduced to avoid the 'object may be undefined' typescript error.

[KrutikaBhatt]

@noman2002, I have covered the remaining branch also. The code coverage is shown below -

[noman2002]

package-lock.json is still not restored. please restore.

[KrutikaBhatt]

package-lock.json is still not restored. please restore.

I just copied the package-lock.json from main file. I will just create a new pull request with the same tests. to avoid any problems with package-lock.json

[KrutikaBhatt]

I am closing this PR. The same tests are covered in new PR - #1113