<div dir="ltr">I concur with Christoph; -1<div><br></div><div>Even if it were not possible to open the tool with a preselection, you will still assemble a test suite through manual selection. Then you may want to run a different test suite, but keep the test suite/selection in the first browser to come back later.</div><div><br></div><div>Kind regards,<br>Jakob</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Am Sa., 28. Dez. 2019 um 14:27 Uhr schrieb Thiede, Christoph <<a href="mailto:Christoph.Thiede@student.hpi.uni-potsdam.de">Christoph.Thiede@student.hpi.uni-potsdam.de</a>>:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">





<div>


<div dir="ltr">
<div id="gmail-m_7311282277882344719x_divtagdefaultwrapper" dir="ltr" style="font-size:12pt;color:rgb(0,0,0);font-family:Calibri,Helvetica,sans-serif">
<p>-1 for that change :-)</p>
<p><br>
</p>
<p>This commit makes it impossible to have multiple TestRunners opened side by side. For me, this is clearly a drawback. Why force unnecessary singletons?</p>
<p>In my opinion, the old implementation was simply correct - I can open as many TestRunners as I would like to do, but if there is already an open TestRunner with the same configuration, it will be focused instead.</p>
<p><br>
</p>
<p>> <span>There's no way to spawn that model with any pre-selection</span></p>
<p><br>
</p>
<p>How do you mean this? Try the following - before loading this commit - line by line:</p>
<p></p>
<div>TestRunner openForSuite: StringTest suite.</div>
<div>TestRunner openForSuite: CharacterTest suite.</div>
<div>TestRunner openForSuite: StringTest suite.</div>
<br>
<p></p>
<p>Best,</p>
<p>Christoph</p>
<div id="gmail-m_7311282277882344719x_Signature">
<div id="gmail-m_7311282277882344719x_divtagdefaultwrapper" dir="ltr" style="font-size:12pt;color:rgb(0,0,0);font-family:Calibri,Helvetica,sans-serif,EmojiFont,"Apple Color Emoji","Segoe UI Emoji",NotoColorEmoji,"Segoe UI Symbol","Android Emoji",EmojiSymbols">
<div name="x_divtagdefaultwrapper">
<div><font size="2" color="#808080"></font></div>
</div>
</div>
</div>
</div>
<hr style="display:inline-block;width:98%">
<div id="gmail-m_7311282277882344719x_divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>Von:</b> Squeak-dev <<a href="mailto:squeak-dev-bounces@lists.squeakfoundation.org" target="_blank">squeak-dev-bounces@lists.squeakfoundation.org</a>> im Auftrag von <a href="mailto:commits@source.squeak.org" target="_blank">commits@source.squeak.org</a> <<a href="mailto:commits@source.squeak.org" target="_blank">commits@source.squeak.org</a>><br>
<b>Gesendet:</b> Samstag, 28. Dezember 2019 05:59:42<br>
<b>An:</b> <a href="mailto:squeak-dev@lists.squeakfoundation.org" target="_blank">squeak-dev@lists.squeakfoundation.org</a>; <a href="mailto:packages@lists.squeakfoundation.org" target="_blank">packages@lists.squeakfoundation.org</a><br>
<b>Betreff:</b> [squeak-dev] The Trunk: SUnitGUI-cmm.78.mcz</font>
<div> </div>
</div>
</div>
<font size="2"><span style="font-size:10pt">
<div>Chris Muller uploaded a new version of SUnitGUI to project The Trunk:<br>
<a href="http://source.squeak.org/trunk/SUnitGUI-cmm.78.mcz" target="_blank">http://source.squeak.org/trunk/SUnitGUI-cmm.78.mcz</a><br>
<br>
==================== Summary ====================<br>
<br>
Name: SUnitGUI-cmm.78<br>
Author: cmm<br>
Time: 27 December 2019, 10:59:41.297401 pm<br>
UUID: 1c7d7b32-cdec-4875-80d8-7d2413e01a05<br>
Ancestors: SUnitGUI-cmm.77<br>
<br>
Fix TestRunner browser for use with Reuse Windows preference.  There's no way to spawn that model with any pre-selection, it's essentially a singleton browser in Squeak.<br>
<br>
=============== Diff against SUnitGUI-cmm.77 ===============<br>
<br>
Item was changed:<br>
  ----- Method: TestRunner>>representsSameBrowseeAs: (in category 'accessing-ui') -----<br>
  representsSameBrowseeAs: anotherModel <br>
+        ^ true!<br>
-        ^ self class = anotherModel class<br>
-        and: [ classesSelected = anotherModel classesSelected ]!<br>
<br>
<br>
</div>
</span></font>
</div>

<br>
</blockquote></div>