Don't show properties on uninstantiated classes outside verbose mode / completion scripts.

PiperOrigin-RevId: 260042848
Change-Id: Ia773c9922a1138693b8fb17d7f5595289ea5d35b

Author: dbieber

fire/completion.py

@@ -328,7 +328,9 @@ def MemberVisible(component, name, member, class_attrs=None, verbose=False):
     if class_attrs is None:
       class_attrs = GetClassAttrsDict(class_attrs)
     class_attr = class_attrs.get(name)
-    if class_attr and class_attr.kind == 'method':
+    if class_attr and class_attr.kind in ('method', 'property'):
+      # methods and properties should be accessed on instantiated objects,
+      # not uninstantiated classes.
       return False
   if (six.PY2 and inspect.isfunction(component)
       and name in ('func_closure', 'func_code', 'func_defaults',

fire/helptext_test.py

@@ -427,11 +427,10 @@ def testUsageOutputNone(self):
         textwrap.dedent(expected_output).lstrip('\n'),
         usage_output)
 
-  @testutils.skip('Only passes in Python 3 for now.')
   def testInitRequiresFlagSyntaxSubclassNamedTuple(self):
     component = tc.SubPoint
     t = trace.FireTrace(component, name='SubPoint')
-    usage_output = helptext.UsageText(component, trace=t, verbose=True)
+    usage_output = helptext.UsageText(component, trace=t, verbose=False)
     expected_output = 'Usage: SubPoint --x=X --y=Y'
     self.assertIn(expected_output, usage_output)