augustjanse / opencensus-java Goto Github PK

Something else that is extraordinary, at the discretion of the examiner.

Each group member writes at least two new tests that improve coverage, as evidenced by the coverage
tool. All tests have at least one meaningful assertion (other than “assert(true)” or equivalent).

You manually count the complexity of five functions (on paper or as comments).

You create a working ad-hoc coverage tool that at least measures coverage of normal branches (if,
while) for ten functions.

https://github.com/augustjanse/opencensus-java/blob/master/exporters/trace/zipkin/src/main/java/io/opencensus/exporter/trace/zipkin/ZipkinExporterHandler.java#L105

The purpose of each of the high-complexity methods is documented in detail w.r.t. the different outcomes resulting in branches in the code.

Each group member writes at least four new or enhanced unit tests (twice as many as for P).

You clearly describe how to refactor five complex functions into smaller functions.

You get a patch with new tests or a refactoring accepted. Note: the patch must be submitted by the
assignment deadline, but it may be accepted later; as long as it is accepted by the end of the course,
this point is counted. (Please notify us if this extra point is necessary for a P+.)

You carry out some of Task 3: you refactor at least two of the functions with high cyclomatic complexity
to reduce it by at least 35 % (for example, by splitting the functions).

You identify ten functions/methods with high complexity, and document the purpose of them, and
why the complexity should be high (or not).

https://github.com/augustjanse/opencensus-java/blob/master/impl_core/src/main/java/io/opencensus/implcore/trace/propagation/BinaryFormatImpl.java#L109

Compare report to report template when done to see that we didn't miss anything that was in the template but not grading criteria, such as #17. Unclear if mandatory or not.

https://github.com/augustjanse/opencensus-java/blob/master/api/src/main/java/io/opencensus/common/Duration.java#L53

You use your issue tracker and systematic commit messages to manage your project.

https://github.com/augustjanse/opencensus-java/blob/master/api/src/main/java/io/opencensus/tags/TagContext.java#L63