Code Coverage: Branches

[theschitz]

Code coverage is looking really good. But the testing of branches is falling behind so we should make an effort to try and raise that number a bit.

Low hanging fruits

• PermissionSet

  • src/PermissionSet/XmlPermissionSet.ts
  • src/PermissionSet/PermissionSetFunctions.ts

• Xliff

  • src/Xliff/XLIFFDocument.ts

• Template (preview)

  • src/Template/TemplateTypes.ts
  • src/Template/TemplateFunctions.ts

• Test branches in caches: XLIFFCache & SymbolReferenceCache #375

  • src/SymbolReference/SymbolReferenceCache.ts
  • src/Xliff/XLIFFCache.ts

• Improve testing of branches in CSV functions #373

  • src/CSV/ImportXliffCSV.ts
  • src/CSV/ExportXliffCSV.ts
  • src/CSV/CSV.ts

Not so low hanging fruit

The ALObject and Documentation parts might require special consideration

• ALObject

  • src/ALObject/ALPagePart.ts

• Documentation
  -src/Documentation.ts.

[jwikman]

Good idea!

But I believe that ALObject & Documentation should be quite easy to fix, since you've added the documentation files test. We "only" need to create all different kinds of AL files and do a manual verification that the generated .md files are correct. Then your existing test will verify that we don't get any regressions.

That would at least be a way to get a very high coverage on the documentation parts. Coverage on objects and parsing will probably get high as well, but not completely verified since Documentation does not output everything...

[jwikman]

I'll take a closer look at the src/Documentation.ts part to see if I could improve that one.