Opened 15 years ago
Closed 15 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)
Change History (2)
Changed 15 years ago by
| Attachment: | sassy.patch added | 
|---|
comment:1 Changed 15 years ago by
| Resolution: | → fixed | 
|---|---|
| Status: | new → closed | 
Note: See
        TracTickets for help on using
        tickets.
    

Test fixes