Python framework: Remove default from get_endpoint (project-chip#41711)

* Python framework: Remove default from get_endpoint

In order to do test selection we need to know the endpoint
before the test. if the default is set by the test explicitly
outside of the test function call, we know.
Ideally, all tests would be explicit about asking the tester
to set the endpoint to avoid accidentally testing the wrong
thing. However, the intent of this PR is not to change that
policy, but rather to make the information available before
the test function itself is run, either through the user
command line parameter or through the static default.

The functions inside tests were adjusted according to the following
rules
- get_endpoint calls in test_ functions with @run_if_endpoint_matches
  decorators just have the defaults removed. Those decorators
  already enforce that the endpoint parameter is given on the
  command line so the default would never be used.
- get_endpoint calls in utility functions have the defaults
  removed since these should use the endpoint of the caller
  in the first place
- get_endpoint calls with a default in functions decorated with
  the async_test_body decorator have an explicit default_endpoint
  property function defined to match the given default if the
  default is not 0

* Fix type annotation

* A couple more

* Fix doorlock

* fix webrtcp-2.9

* fix a couple more

* Fix two additional tests added after this PR was raised

* missed one more

* add default endpoints to door lock

* I liked the function so much I thought...why not two?

Author: cecille

src/python_testing/TC_AVSMTestBase.py

@@ -49,7 +49,7 @@ def predicate(report: AttributeValue) -> bool:
 
 class AVSMTestBase:
     async def precondition_one_allocated_snapshot_stream(self):
-        endpoint = self.get_endpoint(default=1)
+        endpoint = self.get_endpoint()
         cluster = Clusters.CameraAvStreamManagement
         attr = Clusters.CameraAvStreamManagement.Attributes
         commands = Clusters.CameraAvStreamManagement.Commands
@@ -100,7 +100,7 @@ async def precondition_one_allocated_snapshot_stream(self):
             pass
 
     async def precondition_one_allocated_audio_stream(self, streamUsage: Globals.Enums.StreamUsageEnum = None):
-        endpoint = self.get_endpoint(default=1)
+        endpoint = self.get_endpoint()
         cluster = Clusters.CameraAvStreamManagement
         attr = Clusters.CameraAvStreamManagement.Attributes
         commands = Clusters.CameraAvStreamManagement.Commands
@@ -154,7 +154,7 @@ async def precondition_one_allocated_audio_stream(self, streamUsage: Globals.Enu
             pass
 
     async def precondition_one_allocated_video_stream(self, streamUsage: Globals.Enums.StreamUsageEnum = None):
-        endpoint = self.get_endpoint(default=1)
+        endpoint = self.get_endpoint()
         cluster = Clusters.CameraAvStreamManagement
         attr = Clusters.CameraAvStreamManagement.Attributes
         commands = Clusters.CameraAvStreamManagement.Commands

src/python_testing/TC_AVSM_2_1.py

@@ -150,9 +150,13 @@ def steps_TC_AVSM_2_1(self) -> list[TestStep]:
             ),
         ]
 
+    @property
+    def default_endpoint(self) -> int:
+        return 1
+
     @async_test_body
     async def test_TC_AVSM_2_1(self):
-        endpoint = self.get_endpoint(default=1)
+        endpoint = self.get_endpoint()
         cluster = Clusters.CameraAvStreamManagement
         attr = Clusters.CameraAvStreamManagement.Attributes
 

src/python_testing/TC_AVSM_2_10.py

@@ -114,7 +114,7 @@ def steps_TC_AVSM_2_10(self) -> list[TestStep]:
         has_feature(Clusters.CameraAvStreamManagement, Clusters.CameraAvStreamManagement.Bitmaps.Feature.kSnapshot)
     )
     async def test_TC_AVSM_2_10(self):
-        endpoint = self.get_endpoint(default=1)
+        endpoint = self.get_endpoint()
         cluster = Clusters.CameraAvStreamManagement
         attr = Clusters.CameraAvStreamManagement.Attributes
         commands = Clusters.CameraAvStreamManagement.Commands

src/python_testing/TC_AVSM_2_11.py

@@ -116,7 +116,7 @@ def steps_TC_AVSM_2_11(self) -> list[TestStep]:
         and has_feature(Clusters.CameraAvStreamManagement, Clusters.CameraAvStreamManagement.Bitmaps.Feature.kVideo)
     )
     async def test_TC_AVSM_2_11(self):
-        endpoint = self.get_endpoint(default=1)
+        endpoint = self.get_endpoint()
         cluster = Clusters.CameraAvStreamManagement
         attr = Clusters.CameraAvStreamManagement.Attributes
         commands = Clusters.CameraAvStreamManagement.Commands

src/python_testing/TC_AVSM_2_12.py

@@ -160,7 +160,7 @@ def steps_TC_AVSM_2_12(self) -> list[TestStep]:
 
     @run_if_endpoint_matches(has_cluster(Clusters.CameraAvStreamManagement))
     async def test_TC_AVSM_2_12(self):
-        endpoint = self.get_endpoint(default=1)
+        endpoint = self.get_endpoint()
         cluster = Clusters.CameraAvStreamManagement
         attr = Clusters.CameraAvStreamManagement.Attributes
 

src/python_testing/TC_AVSM_2_13.py

@@ -117,7 +117,7 @@ def steps_TC_AVSM_2_13(self) -> list[TestStep]:
         has_feature(Clusters.CameraAvStreamManagement, Clusters.CameraAvStreamManagement.Bitmaps.Feature.kVideo)
     )
     async def test_TC_AVSM_2_13(self):
-        endpoint = self.get_endpoint(default=1)
+        endpoint = self.get_endpoint()
         cluster = Clusters.CameraAvStreamManagement
         attr = Clusters.CameraAvStreamManagement.Attributes
         commands = Clusters.CameraAvStreamManagement.Commands

src/python_testing/TC_AVSM_2_14.py

@@ -102,7 +102,7 @@ def steps_TC_AVSM_2_14(self) -> list[TestStep]:
         has_feature(Clusters.CameraAvStreamManagement, Clusters.CameraAvStreamManagement.Bitmaps.Feature.kAudio)
     )
     async def test_TC_AVSM_2_14(self):
-        endpoint = self.get_endpoint(default=1)
+        endpoint = self.get_endpoint()
         cluster = Clusters.CameraAvStreamManagement
         attr = Clusters.CameraAvStreamManagement.Attributes
         commands = Clusters.CameraAvStreamManagement.Commands

src/python_testing/TC_AVSM_2_15.py

@@ -97,7 +97,7 @@ def steps_TC_AVSM_2_15(self) -> list[TestStep]:
         has_feature(Clusters.CameraAvStreamManagement, Clusters.CameraAvStreamManagement.Bitmaps.Feature.kSnapshot)
     )
     async def test_TC_AVSM_2_15(self):
-        endpoint = self.get_endpoint(default=1)
+        endpoint = self.get_endpoint()
         cluster = Clusters.CameraAvStreamManagement
         attr = Clusters.CameraAvStreamManagement.Attributes
         commands = Clusters.CameraAvStreamManagement.Commands

src/python_testing/TC_AVSM_2_16.py

@@ -144,7 +144,7 @@ def steps_TC_AVSM_2_16(self) -> list[TestStep]:
         has_feature(Clusters.CameraAvStreamManagement, Clusters.CameraAvStreamManagement.Bitmaps.Feature.kVideo) and
         has_feature(Clusters.CameraAvStreamManagement, Clusters.CameraAvStreamManagement.Bitmaps.Feature.kAudio))
     async def test_TC_AVSM_2_16(self):
-        endpoint = self.get_endpoint(default=1)
+        endpoint = self.get_endpoint()
         cluster = Clusters.CameraAvStreamManagement
         attr = Clusters.CameraAvStreamManagement.Attributes
         commands = Clusters.CameraAvStreamManagement.Commands

src/python_testing/TC_AVSM_2_17.py

@@ -173,7 +173,7 @@ def steps_TC_AVSM_2_17(self) -> list[TestStep]:
         and has_feature(Clusters.CameraAvStreamManagement, Clusters.CameraAvStreamManagement.Bitmaps.Feature.kPrivacy)
     )
     async def test_TC_AVSM_2_17(self):
-        endpoint = self.get_endpoint(default=1)
+        endpoint = self.get_endpoint()
         cluster = Clusters.CameraAvStreamManagement
         attr = Clusters.CameraAvStreamManagement.Attributes
         commands = Clusters.CameraAvStreamManagement.Commands