Always print summary as last thing in validation

[CasperWA]

Fixes #699.

This minor change ensures the print_summary() method is always called before returning out of the validate_implementation() method.
Two changes were needed:

• Add method call to as_type "branch".
• Move around order of setting the valid property and printing the summary for the "no base info-branch".

I'm not completely sure about the latter change, but it seems to me the valid property is used in the printed summary, hence I thought it best to switch these statements around?

[codecov]

Codecov Report

Merging #700 (44fd911) into master (f9d3b8a) will increase coverage by 0.00%.
The diff coverage is 50.00%.

@@           Coverage Diff           @@
##           master     #700   +/-   ##
=======================================
  Coverage   93.18%   93.18%           
=======================================
  Files          61       61           
  Lines        3361     3362    +1     
=======================================
+ Hits         3132     3133    +1     
  Misses        229      229           

Flag	Coverage Δ
project	93.18% <50.00%> (+<0.01%)	⬆️
validator	66.15% <50.00%> (+0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
optimade/validator/validator.py	82.31% <50.00%> (+0.03%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f9d3b8a...44fd911. Read the comment docs.

[ml-evs]

Thanks for this @CasperWA, could you add a check to the end of tests.test_server_validation.test_as_type_with_validator?

[CasperWA]

I did something. I don't know if it's exactly what you had in mind?

[ml-evs]

Looks good! Merge/release as you please