remove tee
Author: ArthurZucker
Pull request: yes

# What does this PR do?
Remove the piping with `| tee` when running pytest. 
1. No one looks at the artifacts, and `tee` makes the output un-readable, color less etc. 
2. I checked that even if you remove this, the outputs are still visible because Circle CI uses a custom file system handling for this and not the `output.txt`. 
3. Subtests are omitted from the output this it does not include everything

Comments:
Example test that does not have tee: https://app.circleci.com/pipelines/github/huggingface/transformers/63142/workflows/4754f622-1e12-40bb-be2f-0dcb363a216b/jobs/778648/steps?invite=true#step-111-6582 



CI outputs without tee: 
![image (6)](https://user-images.githubusercontent.com/48595927/234875310-3d08d251-b7fc-49a6-8c21-8453224b7712.png)
With tee: 
<img width="995" alt="image" src="https://user-images.githubusercontent.com/48595927/234875612-209fd9fe-8961-4f25-abb4-11551402ee73.png">


Other files are still available: 
[~/transformers/installed.txt](https://output.circle-artifacts.com/output/job/c12dd81b-f182-4b6a-bdaf-051887b78948/artifacts/0/~/transformers/installed.txt)
[~/transformers/reports/tests_onnx/durations.txt](https://output.circle-artifacts.com/output/job/c12dd81b-f182-4b6a-bdaf-051887b78948/artifacts/0/~/transformers/reports/tests_onnx/durations.txt)
[~/transformers/reports/tests_onnx/stats.txt](https://output.circle-artifacts.com/output/job/c12dd81b-f182-4b6a-bdaf-051887b78948/artifacts/0/~/transformers/reports/tests_onnx/stats.txt)
[~/transformers/reports/tests_onnx/summary_short.txt](https://output.circle-artifacts.com/output/job/c12dd81b-f182-4b6a-bdaf-051887b78948/artifacts/0/~/transformers/reports/tests_onnx/summary_short.txt)
[~/transformers/reports/tests_onnx/warnings.txt](https://output.circle-artifacts.com/output/job/c12dd81b-f182-4b6a-bdaf-051887b78948/artifacts/0/~/transformers/reports/tests_onnx/warnings.txt) 

The docs for this PR live [here](https://moon-ci-docs.huggingface.co/docs/transformers/pr_23027). All of your documentation changes will be reflected on that endpoint.

> No one looks at the artifacts

Speak for yourself ;-) , I only look at artifacts since it's impossible to get the traceback in the output. I do not look at the output artifact however, so that change wouldn't impact how I use the reports. However I'm not alone so make sure @LysandreJik @amyeroberts and @ydshieh all agree before merging this.

(The full traceback can still be seen!)

It seems expanding the run test step is still fast, and personally I don't read `test_output.txt` but just the reports given by `--make_reports`, I am OK for this change.

One thing remaining is to remove the upload artifact step if we don't produce it. So far, we get

```bash

Uploading /home/circleci/transformers/tests_output.txt to ~/transformers/tests_output.txt
  No artifact files found at /home/circleci/transformers/tests_output.txt
Total size uploaded: 0 B
```

I use the artefacts all the time :D! 

I think it's fine for `test_outputs.txt` to go though, as I rarely look at it and I think all the other info can be found in the other .txt files 👍 

This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread.

Please note that issues that do not follow the [contributing guidelines](https://github.com/huggingface/transformers/blob/main/CONTRIBUTING.md) are likely to be ignored.