Issue 4855 in pharo: Add warning handling for tests

Status: New
Owner: [hidden email]
Labels: Milestone-1.4 Type-Bug

New issue 4855 by [hidden email]: Add warning handling for tests
http://code.google.com/p/pharo/issues/detail?id=4855

Now if a test raise a warning (as deprecation), now the test fails


Slice incoming


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: FixProposed

Comment #1 on issue 4855 by [hidden email]: Add warning handling for  
tests
http://code.google.com/p/pharo/issues/detail?id=4855

SLICE-Issue-4855-Add-warning-handling-for-tests-MonkeyGalactikalIntegrator.1  
In  
PharoInbox


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Comment #2 on issue 4855 by [hidden email]: Add warning handling for  
tests
http://code.google.com/p/pharo/issues/detail?id=4855

Run this class tests SystemChangeErrorHandlingTest to see the difference :)


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: MonkeyIsChecking

Comment #3 on issue 4855 by [hidden email]: Add warning handling for  
tests
http://code.google.com/p/pharo/issues/detail?id=4855#c3

The Monkey is currently checking this issue. Please don't change it!


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: ReviewNeeded

Comment #4 on issue 4855 by [hidden email]: Add warning handling for  
tests
http://code.google.com/p/pharo/issues/detail?id=4855#c4

This Issue has been marked for review by Ulysse the Monkey:

Test Results:
-------------
Failures:
        ClosureCompilerTest>>#testBlockNumbering
        ClosureCompilerTest>>#testDecompiledDoitMethodTempNames
        ClosureCompilerTest>>#testMethodAndNodeTempNames

----------------------------------------------------------
Loaded Source:  
SLICE-Issue-4855-Add-warning-handling-for-tests-MonkeyGalactikalIntegrator.1  
from http://ss3.gemstone.com/ss/PharoInbox
Tested using Pharo-1.4-14163-a on Croquet Closure Stack VM  
[StackInterpreter VMMaker-oscog-EstebanLorenzano.109] 21.0


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: FixToInclude

Comment #5 on issue 4855 by [hidden email]: Add warning handling for  
tests
http://code.google.com/p/pharo/issues/detail?id=4855

Tests were failing before the integration of the slice (monkey behavior  
will be changed :) )


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: MonkeyIsChecking

Comment #6 on issue 4855 by [hidden email]: Add warning handling for  
tests
http://code.google.com/p/pharo/issues/detail?id=4855#c6

The Monkey is currently checking this issue. Please don't change it!


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Comment #7 on issue 4855 by [hidden email]: Add warning handling for  
tests
http://code.google.com/p/pharo/issues/detail?id=4855#c7

The Monkey is currently checking this issue. Please don't change it!


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: ReviewNeeded

Comment #8 on issue 4855 by [hidden email]: Add warning handling for  
tests
http://code.google.com/p/pharo/issues/detail?id=4855#c8

This Issue has been marked for review by Ulysse the Monkey:

Test Results:
-------------
Failures:
        ClosureCompilerTest>>#testDecompiledDoitMethodTempNames
        ClosureCompilerTest>>#testMethodAndNodeTempNames
        ClosureCompilerTest>>#testBlockNumbering

----------------------------------------------------------
Loaded Source:  
SLICE-Issue-4855-Add-warning-handling-for-tests-MonkeyGalactikalIntegrator.1  
from http://ss3.gemstone.com/ss/PharoInbox
Tested using Pharo-1.4-14163-a on Croquet Closure Stack VM  
[StackInterpreter VMMaker-oscog-EstebanLorenzano.109] 21.0


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: MonkeyIsChecking

Comment #9 on issue 4855 by [hidden email]: Add warning handling for  
tests
http://code.google.com/p/pharo/issues/detail?id=4855#c9

The Monkey is currently checking this issue. Please don't change it!


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: ReviewNeeded
        Cc: [hidden email] [hidden email]

Comment #10 on issue 4855 by [hidden email]: Add warning handling  
for tests
http://code.google.com/p/pharo/issues/detail?id=4855

Those tests are failing, I have no clue why ... I just spent 2 hours trying  
to understand how it works ... When I debug the test method  
ClosureCompilerTest debug: #testBlockNumbering, the assert at the end is  
true, but when I use the test runner, it fails ...

Also, if I had an halt after the parse:class: and use the testrunner, I am  
not stopped, but if I put here before, the debugger pops up :s

If someone can help :s


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: FixProposed

Comment #11 on issue 4855 by [hidden email]: Add warning handling  
for tests
http://code.google.com/p/pharo/issues/detail?id=4855

New version of the slice :)


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: MonkeyIsChecking

Comment #12 on issue 4855 by [hidden email]: Add warning handling  
for tests
http://code.google.com/p/pharo/issues/detail?id=4855#c12

The Monkey is currently checking this issue. Please don't change it!


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: FixToInclude

Comment #13 on issue 4855 by [hidden email]: Add warning handling  
for tests
http://code.google.com/p/pharo/issues/detail?id=4855#c13

This Issue has been checked by Ulysse the Monkey
       
6239 tests passed in 00:01:06s:
===============================
        CollectionsTests-Arrayed (562)
        CollectionsTests-Atomic (12)
        CollectionsTests-Sequenceable (912)
        CollectionsTests-SplitJoin (27)
        CollectionsTests-Stack (16)
        CollectionsTests-Streams (37)
        CollectionsTests-Strings (590)
        CollectionsTests-Support (12)
        CollectionsTests-Text (40)
        CollectionsTests-Unordered (1977)
        CollectionsTests-Weak (751)
        CompilerTests (60)
        KernelTests-Chronology (574)
        KernelTests-Classes (59)
        KernelTests-Exception (2)
        KernelTests-Methods (177)
        KernelTests-Numbers (232)
        KernelTests-Objects (85)
        KernelTests-Pragmas (3)
        KernelTests-Processes (22)
        SUnit-Core-Extensions (6)
        SUnit-Core-Utilities (6)
        SUnit-Tests-Core (77)

----------------------------------------------------------
Loaded Source:  
SLICE-Issue-4855-Add-warning-handling-for-tests-BenjaminVanRyseghem.2 from  
http://ss3.gemstone.com/ss/PharoInbox
Tested using Pharo-1.4-14167-a on Croquet Closure Stack VM  
[StackInterpreter VMMaker-oscog-EstebanLorenzano.109] 21.0


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker

Updates:
        Status: Integrated

Comment #14 on issue 4855 by [hidden email]: Add warning handling  
for tests
http://code.google.com/p/pharo/issues/detail?id=4855

in 14176


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker