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

Still need more test coverage #152

Open
jankapunkt opened this issue Feb 14, 2024 · 1 comment · May be fixed by #156
Open

Still need more test coverage #152

jankapunkt opened this issue Feb 14, 2024 · 1 comment · May be fixed by #156

Comments

@jankapunkt
Copy link
Member

Is your feature request related to a problem? Please describe.
We still do not catch all potential regressions as we still have insufficient coverage:

-------------------------|---------|----------|---------|---------|-----------------------------------------------------------------------------------------------------
File                     | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s                                                                                   
-------------------------|---------|----------|---------|---------|-----------------------------------------------------------------------------------------------------
All files                |    70.6 |    63.81 |    67.9 |   71.36 |                                                                                                     
 helpers                 |   63.63 |      100 |   33.33 |      75 |                                                                                                     
  reactNativeBindings.js |   63.63 |      100 |   33.33 |      75 | 14-16                                                                                               
 lib                     |   93.23 |       88 |   96.96 |   93.38 |                                                                                                     
  Random.js              |     100 |      100 |     100 |     100 |                                                                                                     
  ddp.js                 |   87.14 |    83.33 |      90 |    88.4 | 80-101                                                                                              
  mongo-id.js            |     100 |      100 |     100 |     100 |                                                                                                     
  queue.js               |     100 |      100 |     100 |     100 |                                                                                                     
  sha256.js              |   92.59 |       40 |     100 |      92 | 211-217                                                                                             
  socket.js              |     100 |      100 |     100 |     100 |                                                                                                     
  utils.js               |   84.61 |       88 |     100 |   84.61 | 59,104,110-111                                                                                      
 src                     |   63.84 |    59.15 |   60.44 |   64.44 |                                                                                                     
  Call.js                |     100 |       75 |     100 |     100 | 10                                                                                                  
  Collection.js          |   76.77 |       72 |   73.68 |   77.85 | 39,49-57,215,234-258,384,422,440-455                                                                
  Data.js                |   25.64 |    35.71 |    6.25 |   25.64 | 30-32,75-167,183-184                                                                                
  Meteor.js              |   19.55 |    16.03 |   21.21 |   20.34 | 28-38,71-85,114,121-126,146-163,168-190,194-208,214-225,231-255,262-276,282-286,292-298,308,325-460 
  Mongo.js               |       0 |        0 |       0 |       0 |                                                                                                     
  ReactiveDict.js        |     100 |      100 |     100 |     100 |                                                                                                     
  Tracker.js             |   91.58 |    81.63 |   94.44 |    92.7 | 58,78-79,99-100,125-127,181,250,437,538,560,646                                                     
 src/components          |    6.25 |        0 |       0 |    6.89 |                                                                                                     
  useTracker.js          |    8.33 |        0 |       0 |    9.09 | 24-60                                                                                               
  withTracker.js         |       0 |        0 |       0 |       0 | 23-33                                                                                               
-------------------------|---------|----------|---------|---------|-----------------------------------------------------------------------------------------------------

Describe the solution you'd like
Complete coverage between 90% and 100%

@jankapunkt jankapunkt linked a pull request Feb 15, 2024 that will close this issue
Copy link

Closing this issue due to no activity. Feel free to reopen.

@jankapunkt jankapunkt linked a pull request Apr 17, 2024 that will close this issue
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 a pull request may close this issue.

1 participant