Skip to content
This repository was archived by the owner on Oct 5, 2023. It is now read-only.

Fix blank strings test #7

Closed
wants to merge 1 commit into from
Closed

Fix blank strings test #7

wants to merge 1 commit into from

Conversation

ikokostya
Copy link
Contributor

$ make test


  ․

  0 passing (9ms)
  1 failing

  1)  should support blank strings:

      AssertionError: expected { declaration: undefined, root: undefined } to equal { declaration: null, root: null } (at declaration, A has undefined and B has null)
      + expected - actual

      +{
      +  "declaration": null
      +  "root": null
      +}
      -{}

      at Assertion.fail (/home/kostya/proj/xml-parser/node_modules/should/lib/assertion.js:113:17)
      at Assertion.prop.(anonymous function) [as eql] (/home/kostya/proj/xml-parser/node_modules/should/lib/assertion.js:39:14)
      at Context.<anonymous> (/home/kostya/proj/xml-parser/test/index.js:7:15)
      at callFn (/home/kostya/proj/xml-parser/node_modules/mocha/lib/runnable.js:250:21)
      at Test.Runnable.run (/home/kostya/proj/xml-parser/node_modules/mocha/lib/runnable.js:243:7)
      at Runner.runTest (/home/kostya/proj/xml-parser/node_modules/mocha/lib/runner.js:373:10)
      at /home/kostya/proj/xml-parser/node_modules/mocha/lib/runner.js:451:12
      at next (/home/kostya/proj/xml-parser/node_modules/mocha/lib/runner.js:298:14)
      at /home/kostya/proj/xml-parser/node_modules/mocha/lib/runner.js:308:7
      at next (/home/kostya/proj/xml-parser/node_modules/mocha/lib/runner.js:246:23)
      at Immediate._onImmediate (/home/kostya/proj/xml-parser/node_modules/mocha/lib/runner.js:275:5)
      at processImmediate [as _immediateCallback] (timers.js:358:17)



Makefile:3: recipe for target 'test' failed
make: *** [test] Error 1

@gjohnson
Copy link
Contributor

gjohnson commented Jun 5, 2015

Fixed in #11 (sorry I saw theirs first)

@gjohnson gjohnson closed this Jun 5, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants