Skip to content

Commit

Permalink
Add worker assertions resource_processing_integration_test
Browse files Browse the repository at this point in the history
Adding a few assertions to the worker and multiplex worker tests to ensure that the expected workers are in fact coming up.

Closes #15952.

PiperOrigin-RevId: 505690186
Change-Id: If732c9f040ec144395b5525a48d3a7d367fe244b
  • Loading branch information
Bencodes authored and ted-xie committed Jan 31, 2023
1 parent 4b4c9c0 commit e93c61e
Showing 1 changed file with 14 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,13 @@ function test_persistent_resource_processor() {
create_android_binary
setup_font_resources

assert_build //java/bazel:bin --persistent_android_resource_processor
assert_build //java/bazel:bin --persistent_android_resource_processor \
--worker_verbose &> $TEST_log
expect_log "Created new non-sandboxed AndroidResourceParser worker (id [0-9]\+)"
expect_log "Created new non-sandboxed AndroidResourceCompiler worker (id [0-9]\+)"
expect_log "Created new non-sandboxed AndroidCompiledResourceMerger worker (id [0-9]\+)"
expect_log "Created new non-sandboxed AndroidAapt2 worker (id [0-9]\+)"
expect_log "Created new non-sandboxed ManifestMerger worker (id [0-9]\+)"
}

function test_persistent_multiplex_resource_processor() {
Expand All @@ -121,7 +127,13 @@ function test_persistent_multiplex_resource_processor() {
setup_font_resources

assert_build //java/bazel:bin --experimental_worker_multiplex \
--persistent_multiplex_android_tools
--persistent_multiplex_android_tools \
--worker_verbose &> $TEST_log
expect_log "Created new non-sandboxed AndroidResourceParser multiplex-worker (id [0-9]\+)"
expect_log "Created new non-sandboxed AndroidResourceCompiler multiplex-worker (id [0-9]\+)"
expect_log "Created new non-sandboxed AndroidCompiledResourceMerger multiplex-worker (id [0-9]\+)"
expect_log "Created new non-sandboxed AndroidAapt2 multiplex-worker (id [0-9]\+)"
expect_log "Created new non-sandboxed ManifestMerger multiplex-worker (id [0-9]\+)"
}

run_suite "Resource processing integration tests"

0 comments on commit e93c61e

Please sign in to comment.