Opened 13 years ago

Closed 13 years ago

#536 closed defect (fixed)

sassy - broken tests and missing export

Reported by: sjamaan Owned by:
Priority: major Milestone:
Component: extensions Version: 4.6.x
Keywords: sassy Cc:
Estimated difficulty:

Description

The Sassy egg fails to test properly. See http://tests.call-cc.org/experimental/openbsd/x86/2011/03/12/salmonella-report/tests/sassy.html

The fix is simple: the run.scm wrapper should not just call require-library but also import the libraries.

After applying this fix, I get another error:

Error: unbound variable: sassy-data-size

        Call history:

        <eval>    [test-misc] (make-list 24 144)
        <eval>    [test-misc] (list 80)
        <eval>    [test-misc] (succeed "text-align")
        <eval>    [succeed] (display "test passed: ")
        <eval>    [succeed] (display x)
        <eval>    [succeed] (newline)
        <eval>    [direc-test] (test-data)
        <eval>    [test-data] (sassy (quote ((data (label foo (dwords "ab")) (align 8) (label bar (dwords 100 quux)) (dwords -3242......
        <eval>    [test-data] (= 40 (sassy-data-size o))
        <eval>    [test-data] (sassy-data-size o)       <--

Looking at sassy-48.scm and comparing it with sassy-chicken.scm, it seems that sassy-data-size is simply missing from the export list.

Here's a patch that fixes both issues.

Attachments (1)

sassy.patch (937 bytes) - added by sjamaan 13 years ago.
Test fixes

Download all attachments as: .zip

Change History (2)

Changed 13 years ago by sjamaan

Attachment: sassy.patch added

Test fixes

comment:1 Changed 13 years ago by felix winkelmann

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.