slapos_accounting: Use specific trade condition if Hosting Subscription comes from a Subscription Request

This allow the system differ the 2 scenarios.

When the Hosting Subscription comes from a Subscription Request, the Trade Condition applied is a different one (slapos_aggregated_subscription_trade_condition).

Update all related scripts to work accordinally.

SPL on this case are started as soon they are created and a payment generated.

This is used for subscription mode, like rapid.space VM-Like services.

Subscription and Trial uses setDefaultEmailText when create a Trial/Subscription as annonymous which will trigger this workflow and raise Unauthorized when invoke Coordinate_beforeCoordinateTextChange.

As Reacheability/Unreacheability is not tracked on SlapOS master for now, so this commits remove the interaction workflow for Email (only).

Code migrated from testSlapOSERP5DefaultScenario.

This would guarantee that the ungenerated payment from others tests are generated, while you are running live tests.

/reviewed-on nexedi/slapos.core!99

This fixes a regression introduced in commit 7467ef4c
("Add Python 3 support").

/reviewed-on nexedi/slapos.core!98

The reload part is mainly used for tests, module is reloaded if the promise file has changed. Here, __file__ ends with pyc when promise_path ends with py.

Since the compatibility with python3 the code change to not encode the string anymore.

@jm @bminusl can you take a look if it is enough (I will improve commit message after)

This at least make the test pass.

/reviewed-on nexedi/slapos.core!94

Just like reference implementation support empty keys.

/cc @rafael 

This makes slapproxy behave the same way as SlapOS Master for empty keys while requesting slaves.

/reviewed-on nexedi/slapos.core!96

There is now `EmptyResult` object which is an empty promise result. This result is send if the promise test or anomaly was skiped (if disabled).
To prevent raise because `EmptyResult` is unknonwn on old slapos-node with `slapos.core < 1.4.17` we will send an empty TestResult instead.

Support for promise test-less or anomaly-less allow to disable either the test phase or the anomaly phase.

by default, test and anomaly are enabled, `__is_tested = True` and `__is_anomaly_detected = True`. Test will run when buildout is processing the computer partition, and anomaly when the partition is processed.

Call `self.setTestLess()` in `__init__.py` of the promise will disable Test and set `__is_tested = False`. Call `self.setAnomalyLess()` will rather disable Anomaly and set `__is_anomaly_detected = False`.


    def __init__(self, config):
        GenericPromise.__init__(self, config)
        # Skip test check on this promise
        self.setTestLess()

If the promise is test less, then slapgrid will not run the promise while processing the partition which mean that the promise will not report error, only Anomaly will be checked. Samething, if promise is anomaly less, only test will run when slapgrid is processing the partition.

/reviewed-on nexedi/slapos.core!93

Support for promise test-less or anomaly-less allow to disable either the test phase or the anomaly phase.

by default, test and anomaly are enabled. Test will run when buildout is processing the computer partition, and anomaly when the partition is processed.

Call `self.setTestLess()` in __init__.py of the promise will disable Test and set `__is_tested = False`. Call `self.setAnomalyLess()` will rather disable Anomaly and set `__is_anomaly_detected = False`.

If the promise is test less, then slapgrid will not run the promise while processing the partition which mean that the promise will not report error, only Anomaly will be checked. Samething, if promise is anomaly less, only test will run when slapgrid is processing the partition.

This can hide problems when tests are not isolated correctly.

/reviewed-on !84

 - rename test files as test_*.py, this is how tests are collected by
   py.test

 - take into account that test runner might disable log level and patch
   sys.stdout already

/reviewed-on !92

  It incldudes minor fixes on the scripts

This may reduce code duplication across tests of several bt5 tests

this will be usefull when looking for all HS belonging to someone.

/cc @alain.takoudjou @rafael @guillaume.hervier

/reviewed-on nexedi/slapos.core!90

the related keys were missing in master/bt5/slapos_cloud/bt/template_catalog_method_id_list

the related keys were correctly commited in master/bt5/slapos_cloud/CatalogRelatedKeyTemplateItem/related_key_list.xml but not in master/bt5/slapos_cloud/bt/template_catalog_related_key_list

- fix upgrade decision duplicated when an hosting subscription is deployed in many partitions
- Prevent concurrent transaction to create 2 upgrade decision for the same Hosting Subscription

/reviewed-on nexedi/slapos.core!79

I added and fixed some URL columns on some views:
- `slapos_cloud/ComputerUsage_view`
- `slapos_cloud/HostingSubscription_view`

The goal is to make the process to look for the needed informations faster by reducing clicks between views on listbox objects, by making some changes such as clicking on an object name on a column should redirect to the object view page instead of the row's object view page.

Additionally, these changes are made to be compatible with the RenderJS UI.

As this is not the default behaviour in ERP5, it may sound counter-intuitive, but this special behaviour is made to improve efficiency/productivity over intuitivity.

Personally, I wondered about having some "compromise" which is having the "Plane" icon on these columns beside the content to clearly indicate that this icon will redirect to the object's view (as the current "Plane icon" behaviour in ERP5)

/reviewed-on nexedi/slapos.core!89