Support HS w/o request_instance.

Also commit test skeleton.

master/bt5/slapos_accounting/SkinTemplateItem/portal_skins/slapos_accounting/Person_storeOpenSaleOrderJournal.xml

@@ -87,9 +87,8 @@ for hosting_subscription in portal.portal_catalog(\n
   try:\n
     start_date = getWorkflowDate(hosting_subscription, \'request_instance\')\n
   except ValueError:\n
-    pass\n
-  else:\n
-    start_date_tuple_list.append((hosting_subscription.getRelativeUrl(), start_date, hosting_subscription.getPath()))\n
+    start_date = hosting_subscription.getCreationDate()\n
+  start_date_tuple_list.append((hosting_subscription.getRelativeUrl(), start_date, hosting_subscription.getPath()))\n
 \n
 \n
 result = context.Person_updateOpenSaleOrder(src__=src__, remove_hosting_list=remove_hosting_list, add_kw_list=add_kw_list, start_date_tuple_list=start_date_tuple_list, stop_date=stop_date)\n

master/bt5/slapos_accounting/TestTemplateItem/testSlapOSAccountingAlarm.py

@@ -1154,6 +1154,11 @@ class TestHostingSubscription_requestUpdateOpenSaleOrder(testSlapOSMixin):
     self.assertEqual(request_time_2, validated_line_2.getStartDate())
     self.assertEqual(stop_date_2, validated_line_2.getStopDate())
 
+  def test_hosting_subscription_start_date_not_changed(self):
+    # if there was no request_instance the getCreationDate has been used
+    # but if request_instance appeared start_date is not changed
+    raise NotImplementedError
+
 class TestSlapOSTriggerBuildAlarm(testSlapOSMixin):
   @simulateByTitlewMark('SimulationMovement_buildSlapOS')
   def test_SimulationMovement_withoutDelivery(self):

master/bt5/slapos_accounting/bt/revision

-212
+213
\ No newline at end of file