Internal change (#1004)

[OpenHTF] Ignore the GitHub Actions config file in *..

[OpenHTF] Add util function for getting the phases from a list of phase nodes and phase node collections.

Cleanup (3p) LSC: Remove `Lint as` gpylint comment directives.

Cleanup (3p) LSC: Remove `Lint as` gpylint comment directives.

Support test-stopping failures with a phase-scoped option.

Add an explicit Literal annotation to test_state.INFER_MIMETYPE.

Add get_test_measurement_strict() into test api.

Cleanup Fix or silence upcoming type errors in Python 3.9.

Cleanup LSC: Remove `//net/proto2/python/public:use_pure_python` dep, migrate to protobuf

Cleanup Silence type errors generated by enabling a new pytype feature.

Cleanup Migrate away from Angular's deprecated `async` symbol to use `waitForAsync`

Cleanup Fix or ignore type errors generated by the next release of pytype.

Iteratively upload attachments in when partial uploads are in use

Give mfg_event_converter access to the existing blobref cache

Add a size attribute for Attachments

Cleanup Remove all usages of //third_party/py/enum library.

Cleanup Silence type errors generated by enabling a new pytype feature.

Migrate partial upload code from MfgInspector to *

Add a mechanism to profile unit tests.

Add case formatting options for phase names in OpenHTF Phase Options.

Cleanup Remove unused python binaries + libraries from //third_party/py (as reported dead by sensenmann)

Cleanup Fix or ignore type errors generated by the next release of pytype.

make the mfg_event_converter to handle datetime objects in JSON conversion. (see cl/396863619 for some context)

Cleanup Infer `__init__.py` attributes for Python strict deps.

validators.py: Use f-string to format Equals str representation.

Enable partial upload of attachments from in progress phases

Detect changed attachments (and skip cache) to support OpenHTF_record.json

Rollback of changelist 392091102. Reason: Rollforward of cl/391826358: Reuse existing blob refs in partial_upload by caching protos

Rollback of changelist 391826358. Reason: Seems that these blobrefs are not the correct values to be using at the moment or at least there are occasional issues. See b/197352227 for details.

Reuse existing blob refs in partial_upload by caching protos

Add alternative for conf.inject_positional_args.

Cleanup Fix or ignore type errors generated by the newest pytype release.

Move Configuration (openhtf.conf) class attributes to module attributes

Treat openhtf.util.conf as an object instead of a module

Move Configuration to its own module instead of living in conf

Update typescript web_gui files to remove deprecated bypassSecurityTrustHtml

Remove __init__.py file that was left behind in an otherwise empty package.

Add a protocol and implementation for conf.declare return value.

Cleanup Fix or ignore type errors generated by the next release of pytype.

PiperOrigin-RevId: 450548096

Author: dieppedalle

contrib/plugs/__init__.py

@@ -32,7 +32,7 @@
 from six.moves import zip
 
 
-@htf.plug(example=example_plugs.ExamplePlug)
+@htf.plug(example=example_plugs.example_plug_configured)
 @htf.plug(frontend_aware=example_plugs.ExampleFrontendAwarePlug)
 def example_monitor(example, frontend_aware):
   time.sleep(.2)
@@ -50,7 +50,7 @@ def example_monitor(example, frontend_aware):
     docstring='Helpful docstring',
     units=units.HERTZ,
     validators=[util.validators.matches_regex('Measurement')])
-@htf.plug(example=example_plugs.ExamplePlug)
+@htf.plug(example=example_plugs.example_plug_configured)
 @htf.plug(prompts=user_input.UserInput)
 def hello_world(test, example, prompts):
   """A hello world test phase."""

examples/all_the_things.py

@@ -14,30 +14,27 @@
 """Example plugs for OpenHTF."""
 
 from openhtf.core import base_plugs
-from openhtf.util import conf
+from openhtf.util import configuration
 
-conf.declare(
+CONF = configuration.CONF
+
+EXAMPLE_PLUG_INCREMENT_SIZE = CONF.declare(
     'example_plug_increment_size',
     default_value=1,
     description='increment constant for example plug.')
 
 
-class ExamplePlug(base_plugs.BasePlug):  # pylint: disable=no-init
+class ExamplePlug(base_plugs.BasePlug):
   """Example of a simple plug.
 
   This plug simply keeps a value and increments it each time increment() is
   called.  You'll notice a few paradigms here:
 
-    - @conf.inject_positional_args
+    - configuration.bind_init_args
       This is generally a good way to pass in any configuration that your
       plug needs, such as an IP address or serial port to connect to.  If
-      You want to use your plug outside of the OpenHTF framework, you can
-      still manually instantiate it, but you must pass the arguments by
-      keyword (as a side effect of the way inject_positional_args is
-      implemented).
-
-      For example, if you had no openhtf.conf loaded, you could do this:
-        my_plug = ExamplePlug(example_plug_increment_size=4)
+      you want to use your plug outside of the OpenHTF framework, you can
+      still manually instantiate it.
 
     - tearDown()
       This method will be called automatically by the OpenHTF framework at
@@ -61,7 +58,6 @@ class ExamplePlug(base_plugs.BasePlug):  # pylint: disable=no-init
       a with: block at the beginning of every phase where it is used.
   """
 
-  @conf.inject_positional_args
   def __init__(self, example_plug_increment_size):
     self.increment_size = example_plug_increment_size
     self.value = 0
@@ -79,6 +75,10 @@ def increment(self):
     return self.value - self.increment_size
 
 
+example_plug_configured = configuration.bind_init_args(
+    ExamplePlug, EXAMPLE_PLUG_INCREMENT_SIZE)
+
+
 class ExampleFrontendAwarePlug(base_plugs.FrontendAwareBasePlug):
   """Example of a simple frontend-aware plug.
 

examples/example_plugs.py

@@ -17,7 +17,9 @@
 from openhtf.output.servers import station_server
 from openhtf.output.web_gui import web_launcher
 from openhtf.plugs import user_input
-from openhtf.util import conf
+from openhtf.util import configuration
+
+CONF = configuration.CONF
 
 
 @htf.measures(htf.Measurement('hello_world_measurement'))
@@ -27,7 +29,7 @@ def hello_world(test):
 
 
 def main():
-  conf.load(station_server_port='4444')
+  CONF.load(station_server_port='4444')
   with station_server.StationServer() as server:
     web_launcher.launch('http://localhost:4444')
     for _ in range(5):

examples/frontend_example.py

@@ -22,15 +22,17 @@
 test.configure(stop_on_first_failure=True)
 
 2. Using config item. This option lets you toggle this feature dynamically.
-conf.load(stop_on_first_failure=True)
+CONF.load(stop_on_first_failure=True)
 """
 
 import openhtf as htf
 from openhtf.output.callbacks import console_summary
 from openhtf.plugs import user_input
-from openhtf.util import conf  # pylint: disable=unused-import
+from openhtf.util import configuration
 from openhtf.util import validators
 
+CONF = configuration.CONF
+
 
 @htf.measures('number_sum', validators=[validators.in_range(0, 5)])
 def add_numbers_fails(test):
@@ -58,7 +60,7 @@ def main():
 
   # Option 2 : You can disable option 1 and enable below line
   # to get same result
-  # conf.load(stop_on_first_failure=True)
+  # CONF.load(stop_on_first_failure=True)
 
   test.execute(test_start=user_input.prompt_for_test_start())
 

examples/stop_on_first_failure.py

@@ -40,6 +40,7 @@
 from openhtf.core.phase_descriptor import diagnose
 from openhtf.core.phase_descriptor import measures
 from openhtf.core.phase_descriptor import PhaseDescriptor
+from openhtf.core.phase_descriptor import PhaseNameCase
 from openhtf.core.phase_descriptor import PhaseOptions
 from openhtf.core.phase_descriptor import PhaseResult
 from openhtf.core.phase_group import PhaseGroup
@@ -50,16 +51,15 @@
 from openhtf.core.test_record import PhaseRecord
 from openhtf.core.test_record import TestRecord
 from openhtf.plugs import plug
-from openhtf.util import conf
+from openhtf.util import configuration
 from openhtf.util import console_output
 from openhtf.util import data
 from openhtf.util import functions
 from openhtf.util import logs
 from openhtf.util import units
 import pkg_resources
 
-if typing.TYPE_CHECKING:
-  conf: conf.Configuration  # Configuration is only available here in typing.
+conf = configuration.CONF
 
 
 def get_version():

openhtf/__init__.py

@@ -60,30 +60,37 @@ def TestPhase(test, example):
   Tearing down ExamplePlug!
 
 Plugs will often need to use configuration values.  The recommended way
-of doing this is with the conf.inject_positional_args decorator:
+of doing this is with the configuration.inject_positional_args decorator:
 
   from openhtf import plugs
-  from openhtf.util import conf
+  from openhtf.util import configuration
 
-  conf.declare('my_config_key', default_value='my_config_value')
+  CONF = configuration.CONF
+  MY_CONFIG_KEY = CONF.declare('my_config_key', default_value='my_config_value')
+
+  CONF.declare('my_config_key', default_value='my_config_value')
 
   class ExamplePlug(base_plugs.BasePlug):
     '''A plug that requires some configuration.'''
 
-    @conf.inject_positional_args
     def __init__(self, my_config_key)
       self._my_config = my_config_key
 
-Note that Plug constructors shouldn't take any other arguments; the
-framework won't pass any, so you'll get a TypeError.  Any values that are only
-known at run time must be either passed into other methods or set via explicit
-setter methods.  See openhtf/conf.py for details, but with the above
-example, you would also need a configuration .yaml file with something like:
+  example_plug_configured = configuration.bind_init_args(
+      ExamplePlug, MY_CONFIG_KEY)
+
+Here, example_plug_configured is a subclass of ExamplePlug with bound args for
+the initializer, and it can be passed to phases like any other plug. See
+openhtf/conf.py for details, but with the above example, you would also need a
+configuration .yaml file with something like:
 
   my_config_key: my_config_value
 
-This will result in the ExamplePlug being constructed with
+This will result in the example_plug_configured being constructed with
 self._my_config having a value of 'my_config_value'.
+
+Note that Plug constructors shouldn't take any other arguments; the
+framework won't pass any, so you'll get a TypeError.
 """
 
 import logging
@@ -103,6 +110,18 @@ class InvalidPlugError(Exception):
 class BasePlug(object):
   """All plug types must subclass this type.
 
+  Okay to use with multiple inheritance when subclassing an existing
+  implementation that you want to convert into a plug. Place BasePlug last in
+  the parent list. For example:
+
+  class MyExistingDriver:
+    def do_something(self):
+      pass
+
+  class MyExistingDriverPlug(MyExistingDriver, BasePlug):
+    def tearDown(self):
+      ...  # Implement the BasePlug interface as desired.
+
   Attributes:
     logger: This attribute will be set by the PlugManager (and as such it
       doesn't appear here), and is the same logger as passed into test phases
@@ -147,14 +166,12 @@ def _asdict(self) -> Dict[Text, Any]:
 
   def tearDown(self) -> None:
     """This method is called automatically at the end of each Test execution."""
-    pass
 
   @classmethod
   def uses_base_tear_down(cls) -> bool:
     """Checks whether the tearDown method is the BasePlug implementation."""
-    this_tear_down = getattr(cls, 'tearDown')
-    base_tear_down = getattr(BasePlug, 'tearDown')
-    return this_tear_down.__code__ is base_tear_down.__code__
+    this_tear_down = getattr(cls, BasePlug.tearDown.__name__)
+    return this_tear_down.__code__ is BasePlug.tearDown.__code__
 
 
 class FrontendAwareBasePlug(BasePlug, util.SubscribableStateMixin):

openhtf/core/base_plugs.py

@@ -12,7 +12,6 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-# Lint as: python3
 """Diagnoses: Measurement and meta interpreters.
 
 Diagnoses are higher level signals that result from processing multiple

openhtf/core/diagnoses_lib.py

@@ -439,7 +439,7 @@ def validate(self) -> 'Measurement':
       raise
     finally:
       if self._cached:
-        self._cached['outcome'] = self.outcome.name
+        self._cached['outcome'] = self.outcome.name  # pytype: disable=bad-return-type
 
   def as_base_types(self) -> Dict[Text, Any]:
     """Convert this measurement to a dict of basic types."""

openhtf/core/measurements.py

@@ -12,7 +12,6 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-# Lint as: python3
 """Implements phase node branches.
 
 A BranchSequence is a phase node sequence that runs conditiionally based on the