Opened 5 years ago

Last modified 5 years ago

#246 testing defect

Changing a test, clicking on other test and clicking on accept shows a debugger window

Reported by: patrik.svestka@… Owned by:
Priority: major Milestone:
Component: default Keywords:
Cc: Also affects CVS HEAD (eXept version): no

Description

1) Create a simple test:

bla
   "test"
     
    self assert: (false)
  
   "/self assert: (false)      


2) The delete something e.g. the comment line "/self assert: (false)

3) then click on other test in your test class.

4) An Attention pop-up window appears

5) click on accept button

6) Debugger window with message:

[MessageNotUnderstood]: UndefinedObject does not understand: #notEmpty in process NewSystemBrowser [35]

appears.

7) Clicking on continue does save the testing method, but the debugger window should not appear.

Change History (7)

comment:1 by patrik.svestka@…, 5 years ago

Summary: Changing a test, clicking on other test and clcking on accept shows a debugger windowChanging a test, clicking on other test and clicking on accept shows a debugger window

comment:2 by jan vrany, 5 years ago

Could you please attach a full backtrace (in debugger, Context > Copy Walkback Text)?

comment:3 by patrik.svestka@…, 5 years ago

Of course.

Here is the whole Walkback Text:

[MessageNotUnderstood]: UndefinedObject does not understand: #notEmpty in process NewSystemBrowser [50394]

UndefinedObject(Object) >> doesNotUnderstand: Message(#notEmpty) {2125824} [16]
UndefinedObject(**NONE**) >> notEmpty {2129920} [48]
Tools::MethodList >> delayedUpdate:with:from: #methodInClass #(ASUserInputTests #bla a Method(unbound)) Smalltalk {2134016} [48]
Tools::MethodList(Object) >> perform:withArguments: #delayedUpdate:with:from: #(#methodInClass #(ASUserInputTests #bla a Method(unbound)) Smalltalk) {2138112} [170]
WindowEvent::MessageSendEvent(MessageSend) >> value {2142208} [4]
[] in WindowGroup>>processEventsWithModalGroup: >> value  {2146304} [201]
Block >> on:do:ensure: LastEventQuery (private in WindowGroup) [] in WindowGroup>>processEventsWithModalGroup: [] in WindowGroup>>processEventsWithModalGroup: {2150400} [15]
WindowGroup >> processEventsWithModalGroup: nil {2154496} [232]
[] in WindowGroup>>eventLoopWhile:onLeave: >> value  {2158592} [152]
SignalSet >> handle:do: [] in WindowGroup>>eventLoopWhile:onLeave: [] in WindowGroup>>eventLoopWhile:onLeave: {2162688} [14]
[] in WindowGroup>>eventLoopWhile:onLeave: >> value  {2166784} [81]
Block >> ensure: [] in WindowGroup>>startupWith: (optimized) {2170880} [12]
[] in WindowGroup>>eventLoopWhile:onLeave: >> value {2174976} [184]
WindowGroup::WindowGroupQuery class(Notification class) >> answer:do: WindowGroup(NewSystemBrowser) [] in WindowGroup>>eventLoopWhile:onLeave: {2179072} [10]
[] in WindowGroup::WindowGroupQuery class>>answer:do: >> value {2183168} [4]
Block >> ensure: [] in WindowGroup::WindowGroupQuery class>>answer:do: (optimized) {2187264} [12]
WindowGroup::WindowGroupQuery class >> answer:do: WindowGroup(NewSystemBrowser) [] in WindowGroup>>eventLoopWhile:onLeave: {2191360} [5]
WindowGroup >> eventLoopWhile:onLeave: [] in WindowGroup>>startupWith: (optimized) [] in WindowGroup>>startupWith: (optimized) {2195456} [25]
[] in WindowGroup>>startupWith: >> value {2199552} [26]
Block >> ensure: [] in WindowGroup>>startupWith: {2203648} [12]
[] in WindowGroup>>startupWith: >> value  {2207744} [27]
ExceptionHandlerSet >> handleDo: [] in WindowGroup>>startupWith: {2211840} [14]
[] in Process>>start >> value {2215936} [34]
Block >> on:do:ensure: SignalSet(AbortAllOperationRequest TerminateProcessRequest RestartProcessRequest) [] in Process>>start [] in Process>>start {2220032} [15]
Process >> start  {2224128} [35]
UndefinedObject >> nil  {2228224} [0]

comment:4 by jan vrany, 5 years ago

Status: newtesting

Attached patch 73c795373547 should fix the problem.

Please test it, but even if works, DON'T CLOSE this issue. I'd like to have a test but did not manage to write one. You may find an unfinished attempt attached as well.

comment:5 by patrik.svestka@…, 5 years ago

The patch works. Now presssing on accept does not bring debugger window forward. I'm leaving the issue opened.

Note: See TracTickets for help on using tickets.