more robust tests

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

more robust tests

stepharo
Hi

While integrating some changes in P6.0 I got such tests failing. Only
failing on mac.

Could you check?

Stef


Regression

GT.EventRecorder.Tests.Core.GTEventRecorderTest.testAddCollector3 (from
GT-EventRecorder-Tests)

Failing for the past 1 build (Since Unstable#7 )
Took 0.96 sec.
add description
Error Message

Assertion failed
Stacktrace

GTEventRecorderTest(TestAsserter)>>assert:
GTEventRecorderTest(TestAsserter)>>deny:
GTEventRecorderTest>>assertReceiverSemaphore
GTEventRecorderTest>>waitForDelivery
GTEventRecorderTest>>testAddCollector3
GTEventRecorderTest(TestCase)>>performTest


Stef


Reply | Threaded
Open this post in threaded view
|

Re: more robust tests

Andrei Chis
We'll have a look.

Cheers,
Andrei

On Sun, May 15, 2016 at 3:06 PM, stepharo <[hidden email]> wrote:
Hi

While integrating some changes in P6.0 I got such tests failing. Only failing on mac.

Could you check?

Stef


Regression

GT.EventRecorder.Tests.Core.GTEventRecorderTest.testAddCollector3 (from GT-EventRecorder-Tests)

Failing for the past 1 build (Since Unstable#7 )
Took 0.96 sec.
add description
Error Message

Assertion failed
Stacktrace

GTEventRecorderTest(TestAsserter)>>assert:
GTEventRecorderTest(TestAsserter)>>deny:
GTEventRecorderTest>>assertReceiverSemaphore
GTEventRecorderTest>>waitForDelivery
GTEventRecorderTest>>testAddCollector3
GTEventRecorderTest(TestCase)>>performTest


Stef



Reply | Threaded
Open this post in threaded view
|

Re: more robust tests

stepharo

thanks.

I would like to have the system in a state where we can release the next day :)


Le 17/5/16 à 21:43, Andrei Chis a écrit :
We'll have a look.

Cheers,
Andrei

On Sun, May 15, 2016 at 3:06 PM, stepharo <[hidden email]> wrote:
Hi

While integrating some changes in P6.0 I got such tests failing. Only failing on mac.

Could you check?

Stef


Regression

GT.EventRecorder.Tests.Core.GTEventRecorderTest.testAddCollector3 (from GT-EventRecorder-Tests)

Failing for the past 1 build (Since Unstable#7 )
Took 0.96 sec.
add description
Error Message

Assertion failed
Stacktrace

GTEventRecorderTest(TestAsserter)>>assert:
GTEventRecorderTest(TestAsserter)>>deny:
GTEventRecorderTest>>assertReceiverSemaphore
GTEventRecorderTest>>waitForDelivery
GTEventRecorderTest>>testAddCollector3
GTEventRecorderTest(TestCase)>>performTest


Stef




Reply | Threaded
Open this post in threaded view
|

File List -- MNU: RubTextEditor>>doIt

Martin McClure-2
So I open Pharo 5, and the *very* *first* thing I do is to open a file
list, select a file that contains the Metacello expression to load my
project, and select "Do it" from the menu.

And get MNU: RubTextEditor>>doIt

:-(

This was on Linux, in case that makes a difference.

I couldn't quickly track down how this is *supposed* to work.

Regards,

-Martin

Reply | Threaded
Open this post in threaded view
|

Re: File List -- MNU: RubTextEditor>>doIt

hernanmd
In Windows if you select a file with Smalltalk code and right click you get

MessageNotUnderstood: FileList class>>allRegisteredServices

without menu at all.

I think you have to open a Workspace with contents and then you can evaluate.

I don't see any "do-it" right clicking on the file from the FileList.

Cheers,

Hernán






2016-05-18 2:42 GMT-03:00 Martin McClure <[hidden email]>:
So I open Pharo 5, and the *very* *first* thing I do is to open a file list, select a file that contains the Metacello expression to load my project, and select "Do it" from the menu.

And get MNU: RubTextEditor>>doIt

:-(

This was on Linux, in case that makes a difference.

I couldn't quickly track down how this is *supposed* to work.

Regards,

-Martin


Reply | Threaded
Open this post in threaded view
|

Re: File List -- MNU: RubTextEditor>>doIt

Martin McClure-2
On 05/29/2016 12:52 PM, Hernán Morales Durand wrote:
In Windows if you select a file with Smalltalk code and right click you get

MessageNotUnderstood: FileList class>>allRegisteredServices

without menu at all.

I think you have to open a Workspace with contents and then you can evaluate.

I don't see any "do-it" right clicking on the file from the FileList.

Cheers,

Hernán


Thanks for the reply, Hernán.

I'm sorry if I wasn't clear -- what I'm doing in the file browser is selecting a file, selecting some text in the text pane, and selecting "Do it" from the right-click menu in the text pane.

I'm still seeing the MNU in 50760.

Regards,

-Martin
 

2016-05-18 2:42 GMT-03:00 Martin McClure <[hidden email]>:
So I open Pharo 5, and the *very* *first* thing I do is to open a file list, select a file that contains the Metacello expression to load my project, and select "Do it" from the menu.

And get MNU: RubTextEditor>>doIt

:-(

This was on Linux, in case that makes a difference.

I couldn't quickly track down how this is *supposed* to work.

Regards,

-Martin



Reply | Threaded
Open this post in threaded view
|

Re: File List -- MNU: RubTextEditor>>doIt

stepharo

You found a bug due to the PluggableTextMorph removal.

Since nobody use the fileList it went unnoticed.

Can you open a bug entry?

Stef


Le 17/7/16 à 20:32, Martin McClure a écrit :
On 05/29/2016 12:52 PM, Hernán Morales Durand wrote:
In Windows if you select a file with Smalltalk code and right click you get

MessageNotUnderstood: FileList class>>allRegisteredServices

without menu at all.

I think you have to open a Workspace with contents and then you can evaluate.

I don't see any "do-it" right clicking on the file from the FileList.

Cheers,

Hernán


Thanks for the reply, Hernán.

I'm sorry if I wasn't clear -- what I'm doing in the file browser is selecting a file, selecting some text in the text pane, and selecting "Do it" from the right-click menu in the text pane.

I'm still seeing the MNU in 50760.

Regards,

-Martin
 

2016-05-18 2:42 GMT-03:00 Martin McClure <[hidden email]>:
So I open Pharo 5, and the *very* *first* thing I do is to open a file list, select a file that contains the Metacello expression to load my project, and select "Do it" from the menu.

And get MNU: RubTextEditor>>doIt

:-(

This was on Linux, in case that makes a difference.

I couldn't quickly track down how this is *supposed* to work.

Regards,

-Martin




Reply | Threaded
Open this post in threaded view
|

Re: File List -- MNU: RubTextEditor>>doIt

Henrik Nergaard

It looks like it is related some refactoring of the FileList, not the removal of the old textMorph.

 

Right clicking a .st file In Pharo 4.0 works fine, but the FileList class (and others ex; FileServices) looks quite different from the latest Pharo 5 and 6.

 

Best regards,

Henrik

 

 

From: Pharo-dev [mailto:[hidden email]] On Behalf Of stepharo
Sent: Sunday, July 17, 2016 10:58 PM
To: [hidden email]
Subject: Re: [Pharo-dev] File List -- MNU: RubTextEditor>>doIt

 

You found a bug due to the PluggableTextMorph removal.

Since nobody use the fileList it went unnoticed.

Can you open a bug entry?

Stef

 

Le 17/7/16 à 20:32, Martin McClure a écrit :

On 05/29/2016 12:52 PM, Hernán Morales Durand wrote:

In Windows if you select a file with Smalltalk code and right click you get

MessageNotUnderstood: FileList class>>allRegisteredServices

without menu at all.

I think you have to open a Workspace with contents and then you can evaluate.

I don't see any "do-it" right clicking on the file from the FileList.

Cheers,

Hernán

 


Thanks for the reply, Hernán.

I'm sorry if I wasn't clear -- what I'm doing in the file browser is selecting a file, selecting some text in the text pane, and selecting "Do it" from the right-click menu in the text pane.

I'm still seeing the MNU in 50760.

Regards,

-Martin
 

 

2016-05-18 2:42 GMT-03:00 Martin McClure <[hidden email]>:

So I open Pharo 5, and the *very* *first* thing I do is to open a file list, select a file that contains the Metacello expression to load my project, and select "Do it" from the menu.

And get MNU: RubTextEditor>>doIt

:-(

This was on Linux, in case that makes a difference.

I couldn't quickly track down how this is *supposed* to work.

Regards,

-Martin

 

 

 

Reply | Threaded
Open this post in threaded view
|

Re: File List -- MNU: RubTextEditor>>doIt

Nicolai Hess-3-2

Take a look at the Bugtracker. This Bugs are already reported.


Am 17.07.2016 23:18 schrieb "Henrik Nergaard" <[hidden email]>:

It looks like it is related some refactoring of the FileList, not the removal of the old textMorph.

 

Right clicking a .st file In Pharo 4.0 works fine, but the FileList class (and others ex; FileServices) looks quite different from the latest Pharo 5 and 6.

 

Best regards,

Henrik

 

 

From: Pharo-dev [mailto:[hidden email]] On Behalf Of stepharo
Sent: Sunday, July 17, 2016 10:58 PM
To: [hidden email]
Subject: Re: [Pharo-dev] File List -- MNU: RubTextEditor>>doIt

 

You found a bug due to the PluggableTextMorph removal.

Since nobody use the fileList it went unnoticed.

Can you open a bug entry?

Stef

 

Le 17/7/16 à 20:32, Martin McClure a écrit :

On 05/29/2016 12:52 PM, Hernán Morales Durand wrote:

In Windows if you select a file with Smalltalk code and right click you get

MessageNotUnderstood: FileList class>>allRegisteredServices

without menu at all.

I think you have to open a Workspace with contents and then you can evaluate.

I don't see any "do-it" right clicking on the file from the FileList.

Cheers,

Hernán

 


Thanks for the reply, Hernán.

I'm sorry if I wasn't clear -- what I'm doing in the file browser is selecting a file, selecting some text in the text pane, and selecting "Do it" from the right-click menu in the text pane.

I'm still seeing the MNU in 50760.

Regards,

-Martin
 

 

2016-05-18 2:42 GMT-03:00 Martin McClure <[hidden email]>:

So I open Pharo 5, and the *very* *first* thing I do is to open a file list, select a file that contains the Metacello expression to load my project, and select "Do it" from the menu.

And get MNU: RubTextEditor>>doIt

:-(

This was on Linux, in case that makes a difference.

I couldn't quickly track down how this is *supposed* to work.

Regards,

-Martin

 

 

 

Reply | Threaded
Open this post in threaded view
|

Re: File List -- MNU: RubTextEditor>>doIt

Martin McClure-2
In reply to this post by stepharo
On 07/17/2016 01:58 PM, stepharo wrote:
>
> You found a bug due to the PluggableTextMorph removal.
>
> Since nobody use the fileList it went unnoticed.
>
> Can you open a bug entry?
>

Done, case 18781.

Regards,

-Martin