3. **Actively collaborate with us.** We commit real time and attention to everything we review. Low-effort submissions—where code is generated, submitted, and abandoned—waste our limited review capacity.

## What contributors can expect from us

            capturedOption.set(option);
            return null;
        });

        final ApiAdminSchedulerAction action = createActionWithMockService(scheduledJob);

        // Execute the real put$start() method
        final JsonResponse<ApiResult> jsonResponse = action.put$start("job-1");

        // Verify: response contains non-null jobLogId
        assertNotNull(jsonResponse);

                case "test.property":
                    return "config-value";
                case "filter.null":
                    // This would be filtered to null by PropertyFilter in real scenario
                    throw new ConfigPropertyNotFoundException("filter.null");
                default:
                    throw new ConfigPropertyNotFoundException(propertyKey);
                }
            }

    @Test
    public void test_updateDocument_noLanguageDetected() {
        Map<String, Object> doc = new HashMap<>();
        doc.put("title", "xyz");
        doc.put("content", "xyz");

        // Without a real detector, this will throw NullPointerException
        try {
            languageHelper.updateDocument(doc);
            fail("Should throw NullPointerException without detector");
        } catch (NullPointerException e) {

    suite.addTestSuite(ImmutableClassToInstanceMapTest.class);

    suite.addTest(
        MapTestSuiteBuilder.using(
                new TestClassToInstanceMapGenerator() {
                  // Other tests will verify what real, warning-free usage looks like
                  // but here we have to do some serious fudging
                  @Override
                  @SuppressWarnings({"unchecked", "rawtypes"})

    suite.addTestSuite(MutableClassToInstanceMapTest.class);

    suite.addTest(
        MapTestSuiteBuilder.using(
                new TestClassToInstanceMapGenerator() {
                  // Other tests will verify what real, warning-free usage looks like
                  // but here we have to do some serious fudging
                  @Override
                  @SuppressWarnings({"unchecked", "rawtypes"})

        // The real getDocumentCount returns -1 on error; our test double tracks this
        testClient.documentCountError = true;
        final long count = testClient.getDocumentCount("nonexistent_index");
        assertEquals(-1L, count);
    }

    @Test
    public void test_getAliasCount_returnsZeroOnError() {
        // The real getAliasCount returns 0 on error; our test double tracks this

Caching
=======

OkHttp implements an optional, off by default, Cache. OkHttp aims for RFC correct and
pragmatic caching behaviour, following common real-world browser like Firefox/Chrome and
server behaviour when ambiguous.

# Basic Usage

```kotlin
  private val client: OkHttpClient = OkHttpClient.Builder()
      .cache(Cache(
          directory = File(application.cacheDir, "http_cache"),
          // $0.05 worth of phone storage in 2020

   */
  /*
   * This test hangs (or maybe is just *very* slow) under Android.
   *
   * TODO(b/218700094): Ideally, get this to pass under Android. Failing that, convince ourselves
   * that the test isn't exposing a real problem with InterruptibleTask, one that could matter in
   * prod.
   */
  @AndroidIncompatible
  public void testInterruptIsSlow() throws Exception {
    CountDownLatch isInterruptibleRegistered = new CountDownLatch(1);

   */
  /*
   * This test hangs (or maybe is just *very* slow) under Android.
   *
   * TODO(b/218700094): Ideally, get this to pass under Android. Failing that, convince ourselves
   * that the test isn't exposing a real problem with InterruptibleTask, one that could matter in
   * prod.
   */
  @AndroidIncompatible
  public void testInterruptIsSlow() throws Exception {
    CountDownLatch isInterruptibleRegistered = new CountDownLatch(1);