/*
 * Copyright 2010 the original author or authors.
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *      http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */

package org.gradle.api.internal.tasks.testing.processors;

import org.gradle.api.internal.tasks.testing.*;
import org.gradle.api.logging.StandardOutputListener;
import org.gradle.api.tasks.testing.TestOutputEvent;
import org.gradle.logging.StandardOutputRedirector;

/**
 * A {@link org.gradle.api.internal.tasks.testing.TestResultProcessor} which redirect stdout and stderr during the
 * execution of a test suite.
 */
public class CaptureTestOutputTestResultProcessor implements TestResultProcessor {
    private final TestResultProcessor processor;
    private final StandardOutputRedirector outputRedirector;
    private Object suiteId;

    public CaptureTestOutputTestResultProcessor(TestResultProcessor processor, StandardOutputRedirector outputRedirector) {
        this.processor = processor;
        this.outputRedirector = outputRedirector;
    }

    public void started(final TestDescriptorInternal test, TestStartEvent event) {
        processor.started(test, event);

        //should redirect output for every particular test
        redirectOutputFor(test.getId());

        //currently our test reports include std out/err per test class (aka suite) not per test method (aka test)
        //for historical reasons. Therefore we only start/stop redirector per suite.
        if (suiteId != null) {
            return;
        }
        suiteId = test.getId();
        outputRedirector.start();
    }

    public void completed(Object testId, TestCompleteEvent event) {
        if (testId.equals(suiteId)) {
            //when suite is completed we no longer redirect for this suite
            try {
                outputRedirector.stop();
            } finally {
                suiteId = null;
            }
        } else {
            //when test is completed, should redirect output for the 'suite' to log things like @AfterSuite, etc.
            //TODO SF throw exception early about null suiteId
            redirectOutputFor(suiteId);
        }
        processor.completed(testId, event);
    }

    private void redirectOutputFor(final Object testId) {
        outputRedirector.redirectStandardOutputTo(new StdOutForwarder(testId));
        outputRedirector.redirectStandardErrorTo(new StdErrForwarder(testId));
    }

    public void output(Object testId, TestOutputEvent event) {
        processor.output(testId, event);
    }

    public void failure(Object testId, Throwable result) {
        processor.failure(testId, result);
    }


    class StdOutForwarder implements StandardOutputListener {
        private final Object testId;

        public StdOutForwarder(Object testId) {
            this.testId = testId;
        }

        public void onOutput(CharSequence output) {
            processor.output(testId, new DefaultTestOutputEvent(TestOutputEvent.Destination.StdOut, output.toString()));
        }
    }

    class StdErrForwarder implements StandardOutputListener {
        private final Object testId;

        public StdErrForwarder(Object testId) {
            this.testId = testId;
        }

        public void onOutput(CharSequence output) {
            processor.output(testId, new DefaultTestOutputEvent(TestOutputEvent.Destination.StdErr, output.toString()));
        }
    }
}

@@ -2473,71 +2473,8 @@
 tc.%0A
-            //TODO SF throw exception early about null suiteId%0A
     
@@ -2682,22 +2682,16 @@
 tTo(new 
-StdOut
 Forwarde
@@ -2690,32 +2690,68 @@
 Forwarder(testId
+, TestOutputEvent.Destination.StdOut
 ));%0A        outp
@@ -2791,22 +2791,16 @@
 rTo(new 
-StdErr
 Forwarde
@@ -2807,16 +2807,52 @@
 r(testId
+, TestOutputEvent.Destination.StdErr
 ));%0A    
@@ -3074,17 +3074,16 @@
 %0A    %7D%0A%0A
-%0A
     clas
@@ -3084,22 +3084,16 @@
   class 
-StdOut
 Forwarde
@@ -3158,32 +3158,88 @@
 l Object testId;
+%0A        private final TestOutputEvent.Destination dest;
 %0A%0A        public
@@ -3239,22 +3239,16 @@
  public 
-StdOut
 Forwarde
@@ -3254,32 +3254,66 @@
 er(Object testId
+, TestOutputEvent.Destination dest
 ) %7B%0A            
@@ -3326,32 +3326,62 @@
 estId = testId;%0A
+            this.dest = dest;%0A
         %7D%0A%0A     
@@ -3443,116 +3443,29 @@
     
-processor.output(testId, new DefaultTestOutputEvent(TestOutputEvent.Destination.StdOut, output.toString()));
+if (testId == null) %7B
 %0A   
@@ -3473,261 +3473,188 @@
     
-%7D%0A
     
-%7D%0A%0A
     
-class StdErrForwarder implements StandardOutputListener %7B%0A        private final Object testId;%0A%0A        public StdErrForwarder(Object testId) %7B%0A            this.testId = testId;%0A        %7D%0A%0A        public void onOutput(CharSequence output) %7B
+throw new RuntimeException(%22Unable send output event from test executor. Please report this problem. Destination: %22 + dest + %22, event: %22 + output.toString());%0A            %7D
 %0A   
@@ -3718,42 +3718,12 @@
 ent(
-TestOutputEvent.Destination.StdErr
+dest
 , ou

Test infrastructure changes - failing fast when output event cannot be dispatched