search

RedHatSatellite / katello-attach-subscription

automatically assign subscriptions to hypervisors based on certain rules
GNU General Public License v2.0
10 stars 5 forks source link

Possible layering for subscription rules #43

Closed mossicrue closed 5 years ago

evgeni commented 5 years ago

after applying my test-change suggestions, the following failures remain (no matter which merge_subs implementation`):

  1) Failure:
SubMergerTest#test_keep_parse [/home/egolov/Devel/katello-attach-subscription/test/sub_merger_test.rb:137]:
--- expected
+++ actual
@@ -1 +1 @@
-{"rhel"=>["b1a5d251fa4fe598cb947ffc42b9cbed", "1337d38747e659ed836548ae6cda7cc2"], "smartmanagement"=>["f180623caa42379bc4518d06c9c9be05"]}
+{"rhel"=>["bb98d4e9c281b175ea84c517b59308ea"], "smartmanagement"=>["af03af10d57b7b17f26a0130562d6b6e"]}

  2) Failure:
SubMergerTest#test_merge_parse [/home/egolov/Devel/katello-attach-subscription/test/sub_merger_test.rb:131]:
--- expected
+++ actual
@@ -1 +1 @@
-{"rhel"=>["b1a5d251fa4fe598cb947ffc42b9cbed", "1337d38747e659ed836548ae6cda7cc2"], "smartmanagement"=>["f180623caa42379bc4518d06c9c9be05"], "els"=>["523af537946b79c4f8369ed39ba78605"]}
+{"rhel"=>["bb98d4e9c281b175ea84c517b59308ea"], "smartmanagement"=>["af03af10d57b7b17f26a0130562d6b6e"]}