Updates to run tests and fixes

Author: JustCallMeRay

.github/workflows/cmake.yml

@@ -2,9 +2,7 @@ name: CMake
 
 on:
   push:
-    branches: [ master ]
   pull_request:
-    branches: [ master ]
 
 env:
   # Customize the CMake build type here (Release, Debug, RelWithDebInfo, etc.)

include/CXXGraph/Node/Node_decl.h

@@ -28,8 +28,8 @@
 namespace CXXGraph {
 template <typename T>
 class Node;
-template <typename T, typename Stream>
-constexpr Stream &operator<<(Stream &os, const Node<T> &node);
+template <typename T>
+std::ostream &operator<<(std::ostream &os, const Node<T> &node);
 template <typename T>
 class Node {
  private:
@@ -45,15 +45,15 @@ class Node {
   // Move constructor
   explicit Node(const std::string &, T &&data) noexcept;
   ~Node() = default;
-  const CXXGraph::id_t &getId() const;
+  constexpr const CXXGraph::id_t &getId() const;
   const std::string &getUserId() const;
-  const T &getData() const;
+  constexpr const T &getData() const;
   void setData(T &&new_data);
   // operator
-  bool operator==(const Node<T> &b) const;
-  bool operator<(const Node<T> &b) const;
-  template<typename Stream>
-  friend Stream &operator<< <>(Stream &os, const Node<T> &node);
+  constexpr bool operator==(const Node<T> &b) const;
+  constexpr bool operator<(const Node<T> &b) const;
+  
+  friend std::ostream &operator<< <>(std::ostream &os, const Node<T> &node);
 };
 
 }  // namespace CXXGraph

include/CXXGraph/Node/Node_impl.hpp

@@ -83,8 +83,8 @@ constexpr bool Node<T>::operator<(const Node<T> &b) const {
 
 // ostream overload
 // The data type T must have an overload of the ostream operator
-template <typename T, typename Stream>
-constexpr Stream &operator<<(Stream &os, const Node<T> &node) {
+template <typename T>
+std::ostream &operator<<(std::ostream &os, const Node<T> &node) {
   os << "Node: {\n"
      << "  Id:\t" << node.userId << "\n  Data:\t" << node.data << "\n}";
   return os;