WAFormCharEncodingTest

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

WAFormCharEncodingTest

Michael Lucas-Smith-3
Hi All,

Do #testBasicForm, #testDefaultAction, #testDefaultActionGet and
#testIsGet on WAFormCharEncodingTest fail for anyone else? For me it
fails bceause the form accept-charset becomes utf-16, not the expected
utf-8 -- but it seems as though it *should* be utf-16 because
#createHandlers specifically sets the charSet to be utf-16.

Cheers,
Michael
_______________________________________________
seaside-dev mailing list
[hidden email]
http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev
Reply | Threaded
Open this post in threaded view
|

Re: WAFormCharEncodingTest

Philippe Marschall
2010/5/15 Michael Lucas-Smith <[hidden email]>:
> Hi All,
>
> Do #testBasicForm, #testDefaultAction, #testDefaultActionGet and #testIsGet
> on WAFormCharEncodingTest fail for anyone else? For me it fails bceause the
> form accept-charset becomes utf-16, not the expected utf-8 -- but it seems
> as though it *should* be utf-16 because #createHandlers specifically sets
> the charSet to be utf-16.

The version in my image (lr.47) expects utf-16 and gets utf-16. What
version do you have?

Cheers
Philippe
_______________________________________________
seaside-dev mailing list
[hidden email]
http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev
Reply | Threaded
Open this post in threaded view
|

Re: WAFormCharEncodingTest

Michael Lucas-Smith-3
On 5/15/10 4:14 AM, Philippe Marschall wrote:

> 2010/5/15 Michael Lucas-Smith<[hidden email]>:
>    
>> Hi All,
>>
>> Do #testBasicForm, #testDefaultAction, #testDefaultActionGet and #testIsGet
>> on WAFormCharEncodingTest fail for anyone else? For me it fails bceause the
>> form accept-charset becomes utf-16, not the expected utf-8 -- but it seems
>> as though it *should* be utf-16 because #createHandlers specifically sets
>> the charSet to be utf-16.
>>      
> The version in my image (lr.47) expects utf-16 and gets utf-16. What
> version do you have?
>    
lr.47

WAFormCharEncodingTest inherits the tests from WAFormTagTest which have
'utf-8' hard coded in their expected strings.

Another surprise on this class is #testAnchorAttributes inherited from
WACanvasBrushTest. It breaks for me in
#urlForDocument:mimeType:fileName: with #registry returning nil.

Michael
_______________________________________________
seaside-dev mailing list
[hidden email]
http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev
Reply | Threaded
Open this post in threaded view
|

Re: WAFormCharEncodingTest

Philippe Marschall
2010/5/17 Michael Lucas-Smith <[hidden email]>:

> On 5/15/10 4:14 AM, Philippe Marschall wrote:
>>
>> 2010/5/15 Michael Lucas-Smith<[hidden email]>:
>>
>>>
>>> Hi All,
>>>
>>> Do #testBasicForm, #testDefaultAction, #testDefaultActionGet and
>>> #testIsGet
>>> on WAFormCharEncodingTest fail for anyone else? For me it fails bceause
>>> the
>>> form accept-charset becomes utf-16, not the expected utf-8 -- but it
>>> seems
>>> as though it *should* be utf-16 because #createHandlers specifically sets
>>> the charSet to be utf-16.
>>>
>>
>> The version in my image (lr.47) expects utf-16 and gets utf-16. What
>> version do you have?
>>
>
> lr.47
>
> WAFormCharEncodingTest inherits the tests from WAFormTagTest...

Not in Pharo.

Cheers
Philippe
_______________________________________________
seaside-dev mailing list
[hidden email]
http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev
Reply | Threaded
Open this post in threaded view
|

Re: WAFormCharEncodingTest

Michael Lucas-Smith-3
On 5/17/10 9:38 AM, Philippe Marschall wrote:

> 2010/5/17 Michael Lucas-Smith<[hidden email]>:
>    
>> On 5/15/10 4:14 AM, Philippe Marschall wrote:
>>      
>>> 2010/5/15 Michael Lucas-Smith<[hidden email]>:
>>>
>>>        
>>>> Hi All,
>>>>
>>>> Do #testBasicForm, #testDefaultAction, #testDefaultActionGet and
>>>> #testIsGet
>>>> on WAFormCharEncodingTest fail for anyone else? For me it fails bceause
>>>> the
>>>> form accept-charset becomes utf-16, not the expected utf-8 -- but it
>>>> seems
>>>> as though it *should* be utf-16 because #createHandlers specifically sets
>>>> the charSet to be utf-16.
>>>>
>>>>          
>>> The version in my image (lr.47) expects utf-16 and gets utf-16. What
>>> version do you have?
>>>
>>>        
>> lr.47
>>
>> WAFormCharEncodingTest inherits the tests from WAFormTagTest...
>>      
> Not in Pharo.
>    
How does this work in Pharo? The test* methods are in the superclass,
why aren't they run in the subclass?

Michael
_______________________________________________
seaside-dev mailing list
[hidden email]
http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev
Reply | Threaded
Open this post in threaded view
|

Re: WAFormCharEncodingTest

Philippe Marschall
2010/5/17 Michael Lucas-Smith <[hidden email]>:

> On 5/17/10 9:38 AM, Philippe Marschall wrote:
>>
>> 2010/5/17 Michael Lucas-Smith<[hidden email]>:
>>
>>>
>>> On 5/15/10 4:14 AM, Philippe Marschall wrote:
>>>
>>>>
>>>> 2010/5/15 Michael Lucas-Smith<[hidden email]>:
>>>>
>>>>
>>>>>
>>>>> Hi All,
>>>>>
>>>>> Do #testBasicForm, #testDefaultAction, #testDefaultActionGet and
>>>>> #testIsGet
>>>>> on WAFormCharEncodingTest fail for anyone else? For me it fails bceause
>>>>> the
>>>>> form accept-charset becomes utf-16, not the expected utf-8 -- but it
>>>>> seems
>>>>> as though it *should* be utf-16 because #createHandlers specifically
>>>>> sets
>>>>> the charSet to be utf-16.
>>>>>
>>>>>
>>>>
>>>> The version in my image (lr.47) expects utf-16 and gets utf-16. What
>>>> version do you have?
>>>>
>>>>
>>>
>>> lr.47
>>>
>>> WAFormCharEncodingTest inherits the tests from WAFormTagTest...
>>>
>>
>> Not in Pharo.
>>
>
> How does this work in Pharo? The test* methods are in the superclass, why
> aren't they run in the subclass?

Dunno, I simply get "1 run, 1 passes".

Cheers
Philippe
_______________________________________________
seaside-dev mailing list
[hidden email]
http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev
Reply | Threaded
Open this post in threaded view
|

Re: WAFormCharEncodingTest

Philippe Marschall
In reply to this post by Michael Lucas-Smith-3
2010/5/17 Michael Lucas-Smith <[hidden email]>:
> ....
>
> How does this work in Pharo? The test* methods are in the superclass, why
> aren't they run in the subclass?

shouldInheritSelectors
        "I should inherit from an Abstract superclass but not from a concrete
one by default, unless I have no testSelectors in which case I must be
expecting to inherit them from my superclass.  If a test case with
selectors wants to inherit selectors from a concrete superclass,
override this to true in that subclass."

        ^self superclass isAbstract
                or: [self testSelectors isEmpty]

"$QA Ignore:Sends system method(superclass)$"
                       

Cheers
Philippe
_______________________________________________
seaside-dev mailing list
[hidden email]
http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev
Reply | Threaded
Open this post in threaded view
|

Re: WAFormCharEncodingTest

Philippe Marschall
2010/5/17 Philippe Marschall <[hidden email]>:

> 2010/5/17 Michael Lucas-Smith <[hidden email]>:
>> ....
>>
>> How does this work in Pharo? The test* methods are in the superclass, why
>> aren't they run in the subclass?
>
> shouldInheritSelectors
>        "I should inherit from an Abstract superclass but not from a concrete
> one by default, unless I have no testSelectors in which case I must be
> expecting to inherit them from my superclass.  If a test case with
> selectors wants to inherit selectors from a concrete superclass,
> override this to true in that subclass."
>
>        ^self superclass isAbstract
>                or: [self testSelectors isEmpty]
>
> "$QA Ignore:Sends system method(superclass)$"

Maybe we should do WATestCase ;-)

Cheers
Philippe
_______________________________________________
seaside-dev mailing list
[hidden email]
http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev
Reply | Threaded
Open this post in threaded view
|

Re: WAFormCharEncodingTest

Michael Lucas-Smith-3
On 5/17/10 10:02 AM, Philippe Marschall wrote:

> 2010/5/17 Philippe Marschall<[hidden email]>:
>    
>> 2010/5/17 Michael Lucas-Smith<[hidden email]>:
>>      
>>> ....
>>>
>>> How does this work in Pharo? The test* methods are in the superclass, why
>>> aren't they run in the subclass?
>>>        
>> shouldInheritSelectors
>>         "I should inherit from an Abstract superclass but not from a concrete
>> one by default, unless I have no testSelectors in which case I must be
>> expecting to inherit them from my superclass.  If a test case with
>> selectors wants to inherit selectors from a concrete superclass,
>> override this to true in that subclass."
>>
>>         ^self superclass isAbstract
>>                 or: [self testSelectors isEmpty]
>>
>> "$QA Ignore:Sends system method(superclass)$"
>>      
>    
Ah that makes sense. SUnitToo and SUnit deviate at this point.
> Maybe we should do WATestCase ;-)
>    
Or restructure the tests so they don't inherit as much. Or I could
ignore those failing tests. At least now I understand why it is so.

Michael
_______________________________________________
seaside-dev mailing list
[hidden email]
http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev
Reply | Threaded
Open this post in threaded view
|

Re: WAFormCharEncodingTest

Lukas Renggli
> Ah that makes sense. SUnitToo and SUnit deviate at this point.
>
> Maybe we should do WATestCase ;-)
>
>
> Or restructure the tests so they don't inherit as much. Or I could ignore those failing tests. At least now I understand why it is so.

The easiest would be if everybody used normal SUnit. We never had any
compatibility problems with that.

Lukas

--
Lukas Renggli
www.lukas-renggli.ch
_______________________________________________
seaside-dev mailing list
[hidden email]
http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev
Reply | Threaded
Open this post in threaded view
|

Re: WAFormCharEncodingTest

Philippe Marschall
In reply to this post by Michael Lucas-Smith-3
2010/5/17 Michael Lucas-Smith <[hidden email]>:
> ....
>
> Ah that makes sense. SUnitToo and SUnit deviate at this point.

It would be cool if you could run them with SUnit instead of SUnitToo.
SUnit has pretty well defined semantics across all dialects and we
never had any problems with it.

>>
>> Maybe we should do WATestCase ;-)
>>
>
> Or restructure the tests so they don't inherit as much. Or I could ignore
> those failing tests. At least now I understand why it is so.

Seaside-Tests-Canvas-pmm.48

Cheers
Philippe
_______________________________________________
seaside-dev mailing list
[hidden email]
http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev
Reply | Threaded
Open this post in threaded view
|

Re: WAFormCharEncodingTest

Michael Lucas-Smith-3
On 5/18/10 1:03 PM, Philippe Marschall wrote:

> 2010/5/17 Michael Lucas-Smith<[hidden email]>:
>    
>> ....
>>
>> Ah that makes sense. SUnitToo and SUnit deviate at this point.
>>      
> It would be cool if you could run them with SUnit instead of SUnitToo.
> SUnit has pretty well defined semantics across all dialects and we
> never had any problems with it.
>
>    
We may do that, but SUnitToo also gives us some other advantages.
>>> Maybe we should do WATestCase ;-)
>>>
>>>        
>> Or restructure the tests so they don't inherit as much. Or I could ignore
>> those failing tests. At least now I understand why it is so.
>>      
> Seaside-Tests-Canvas-pmm.48
>    
Thanks for this, I really appreciate it.

Michael
_______________________________________________
seaside-dev mailing list
[hidden email]
http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev