Fix a couple of tests

Summary: One didn't work if you borrowed a machine that someone else
occasionally ran tests on (due to permission issues), another doesn't
work on internal FB build servers which have an older libxml.

Reviewed By: @paulbiss

Differential Revision: D1680048

Signature: t1:1680048:1415916748:8fbddd359d35672b03b697de4eb10e527e144143
Remove bad assert from BaseVector::data()

Summary: BaseVector::data() had an assert that was checking if m_data[0] was a Cell.
This assert is bogus because the Vector could be empty, and in such cases
it is not valid to dereference m_data[0]. It turns out that this assert was
redundant anywhere because the only place that uses BaseVector::data()
already asserts that each element in the Vector is a Cell (also, there are
a bunch of asserts throughout ext_collections that make sure that each
element is a Cell).

Reviewed By: @elgenie

Differential Revision: D1680393

Signature: t1:1680393:1415922724:0331eb4ed100231a24517cf4870736a07f56e275