Skip to content

Commit c1920da

Browse files
xtqqczzeadityapatwardhan
authored andcommitted
Update code coverage analysis document (PowerShell#8543)
1 parent 64fcddc commit c1920da

2 files changed

Lines changed: 25 additions & 51 deletions

File tree

.spelling

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -379,6 +379,7 @@ metadata
379379
miaromero
380380
microsoft
381381
microsoft.com
382+
microsoft.management.infrastructure.cimcmdlets
382383
microsoft.management.infrastructure.native
383384
microsoft.powershell.archive
384385
microsoft.powershell.commands.diagnostics
@@ -391,6 +392,7 @@ microsoft.powershell.coreclr.eventing
391392
microsoft.powershell.diagnostics
392393
microsoft.powershell.localaccounts
393394
microsoft.powershell.management
395+
microsoft.powershell.markdownrender
394396
microsoft.powershell.psreadline
395397
microsoft.powershell.security
396398
microsoft.powershell.utility
@@ -813,6 +815,8 @@ yurko7
813815
zhenggu
814816
- docs/debugging/README.md
815817
corehost
818+
- docs/testing-guidelines/CodeCoverageAnalysis.md
819+
de5f69c
816820
- docs/testing-guidelines/TestRoadmap.md
817821
_no_
818822
- docs/testing-guidelines/WritingPesterTests.md

docs/testing-guidelines/CodeCoverageAnalysis.md

Lines changed: 21 additions & 51 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
# Code coverage analysis for commit [2ae5d07](https://codecov.io/gh/PowerShell/PowerShell/tree/c7b959bd6e5356fbbd395f22ba0c6cba49f354f6/src)
1+
# Code coverage analysis for commit [de5f69c](https://codecov.io/gh/PowerShell/PowerShell/tree/de5f69cf942a85839c907f11a29cf9c09f9de8b4/src)
22

33
Code coverage runs are enabled on daily Windows builds for PowerShell Core 6.
44
The results of the latest build are available at: [CodeCov.io](https://codecov.io/gh/PowerShell/PowerShell)
@@ -8,41 +8,33 @@ The metrics used for selection of these hot spots were: # missing lines and like
88

99
## Coverage Status
1010

11-
The following table shows the status for the above commit, dated 06/18/2017
11+
The following table shows the status for the above commit, dated 2018-11-28
1212

1313
| Assembly | Hit % |
1414
| -------- |:-----:|
15-
| Microsoft.PowerShell.Commands.Diagnostics | 58.01% |
16-
| Microsoft.PowerShell.Commands.Management | 32.02% |
17-
| Microsoft.PowerShell.Commands.Utility | 67.55% |
18-
| Microsoft.PowerShell.ConsoleHost | 41.15% |
19-
| Microsoft.PowerShell.CoreCLR.AssemblyLoadContext | 97.65% |
20-
| Microsoft.PowerShell.CoreCLR.Eventing | 29.91% |
21-
| Microsoft.PowerShell.LocalAccounts | 86.35% |
22-
| Microsoft.PowerShell.PSReadLine | 10.18% |
23-
| Microsoft.PowerShell.Security | 44.44% |
24-
| Microsoft.WSMan.Management | 4.91% |
25-
| System.Management.Automation | 50.42% |
26-
| Microsoft.WSMan.Runtime/WSManSessionOption.cs | 100% |
27-
| powershell/Program.cs | 100% |
15+
| Microsoft.Management.Infrastructure.CimCmdlets | 48.18% |
16+
| Microsoft.PowerShell.Commands.Diagnostics | 47.58% |
17+
| Microsoft.PowerShell.Commands.Management | 61.06% |
18+
| Microsoft.PowerShell.Commands.Utility | 70.76% |
19+
| Microsoft.PowerShell.ConsoleHost | 46.39% |
20+
| Microsoft.PowerShell.CoreCLR.Eventing | 37.84% |
21+
| Microsoft.PowerShell.MarkdownRender | 70.68% |
22+
| Microsoft.PowerShell.Security | 49.36% |
23+
| Microsoft.WSMan.Management | 62.36% |
24+
| System.Management.Automation | 63.35% |
25+
| Microsoft.WSMan.Runtime/WSManSessionOption.cs | 100.00% |
26+
| powershell/Program.cs | 100.00% |
2827

2928
## Hot Spots with missing coverage
3029

3130
### Microsoft.PowerShell.Commands.Management
3231

33-
- [ ] CDXML cmdlet coverage. It is 0% as CDXML is broken due to CoreCLR [issue](https://github.com/dotnet/corefx/issues/18877).
34-
- [ ] Add tests for *-Computer cmdlets [#4146](https://github.com/PowerShell/PowerShell/issues/4146)
35-
- [ ] Add tests for *-Service cmdlets [#4147](https://github.com/PowerShell/PowerShell/issues/4147)
3632
- [ ] Add tests for *-Item cmdlets. Especially for literal paths and error cases. [#4148](https://github.com/PowerShell/PowerShell/issues/4148)
37-
- [ ] Add tests for Get-Content -Tail. [#4150](https://github.com/PowerShell/PowerShell/issues/4150)
3833
- [ ] Lots of resource strings not covered. Will probably get covered when coverage is added for error cases. [#4148](https://github.com/PowerShell/PowerShell/issues/4148)
3934

4035
### Microsoft.PowerShell.Commands.Utility
4136

42-
- [ ] Add tests for Trace-Command. Especially Trace-Command -Expression [#4151](https://github.com/PowerShell/PowerShell/issues/4151)
43-
- [ ] Add tests for ConvertTo-XML serialization of PSObjects [#4152](https://github.com/PowerShell/PowerShell/issues/4152)
4437
- [ ] Add tests for Debug-Runspace [#4153](https://github.com/PowerShell/PowerShell/issues/4153)
45-
- [ ] Add tests for New-Object for ArgumentList, ComObject [#4154](https://github.com/PowerShell/PowerShell/issues/4154)
4638

4739
### Microsoft.PowerShell.ConsoleHost
4840

@@ -52,15 +44,11 @@ The following table shows the status for the above commit, dated 06/18/2017
5244

5345
- [ ] Add tests for ETW events. [#4156](https://github.com/PowerShell/PowerShell/issues/4156)
5446

55-
### Microsoft.PowerShell.PSReadLine
56-
57-
- [ ] We need tests from PSReadline repo or ignore coverage data for this module. (This will be filtered out.)
58-
5947
### Microsoft.PowerShell.Security
6048

61-
- [ ] Add tests for *-Acl cmdlets. [4157] (https://github.com/PowerShell/PowerShell/issues/4157)
62-
- [ ] Add tests for *-AuthenticodeSignature cmdlets. [4157] (https://github.com/PowerShell/PowerShell/issues/4157)
63-
- [ ] Add coverage to various utility methods under src/Microsoft.PowerShell.Security/security/Utils.cs [4157] (https://github.com/PowerShell/PowerShell/issues/4157)
49+
- [ ] Add tests for *-Acl cmdlets. [#4157](https://github.com/PowerShell/PowerShell/issues/4157)
50+
- [ ] Add tests for *-AuthenticodeSignature cmdlets. [#4157](https://github.com/PowerShell/PowerShell/issues/4157)
51+
- [ ] Add coverage to various utility methods under src/Microsoft.PowerShell.Security/security/Utils.cs [#4157](https://github.com/PowerShell/PowerShell/issues/4157)
6452

6553
### Microsoft.WSMan.Management
6654

@@ -72,24 +60,14 @@ The following table shows the status for the above commit, dated 06/18/2017
7260

7361
#### CoreCLR
7462

75-
- [ ] Lots of non-windows code can be ifdef'ed out. Issue #[3565](https://github.com/PowerShell/PowerShell/issues/3565)
76-
77-
#### CIMSupport
78-
79-
- [ ] Missing coverage possibly due to: CoreCLR [issue](https://github.com/dotnet/corefx/issues/18877).
80-
[4159](https://github.com/PowerShell/PowerShell/issues/4159)
63+
- [ ] Lots of non-windows code can be ifdef'ed out. [#3565](https://github.com/PowerShell/PowerShell/issues/3565)
8164

8265
#### Engine
8366

84-
- [ ] Add tests for COM. [#4154](https://github.com/PowerShell/PowerShell/issues/4154)
8567
- [ ] Add tests for Tab Completion of various types of input. [#4160](https://github.com/PowerShell/PowerShell/issues/4160)
86-
- [ ] Add tests for Import-Module / Get-Module over PSRP and CIMSession. [#4161](https://github.com/PowerShell/PowerShell/issues/4161)
87-
- [ ] Add tests for debugging PS Jobs.[#4153](https://github.com/PowerShell/PowerShell/issues/4153)
88-
- [ ] Add test for -is, -isnot, -contains, -notcontains and -like operators.[#4162](https://github.com/PowerShell/PowerShell/issues/4162)
89-
- [ ] Remove Snapin code from CommandDiscovery. Issue #[4118](https://github.com/PowerShell/PowerShell/issues/4118)
68+
- [ ] Add tests for debugging PS Jobs. [#4153](https://github.com/PowerShell/PowerShell/issues/4153)
69+
- [ ] Remove Snapin code from CommandDiscovery. [#4118](https://github.com/PowerShell/PowerShell/issues/4118)
9070
- [ ] Add tests SessionStateItem, SessionStateContainer error cases, dynamic parameters. Coverage possibly added by *-Item, *-ChildItem error case tests. [#4148](https://github.com/PowerShell/PowerShell/issues/4148)
91-
- [ ] Add tests for Get-Command -ShowCommandInfo [#4163](https://github.com/PowerShell/PowerShell/issues/4163)
92-
- [ ] Add tests for Proxy Commands [#4164](https://github.com/PowerShell/PowerShell/issues/4164)
9371
- [ ] Add more tests using PSCredential [#4165](https://github.com/PowerShell/PowerShell/issues/4165)
9472

9573
#### Remoting
@@ -101,17 +79,9 @@ The following table shows the status for the above commit, dated 06/18/2017
10179
- [ ] Add tests for Connect/Disconnect session [#4166](https://github.com/PowerShell/PowerShell/issues/4166)
10280
- [ ] Add more tests for Start-Job's various options [#4166](https://github.com/PowerShell/PowerShell/issues/4166)
10381

104-
#### HelpSystem
105-
106-
- [ ] Add tests for Alias help [#4167](https://github.com/PowerShell/PowerShell/issues/4167)
107-
- [ ] Add tests for Class help [#4167](https://github.com/PowerShell/PowerShell/issues/4167)
108-
- [ ] Add tests for SaveHelp [#4167](https://github.com/PowerShell/PowerShell/issues/4167)
109-
- [ ] Add tests for HelpProviderWithCache [#4167](https://github.com/PowerShell/PowerShell/issues/4167)
110-
- [ ] HelpProviderWithFullCache, potential dead code. [#4167](https://github.com/PowerShell/PowerShell/issues/4167)
111-
11282
#### Security
11383

114-
- [ ] Add more tests under various ExecutionPolicy modes. [4168](https://github.com/PowerShell/PowerShell/issues/4168)
84+
- [ ] Add more tests under various ExecutionPolicy modes. [#4168](https://github.com/PowerShell/PowerShell/issues/4168)
11585

11686
#### Utils
11787

0 commit comments

Comments
 (0)