-
Notifications
You must be signed in to change notification settings - Fork 107
#555 - RSS feed parser not grabbing entire content #555
Comments
Nice find. Perhaps it would be better to have the test failing since it turns out that the parser has issues. Would you or @jh0720 be interested in further investigating the issue? |
The test now fails - |
@jameskunjoolee please do investigate, I've assigned you to the issue. I recommend putting the CLI tests on hold on upping the priority on this. Tests let us discover issues like this, and when a test fails, the problem should be fixed as soon possible (all tests should be passing). |
@jameskunjoolee to respond to your question on IRC, the branch for this fix should be based off of upstream/master, not your rss testing branch. The fix is independent of the tests. |
Update: |
This test will be run but no traceback will be reported when it fails. Instead, terminal reporting will list it as "expected to fail". Related to #555 http://pytest.org/latest/skipping.html#xfail
There are two instances in which the parser does not grab the entire content of a feed. They are both lists.
Instance 1
Expected:
Actual:
Instance 2
Expected:
Actual:
Relevant: #506.
The text was updated successfully, but these errors were encountered: