openwhisk icon indicating copy to clipboard operation
openwhisk copied to clipboard

Add test cases that guarantee the main execution flows of FPCScheduler.

Open style95 opened this issue 3 years ago • 2 comments

Description

This is to guarantee the main execution flows with system tests.

Related issue and scope

  • [ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

  • [ ] API
  • [ ] Controller
  • [ ] Message Bus (e.g., Kafka)
  • [ ] Loadbalancer
  • [x] Scheduler
  • [ ] Invoker
  • [ ] Intrinsic actions (e.g., sequences, conductors)
  • [ ] Data stores (e.g., CouchDB)
  • [x] Tests
  • [ ] Deployment
  • [ ] CLI
  • [ ] General tooling
  • [ ] Documentation

Types of changes

  • [ ] Bug fix (generally a non-breaking change which closes an issue).
  • [x] Enhancement or new feature (adds new functionality).
  • [ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

  • [x] I signed an Apache CLA.
  • [x] I reviewed the style guides and followed the recommendations (Travis CI will check :).
  • [x] I added tests to cover my changes.
  • [ ] My changes require further changes to the documentation.
  • [ ] I updated the documentation where necessary.

style95 avatar Aug 08 '22 11:08 style95

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 69.39%. Comparing base (8af2092) to head (164da32).

:exclamation: Current head 164da32 differs from pull request most recent head b9386f6. Consider uploading reports for the commit b9386f6 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5308      +/-   ##
==========================================
- Coverage   76.78%   69.39%   -7.40%     
==========================================
  Files         241      241              
  Lines       14650    14650              
  Branches      644      644              
==========================================
- Hits        11249    10166    -1083     
- Misses       3401     4484    +1083     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov-commenter avatar Aug 08 '22 11:08 codecov-commenter

LGTM

bdoyle0182 avatar Feb 12 '23 01:02 bdoyle0182