Unusual case in test_load

[prescod]

David, do you remember what you were trying to test here? It seems odd to have a Lookup declaration on an Id column. (i.e. the column is both a primary key and a lookup to another table)

Am I missing something or is this a typo?

https://github.com/SFDO-Tooling/CumulusCI/blob/158a2d4356f1ac149dd72a13e961ccef8d0f9ef9/cumulusci/tasks/bulkdata/tests/test_load.py#L352

[davidmreed]

Yeah, that is a horrible hack. I think the original intent was a negative test - to include a lookup that was guaranteed not to generate an after step. No idea why I called it Id. I'll fix it.

[prescod]

I think that there were a couple of places the pattern occurred in that file. Thanks for looking at it.

[davisagli]

@davidmreed this is still outstanding, right?

[davidmreed]

Yes, @davisagli, I need to revisit that test case.