Helping Nokogiri. Take II

Nokogiri and Pizza, what else can you ask for?

Ok. My fault. Now, let’s go get some work done.

First, if you haven’t done it, read the previous post about helping nokogiri and forgot about the script and memory leak. It seems that there are more important issues, so let’s fix them first. First run jruby test/test_jruby.rb from Nokogiri root. You’ll see a lot of errors (27 by now) and failures (14). Choose one, and get it green. After that, send me a pull request.

Ok. That sounds simple, but what if  the number of errors or failures raises? The rule I use is simple: keep the sum of both numbers going down and having a failure is better than having an error.

On the other hand, if you take a look at test/test_jruby.rb, you’ll see that not every test is in there. There is a reason for that. Even keeping the number of test low, you get a lot of errors/failures. If that annoys me with just 50 failures, imagine if I had a couple of hundreds errors. When all is right, I’ll add some more to keep the fun on.

Photo by Paul Johnston.

Author: Serabe

Mathematician, and Ruby and JavaScript programmer. Sometimes I speak at conferences and local meetups.

9 thoughts on “Helping Nokogiri. Take II”

  1. Maybe i did something wrong, but it says to me:

    /home/nils/HelpingJRubyNokogiri/nokogiri/lib/nokogiri.rb:17:in `require’: no such file to load — nokogiri/nokogiri (LoadError)
    from /home/nils/HelpingJRubyNokogiri/nokogiri/lib/nokogiri.rb:17
    from /home/nils/HelpingJRubyNokogiri/nokogiri/lib/nokogiri.rb:8:in `require’
    from /home/nils/HelpingJRubyNokogiri/nokogiri/test/helper.rb:8
    from /home/nils/HelpingJRubyNokogiri/nokogiri/test/helper.rb:2:in `require’
    from ./test/test_reader.rb:2
    from ./test/test_reader.rb:10:in `load’
    from test/test_jruby.rb:10

    Did i something wrong, or is there a file missing ? Obviously i am a noob, but i wanted so have look at how difficult it is to help, and if i am able to perhaps.

  2. Thank to you. Take a look at it and, if you have any other question, late me know. You can contact me here, gmail or github.

  3. Hrm. I have had a look at the tests now, but i guess i need a short briefing about the way to fix them, espacially WHERE the missing methods need to be added. Could you give a short example on this one perhaps?

    18) Error:
    NoMethodError: undefined method `line’ for #
    ./test/xml/test_node.rb:735:in `test_line’

    I have never done or tried something like this before, but i will be a good learner. 🙂

Leave a Reply

Your email address will not be published. Required fields are marked *