Fix unit tests:

[lemirep]

• TEXCOORD_1 and TEXCOORD_2 to Qt3DRender::QAttribute name was missing
• Context was missing accessor info for checkForAssetPipelineEditorProperties
• Rewrote geometryIsGLTF2Valid to make it easier to understand

[jjcasmar]

I would say commit message and changes are unrelated. The change in the is valid function has nothing to do with the tests. Same happens for the TEXCOOR_1/2.

Regarding the refactor of the is valid function, I wouldn't say its easier to understand now. You are defining a lambda, which is invoked directly. Also, you are capturing the attributes and returning from the lambda. I found the current implementation (based on std::find_if) easy, but if you prefer to modify it, I would suggest going for a raw loop instead of a lambda