diff options
| author | Jonathan Pryor <jonpryor@vt.edu> | 2013-05-29 19:53:22 +0400 |
|---|---|---|
| committer | Duncan Mak <duncanm@microsoft.com> | 2016-10-08 03:30:14 +0300 |
| commit | c862d9e0d6f88dbd27930fdcb8b828450d1933c7 (patch) | |
| tree | 9a9bead303e938538e47e4eb51f4c088f1535443 /mdoc/Test/en.expected.importecmadoc | |
| parent | 4365bcd902dfff245bfcd4386aea0e955423bcb4 (diff) | |
[mdoc] Fix unit tests.
3d3fe6af (among other commits...) "broke" the `mdoc` unit tests, as it
did two things:
1. It added a new codepath calling string.Format(), thus introducing
a new set of exceptions.
2. Due to an `mdoc` bug, (1) resulted in a stack overflow on OS X.
(1) demonstrates that the mdoc unit tests might be too brittle.
(2) was fixed in 04bf9ad.
The problem with 04bf9ad, though, is that it resulted in a gigantic
~212MB diff, because while the recursion was "fixed", we could still
process the same method multiple times, resulting in _lots_ of
duplication, e.g. System.String.get_Chars() would be repeated hundreds
of times for a given exception.
Improve the source reporting mechanism so that instead of a
List<MemberReference> we use a HashSet<MemberReference>, listing each
source location _once_ instead of repeating them.
The _formatting_ of the <exception/> element is also changed so that
each source location is on its own line, making it easier to read and
review the locations.
Diffstat (limited to 'mdoc/Test/en.expected.importecmadoc')
| -rw-r--r-- | mdoc/Test/en.expected.importecmadoc/System/Array.xml | 9 | ||||
| -rw-r--r-- | mdoc/Test/en.expected.importecmadoc/System/Environment.xml | 3 |
2 files changed, 8 insertions, 4 deletions
diff --git a/mdoc/Test/en.expected.importecmadoc/System/Array.xml b/mdoc/Test/en.expected.importecmadoc/System/Array.xml index b0826886..96aee607 100644 --- a/mdoc/Test/en.expected.importecmadoc/System/Array.xml +++ b/mdoc/Test/en.expected.importecmadoc/System/Array.xml @@ -131,7 +131,8 @@ and the second dimension indexed by 1, 2, and 3. </para> <summary>To be added.</summary> <returns>To be added.</returns> <remarks>To be added.</remarks> - <exception cref="T:System.NotImplementedException">To be added; from: <see cref="M:System.Array.AsReadOnly``1(``0[])" /></exception> + <exception cref="T:System.NotImplementedException">To be added; from: + <see cref="M:System.Array.AsReadOnly``1(``0[])" /></exception> </Docs> </Member> <Member MemberName="ConvertAll<T,U>"> @@ -168,7 +169,8 @@ and the second dimension indexed by 1, 2, and 3. </para> <remarks> <para>The <see cref="T:System.Converter<T,U>" /> is a delegate that converts an array element to the target type. The elements of <paramref name="array" /> are individually passed to this converter, and the converted elements are saved in the new array. The source array remains unchanged.</para> </remarks> - <exception cref="T:System.InvalidOperationException">To be added; from: <see cref="M:System.Array.ConvertAll``2(``0[],System.Converter{``0,``1})" /></exception> + <exception cref="T:System.InvalidOperationException">To be added; from: + <see cref="M:System.Array.ConvertAll``2(``0[],System.Converter{``0,``1})" /></exception> <exception cref="T:System.ArgumentNullException"> <paramref name="array" /> is <see langword="null" /> or <paramref name="converter" /> is <see langword="null" />.</exception> </Docs> @@ -196,7 +198,8 @@ and the second dimension indexed by 1, 2, and 3. </para> <param name="newSize">To be added.</param> <summary>To be added.</summary> <remarks>To be added.</remarks> - <exception cref="T:System.Exception">To be added; from: <see cref="M:System.Array.Resize``1(``0[]@,System.Int32)" /></exception> + <exception cref="T:System.Exception">To be added; from: + <see cref="M:System.Array.Resize``1(``0[]@,System.Int32)" /></exception> </Docs> </Member> </Members> diff --git a/mdoc/Test/en.expected.importecmadoc/System/Environment.xml b/mdoc/Test/en.expected.importecmadoc/System/Environment.xml index 5ce89496..f7a5f40c 100644 --- a/mdoc/Test/en.expected.importecmadoc/System/Environment.xml +++ b/mdoc/Test/en.expected.importecmadoc/System/Environment.xml @@ -78,7 +78,8 @@ property.</para> </block> </remarks> - <exception cref="T:System.NotSupportedException">To be added; from: <see cref="M:System.Environment.GetFolderPath(System.Environment.SpecialFolder)" /></exception> + <exception cref="T:System.NotSupportedException">To be added; from: + <see cref="M:System.Environment.GetFolderPath(System.Environment.SpecialFolder)" /></exception> <exception cref="T:System.ArgumentException">foo</exception> <permission cref="T:System.SomePermission">bar</permission> <altmember cref="T:System.SomeMember">alt member</altmember> |