Use Executor in AbstractWaitStrategy

Đang mở
#5,253 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Đánh giá

Độ khó
5/5
Thời gian dự kiến
Hơn một tuần
Mức phù hợp với người mới
20/100
Loại issue
Tính năng
Độ rõ ràng
Cần làm rõ
Mức độ hoạt động
Đình trệ
Công nghệ
java, kotlin
Lĩnh vực
testing

Hướng nghiên cứu

Bắt đầu bằng cách đọc core/src/main/java/org/testcontainers/containers/wait/strategy/AbstractWaitStrategy.java, đặc biệt là phần khai báo EXECUTOR và tích hợp waitUntilReady. Issue cung cấp các ví dụ về coroutine của Kotlin nhưng không có các bài kiểm thử được đặt tên hay tiêu chí chấp nhận; một thay đổi hoàn chỉnh sẽ cần một phương án đã được thống nhất để expose hoặc thay thế executor, cùng với một cách được xác định rõ để xác minh các wait strategy bị suspend.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

I am trying to bridge to a suspend version of waitUntilReady in AbstractWaitStrategy with something along the lines of:

suspend fun strategy(waitingStrategyTarget: WaitingStrategyTarget): Unit {
  // run your strategy
}

class MyWaitStrategy : AbstractWaitStrategy() {
  override fun waitUntilReady() {
    val executor: ExecutorService = EXECUTOR // executor from  AbstractWaitStrategy
    runBlocking(executor.asCoroutineDispatcher()) {
      strategy(waitingStrategyTarget)
    }
  }
}

or in a TestScope with JUnit this may look like:

@Test
    fun testWithSuspendedWaitStrategy() = 
      runTest {
        val strategy = 
            waitStrategy { target -> 
               // run my strategy
            }

       // MyContainer.waitUntil(strategy, ....)
      }

fun TestScope.waitStrategy(
  f: CoroutineScope.(target: WaitStrategyTarget) -> Unit
): AbstractWaitStrategy =
  object : AbstractWaitStrategy() {
    override fun waitUntilReady() {
      launch(EXECUTOR.asCoroutineDispatcher()) {
        f(this, waitStrategyTarget)
      }
    }
  }

Is there a way how to implement this with EXECUTOR, being public instead of package private, or with another Executor, ExecutorService or something.

Any Feedback is welcome :D

Ngôn ngữ chính
Java
Star
8.7k
Fork
1.9k
Merge trung bình
2 ngày 17 giờ
Pull request đã merge (30 ngày)
9

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của testcontainers/testcontainers-java

Tất cả issue của testcontainers/testcontainers-java

Issue tương tự

Thêm issue về Java

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.