Issue 6934 in pharo: move encodeForHTTP tests to own class

Status: Accepted
Owner: [hidden email]
Labels: Type-Bug Milestone-2.0

New issue 6934 by [hidden email]: move encodeForHTTP tests to own  
class
http://code.google.com/p/pharo/issues/detail?id=6934

encodeForHTTP and similar methods are placed in Network-Protocols method  
but the tests for it are part of standard kernel string tests. So this  
tests are failing on Pharo Kernel:
CollectionsTests.Strings.StringTest.testEncodeForHTTP
CollectionsTests.Strings.StringTest.testPercentEncodingJa

Move the tests to own class HTTPEncodingTest placed in  
NetworkTests-Protocols.


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

Updates:
        Status: FixReviewNeeded

Comment #1 on issue 6934 by [hidden email]: move encodeForHTTP  
tests to own class
http://code.google.com/p/pharo/issues/detail?id=6934

SLICE-Issue-6934-move-encodeForHTTP-tests-to-own-class-PavelKrivanek.1


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

Updates:
        Status: MonkeyIsChecking

Comment #2 on issue 6934 by [hidden email]: move encodeForHTTP tests  
to own class
http://code.google.com/p/pharo/issues/detail?id=6934#c2

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: WorkNeeded

Comment #3 on issue 6934 by [hidden email]: move encodeForHTTP tests  
to own class
http://code.google.com/p/pharo/issues/detail?id=6934#c3

Monkey went bananas:
--------------------
Error while loading  
SLICE-Issue-6934-move-encodeForHTTP-tests-to-own-class-PavelKrivanek.1 from  
http://ss3.gemstone.com/ss/PharoInbox:
        MessageNotUnderstood: GoferResolvedReference>>merge
  1: GoferResolvedReference(Object)>>doesNotUnderstand: #merge
  2: [self slice merge] in UlysseTheMonkey>>loadSlice
  3: BlockClosure>>on:do:
  4: UlysseTheMonkey>>loadSlice
  5: [self loadSlice] in UlysseTheMonkey>>loadAndTestSlice
  6: [loadStopwatch reset activate.
                aBlock value.
                loadStopwatch suspend.
                testResult := self runTests.
                self removeFalseErrors] in UlysseTheMonkey>>testChangeImpact:
  7: BlockClosure>>on:do:
  8: UlysseTheMonkey>>testChangeImpact:
  9: UlysseTheMonkey>>loadAndTestSlice
10: UlysseTheMonkey>>loadAndTest
        ...
----------------------------------------------------------
Loaded Source:  
SLICE-Issue-6934-move-encodeForHTTP-tests-to-own-class-PavelKrivanek.1 from  
http://ss3.gemstone.com/ss/PharoInbox
Tested using Pharo-2.0-20384-a on CoInterpreter  
VMMaker-oscog-EstebanLorenzano.166 uuid:  
5773fcb9-2982-4507-8a9e-4308ec33731e Nov  6 2012
StackToRegisterMappingCogit VMMaker-oscog-EstebanLorenzano.166 uuid:  
5773fcb9-2982-4507-8a9e-4308ec33731e Nov  6 2012
https://git.gitorious.org/cogvm/blessed.git Commit:  
a7bf128cd3b9389e77ea0cdec9a515282926a483 Date: 2012-11-02 14:17:44 +0100  
By: Esteban Lorenzano <[hidden email]> Jenkins build #136


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

Updates:
        Status: HumanReviewNeeded

Comment #4 on issue 6934 by [hidden email]: move encodeForHTTP tests  
to own class
http://code.google.com/p/pharo/issues/detail?id=6934

(No comment was entered for this change.)


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

Updates:
        Status: FixToInclude

Comment #5 on issue 6934 by [hidden email]: move encodeForHTTP tests  
to own class
http://code.google.com/p/pharo/issues/detail?id=6934

(No comment was entered for this change.)


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

Updates:
        Status: Integrated

Comment #6 on issue 6934 by [hidden email]: move encodeForHTTP tests  
to own class
http://code.google.com/p/pharo/issues/detail?id=6934

in 2.0 385


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