It's nice of you to defer requesting a review until all tests are passing, but I think a lot of us don't observe that.
Was there a preimplementation call?
Was there any lint?
AIUI PEP8 requires that items within a class definition be separated by a single blank line, so a bunch of your webservice_error(400) lines need a blank line in front of them.
Why are you using export_write_operation rather than export_factory_operation on branchtarget?
On IHasCodeImports, you claim that newCodeImport returns BranchMergeProposal. I think this is inaccurate.
What would you think about expressing xx-code-import.txt's "Exceptions" heading as a unit test? Failing that, it needs two blank lines separating it from the previous section.
It's nice of you to defer requesting a review until all tests are passing, but I think a lot of us don't observe that.
Was there a preimplementation call?
Was there any lint?
AIUI PEP8 requires that items within a class definition be separated by a single blank line, so a bunch of your webservice_ error(400) lines need a blank line in front of them.
Why are you using export_ write_operation rather than export_ factory_ operation on branchtarget?
On IHasCodeImports, you claim that newCodeImport returns BranchMergeProp osal. I think this is inaccurate.
What would you think about expressing xx-code- import. txt's "Exceptions" heading as a unit test? Failing that, it needs two blank lines separating it from the previous section.