Code review comment for lp:~benji/landscape-client/better-self-signed-cert-ux-3

Revision history for this message
Данило Шеган (danilo) wrote :

Seems to work fine against my self-signed Landscape. A few comments inline.

I am not too happy with the tests, since you seem to be mostly testing the wiring, and never that it actually works well together. I do understand that coming up with proper tests would require even more work, and with the understanding that this does not make the code base any worse, I am approving it.

Finally, I am not sure it's such a great idea to have Chris do one of the reviews _if_ he has contributed a lot of code/opinions while you were developing this branch. But you guys know best how much his mind is already "tainted" with the approach :)

review: Approve

« Back to merge proposal