The Trunk: ST80Tests-dtl.5.mcz

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

The Trunk: ST80Tests-dtl.5.mcz

commits-2
David T. Lewis uploaded a new version of ST80Tests to project The Trunk:
http://source.squeak.org/trunk/ST80Tests-dtl.5.mcz

==================== Summary ====================

Name: ST80Tests-dtl.5
Author: dtl
Time: 13 November 2017, 9:26:52.125552 pm
UUID: 5bcbeaa0-c4f2-46c0-b555-d0312fff6f91
Ancestors: ST80Tests-pre.4

Remove unnecessary references to global world.
Do not test nil World for project type, instead use Project current isMVC

=============== Diff against ST80Tests-pre.4 ===============

Item was changed:
  ----- Method: MVCToolBuilderTests>>testWindowCloseAction (in category 'tests-not applicable') -----
  testWindowCloseAction
  "This can only work if we're actually run in MVC"
+ Project current isMVC ifTrue: [super testWindowCloseAction]!
- World isNil ifTrue: [super testWindowCloseAction]!


Reply | Threaded
Open this post in threaded view
|

Re: The Trunk: ST80Tests-dtl.5.mcz

Hannes Hirzel
Minor issue

On Tue, 14 Nov 2017 02:26:56 0000, [hidden email]
<[hidden email]> wrote:

> David T. Lewis uploaded a new version of ST80Tests to project The Trunk:
> http://source.squeak.org/trunk/ST80Tests-dtl.5.mcz
>
> ==================== Summary ====================
>
> Name: ST80Tests-dtl.5
> Author: dtl
> Time: 13 November 2017, 9:26:52.125552 pm
> UUID: 5bcbeaa0-c4f2-46c0-b555-d0312fff6f91
> Ancestors: ST80Tests-pre.4
>
> Remove unnecessary references to global world.
> Do not test nil World for project type, instead use Project current isMVC
>
> =============== Diff against ST80Tests-pre.4 ===============
>
> Item was changed:
>   ----- Method: MVCToolBuilderTests>>testWindowCloseAction (in category
> 'tests-not applicable') -----
>   testWindowCloseAction
>   "This can only work if we're actually run in MVC"
Minor issue: the comment above is no longer necessary. Because now the
code is clear enough.
> + Project current isMVC ifTrue: [super testWindowCloseAction]!
> - World isNil ifTrue: [super testWindowCloseAction]!
I assume the comment was there because it is not obvious at all that
    World isNil
is the test for an MVC project.


>
>

Reply | Threaded
Open this post in threaded view
|

Re: The Trunk: ST80Tests-dtl.5.mcz

David T. Lewis
On Tue, Nov 14, 2017 at 05:06:14AM +0100, H. Hirzel wrote:

> Minor issue
>
> On Tue, 14 Nov 2017 02:26:56 0000, [hidden email]
> <[hidden email]> wrote:
> > David T. Lewis uploaded a new version of ST80Tests to project The Trunk:
> > http://source.squeak.org/trunk/ST80Tests-dtl.5.mcz
> >
> > ==================== Summary ====================
> >
> > Name: ST80Tests-dtl.5
> > Author: dtl
> > Time: 13 November 2017, 9:26:52.125552 pm
> > UUID: 5bcbeaa0-c4f2-46c0-b555-d0312fff6f91
> > Ancestors: ST80Tests-pre.4
> >
> > Remove unnecessary references to global world.
> > Do not test nil World for project type, instead use Project current isMVC
> >
> > =============== Diff against ST80Tests-pre.4 ===============
> >
> > Item was changed:
> >   ----- Method: MVCToolBuilderTests>>testWindowCloseAction (in category
> > 'tests-not applicable') -----
> >   testWindowCloseAction
> >   "This can only work if we're actually run in MVC"
> Minor issue: the comment above is no longer necessary. Because now the
> code is clear enough.
> > + Project current isMVC ifTrue: [super testWindowCloseAction]!
> > - World isNil ifTrue: [super testWindowCloseAction]!
> I assume the comment was there because it is not obvious at all that
>     World isNil
> is the test for an MVC project.
>

I think you are right, but I did not change the comment because I do not
really know what that test is intended to do. Mainly the MVCToolBuilderTests
class seems to do nothing except disable the ToolBuilderTests that are not
applicable to MVC.

Dave


Reply | Threaded
Open this post in threaded view
|

Re: The Trunk: ST80Tests-dtl.5.mcz

marcel.taeumel
Cool! It is much clearer now. :) I think there are more places with "World isNil"-like-checks in MVC code.

Best,
Marcel

Am 14.11.2017 05:26:28 schrieb David T. Lewis <[hidden email]>:

On Tue, Nov 14, 2017 at 05:06:14AM +0100, H. Hirzel wrote:
> Minor issue
>
> On Tue, 14 Nov 2017 02:26:56 0000, [hidden email]
> wrote:
> > David T. Lewis uploaded a new version of ST80Tests to project The Trunk:
> > http://source.squeak.org/trunk/ST80Tests-dtl.5.mcz
> >
> > ==================== Summary ====================
> >
> > Name: ST80Tests-dtl.5
> > Author: dtl
> > Time: 13 November 2017, 9:26:52.125552 pm
> > UUID: 5bcbeaa0-c4f2-46c0-b555-d0312fff6f91
> > Ancestors: ST80Tests-pre.4
> >
> > Remove unnecessary references to global world.
> > Do not test nil World for project type, instead use Project current isMVC
> >
> > =============== Diff against ST80Tests-pre.4 ===============
> >
> > Item was changed:
> > ----- Method: MVCToolBuilderTests>>testWindowCloseAction (in category
> > 'tests-not applicable') -----
> > testWindowCloseAction
> > "This can only work if we're actually run in MVC"
> Minor issue: the comment above is no longer necessary. Because now the
> code is clear enough.
> > + Project current isMVC ifTrue: [super testWindowCloseAction]!
> > - World isNil ifTrue: [super testWindowCloseAction]!
> I assume the comment was there because it is not obvious at all that
> World isNil
> is the test for an MVC project.
>

I think you are right, but I did not change the comment because I do not
really know what that test is intended to do. Mainly the MVCToolBuilderTests
class seems to do nothing except disable the ToolBuilderTests that are not
applicable to MVC.

Dave