This weekend, I’m overhauling some of the names code in citeproc-js,
to fix some long-standing bugs in attribute inheritance. One of the
test failures doesn’t actually look like a failure, and I’d like to
confirm that I’m not misreading the spec.
The test is this one:
In the fixture, a value for et-al-min is set on separate cs:name nodes
inside cs:citation and cs:bibliography, and a value of 2 for
et-al-use-first is set on the cs:citation node. The test renders the
The bibliography comes out with all three names given in the input.
The fixture RESULT has one name, truncated by et al.
I don’t see anything in the spec about defaulting to a value of 1 for
et-al-use-first, and it says that “[u]se of these two attributes
enables et-al abbreviation.” Since the test provides a value for only
one of the attributes inside cs:bibliography, it sounds like listing
all three names there is the right thing to do (and the original
RESULT string was wrong).
So … should I amend the test there?