We discussed a few style improvements on IRC:
* Keeping the ISourcePackageRecipeSource utility in a variable to avoid breaking the "if" line in an awkward way.
* Double-quoting rather than single-quoting stirngs that may contain apostrophes.
* A single data.get('name') instead of separate existence check and retrieval.
Unfortunately there seems to be no easy way to ask the test browser for the error message associated with the "name" field, so we'll continue to find them by class and then use list indexing.
We discussed a few style improvements on IRC: ecipeSource utility in a variable to avoid breaking the "if" line in an awkward way.
* Keeping the ISourcePackageR
* Double-quoting rather than single-quoting stirngs that may contain apostrophes.
* A single data.get('name') instead of separate existence check and retrieval.
Unfortunately there seems to be no easy way to ask the test browser for the error message associated with the "name" field, so we'll continue to find them by class and then use list indexing.