<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 30 April 2018 at 13:23, David T. Lewis <span dir="ltr"><<a href="mailto:lewis@mail.msen.com" target="_blank">lewis@mail.msen.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-">On Fri, Apr 27, 2018 at 10:36:17PM -0500, Tim Johnson wrote:<br>
> Hi,<br>
> <br>
> I ran some tests and found a couple that are failing.  I also checked <br>
> and found these tests don't fail in my 5.1 image.<br>
> <br>
> BrowserTest>>#<wbr>testBuildMessageCategoryBrowse<wbr>rEditString<br>
> DebuggerExtensionsTest>>#<wbr>testCollectionsGeneralise<br>
> <br>
> I think the first one might actually be a case where a bug was fixed.  <br>
> The test fails because of a timeout, because there is a dialog on-screen <br>
> which is not returning.  Not sure, but the test may be intended to <br>
> declare that the dialog should appear, and now it is (?).<br>
> <br>
> The second one, I don't understand.  There is no comment.<br>
<br>
</span>I don't understand it either, and strangely it has no method timestamp.<br>
But the test was was introduced in April 2013 in this update:<br>
<br>
  Name: ToolsTests-fbs.62<br>
  Author: fbs<br>
  Time: 19 April 2013, 8:43:40.116 am<br>
  UUID: 926d563e-d57b-44ec-b4e7-<wbr>672010293c2b<br>
  Ancestors: ToolsTests-nice.61<br>
<br>
  Tests for the new #canonicalArgumentName Debugger methods.<br>
<br>
This is part of test coverage for #canonicalArgumentName, which is implemented<br>
in Object and also has no method timestamp or comment (!!!).<br>
<br>
The canonicalArgumentName method is sent by many unit tests, so it has very<br>
good coverage (even though I don't know the significance of this failing test).<br>
<br>
Aside from unit tests, it is sent by Message>>createStubMethod and appears<br>
to be used when the debugger provides a template for implementing a missing<br>
method, or for implementing a method override.<br>
<br>
The test does this:<br>
<br>
  testCollectionsGeneralise<br>
      self assert: Collection name equals: Array new canonicalArgumentName.<br>
      self assert: Collection name equals: OrderedCollection new canonicalArgumentName.<br>
      self assert: Collection name equals: LinkedList new canonicalArgumentName<br>
<br>
<br>
This looks like a regression that should be fixed such that the tests pass<br>
again, and that also deserves a good method comment in Object>>cononicalArgumentName.<br>
<br>
I think that some more background and explanation can probably be found on<br>
squeak-dev circa April 2013, notably this reference:<br>
<br>
  <a href="http://lists.squeakfoundation.org/pipermail/squeak-dev/2013-April/170506.html" rel="noreferrer" target="_blank">http://lists.squeakfoundation.<wbr>org/pipermail/squeak-dev/2013-<wbr>April/170506.html</a><br>
<br>
Which points to this:<br>
<br>
  Name: Tools-fbs.460<br>
  Author: fbs<br>
  Time: 19 April 2013, 8:40:24.143 am<br>
  UUID: d5cf82c4-bda7-48ff-bfbd-<wbr>e8b27d0a07d7<br>
  Ancestors: Tools-fbs.459<br>
<br>
  When creating a stub method, give the argument names that represent the<br>
  (usual) desired name more accurately. For instance, Arrays, OrderedCollections<br>
  and Sets all result in 'aCollection', ByteStrings and WideStrings in 'aString',<br>
  and so on.<br>
<br>
So perhaps that last paragraph about "when creating a stub method..." might serve<br>
as a method comment for Object>>cononicalArgumentName?<br>
<br>
Dave<br></blockquote><div><br></div><div>It was part of my work to improve the "JIT development" workflow, aka "debugger driven development". See <a href="http://lists.squeakfoundation.org/pipermail/squeak-dev/2013-April/170693.html">http://lists.squeakfoundation.org/pipermail/squeak-dev/2013-April/170693.html</a> and <a href="http://bugs.squeak.org/view.php?id=7761">http://bugs.squeak.org/view.php?id=7761</a></div><div><br></div><div>frank</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div class="gmail-HOEnZb"><div class="gmail-h5"><br>
> <br>
> Image/VM info:<br>
> <br>
> Squeak6.0alpha<br>
> latest update: #17922<br>
> Image format 6521 (32 bit)<br>
> <br>
> Virtual Machine<br>
> ---------------<br>
> C:\Users\tcj\Desktop\squeak.<wbr>cog.spur_win32x86_<wbr>201803080952\Squeak.exe<br>
> Croquet Closure Cog[Spur] VM [CoInterpreterPrimitives <br>
> VMMaker.oscog-eem.2347]<br>
> Win32 built on Mar  8 2018 11:08:39 GMT Compiler: 6.4.0<br>
> platform sources revision VM: 201803080952<br>
> <br>
> Cheers,<br>
> Tim<br>
> <br>
<br>
</div></div></blockquote></div><br></div></div>