Chiebot-Mirror/grpc - grpc - Gitea: Git with a cup of tea

53912 Commits

300 Branches

385 Tags

1.4 GiB

Tag: Branch: Tree: 11d9b083f3

Commit Graph

Author SHA1 Message Date

Author	SHA1	Message	Date
Xuan Wang	11d9b083f3	[Python run_test] Fix run_test (#34292 ) We're not running any test at all from `run_test.py` because of the way we filter test cases: `1d136fd05f/src/python/grpcio_tests/tests/_runner.py (L137)` * `testcase_filter` is read from a json file (like [this one](https://github.com/grpc/grpc/blob/master/src/python/grpcio_tests/tests/tests.json)) and test name is similar to `unit._metadata_test.MetadataTest`. * `case.id()` is loaded by `iterate_suite_cases` and will always have a prefix of `tests`, an example of case id will be: `tests.unit._metadata_test.MetadataTest`. Because of the prefix, none of the test case will be matched thus we're not running any of the tests. This PR fixes the prefix issue and all the regressions comes from not running tests using `run_test.py`. #### Other Changes * Added couple of `__init__.py` file since it's required to load tests. * Added `py_status_code` to Aio rpc state. * `code()` is expecting to return a python gRPC code but current `status_code` is a Cython code. * Added `libsqlite3-dev` to our dockers because it's required for `coverage==7.2.0`. * Renamed csds and admin test because test case file have to end with `_test`: `1d136fd05f/src/python/grpcio_tests/tests/_loader.py (L26)` * Removed gevent test from `run_test.py` because Bazel gevent tests should be good enough for us. <!-- If you know who should review your pull request, please assign it to that person, otherwise the pull request would get assigned randomly. If your pull request is for a specific language, please add the appropriate lang label. -->	2 years ago
Richard Belleville	498fc99479	[packaging] Publish xds-protos as part of the standard package pipeline (#33797 ) This PR: - Fixes the xds-protos Python package, which was broken when the `udpa` submodule was removed - This required re-adding the protoc-gen-validate submodule - Adds non-Bazel tests for xds-protos and all of its dependent packages - Versions xds-protos the same way as the rest of the Python packages - Fixes Python 3.11 support in `run_tests.py`, which is necessary for the testing mentioned above CC @sergiitk You won't be able to consume this in the interop tests until it makes it into a release. I'm thinking I'll want to backport this to the 1.57.x branch to make that happen faster. CC @drfloob to inform him about the likely backport.	2 years ago

Xuan Wang

11d9b083f3

[Python run_test] Fix run_test (#34292 )

We're not running any test at all from `run_test.py` because of the way
we filter test cases:

1d136fd05f/src/python/grpcio_tests/tests/_runner.py (L137)

* `testcase_filter` is read from a json file (like [this
one](https://github.com/grpc/grpc/blob/master/src/python/grpcio_tests/tests/tests.json))
and test name is similar to `unit._metadata_test.MetadataTest`.
* `case.id()` is loaded by `iterate_suite_cases` and will always have a
prefix of `tests`, an example of case id will be:
`tests.unit._metadata_test.MetadataTest`.

Because of the prefix, none of the test case will be matched thus we're
not running any of the tests.

This PR fixes the prefix issue and all the regressions comes from not
running tests using `run_test.py`.

#### Other Changes
* Added couple of `__init__.py` file since it's required to load tests.
* Added `py_status_code` to Aio rpc state.
* `code()` is expecting to return a python gRPC code but current
`status_code` is a Cython code.
* Added `libsqlite3-dev` to our dockers because it's required for
`coverage==7.2.0`.
* Renamed csds and admin test because test case file have to end with
`_test`:
1d136fd05f/src/python/grpcio_tests/tests/_loader.py (L26)
* Removed gevent test from `run_test.py` because Bazel gevent tests
should be good enough for us.

<!--

If you know who should review your pull request, please assign it to
that
person, otherwise the pull request would get assigned randomly.

If your pull request is for a specific language, please add the
appropriate
lang label.

-->

2 years ago

Richard Belleville

498fc99479

[packaging] Publish xds-protos as part of the standard package pipeline (#33797 )

This PR:

- Fixes the xds-protos Python package, which was broken when the `udpa`
submodule was removed
  - This required re-adding the protoc-gen-validate submodule
- Adds non-Bazel tests for xds-protos and all of its dependent packages
- Versions xds-protos the same way as the rest of the Python packages
- Fixes Python 3.11 support in `run_tests.py`, which is necessary for
the testing mentioned above


CC @sergiitk You won't be able to consume this in the interop tests
until it makes it into a release. I'm thinking I'll want to backport
this to the 1.57.x branch to make that happen faster.

CC @drfloob to inform him about the likely backport.

2 years ago

2 Commits (11d9b083f31e7025a511dda954d317fd55c36f83)