<html>
<head>
<style><!--
.hmmessage P
{
margin:0px;
padding:0px
}
body.hmmessage
{
font-size: 12pt;
font-family:Calibri
}
--></style></head>
<body class='hmmessage'><div dir='ltr'><font color="#000000" face="Calibri,sans-serif">You can run "make check FIXTURE=System.Web.Script.Serialization.JavaScriptSerializerTest" in mcs/class/System.Web.Extensions to only run the tests that are now failing.<br id="FontBreak"></font><BR>As for ArrayList vs List<object>: can you add a unit test that verifies this?<BR> <BR>-- Alex<br> <BR><div><hr id="stopSpelling">Date: Thu, 30 Oct 2014 08:38:27 -0500<br>Subject: Re: [Mono-dev] PR #1337 Review<br>From: ryan.melena@noesislabs.com<br>To: alex.koeplinger@outlook.com<br>CC: miguel@xamarin.com; mono-devel-list@lists.ximian.com<br><br><div dir="ltr">I can take a look at the unit test problems but any help is appreciated.  I originally ran 'make check' and everything passed so I'm guessing these tests must not have been part of that script.<div><br></div><div>The change from ArrayList to List<object> is crucial because calling MS code assumes that the final result is an IEnumerable<object> (which an ArrayList is not).  It could conceivably be some other IEnumerable other than a List but I doubt that would fix the unit test issues.</div></div><div class="ecxgmail_extra"><br clear="all"><div><div dir="ltr">Owner<br>Cell: (612) 293-9161<br><a href="mailto:ryan.melena@noesislabs.com" target="_blank">ryan.melena@noesislabs.com</a><br><a href="http://noesislabs.com" target="_blank">noesislabs.com</a></div></div>
<br><div class="ecxgmail_quote">On Thu, Oct 30, 2014 at 8:23 AM, Alexander Köplinger <span dir="ltr"><<a href="mailto:alex.koeplinger@outlook.com" target="_blank">alex.koeplinger@outlook.com</a>></span> wrote:<br><blockquote class="ecxgmail_quote" style="padding-left: 1ex; border-left-color: rgb(204, 204, 204); border-left-width: 1px; border-left-style: solid;">


<div><div dir="ltr"><font color="#000000" face="Calibri,sans-serif">Hey, this breaks a few tests:</font><br><font color="#000000" face="Calibri,sans-serif"></font> <br><font color="#000000" face="Calibri,sans-serif"><code>MonoTests.System.Web.Script.Serialization.JavaScriptSerializerTest.DeserializeDictionaryOfArrayList MonoTests.System.Web.Script.Serialization.JavaScriptSerializerTest.DeserializeObject MonoTests.System.Web.Script.Serialization.JavaScriptSerializerTest.InfinityAndNaN MonoTests.System.Web.Script.Serialization.JavaScriptSerializerTest.TestDeserialize MonoTests.System.Web.Script.Serialization.JavaScriptSerializerTest.TestDeserializeConverter1 MonoTests.System.Web.Script.Serialization.JavaScriptSerializerTest.TestDeserializeNonGenericOverload MonoTests.System.Web.Script.Serialization.JavaScriptSerializerTest.TestDeserializeTypeResolver MonoTests.System.Web.Script.Serialization.JavaScriptSerializerTest.TestSerialize1 <br> <br></code>The TestDeserializeNonGenericOverload is a new one added by the PR (it passes on <a href="http://MS.NET" target="_blank">MS.NET</a> though, so we should make it work on Mono).<br> <br>I'm not sure the switch from ArrayList to List<object> is correct, if I change that back all tests pass again except the new TestDeserializeNonGenericOverload.<br> <br>-- Alex<br></font><br> <br><div><hr>From: <a href="mailto:miguel@xamarin.com" target="_blank">miguel@xamarin.com</a><br>Date: Wed, 29 Oct 2014 15:02:39 -0400<br>To: <a href="mailto:Ryan.Melena%2BMono-Devel-List@noesislabs.com" target="_blank">Ryan.Melena+Mono-Devel-List@noesislabs.com</a><br>CC: <a href="mailto:mono-devel-list@lists.ximian.com" target="_blank">mono-devel-list@lists.ximian.com</a><br>Subject: Re: [Mono-dev] PR #1337 Review<div><div class="h5"><br><br><div dir="ltr">Thanks for pointing this out.<div><br></div><div>Imlemented.</div><div><br></div><div>Miguel</div></div><div><br><div>On Wed, Oct 29, 2014 at 2:52 PM, RyanMelenaNoesis <span dir="ltr"><<a href="mailto:Ryan.Melena+Mono-Devel-List@noesislabs.com" target="_blank">Ryan.Melena+Mono-Devel-List@noesislabs.com</a>></span> wrote:<br><blockquote style="padding-left: 1ex; border-left-color: rgb(204, 204, 204); border-left-width: 1px; border-left-style: solid;"><div dir="ltr"><span style="font-family: arial,sans-serif; font-size: 13px;">It was requested that I notify the mailing list about this pull request ( </span><a style="font-family: arial,sans-serif; font-size: 13px;" href="https://github.com/mono/mono/pull/1337" target="_blank">https://github.com/mono/mono/pull/<span style="background: rgb(255, 255, 204); color: rgb(34, 34, 34);">1337</span></a><span style="font-family: arial,sans-serif; font-size: 13px;"> ) in order to facilitate conversation.  The changes were implemented to support JWT authentication in our web application and required a couple changes to existing Mono code.  The changes to existing code were necessary to make Mono compatible with MS calling code and are explained in line comments.  Please let me know if anyone has particular concerns with this pull request.</span></div>
<br>_______________________________________________<br>
Mono-devel-list mailing list<br>
<a href="mailto:Mono-devel-list@lists.ximian.com" target="_blank">Mono-devel-list@lists.ximian.com</a><br>
<a href="http://lists.ximian.com/mailman/listinfo/mono-devel-list" target="_blank">http://lists.ximian.com/mailman/listinfo/mono-devel-list</a><br>
<br></blockquote></div><br></div>
<br>_______________________________________________
Mono-devel-list mailing list
<a href="mailto:Mono-devel-list@lists.ximian.com" target="_blank">Mono-devel-list@lists.ximian.com</a>
<a href="http://lists.ximian.com/mailman/listinfo/mono-devel-list" target="_blank">http://lists.ximian.com/mailman/listinfo/mono-devel-list</a></div></div></div>                                       </div></div>
</blockquote></div><br></div></div>                                         </div></body>
</html>