close #345 fixed Node<T> move constructor

[AryanGitHub]

used std::move() in Node move constructor

A good pull request has the following information:

• fixed For Node<T> move constructor, T copy constructor is called when the move constructor is not defined but the move assignment operator is defined. #345 by replacing std::swap with std::move
• all tests passed locally.

[----------] Global test environment tear-down
[==========] 246 tests from 33 test suites ran. (357 ms total)
[  PASSED  ] 246 tests.

@ZigRazor

[ghost]

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

Legend

[codecov]

Codecov Report

Merging #346 (b2e99a0) into master (7c03e1a) will not change coverage.
Report is 1 commits behind head on master.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #346   +/-   ##
=======================================
  Coverage   97.31%   97.31%           
=======================================
  Files          55       55           
  Lines        8455     8455           
=======================================
  Hits         8228     8228           
  Misses        227      227           

Flag	Coverage Δ
unittests	97.31% <100.00%> (ø)

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

Files Changed	Coverage Δ
include/Node/Node.hpp	100.00% <100.00%> (ø)