Fix SIGILL error in addEdges

[sbaldu]

This PR:

• adds a typedef member for the Node and Edge classes containing the type of the template parameter
• rewrites the type traits for said classes
• fixes the addEdges and addNodes methods, which produced a SIGILL error (issue [BUG] Illegal Instruction SIGILL in addEgdes_1 #386)
• add tests for type traits

[ghost]

👇 Click on the image for a new way to code review

Legend

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (1fdcff9) 97.54% compared to head (74ac660) 97.55%.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #389      +/-   ##
==========================================
+ Coverage   97.54%   97.55%   +0.01%     
==========================================
  Files          85       85              
  Lines        9357     9358       +1     
==========================================
+ Hits         9127     9129       +2     
+ Misses        230      229       -1     

Flag	Coverage Δ
unittests	97.55% <100.00%> (+0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[sbaldu]

@ZigRazor As a way to test the type traits I added an header file containing a list series of static asserts to test all the possible cases. This header is included in GraphTest.cpp and when everything works correctly it is compiled without any errors.
Do you think it's a good way to test it?

[sbaldu]

Also, I've only implemented variadic methods taking pointers to edges/nodes. I thought that it would be sensible to add another taking them by const ref, but that probably should be in another PR.

[ZigRazor]

Also, I've only implemented variadic methods taking pointers to edges/nodes. I thought that it would be sensible to add another taking them by const ref, but that probably should be in another PR.

Ok, @sbaldu