POETSII / Orchestrator

The Orchestrator is the configuration and run-time management system for POETS platforms.
1 stars 1 forks source link

xml v4 parsing patches #264

Closed m8pple closed 3 years ago

m8pple commented 3 years ago

This is a re-packaging of the v4 parsing patches proposed last month, which should merge straight onto the current development branch.

The main purpose is to just get the PIP20 v4 XML parsing and running.

It contains the test suite and reference XML I suggested previously in #208, but I get the impression there is some reluctance to have unit tests in the Orchestrator? So if having tests in the Orchestrator repo is blocking merging, then just strip them out and take just the syntax patches. I can just maintain my own orchestrator test-suite somewhere else (though I still really don't understand how you develop and maintain something like the Orchestrator without a good test suite).

mvousden commented 3 years ago

I have not performed any timing tests. It should be noted that there may still be some Soton XML (I am thinking Third year projects) that has not been updated for these changes.

I have emailed @kdalal2000 about this.