<div dir="ltr">Hi Hernan,<div>I think this should fix cmd-t. I never used this and I'm not sure how you use it, can you test it and let me know if it behaves as you expect? Notice that I moved #classesAt: from Categorizer to SystemOrganizer because the Categorizer is more general (it is used for method categories too), and then I removed #testCaseClassesAt: and put that line of code in the caller (because it made the SystemOrganizer know about test cases and I thought that was unnecessary). I don't think I broke anything, I searched for senders in other packages, but let me know what you think.</div><div><br></div><div>Also I saw the methods for #linesOfCodeOf:, #instanceMethodCountOf: and #classMethodCountOf: you added to SystemOrganizer to show that information in annotations. Currently, they return 0 for top categories without any classes. I made the experiment of including all classes in all subcategories, but it became too slow so I reverted to the original behavior.</div><div><br></div><div>Regarding cmd-y I think there's not much we can do without getting dirty... at the level of SmalltalkEditor we don't know anything of the Browser, what system category is selected, etc, and I wouldn't want to hack my way into a fix. What do you think?<br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Feb 2, 2022 at 3:08 PM Hernan Wilkinson via Cuis-dev <<a href="mailto:cuis-dev@lists.cuis.st">cuis-dev@lists.cuis.st</a>> wrote:<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 dir="ltr">Hi Luciano,<div> I found another bug... well, I do not know if it is a bug really, maybe a "desired behavior" :-) </div><div> It is related to running the tests for a system category.</div><div> Currently, if you have the mouse over a system category and pressed cmd+t, it runs all the tests of that category. That works ok for the subcategories but if you are in a root category it only runs the tests for that root category and I think it should run the tests of all the categories because they are shown in the class list.</div><div><br></div><div> The same thing should happen when pressing cmd+y in the code pane. cmd+y runs the tests of the system category, so if you are in a root system category it should behave as described in the previous paragraph.</div><div><br></div><div> Does it make sense to you?</div><div><br></div><div>Cheers!</div><div>Hernan.</div></div>
-- <br>
Cuis-dev mailing list<br>
<a href="mailto:Cuis-dev@lists.cuis.st" target="_blank">Cuis-dev@lists.cuis.st</a><br>
<a href="https://lists.cuis.st/mailman/listinfo/cuis-dev" rel="noreferrer" target="_blank">https://lists.cuis.st/mailman/listinfo/cuis-dev</a><br>
</blockquote></div>