On Thu, 11 Nov 2021 16:35:46 GMT, Pavel Rappo <pra...@openjdk.org> wrote:

> This change is mostly about tests. I say "mostly" because while increasing 
> code coverage, which was the primary goal of this exercise, I uncovered a few 
> non-critical bugs and fixed them in-place. The net effect of the change boils 
> down to these code coverage statistics.
> 
> before
> ------
> 
> %method       %block  %branch %line
> jdk.javadoc.internal.doclets.toolkit.taglets.SnippetTaglet    
> 75%(12/16)
> 87%(109/124)
> 88%(99/112)
> 85%(140/164)
> 
> #classes      %method %block  %branch %line
> jdk.javadoc.internal.doclets.toolkit.taglets.snippet  
> 70%(80/114)
> 76%(310/407)
> 65%(178/273)
> 81%(413/508)
> 
> after
> -----
> 
> %method       %block  %branch %line
> jdk.javadoc.internal.doclets.toolkit.taglets.SnippetTaglet    
> 100%(17/17)
> 95%(120/126)
> 93%(103/110)
> 97%(163/168)
> 
> %method       %block  %branch %line
> jdk.javadoc.internal.doclets.toolkit.taglets.snippet  
> 83%(94/112)
> 85%(348/405)
> 73%(202/273)
> 91%(463/505)

This pull request has now been integrated.

Changeset: 2ab43ec2
Author:    Pavel Rappo <pra...@openjdk.org>
URL:       
https://git.openjdk.java.net/jdk/commit/2ab43ec2428edaebfe9a7fb0e716ff7141a28de0
Stats:     1412 lines in 12 files changed: 1123 ins; 179 del; 110 mod

8273544: Increase test coverage for snippets

Reviewed-by: jjg

-------------

PR: https://git.openjdk.java.net/jdk/pull/6359

Reply via email to