Re: [squeak-dev] Seaside on Squeak?
gettimothy <[email protected]>
| Newsgroups | gmane.comp.lang.smalltalk.squeak.seaside,gmane.comp.lang.smalltalk.squeak.general |
|---|---|
| Message-ID | <[email protected]> |
Regarding testGenericCodecMacRoman Within testGenericCodecMacRoman changing mac-roman to macroman fixes the test. I believe this is a valid change based on the following change of reasoning. Those subclasses are: GRNullCodec GRPharoGenericCodec GRPharoUtf8Codec GRPharoLatin1Codec What this test does is loop through each subclass of GRCodec and asks each one if they support the encoding 'mac-roman' The subclasses either hard-code their encodings in the class side supportedEncodingNames method or they define it as a subset of TextConverter allEncodingNames TextConverter allEncodingNames inspect TextConverter allEncodingNames select:[:element | element asString beginsWith:'mac'] MacRomanTextConverter encodingNames ---- On Wed, 05 Dec 2018 08:33:02 -0500 David T. Lewis <[email protected]> wrote ---- Replying on the squeak-dev list to get some more eyes on the issue. Dave On Tue, Dec 04, 2018 at 10:17:17PM +0100, Johan Brichau wrote: > Hi, > > Two tests are currently still failing for Squeak [1]. > - testNamedTempAt -> fails on travis but works in an interactive image??? dunno what???s going on but it???s an expected failure in Pharo for a long time as well??? > - testGenericCodecMacRoman -> it seems the preferential name for the Mac-Roman encoding became ???macroman??? in Squeak instead of ???mac-roman???. Not sure what to think about this one??? > > Anybody from the Squeak community who wants to step in and check on these? > > cheers > Johan > > > [1] https://travis-ci.org/SeasideSt/Seaside/jobs/463542906 > _______________________________________________ > seaside mailing list > [email protected] > http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside _______________________________________________ seaside mailing list [email protected] http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside