string monoid class should define one()
Method 'one' is supposed to return the identity element of a monoid (or I guess of any magma). String monoids should define it to be the empty string. Not only have they not, the default implementation throws an error:
sage: BinaryStrings().one()
TypeError: Argument x (= 1) is of the wrong type.