Test for attr membership on a frequently-used attr

While this can still fail, less likely to fail than querying
on the last-seen attr. In particular, some mechanisms will not
respond to an inquiry for that attr despite claiming to have it.

Signed-off-by: Alexander Scheel <[email protected]>

Author: cipherboy

gssapi/tests/test_raw.py

@@ -660,7 +660,7 @@ def test_rfc5587(self):
         # Since mechs is of type set and thus not indexable, these
         # are used to track the last visited mech for testing
         # purposes, and saves a call to inquire_attrs_for_mech().
-        last_attr = None
+        attrs_dict = {}
         last_mech = None
 
         for mech in mechs:
@@ -691,7 +691,9 @@ def test_rfc5587(self):
                 display_out.short_desc.should_be_a(bytes)
                 display_out.long_desc.should_be_a(bytes)
 
-                last_attr = mech_attr
+                if mech_attr not in attrs_dict:
+                    attrs_dict[mech_attr] = 0
+                attrs_dict[mech_attr] += 1
 
             for mech_attr in known_mech_attrs:
                 mech_attr.shouldnt_be_none()
@@ -705,7 +707,12 @@ def test_rfc5587(self):
                 display_out.short_desc.should_be_a(bytes)
                 display_out.long_desc.should_be_a(bytes)
 
-        attrs = set([last_attr])
+                if mech_attr not in attrs_dict:
+                    attrs_dict[mech_attr] = 0
+                attrs_dict[mech_attr] += 1
+
+        max_attr = max(attrs_dict, key=attrs_dict.get)
+        attrs = set([max_attr])
 
         mechs = gb.indicate_mechs_by_attrs(attrs, None, None)
         mechs.shouldnt_be_empty()