- Pharo-dev - lists.pharo.org

Re: [Pharo-dev] Necessity of TDD with Smalltalk

by Attila Magyar

I think the tests could provide valuable design feedback about the code under test, so it is good to have them. I can respond to those feedbacks effectively only when I'm writing them first, otherwise it might be too late. But this is a long topic. When I write tests first, I like the fact that the initial state of my test is red. So I can catch the point when it changes to green. If this happens when I expect, then I can be sure that I understand what is happening, and I managed to write a meaningful test. Writing tests afterwards makes easier to write tests which are always green. To avoid this I have to break the test after I finished it by changing something in the code. This is an extra step which I can save by writing tests first. -- View this message in context: http://forum.world.st/Pharo-dev-Necessity-of-TDD-with-Smalltalk-tp4691867p4… Sent from the Pharo Smalltalk Developers mailing list archive at Nabble.com.

June 8, 2013

Re: [Pharo-dev] [update 3.0] #30188

by Marcus Denker

On Jun 8, 2013, at 12:31 PM, StÃ©phane Ducasse <stephane.ducasse(a)inria.fr> wrote: > so how do we sync before I do something. > > when I run the scripts of esteban > ./integrate > > the staged file is not empty and I do not understand why > because to me it should be empty. > right now I have and it looks totally bogus. > > #Pharo3.0^M30184-Pha-Update.cs^M30185-Pha-Update.cs^M30186-Pha-Update.cs^M30187-Pha-Update.cs^M30188-Pha-Update.cs^M30188-Pha-Update.cs > Yes, it should be empty but it works even like this. It looks just at the last entry. Marcus

June 8, 2013

Re: [Pharo-dev] [update 3.0] #30188

by Marcus Denker

On Jun 8, 2013, at 12:29 PM, StÃ©phane Ducasse <stephane.ducasse(a)inria.fr> wrote: > when I look at pharo-update I have > > 30186-Pha-Update.cs > 30187-Pha-Update.cs > 30188-Pha-Update.cs > 30189-Pha-Update.cs > updates30.staged > > I do not understand why there is a staged file in there. > The idea is that *only* the staged file should be there, the updates file is just on the server. > #Pharo3.0 > 30184-Pha-Update.cs > 30185-Pha-Update.cs > 30186-Pha-Update.cs > 30187-Pha-Update.cs > 30188-Pha-Update.cs > 30188-Pha-Update.cs > > I do not understand why the staged file is in this folder. > The file should there. There is a bug right now that it contains more than one entry. And there is a bug that the updates30 is wrongly generated (yes, I should have looked at thatâ¦) But it works for the update process. https://pharo.fogbugz.com/f/cases/10412/Pharo-3-updates-list-on-the-server-… > So I'm confused and I looks like that I lost my time. > > > So I can rollback until > 30187 -> ScriptLoader30-StephaneDucasse.214.mcz > > but I do not know how to do it efficiently. No, not needed. I will check. And in general, we should simplify the whole thing. Marcus

June 8, 2013

Re: [Pharo-dev] [update 3.0] #30188

by StÃ©phane Ducasse

so how do we sync before I do something. when I run the scripts of esteban ./integrate the staged file is not empty and I do not understand why because to me it should be empty. right now I have and it looks totally bogus. #Pharo3.0^M30184-Pha-Update.cs^M30185-Pha-Update.cs^M30186-Pha-Update.cs^M30187-Pha-Update.cs^M30188-Pha-Update.cs^M30188-Pha-Update.cs Stef On Jun 8, 2013, at 12:28 PM, StÃ©phane Ducasse <Stephane.Ducasse(a)inria.fr> wrote: > when I look at pharo-update I have > > 30186-Pha-Update.cs > 30187-Pha-Update.cs > 30188-Pha-Update.cs > 30189-Pha-Update.cs > updates30.staged > > I do not understand why there is a staged file in there. > > #Pharo3.0 > 30184-Pha-Update.cs > 30185-Pha-Update.cs > 30186-Pha-Update.cs > 30187-Pha-Update.cs > 30188-Pha-Update.cs > 30188-Pha-Update.cs > > I do not understand why the staged file is in this folder. > > So I'm confused and I looks like that I lost my time. > > > So I can rollback until > 30187 -> ScriptLoader30-StephaneDucasse.214.mcz > > but I do not know how to do it efficiently. > > Stef >

June 8, 2013

Re: [Pharo-dev] [update 3.0] #30188

by StÃ©phane Ducasse

when I look at pharo-update I have 30186-Pha-Update.cs 30187-Pha-Update.cs 30188-Pha-Update.cs 30189-Pha-Update.cs updates30.staged I do not understand why there is a staged file in there. #Pharo3.0 30184-Pha-Update.cs 30185-Pha-Update.cs 30186-Pha-Update.cs 30187-Pha-Update.cs 30188-Pha-Update.cs 30188-Pha-Update.cs I do not understand why the staged file is in this folder. So I'm confused and I looks like that I lost my time. So I can rollback until 30187 -> ScriptLoader30-StephaneDucasse.214.mcz but I do not know how to do it efficiently. Stef

June 8, 2013

Re: [Pharo-dev] SourceCity works

by StÃ©phane Ducasse

may be we should move it under Moose Stef On Jun 8, 2013, at 10:49 AM, Stephan Eggermont <stephan(a)stack.nl> wrote: > > On 8 jun 2013, at 10:30, pharo-dev-request(a)lists.pharo.org wrote: > >> display bindRenderBuffers > > And they need to go in > fillRenderingList: anOpenGLList drawDependencies: aBuilding > too. > > I was not successful in committing to smalltalkhub (500) > > Stephan > >

June 8, 2013

Re: [Pharo-dev] [update 3.0] #30188

by StÃ©phane Ducasse

Shit! So now I do not know what to do :( Especially since I let times in between. So should I rollback both? >> strange: there was already a 30188 >> >> Is this 30189? >> >> because there is a 30189 update file, bit it is not in the updates.staged. >> >> This means 30188 is the last on the server. >> >> So I am a bit confusedâ¦ > > The two 30188 updates are (in this order it seems): > in the git we have in addition a 30189-Pha-Update.cs > > 30188 > ----- > > 10867 enabled: cleans post fix > https://pharo.fogbugz.com/default.asp?10867 > > Diff information: > http://smalltalkhub.com/mc/Pharo/Pharo30/main/Polymorph-Widgets-StephaneDuc… > > 30188 > ----- > > 10745 [ENH]: MethodContext>>#callChainAnySatisfy: > https://pharo.fogbugz.com/default.asp?10745 > > 10838 an MCOrganizationDefinition should only remove empty categories during unload > https://pharo.fogbugz.com/default.asp?10838 > > 10853 AlainTreeModel: Want menu on tree items only > https://pharo.fogbugz.com/default.asp?10853 > > Diff information: > http://smalltalkhub.com/mc/Pharo/Pharo30/main/Spec-Widgets-StephaneDucasse.… > http://smalltalkhub.com/mc/Pharo/Pharo30/main/Monticello-StephaneDucasse.81… > http://smalltalkhub.com/mc/Pharo/Pharo30/main/Kernel-StephaneDucasse.1472.d… > >

June 8, 2013

Re: [Pharo-dev] [update 3.0] #30188

by Marcus Denker

On Jun 8, 2013, at 11:45 AM, Marcus Denker <marcus.denker(a)inria.fr> wrote: > strange: there was already a 30188 > > Is this 30189? > > because there is a 30189 update file, bit it is not in the updates.staged. > > This means 30188 is the last on the server. > > So I am a bit confusedâ¦ The two 30188 updates are (in this order it seems): in the git we have in addition a 30189-Pha-Update.cs 30188 ----- 10867 enabled: cleans post fix https://pharo.fogbugz.com/default.asp?10867 Diff information: http://smalltalkhub.com/mc/Pharo/Pharo30/main/Polymorph-Widgets-StephaneDuc… 30188 ----- 10745 [ENH]: MethodContext>>#callChainAnySatisfy: https://pharo.fogbugz.com/default.asp?10745 10838 an MCOrganizationDefinition should only remove empty categories during unload https://pharo.fogbugz.com/default.asp?10838 10853 AlainTreeModel: Want menu on tree items only https://pharo.fogbugz.com/default.asp?10853 Diff information: http://smalltalkhub.com/mc/Pharo/Pharo30/main/Spec-Widgets-StephaneDucasse.… http://smalltalkhub.com/mc/Pharo/Pharo30/main/Monticello-StephaneDucasse.81… http://smalltalkhub.com/mc/Pharo/Pharo30/main/Kernel-StephaneDucasse.1472.d…

June 8, 2013

Re: [Pharo-dev] [update 3.0] #30188

by Marcus Denker

strange: there was already a 30188 Is this 30189? because there is a 30189 update file, bit it is not in the updates.staged. This means 30188 is the last on the server. So I am a bit confused... On Jun 7, 2013, at 6:37 PM, StÃ©phane Ducasse <stephane.ducasse(a)inria.fr> wrote: > 30188 > ----- > > 10867 enabled: cleans post fix > https://pharo.fogbugz.com/default.asp?10867 > > Diff information: > http://smalltalkhub.com/mc/Pharo/Pharo30/main/Polymorph-Widgets-StephaneDuc… > >

June 8, 2013

Re: [Pharo-dev] SourceCity works

by Stephan Eggermont

On 8 jun 2013, at 10:30, pharo-dev-request(a)lists.pharo.org wrote: > display bindRenderBuffers And they need to go in fillRenderingList: anOpenGLList drawDependencies: aBuilding too. I was not successful in committing to smalltalkhub (500) Stephan

June 8, 2013