Add option to disable the build output completion if are tests not passed

[martonmiklos]

Fixes #5037

[netlify]

✅ Deploy Preview for inventree-web-pui-preview canceled.

Name	Link
🔨 Latest commit	09dc491
🔍 Latest deploy log	https://app.netlify.com/sites/inventree-web-pui-preview/deploys/65d13a4c5c26430007f569b3

[SchrodingersGat]

@martonmiklos we will need some unit tests here to ensure that the new checks work as expected. And to ensure that the correct errors are reported via the API

[SchrodingersGat]

@martonmiklos have you had a chance to look at this?

[martonmiklos]

@martonmiklos have you had a chance to look at this?

Yes I started to write tests, but have not finished them yet.
However I have not started looking into the PUI config page yet.

[SchrodingersGat]

@martonmiklos i am back on deck now, so if you can address the requirements above I'm happy to review again :)

[SchrodingersGat]

@martonmiklos any progress on this one? :)

[SchrodingersGat]

@martonmiklos it would be great to be able to get this into 0.14.0 - can you please address the outstanding issues? :)

[martonmiklos]

@martonmiklos it would be great to be able to get this into 0.14.0 - can you please address the outstanding issues? :)

@SchrodingersGat I do not want to be fussing on this one, but I am still under the impression that the non-trackable (serial number less) parts cannot have tests assigned. If that's true adding the serial number to the error messages makes sense in most cases.

If you say you do not want to see the serial in the error message I can understand and will approve the decision, (and implement changes accordingly) just I have not get exact feedback for my claims (in the thread above) yet ;)

[SchrodingersGat]

@martonmiklos ok I see the point there, the subtlety was lost on me before. I'll reply inline to the error message comments

[SchrodingersGat]

@martonmiklos thanks for the contribution :)