Add methods and unit tests to Shader class

[arrieta]

I've added a prototype for the Shader class. It will be used to contain a graphics shader. It needs methods to compile, link, validate, and so on. Every individual piece of functionality must be tested in the corresponding unit test.

[fayalalebrun]

I've been working on this class, and I've seen you've added both Pixel and Fragment to the Stage enum. According to what I know, Pixel and Fragment shaders refer to the same thing (Although it is called a fragment shader in OpenGL). Should I then remove Pixel out of the Stage enum, or did you mean some other shader stage with Pixel?

[arrieta]

Yeah, it is kind of confusing.

I meant Pixel shaders for Fragment shaders used in 2D graphics (e.g., planetary images) as opposed to the 2D stage of a 3D rendering call.

Maybe it causes more confusion than it’s worth?

Let’s remove it and add it back if/when we ever need it.

Good catch!

[fayalalebrun]

The Shader class seems to be mostly complete, I think this issue should be closed.