openBackhaul / NetExplorerProxy

Interface towards the NetExplorer
Apache License 2.0
1 stars 0 forks source link

Modify testcase collection in regards of collectionName parameter #45

Open kmohr-soprasteria opened 2 months ago

kmohr-soprasteria commented 2 months ago

Currently parameter collectionName is set once for the complete collection. Would be better if this is done in each request

kmohr-soprasteria commented 2 months ago

Note: some services also are offered by MATR, checking those should be possible with the same collection, however: in that case just the collectionName for the run must be changed - this could be done as an additional parameter in the loadFile (with nep being the default).

kmohr-soprasteria commented 2 months ago

Asked Thorsten on his feedback; in case this needs to be changed still, this will be done with spec_v1.0.1.

kmohr-soprasteria commented 2 months ago

Thorsten's feedback:

I only noticed later that the InterfaceValidator has to address different servers. Then of course I was happy that I had such a great, flexible structure ;) :)

I actually see this coming in all applications because:

  • The applications should be tested against mocks or simulators.
  • The TestCaseCollection should be supplemented based on Analysis or FriendlyUser observations
  • New TestCases might require different inputs
  • Other inputs are provided by other simulators
  • Other simulators could mean: o Existing simulators are supplemented with additional paths o Additional simulators are placed next to them and addressed depending on the TestCase

The second variant is more flexible because the simulators can be created by different people without coordination. You don't need to change this now, but I suspect it would be better if the collectionName was set at the collection level.