Added `echo` bool to capture.

The capture method should have an echo parameter. When echo=True (the default) text should be captured and written to the terminal. When echo=False the text should be captured but not written to the terminal.

See https://github.com/Textualize/rich/issues/2172 for context.

[SertugF]

Hello, i have been working on this. If echo is true by default it breaks the test below. Since new capture method prints all captured strings ‘out’ changes.

def test_capture_and_record(capsys):
    recorder = Console(record=True)
    recorder.print("ABC")

    with recorder.capture() as capture:
        recorder.print("Hello")

    assert capture.get() == "Hello\n"

    recorded_text = recorder.export_text()
    out, err = capsys.readouterr()

    assert recorded_text == "ABC\nHello\n"
    assert capture.get() == "Hello\n"
    assert out == "ABC\n"

terminal output :

In order to pass previous tests should i just change default ‘echo’ value to ‘false’ ? Or does this test needs to be replaced ?

[darrenburns]

I think #2172 is unrelated to what actually gets written to the terminal and is instead about record=True not recording things that occur inside a Capture context. Doesn’t impact developing this feature, just thought I’d point it out.