1651432 - Perfherder UI tests need to be updated

Status: ASSIGNED

(Tree Management :: Perfherder, task, P2)

Description

[Sarah Clements [:sclements]]

There's a deprecation notice for at least one test that uses waitForElement for graphs_view_test: "waitForElementhas been deprecated. Use afind*query (preferred: https://testing-library.com/docs/dom-testing-library/api-queries#findby) or usewaitFor` instead: https://testing-library.com/docs/dom-testing-library/api-async#waitfor"

There's also been a lot of react console warnings for this particular test suite that have been occurring for a while and creates a lot of noise when running the tests. It'd be nice to get those cleaned up.

Comment 1

[Acasandrei Beatrice (needinfo me)]

How can I view these console warnings? I have tried to run the tests locally but there is no such warning. Thanks in advance for your help!

Comment 2

[Sarah Clements [:sclements]]

I don't see that particular waitForElement warning anymore. Perhaps someone fixed it... I do see a few different console.error messages for Perfherder components though.

Comment 3

[Sarah Clements [:sclements]]

There's also a really annoying alerts tests (in alerts_test file) that fails intermittently when running yarn test (outside of docker) locally but it never fails on staging/production. It would be wonderful if that could be fixed :)

Comment 4

[Sarah Clements [:sclements]]

Are you running the tests in docker or outside of docker and not seeing any of the red console.error warnings?

Comment 5

[Acasandrei Beatrice (needinfo me)]

(In reply to Sarah Clements [:sclements] from comment #4)

Are you running the tests in docker or outside of docker and not seeing any of the red console.error warnings?

I am running the tests outside of docker and I am also getting all the errors and warnings mentioned by you. I just couldn't find the waitForElement warning and didn't know how to reproduce it.