Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error and garbled output when compiling with -node-test #26

Closed
brow opened this issue Dec 28, 2012 · 1 comment
Closed

Error and garbled output when compiling with -node-test #26

brow opened this issue Dec 28, 2012 · 1 comment
Milestone

Comments

@brow
Copy link
Contributor

brow commented Dec 28, 2012

$ cd src/forml
$ forml -node-test readme.forml
[*] Loading prelude.forml
[*] Loading readme.forml
[*] Closure readme.js  remote][remote]
[-] Closure [tests]
[*] Docs
                        ode.js]
Test failed at line 372

undefined

    Expected true
    Actual   {"error":{}}

Test failed at line 577

undefined

    Expected undefined
    Actual   undefined


[stdin]:98
})}}};this.split=b;var u=0,h={reportRunnerResults:function(){return phantom.ex
                                                                    ^
ReferenceError: phantom is not defined
    at Object.reportRunnerResults ([stdin]:98:321)
    at [object Object].reportRunnerResults ([stdin]:41:317)
    at [object Object].finishCallback ([stdin]:52:267)
    at [object Object].onComplete ([stdin]:51:492)
    at [object Object].next_ ([stdin]:51:19)
    at [stdin]:50:439
    at [object Object].finish ([stdin]:64:269)
    at [object Object].onComplete ([stdin]:65:466)
    at [object Object].next_ ([stdin]:51:19)
    at [stdin]:50:439
[X] Testing readme.js [Node.js]
texodus pushed a commit that referenced this issue Dec 29, 2012
@texodus
Copy link
Owner

texodus commented Dec 29, 2012

Not quite feature parity with the Phantom.js harness, as you will still need to use Phantom.js to pass the rendering portion of the prelude's own test suite (forml src/forml/prelude.forml), but otherwise fixed enough to pass the non-rendering test suites with identical output.

Thanks for the bugs!

@texodus texodus closed this as completed Dec 29, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants