Address comments

* Remove requests/pyproject.toml
* Improve naming of _WAIT_INTERVAL
* Fix type hint
* Improve comment

Author: thpierce

contract-tests/images/applications/requests/pyproject.toml

@@ -23,7 +23,7 @@
 
 _logger: Logger = getLogger(__name__)
 _TIMEOUT_DELAY: timedelta = timedelta(seconds=20)
-_WAIT_INTERVAL: float = 0.1
+_WAIT_INTERVAL_SEC: float = 0.1
 T: TypeVar = TypeVar("T")
 
 
@@ -133,7 +133,7 @@ def _wait_for_content(get_export: Callable[[], List[T]], wait_condition: Callabl
                 return current_exported
             exported = current_exported
 
-            sleep(_WAIT_INTERVAL)
+            sleep(_WAIT_INTERVAL_SEC)
         # pylint: disable=broad-exception-caught
         except Exception:
             _logger.exception("Error while reading content")

contract-tests/images/mock-collector/mock_collector_client.py

@@ -15,7 +15,6 @@ rm -rf dist/contract_tests*
 cd contract-tests/images/mock-collector
 docker build . -t aws-appsignals-mock-collector-python
 
-
 # Find and store aws_opentelemetry_distro whl file
 cd ../../../dist
 DISTRO=(aws_opentelemetry_distro-*-py3-none-any.whl)

contract-tests/set-up-contract-tests.sh

@@ -40,7 +40,7 @@ def get_application_network_aliases(self) -> List[str]:
     def get_application_extra_environment_variables(self) -> Dict[str, str]:
         """
         This does not appear to do anything, as it does not seem that OTEL supports peer service for Python. Keeping
-        for consistency at this time.
+        for consistency with Java contract tests at this time.
         """
         return {"OTEL_INSTRUMENTATION_COMMON_PEER_SERVICE_MAPPING": "backend=backend:8080"}
 
@@ -162,7 +162,7 @@ def _assert_metric_attributes(
         method: str,
         path: str,
         metric_name: str,
-        expected_sum: float,
+        expected_sum: int,
     ) -> None:
         target_metrics: List[Metric] = []
         for resource_scope_metric in resource_scope_metrics: