Brett Chabot | b1052c2 | 2011-01-12 19:44:23 -0800 | [diff] [blame] | 1 | /* |
| 2 | * Copyright (C) 2011 The Android Open Source Project |
| 3 | * |
| 4 | * Licensed under the Apache License, Version 2.0 (the "License"); |
| 5 | * you may not use this file except in compliance with the License. |
| 6 | * You may obtain a copy of the License at |
| 7 | * |
| 8 | * http://www.apache.org/licenses/LICENSE-2.0 |
| 9 | * |
| 10 | * Unless required by applicable law or agreed to in writing, software |
| 11 | * distributed under the License is distributed on an "AS IS" BASIS, |
| 12 | * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 13 | * See the License for the specific language governing permissions and |
| 14 | * limitations under the License. |
| 15 | */ |
| 16 | package com.android.tradefed.invoker; |
| 17 | |
Jed Estep | 1db79d1 | 2015-12-14 18:18:51 -0800 | [diff] [blame] | 18 | import com.android.ddmlib.Log.LogLevel; |
Brett Chabot | 2ea6ede | 2014-08-04 11:15:19 -0700 | [diff] [blame] | 19 | import com.android.ddmlib.testrunner.TestResult.TestStatus; |
Jed Estep | 1db79d1 | 2015-12-14 18:18:51 -0800 | [diff] [blame] | 20 | import com.android.tradefed.log.LogUtil.CLog; |
Julien Desprez | 74089e1 | 2018-05-31 01:47:53 -0700 | [diff] [blame] | 21 | import com.android.tradefed.metrics.proto.MetricMeasurement.Metric; |
Brett Chabot | b1052c2 | 2011-01-12 19:44:23 -0800 | [diff] [blame] | 22 | import com.android.tradefed.result.CollectingTestListener; |
Julien Desprez | 8ee0923 | 2020-02-06 10:30:11 -0800 | [diff] [blame] | 23 | import com.android.tradefed.result.FailureDescription; |
jdesprez | 2ffb1bd | 2018-03-19 10:37:23 -0700 | [diff] [blame] | 24 | import com.android.tradefed.result.ILogSaverListener; |
Brett Chabot | b1052c2 | 2011-01-12 19:44:23 -0800 | [diff] [blame] | 25 | import com.android.tradefed.result.ITestInvocationListener; |
Omari Stephens | 41978ee | 2011-02-23 19:03:05 -0800 | [diff] [blame] | 26 | import com.android.tradefed.result.InputStreamSource; |
Brett Chabot | b1052c2 | 2011-01-12 19:44:23 -0800 | [diff] [blame] | 27 | import com.android.tradefed.result.LogDataType; |
jdesprez | 2ffb1bd | 2018-03-19 10:37:23 -0700 | [diff] [blame] | 28 | import com.android.tradefed.result.LogFile; |
jdesprez | 109b1de | 2018-02-07 16:33:40 -0800 | [diff] [blame] | 29 | import com.android.tradefed.result.TestDescription; |
jdesprez | 821a005 | 2018-01-16 03:37:16 -0800 | [diff] [blame] | 30 | import com.android.tradefed.result.TestResult; |
| 31 | import com.android.tradefed.result.TestRunResult; |
Julien Desprez | 9ce5fa3 | 2019-08-02 14:49:29 -0700 | [diff] [blame] | 32 | import com.android.tradefed.result.retry.ISupportGranularResults; |
jdesprez | 0b8464a | 2017-06-26 17:33:58 -0700 | [diff] [blame] | 33 | import com.android.tradefed.util.TimeUtil; |
Brett Chabot | b1052c2 | 2011-01-12 19:44:23 -0800 | [diff] [blame] | 34 | |
Julien Desprez | 9ce5fa3 | 2019-08-02 14:49:29 -0700 | [diff] [blame] | 35 | import java.util.ArrayList; |
jdesprez | 0b8464a | 2017-06-26 17:33:58 -0700 | [diff] [blame] | 36 | import java.util.Collection; |
Julien Desprez | 74089e1 | 2018-05-31 01:47:53 -0700 | [diff] [blame] | 37 | import java.util.HashMap; |
Julien Desprez | 9ce5fa3 | 2019-08-02 14:49:29 -0700 | [diff] [blame] | 38 | import java.util.List; |
Brett Chabot | b1052c2 | 2011-01-12 19:44:23 -0800 | [diff] [blame] | 39 | import java.util.Map; |
Julien Desprez | e6b756b | 2018-04-16 10:56:38 -0700 | [diff] [blame] | 40 | import java.util.Map.Entry; |
Brett Chabot | b1052c2 | 2011-01-12 19:44:23 -0800 | [diff] [blame] | 41 | |
| 42 | /** |
| 43 | * A {@link ITestInvocationListener} that collects results from a invocation shard (aka an |
| 44 | * invocation split to run on multiple resources in parallel), and forwards them to another |
| 45 | * listener. |
| 46 | */ |
Julien Desprez | 9ce5fa3 | 2019-08-02 14:49:29 -0700 | [diff] [blame] | 47 | public class ShardListener extends CollectingTestListener implements ISupportGranularResults { |
Brett Chabot | b1052c2 | 2011-01-12 19:44:23 -0800 | [diff] [blame] | 48 | |
| 49 | private ITestInvocationListener mMasterListener; |
Julien Desprez | f97eb34 | 2019-05-28 11:49:54 -0700 | [diff] [blame] | 50 | private IInvocationContext mModuleContext = null; |
Julien Desprez | 9ce5fa3 | 2019-08-02 14:49:29 -0700 | [diff] [blame] | 51 | private int mAttemptInProgress = 0; |
| 52 | private boolean mEnableGranularResults = false; |
Brett Chabot | b1052c2 | 2011-01-12 19:44:23 -0800 | [diff] [blame] | 53 | |
| 54 | /** |
| 55 | * Create a {@link ShardListener}. |
| 56 | * |
Julien Desprez | 453d889 | 2017-02-06 14:13:34 +0000 | [diff] [blame] | 57 | * @param master the {@link ITestInvocationListener} the results should be forwarded. To prevent |
| 58 | * collisions with other {@link ShardListener}s, this object will synchronize on |
| 59 | * <var>master</var> when forwarding results. And results will only be sent once the |
| 60 | * invocation shard completes. |
Brett Chabot | b1052c2 | 2011-01-12 19:44:23 -0800 | [diff] [blame] | 61 | */ |
Julien Desprez | 453d889 | 2017-02-06 14:13:34 +0000 | [diff] [blame] | 62 | public ShardListener(ITestInvocationListener master) { |
Brett Chabot | b1052c2 | 2011-01-12 19:44:23 -0800 | [diff] [blame] | 63 | mMasterListener = master; |
| 64 | } |
| 65 | |
Julien Desprez | 9ce5fa3 | 2019-08-02 14:49:29 -0700 | [diff] [blame] | 66 | /** {@inheritDoc} */ |
| 67 | @Override |
| 68 | public boolean supportGranularResults() { |
| 69 | return mEnableGranularResults; |
| 70 | } |
| 71 | |
| 72 | public void setSupportGranularResults(boolean enableGranularResults) { |
| 73 | mEnableGranularResults = enableGranularResults; |
| 74 | } |
| 75 | |
Brett Chabot | b1052c2 | 2011-01-12 19:44:23 -0800 | [diff] [blame] | 76 | /** |
| 77 | * {@inheritDoc} |
| 78 | */ |
Brett Chabot | b1052c2 | 2011-01-12 19:44:23 -0800 | [diff] [blame] | 79 | @Override |
Julien Desprez | 53353e6 | 2016-08-12 15:24:33 +0100 | [diff] [blame] | 80 | public void invocationStarted(IInvocationContext context) { |
| 81 | super.invocationStarted(context); |
| 82 | synchronized (mMasterListener) { |
| 83 | mMasterListener.invocationStarted(context); |
| 84 | } |
| 85 | } |
| 86 | |
| 87 | /** |
| 88 | * {@inheritDoc} |
| 89 | */ |
| 90 | @Override |
Brett Chabot | b1052c2 | 2011-01-12 19:44:23 -0800 | [diff] [blame] | 91 | public void invocationFailed(Throwable cause) { |
| 92 | super.invocationFailed(cause); |
| 93 | synchronized (mMasterListener) { |
| 94 | mMasterListener.invocationFailed(cause); |
| 95 | } |
| 96 | } |
| 97 | |
Julien Desprez | 7fc5425 | 2020-04-30 15:08:45 -0700 | [diff] [blame^] | 98 | /** {@inheritDoc} */ |
| 99 | @Override |
| 100 | public void invocationFailed(FailureDescription failure) { |
| 101 | super.invocationFailed(failure); |
| 102 | synchronized (mMasterListener) { |
| 103 | mMasterListener.invocationFailed(failure); |
| 104 | } |
| 105 | } |
| 106 | |
Brett Chabot | b1052c2 | 2011-01-12 19:44:23 -0800 | [diff] [blame] | 107 | /** |
| 108 | * {@inheritDoc} |
| 109 | */ |
| 110 | @Override |
Omari Stephens | 41978ee | 2011-02-23 19:03:05 -0800 | [diff] [blame] | 111 | public void testLog(String dataName, LogDataType dataType, InputStreamSource dataStream) { |
Julien Desprez | e6b756b | 2018-04-16 10:56:38 -0700 | [diff] [blame] | 112 | // forward testLog results immediately, since result reporters might take action on it. |
Brett Chabot | b1052c2 | 2011-01-12 19:44:23 -0800 | [diff] [blame] | 113 | synchronized (mMasterListener) { |
Julien Desprez | e6b756b | 2018-04-16 10:56:38 -0700 | [diff] [blame] | 114 | if (mMasterListener instanceof ShardMasterResultForwarder) { |
| 115 | // If the listener is a log saver, we should simply forward the testLog not save |
| 116 | // again. |
| 117 | ((ShardMasterResultForwarder) mMasterListener) |
| 118 | .testLogForward(dataName, dataType, dataStream); |
| 119 | } else { |
| 120 | mMasterListener.testLog(dataName, dataType, dataStream); |
| 121 | } |
Brett Chabot | b1052c2 | 2011-01-12 19:44:23 -0800 | [diff] [blame] | 122 | } |
| 123 | } |
| 124 | |
jdesprez | 2ffb1bd | 2018-03-19 10:37:23 -0700 | [diff] [blame] | 125 | /** {@inheritDoc} */ |
| 126 | @Override |
| 127 | public void testLogSaved( |
| 128 | String dataName, LogDataType dataType, InputStreamSource dataStream, LogFile logFile) { |
| 129 | super.testLogSaved(dataName, dataType, dataStream, logFile); |
| 130 | // Forward the testLogSaved callback. |
| 131 | synchronized (mMasterListener) { |
| 132 | if (mMasterListener instanceof ILogSaverListener) { |
| 133 | ((ILogSaverListener) mMasterListener) |
| 134 | .testLogSaved(dataName, dataType, dataStream, logFile); |
| 135 | } |
| 136 | } |
| 137 | } |
| 138 | |
| 139 | /** {@inheritDoc} */ |
Brett Chabot | b1052c2 | 2011-01-12 19:44:23 -0800 | [diff] [blame] | 140 | @Override |
Julien Desprez | f97eb34 | 2019-05-28 11:49:54 -0700 | [diff] [blame] | 141 | public void testModuleStarted(IInvocationContext moduleContext) { |
| 142 | super.testModuleStarted(moduleContext); |
| 143 | mModuleContext = moduleContext; |
Jed Estep | 1db79d1 | 2015-12-14 18:18:51 -0800 | [diff] [blame] | 144 | } |
| 145 | |
Julien Desprez | 9ce5fa3 | 2019-08-02 14:49:29 -0700 | [diff] [blame] | 146 | /** {@inheritDoc} */ |
| 147 | @Override |
| 148 | public void testRunStarted(String name, int numTests, int attemptNumber, long startTime) { |
| 149 | super.testRunStarted(name, numTests, attemptNumber, startTime); |
| 150 | mAttemptInProgress = attemptNumber; |
| 151 | } |
| 152 | |
Jed Estep | 1db79d1 | 2015-12-14 18:18:51 -0800 | [diff] [blame] | 153 | /** |
| 154 | * {@inheritDoc} |
| 155 | */ |
| 156 | @Override |
| 157 | public void testRunFailed(String failureMessage) { |
| 158 | super.testRunFailed(failureMessage); |
| 159 | CLog.logAndDisplay(LogLevel.ERROR, "FAILED: %s failed with message: %s", |
| 160 | getCurrentRunResults().getName(), failureMessage); |
| 161 | } |
| 162 | |
Julien Desprez | f97eb34 | 2019-05-28 11:49:54 -0700 | [diff] [blame] | 163 | /** {@inheritDoc} */ |
Jed Estep | 1db79d1 | 2015-12-14 18:18:51 -0800 | [diff] [blame] | 164 | @Override |
Julien Desprez | 8ee0923 | 2020-02-06 10:30:11 -0800 | [diff] [blame] | 165 | public void testRunFailed(FailureDescription failure) { |
| 166 | super.testRunFailed(failure); |
| 167 | CLog.logAndDisplay( |
| 168 | LogLevel.ERROR, |
| 169 | "FAILED: %s failed with message: %s", |
| 170 | getCurrentRunResults().getName(), |
| 171 | failure.toString()); |
| 172 | } |
| 173 | |
| 174 | /** {@inheritDoc} */ |
| 175 | @Override |
Julien Desprez | f97eb34 | 2019-05-28 11:49:54 -0700 | [diff] [blame] | 176 | public void testRunEnded(long elapsedTime, HashMap<String, Metric> runMetrics) { |
| 177 | super.testRunEnded(elapsedTime, runMetrics); |
| 178 | CLog.logAndDisplay( |
| 179 | LogLevel.INFO, "Sharded test completed: %s", getCurrentRunResults().getName()); |
| 180 | if (mModuleContext == null) { |
| 181 | // testRunEnded only forwards if it's not part of a module. If it's a module |
| 182 | // testModuleEnded is in charge of forwarding all run results. |
| 183 | synchronized (mMasterListener) { |
Julien Desprez | 9ce5fa3 | 2019-08-02 14:49:29 -0700 | [diff] [blame] | 184 | forwardRunResults(getCurrentRunResults(), mAttemptInProgress); |
Julien Desprez | f97eb34 | 2019-05-28 11:49:54 -0700 | [diff] [blame] | 185 | } |
Julien Desprez | 9ce5fa3 | 2019-08-02 14:49:29 -0700 | [diff] [blame] | 186 | mAttemptInProgress = 0; |
Julien Desprez | f97eb34 | 2019-05-28 11:49:54 -0700 | [diff] [blame] | 187 | } |
Julien Desprez | 9ce5fa3 | 2019-08-02 14:49:29 -0700 | [diff] [blame] | 188 | |
Julien Desprez | f97eb34 | 2019-05-28 11:49:54 -0700 | [diff] [blame] | 189 | } |
| 190 | |
| 191 | /** {@inheritDoc} */ |
| 192 | @Override |
| 193 | public void testModuleEnded() { |
| 194 | super.testModuleEnded(); |
| 195 | |
Brett Chabot | b1052c2 | 2011-01-12 19:44:23 -0800 | [diff] [blame] | 196 | synchronized (mMasterListener) { |
Julien Desprez | 9ce5fa3 | 2019-08-02 14:49:29 -0700 | [diff] [blame] | 197 | mMasterListener.testModuleStarted(mModuleContext); |
| 198 | List<String> resultNames = new ArrayList<String>(); |
| 199 | if (mEnableGranularResults) { |
| 200 | for (int i = 0; i < mAttemptInProgress + 1; i++) { |
| 201 | List<TestRunResult> runResults = getTestRunForAttempts(i); |
| 202 | for (TestRunResult runResult : runResults) { |
| 203 | forwardRunResults(runResult, i); |
| 204 | resultNames.add(runResult.getName()); |
| 205 | } |
Julien Desprez | f97eb34 | 2019-05-28 11:49:54 -0700 | [diff] [blame] | 206 | } |
Julien Desprez | 9ce5fa3 | 2019-08-02 14:49:29 -0700 | [diff] [blame] | 207 | } else { |
| 208 | for (TestRunResult runResult : getMergedTestRunResults()) { |
| 209 | // Forward the run level results |
| 210 | forwardRunResults(runResult, 0); |
| 211 | resultNames.add(runResult.getName()); |
| 212 | } |
| 213 | } |
Eric Rowe | a6d559e | 2018-07-26 12:01:02 -0700 | [diff] [blame] | 214 | |
Julien Desprez | 9ce5fa3 | 2019-08-02 14:49:29 -0700 | [diff] [blame] | 215 | // Ensure we don't carry results from one module to another. |
| 216 | for (String name : resultNames) { |
| 217 | clearResultsForName(name); |
Brett Chabot | b1052c2 | 2011-01-12 19:44:23 -0800 | [diff] [blame] | 218 | } |
Julien Desprez | 9ce5fa3 | 2019-08-02 14:49:29 -0700 | [diff] [blame] | 219 | mMasterListener.testModuleEnded(); |
Julien Desprez | f97eb34 | 2019-05-28 11:49:54 -0700 | [diff] [blame] | 220 | } |
| 221 | mModuleContext = null; |
| 222 | } |
| 223 | |
| 224 | /** {@inheritDoc} */ |
| 225 | @Override |
| 226 | public void invocationEnded(long elapsedTime) { |
| 227 | super.invocationEnded(elapsedTime); |
| 228 | synchronized (mMasterListener) { |
| 229 | logShardContent(getMergedTestRunResults()); |
Julien Desprez | 69c2620 | 2019-06-04 09:31:24 -0700 | [diff] [blame] | 230 | // Report all logs not associated with test runs |
| 231 | forwardLogAssociation(getNonAssociatedLogFiles(), mMasterListener); |
Brett Chabot | b1052c2 | 2011-01-12 19:44:23 -0800 | [diff] [blame] | 232 | mMasterListener.invocationEnded(elapsedTime); |
| 233 | } |
| 234 | } |
| 235 | |
Julien Desprez | 9ce5fa3 | 2019-08-02 14:49:29 -0700 | [diff] [blame] | 236 | private void forwardRunResults(TestRunResult runResult, int attempt) { |
| 237 | mMasterListener.testRunStarted( |
| 238 | runResult.getName(), |
| 239 | runResult.getExpectedTestCount(), |
| 240 | attempt, |
| 241 | runResult.getStartTime()); |
Julien Desprez | f97eb34 | 2019-05-28 11:49:54 -0700 | [diff] [blame] | 242 | forwardTestResults(runResult.getTestResults()); |
| 243 | if (runResult.isRunFailure()) { |
Julien Desprez | ead2e46 | 2020-02-18 14:54:12 -0800 | [diff] [blame] | 244 | mMasterListener.testRunFailed(runResult.getRunFailureDescription()); |
Julien Desprez | f97eb34 | 2019-05-28 11:49:54 -0700 | [diff] [blame] | 245 | } |
| 246 | |
| 247 | // Provide a strong association of the run to its logs. |
| 248 | forwardLogAssociation(runResult.getRunLoggedFiles(), mMasterListener); |
| 249 | |
| 250 | mMasterListener.testRunEnded(runResult.getElapsedTime(), runResult.getRunProtoMetrics()); |
| 251 | } |
| 252 | |
jdesprez | 109b1de | 2018-02-07 16:33:40 -0800 | [diff] [blame] | 253 | private void forwardTestResults(Map<TestDescription, TestResult> testResults) { |
| 254 | for (Map.Entry<TestDescription, TestResult> testEntry : testResults.entrySet()) { |
jdesprez | 4f7ade0 | 2017-04-03 11:46:11 -0700 | [diff] [blame] | 255 | mMasterListener.testStarted(testEntry.getKey(), testEntry.getValue().getStartTime()); |
Brett Chabot | 2ea6ede | 2014-08-04 11:15:19 -0700 | [diff] [blame] | 256 | switch (testEntry.getValue().getStatus()) { |
| 257 | case FAILURE: |
Julien Desprez | ead2e46 | 2020-02-18 14:54:12 -0800 | [diff] [blame] | 258 | mMasterListener.testFailed( |
| 259 | testEntry.getKey(), testEntry.getValue().getFailure()); |
Brett Chabot | 2ea6ede | 2014-08-04 11:15:19 -0700 | [diff] [blame] | 260 | break; |
| 261 | case ASSUMPTION_FAILURE: |
| 262 | mMasterListener.testAssumptionFailure(testEntry.getKey(), |
| 263 | testEntry.getValue().getStackTrace()); |
| 264 | break; |
| 265 | case IGNORED: |
| 266 | mMasterListener.testIgnored(testEntry.getKey()); |
| 267 | break; |
Julien Desprez | 2763ca0 | 2016-12-06 14:39:33 +0000 | [diff] [blame] | 268 | default: |
| 269 | break; |
Brett Chabot | b1052c2 | 2011-01-12 19:44:23 -0800 | [diff] [blame] | 270 | } |
Julien Desprez | e6b756b | 2018-04-16 10:56:38 -0700 | [diff] [blame] | 271 | // Provide a strong association of the test to its logs. |
| 272 | forwardLogAssociation(testEntry.getValue().getLoggedFiles(), mMasterListener); |
| 273 | |
Brett Chabot | e3b8b9e | 2011-10-21 14:17:06 -0700 | [diff] [blame] | 274 | if (!testEntry.getValue().getStatus().equals(TestStatus.INCOMPLETE)) { |
jdesprez | 4f7ade0 | 2017-04-03 11:46:11 -0700 | [diff] [blame] | 275 | mMasterListener.testEnded( |
| 276 | testEntry.getKey(), |
| 277 | testEntry.getValue().getEndTime(), |
Julien Desprez | 04db83b | 2018-04-25 06:06:36 -0700 | [diff] [blame] | 278 | testEntry.getValue().getProtoMetrics()); |
Brett Chabot | e3b8b9e | 2011-10-21 14:17:06 -0700 | [diff] [blame] | 279 | } |
Brett Chabot | b1052c2 | 2011-01-12 19:44:23 -0800 | [diff] [blame] | 280 | } |
| 281 | } |
jdesprez | 0b8464a | 2017-06-26 17:33:58 -0700 | [diff] [blame] | 282 | |
Julien Desprez | e6b756b | 2018-04-16 10:56:38 -0700 | [diff] [blame] | 283 | /** Forward to the listener the logAssociated callback on the files. */ |
| 284 | private void forwardLogAssociation( |
| 285 | Map<String, LogFile> loggedFiles, ITestInvocationListener listener) { |
| 286 | for (Entry<String, LogFile> logFile : loggedFiles.entrySet()) { |
| 287 | if (listener instanceof ILogSaverListener) { |
| 288 | ((ILogSaverListener) listener).logAssociation(logFile.getKey(), logFile.getValue()); |
| 289 | } |
| 290 | } |
| 291 | } |
| 292 | |
jdesprez | 0b8464a | 2017-06-26 17:33:58 -0700 | [diff] [blame] | 293 | /** Log the content of the shard for easier debugging. */ |
| 294 | private void logShardContent(Collection<TestRunResult> listResults) { |
| 295 | CLog.d("================================================="); |
| 296 | CLog.d( |
| 297 | "========== Shard Primary Device %s ==========", |
| 298 | getInvocationContext().getDevices().get(0).getSerialNumber()); |
| 299 | for (TestRunResult runRes : listResults) { |
| 300 | CLog.d( |
| 301 | "\tRan '%s' in %s", |
| 302 | runRes.getName(), TimeUtil.formatElapsedTime(runRes.getElapsedTime())); |
| 303 | } |
| 304 | CLog.d("================================================="); |
| 305 | } |
Brett Chabot | b1052c2 | 2011-01-12 19:44:23 -0800 | [diff] [blame] | 306 | } |