Merge branch 'alesh/2317/multiawards-remove-winners-from-meta-for-cancelled-auctions' into 'master'

fix(multiAwards): exclude winners from public_meta for cancelled auctions

See merge request !566

src/prozorro_sale/auction/packages/multiAwards/states.py

@@ -159,7 +159,19 @@ class MultiAwardsSimultaneousRound(mixins._CancellationMixin, states.Simultaneou
 
 
 class Cancelled(states.Cancelled):
-    ...
+    def onEnter(self):
+        """
+        On cancelled state enter action.
+        """
+        super().onEnter()
+        self._clean_winners()
+
+    def _clean_winners(self):
+        """
+        Clean winners from public meta because auction is cancelled and there are no winners.
+        """
+        if self._auction.public_meta:
+            self._auction.public_meta.pop('winners', None)
 
 
 class Done(states.Done):

test/unit/test_build_multiAwards_auction.py

@@ -260,3 +260,24 @@ class AuctionBuildTest(unittest.TestCase):
                 auth.AclContext(None, str(auction.bids[bid.id].ownerToken)),
                 validators=['ten_times_lg', ]
             )
+
+    def test_cancel_auction_on_simultaneous_round(self):
+        auction = build_from_procedure_spec(self.PROCEDURE, 'multiAwards')
+        auction.set_state('simultaneous_round')
+        bid = list(auction.bids.values())[0]
+        bid_amount = 0.12
+        auction.place_bid(bid.id, bid_amount, auth.AclContext('test1', str(bid.ownerToken)))
+        bid = auction.bids[bid.id]
+        self.assertEqual(bid.amount, bid_amount * 100)
+
+        auction.cancel_auction()
+
+        with self.assertRaises(errors.ForbiddenStateError):
+            auction.place_bid(
+                bid.id, bid_amount,
+                auth.AclContext(None, str(auction.bids[bid.id].ownerToken)),
+            )
+
+        self.assertEqual(auction.current_state, 'cancelled')
+        self.assertIsNone(auction.endDate)
+        self.assertNotIn('winners', auction.public_meta.keys())