grpc / grpc/grpc-ios

CFStream Test Suite Bazel Migration

Open
#48 1 comment 0 reactions 1 assignee Assigned to @dennycd View on GitHub
area/bazel area/cfstream area/cocoapod kind/test
Dominant language
C
Stars
61
Forks
26
Avg merge
1d 13h
Merged PRs (30d)
11

Description

Our [CFStream test suite](https://github.com/grpc/grpc/tree/master/test/core/iomgr/ios/CFStreamTests) currently have a few issues that need to be improved

* Test hangs and keep retries until timeout if test failure
* Async test logic is using NSRunLoop mainRunLoop instead of XCTestExpectation

In addition, the current test suite is an objc translation from existing c-core test
*

We should revisit the test setup to run c-core's test setup using the corresponding ios unit test target instead
*

---

Additional todos for CFStream Test Update

* Migrate CFStream test to bazel based
* Move CFStream test out of test/core/iomgr and move into src/objective-c so that it is co-located with the rest of the ios/objc test.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.