Gitiles
Code Review Sign In
gerrit-public.fairphone.software / platform / system / tools / aidl / afadec8e456af2b67f415f769192ff07f4d0f249
commitafadec8e456af2b67f415f769192ff07f4d0f249[log] [tgz]
authorSteven Moreland <smoreland@google.com>Wed Feb 17 22:15:27 2021 +0000
committerSteven Moreland <smoreland@google.com>Thu Feb 18 00:06:28 2021 +0000
tree32fb0d3be09483fbb548c7ae1a05658c2d05f871
parentd43bdaa8f6890101a3e0c1a67e20625dd4196a7c [diff]
AIDL test Java: getService -> waitForService

getService is racing with registration of services, especially for Java
clients, since in the Java backend, getService does not wait 5s for
services to appear (like it does for other backends).

This means, if it fails, we'll get timeout instead of a failure. We
could use a constant timeout here, but I've never had a problem with
waiting indefinitely in HIDL tests - it's flake proof! If there is a
server crash causing a timeout, it should be easy to repro locally.

Fixes: 179698026
Test: (runs for a long time w/o repro)
    while ./tests/aidl_integration_test.py; do adb logcat -c; done
Change-Id: I04ee9e87806fa4c5a3f90083e15a6d3c7cb4998d
4 files changed
tree: 32fb0d3be09483fbb548c7ae1a05658c2d05f871
  1. aidl_api/
  2. build/
  3. metadata/
  4. tests/
  5. trace/
  6. .clang-format
  7. aidl-format.sh
  8. aidl.cpp
  9. aidl.h
  10. aidl_checkapi.cpp
  11. aidl_checkapi.h
  12. aidl_const_expressions.cpp
  13. aidl_dumpapi.cpp
  14. aidl_dumpapi.h
  15. aidl_language.cpp
  16. aidl_language.h
  17. aidl_language_l.ll
  18. aidl_language_y.yy
  19. aidl_to_cpp.cpp
  20. aidl_to_cpp.h
  21. aidl_to_cpp_common.cpp
  22. aidl_to_cpp_common.h
  23. aidl_to_java.cpp
  24. aidl_to_java.h
  25. aidl_to_ndk.cpp
  26. aidl_to_ndk.h
  27. aidl_to_rust.cpp
  28. aidl_to_rust.h
  29. aidl_typenames.cpp
  30. aidl_typenames.h
  31. aidl_unittest.cpp
  32. Android.bp
  33. ast_cpp.cpp
  34. ast_cpp.h
  35. ast_cpp_unittest.cpp
  36. ast_java.cpp
  37. ast_java.h
  38. ast_java_unittest.cpp
  39. code_writer.cpp
  40. code_writer.h
  41. code_writer_unittest.cpp
  42. comments.cpp
  43. comments.h
  44. diagnostics.cpp
  45. diagnostics.h
  46. diagnostics.inc
  47. diagnostics_unittest.cpp
  48. generate_aidl_mappings.cpp
  49. generate_aidl_mappings.h
  50. generate_cpp.cpp
  51. generate_cpp.h
  52. generate_cpp_unittest.cpp
  53. generate_java.cpp
  54. generate_java.h
  55. generate_java_binder.cpp
  56. generate_ndk.cpp
  57. generate_ndk.h
  58. generate_rust.cpp
  59. generate_rust.h
  60. hiddenapi-greylist
  61. import_resolver.cpp
  62. import_resolver.h
  63. io_delegate.cpp
  64. io_delegate.h
  65. io_delegate_unittest.cpp
  66. line_reader.cpp
  67. line_reader.h
  68. location.cpp
  69. location.h
  70. logging.cpp
  71. logging.h
  72. main.cpp
  73. NOTICE
  74. options.cpp
  75. options.h
  76. options_unittest.cpp
  77. os.h
  78. OWNERS
  79. parser.cpp
  80. parser.h
  81. PREUPLOAD.cfg
  82. README.md
  83. run_integration_tests.sh
  84. TEST_MAPPING
README.md

Documentation for this project is currently maintained here:

https://source.android.com/devices/architecture/aidl/overview