update(LinkedList): added more tests

Signed-off-by: Rohith-Raju <[email protected]>

.github/workflows/workflow.yaml

@@ -0,0 +1,25 @@
+name: Build
+on:
+  push:
+jobs:
+  build-project:
+    name: Build Project
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout 
+        uses: actions/[email protected]
+        with:
+          submodules: true 
+
+      - name: Configure Project
+        uses: threeal/[email protected]
+        with: 
+          c-compiler: gcc
+          cxx-compiler: g++
+          cxx-flags: "--std=c++17"
+
+      - name: Build Project
+        run: cmake --build build
+
+      - name: Run-Tests
+        run: ./build/test/LinkedList_test

CMakeLists.txt

@@ -1,6 +1,6 @@
 cmake_minimum_required(VERSION 3.27)
 
-project(CppTemplation)
+project(LinkedList)
 
 set(CMAKE_CXX_STANDARD 17)
 
@@ -11,4 +11,4 @@ include_directories(include)
 add_subdirectory(testing)
 
 # Sources
-add_executable(CppTemplation main.cpp)
+add_executable(LinkedList main.cpp)

include/LinkedList.h

@@ -48,6 +48,7 @@ template <class T> class LinkedList {
   // Move constructor
   LinkedList(LinkedList &&other);
 
+  // Destructor
   ~LinkedList();
 
   /* Iterators */
@@ -199,6 +200,11 @@ template <typename T> LinkedList<T> &LinkedList<T>::clean() {
   return *this;
 }
 
+/*
+ * reverse() reverses all the elements where head = prev_tail
+ * and  tail = prev_head
+ */
+
 template <typename T> void LinkedList<T>::reverse() {
   Node *first_ptr = nullptr;
   Node *second_ptr = head;
@@ -212,6 +218,9 @@ template <typename T> void LinkedList<T>::reverse() {
   head = first_ptr;
 }
 
+/*
+ * deep_clean() drills into the entire list and removes all the elements
+ */
 template <typename T> void LinkedList<T>::deep_clean(Node *&list) {
   if (list != tail) {
     deep_clean(list->next);
@@ -222,17 +231,28 @@ template <typename T> void LinkedList<T>::deep_clean(Node *&list) {
   return;
 }
 
+/*
+ * insert_first() inserts element only when there are no elements in the list
+ */
 template <typename T> void LinkedList<T>::insert_first(Node *node) {
-  head = node;
-  size_counter = 1;
-  if (tail == nullptr) {
-    tail = head;
+  if (!is_empty()) {
+    head = node;
+    size_counter = 1;
+    if (tail == nullptr) {
+      tail = head;
+    }
   }
   return;
 }
 
+/*
+ * is_empty() returns true if the list is empty
+ */
 template <typename T> bool LinkedList<T>::is_empty() { return head == nullptr; }
 
+/*
+ * push_back() append new element to the end of the list
+ */
 template <typename T> void LinkedList<T>::push_back(T data) {
   Node *node = new Node(data);
   if (is_empty()) {
@@ -243,9 +263,14 @@ template <typename T> void LinkedList<T>::push_back(T data) {
   tail->next = node;
   tail = node;
 }
-
+/*
+ * size() returns the size of the list
+ */
 template <typename T> size_t LinkedList<T>::size() { return size_counter; }
 
+/*
+ * at() returns the node containing the data at a given position
+ */
 template <typename T>
 typename LinkedList<T>::Node *LinkedList<T>::at(size_t pos) {
   if (pos >= size())
@@ -263,6 +288,9 @@ typename LinkedList<T>::Node *LinkedList<T>::operator[](size_t pos) {
   return at(pos);
 }
 
+/*
+ * delete_at() remove element in the given pos
+ */
 template <typename T> bool LinkedList<T>::delete_at(size_t pos) {
   if (pos >= size()) {
     throw std::out_of_range("Cannot delete. Index out of bounds");
@@ -298,13 +326,27 @@ template <typename T> bool LinkedList<T>::delete_at(size_t pos) {
   }
 }
 
+/*
+ * get_head_val() return the value of head
+ */
 template <typename T> T LinkedList<T>::get_head_val() { return head->data; }
 
+/*
+ * get_tail_val() return the value of tail
+ */
 template <typename T> T LinkedList<T>::get_tail_val() { return tail->data; }
+
+/*
+ * sort() sorts all the elements in the list (default accending)
+ */
 template <typename T> LinkedList<T> &LinkedList<T>::sort() {
   return sort([](T leftVal, T rightVal) { return leftVal < rightVal; });
 }
 
+/*
+ * sort() sorts all the elements in the list by passing in a anonymous function
+ */
+
 template <typename T>
 template <class function>
 LinkedList<T> &LinkedList<T>::sort(function &&func) {

main.cpp

@@ -1,12 +1,6 @@
 //
 // Created by Rohith on 1/10/24.
 //
-#include "LinkedList.h"
-#include "iostream"
 
 int main() {
-  LinkedList<int> list = {1, 2, 3, 4, 5};
-  for (auto it = list.rbegin(); it != list.rend(); it++) {
-    std::cout << *it;
-  }
 }

testing/TestLinkedList.cpp

@@ -82,7 +82,13 @@ TEST(LinkedList, TestDeleteAtPosition) {
   ASSERT_EQ(ptr->data, 3);
   list.delete_at(2);
   ASSERT_NE(ptr->data, 3);
-  // todo: write SIGSEGV test case
+}
+
+TEST(LinkedList, TestDeleteAtEnd) {
+  LinkedList list = {1, 2, 3, 4, 5};
+  ASSERT_EQ(list.get_tail_val(), 5);
+  list.delete_at(4);
+  ASSERT_EQ(list.get_tail_val(), 4);
 }
 
 TEST(LinkedList, TestCustomSort) {