From 1d0818ca3fca7a89dd07e637f14af826d679a3af Mon Sep 17 00:00:00 2001
From: malnadhudga <gourish.hegde@zop.dev>
Date: Wed, 18 Jun 2025 17:33:48 +0530
Subject: [PATCH 1/2] implemented the unit testing

---
 .idea/workspace.xml             |  68 +++++++-----
 assign3/assignment3.go          | 189 ++++++++++++++++++++++----------
 assign5/assign5_1/assign_5_1.go |  17 +++
 assign5/assign5_1/shape_test.go |  45 ++++++++
 4 files changed, 236 insertions(+), 83 deletions(-)
 create mode 100644 assign5/assign5_1/shape_test.go

diff --git a/.idea/workspace.xml b/.idea/workspace.xml
index 300a6e2..8d66291 100644
--- a/.idea/workspace.xml
+++ b/.idea/workspace.xml
@@ -4,7 +4,11 @@
     <option name="autoReloadType" value="ALL" />
   </component>
   <component name="ChangeListManager">
-    <list default="true" id="ed75f871-d04c-4e80-b59b-bdd6ea65a2dc" name="Changes" comment="" />
+    <list default="true" id="ed75f871-d04c-4e80-b59b-bdd6ea65a2dc" name="Changes" comment="">
+      <change afterPath="$PROJECT_DIR$/assign5/assign5_1/shape_test.go" afterDir="false" />
+      <change beforePath="$PROJECT_DIR$/.idea/workspace.xml" beforeDir="false" afterPath="$PROJECT_DIR$/.idea/workspace.xml" afterDir="false" />
+      <change beforePath="$PROJECT_DIR$/assign5/assign5_1/assign_5_1.go" beforeDir="false" afterPath="$PROJECT_DIR$/assign5/assign5_1/assign_5_1.go" afterDir="false" />
+    </list>
     <option name="SHOW_DIALOG" value="false" />
     <option name="HIGHLIGHT_CONFLICTS" value="true" />
     <option name="HIGHLIGHT_NON_ACTIVE_CHANGELIST" value="false" />
@@ -34,37 +38,38 @@
     <option name="hideEmptyMiddlePackages" value="true" />
     <option name="showLibraryContents" value="true" />
   </component>
-  <component name="PropertiesComponent">{
-  &quot;keyToString&quot;: {
-    &quot;DefaultGoTemplateProperty&quot;: &quot;Go File&quot;,
-    &quot;ModuleVcsDetector.initialDetectionPerformed&quot;: &quot;true&quot;,
-    &quot;RunOnceActivity.GoLinterPluginOnboarding&quot;: &quot;true&quot;,
-    &quot;RunOnceActivity.GoLinterPluginStorageMigration&quot;: &quot;true&quot;,
-    &quot;RunOnceActivity.ShowReadmeOnStart&quot;: &quot;true&quot;,
-    &quot;RunOnceActivity.TerminalTabsStorage.copyFrom.TerminalArrangementManager&quot;: &quot;true&quot;,
-    &quot;RunOnceActivity.git.unshallow&quot;: &quot;true&quot;,
-    &quot;RunOnceActivity.go.formatter.settings.were.checked&quot;: &quot;true&quot;,
-    &quot;RunOnceActivity.go.migrated.go.modules.settings&quot;: &quot;true&quot;,
-    &quot;RunOnceActivity.go.modules.go.list.on.any.changes.was.set&quot;: &quot;true&quot;,
-    &quot;git-widget-placeholder&quot;: &quot;main&quot;,
-    &quot;go.import.settings.migrated&quot;: &quot;true&quot;,
-    &quot;go.sdk.automatically.set&quot;: &quot;true&quot;,
-    &quot;last_opened_file_path&quot;: &quot;/Users/zopdev/GolandProjects/assignment&quot;,
-    &quot;node.js.detected.package.eslint&quot;: &quot;true&quot;,
-    &quot;node.js.selected.package.eslint&quot;: &quot;(autodetect)&quot;,
-    &quot;nodejs_package_manager_path&quot;: &quot;npm&quot;
+  <component name="PropertiesComponent"><![CDATA[{
+  "keyToString": {
+    "DefaultGoTemplateProperty": "Go File",
+    "Go Test.TestCosts in assignment/assign5/assign5_1.executor": "Run",
+    "ModuleVcsDetector.initialDetectionPerformed": "true",
+    "RunOnceActivity.GoLinterPluginOnboarding": "true",
+    "RunOnceActivity.GoLinterPluginStorageMigration": "true",
+    "RunOnceActivity.ShowReadmeOnStart": "true",
+    "RunOnceActivity.TerminalTabsStorage.copyFrom.TerminalArrangementManager": "true",
+    "RunOnceActivity.git.unshallow": "true",
+    "RunOnceActivity.go.formatter.settings.were.checked": "true",
+    "RunOnceActivity.go.migrated.go.modules.settings": "true",
+    "RunOnceActivity.go.modules.go.list.on.any.changes.was.set": "true",
+    "git-widget-placeholder": "main",
+    "go.import.settings.migrated": "true",
+    "go.sdk.automatically.set": "true",
+    "last_opened_file_path": "/Users/zopdev/GolandProjects/assignment",
+    "node.js.detected.package.eslint": "true",
+    "node.js.selected.package.eslint": "(autodetect)",
+    "nodejs_package_manager_path": "npm"
   }
-}</component>
+}]]></component>
   <component name="RecentsManager">
     <key name="MoveFile.RECENT_KEYS">
+      <recent name="$PROJECT_DIR$/assign5/assign5_3" />
+      <recent name="$PROJECT_DIR$/assign5/assign5_2" />
+      <recent name="$PROJECT_DIR$/assign5/assign5_1" />
       <recent name="$PROJECT_DIR$" />
       <recent name="$PROJECT_DIR$/assign5" />
-      <recent name="$PROJECT_DIR$/assign4" />
-      <recent name="$PROJECT_DIR$/assign6" />
-      <recent name="$PROJECT_DIR$/assign2" />
     </key>
   </component>
-  <component name="RunManager" selected="Go Build.go build assignment/assign6">
+  <component name="RunManager" selected="Go Test.TestCosts in assignment/assign5/assign5_1">
     <configuration default="true" type="GoApplicationRunConfiguration" factoryName="Go Application">
       <module name="assignment" />
       <working_directory value="$PROJECT_DIR$" />
@@ -92,6 +97,18 @@
       <filePath value="$PROJECT_DIR$/assign6/assign_6_1.go" />
       <method v="2" />
     </configuration>
+    <configuration name="TestCosts in assignment/assign5/assign5_1" type="GoTestRunConfiguration" factoryName="Go Test" temporary="true" nameIsGenerated="true">
+      <module name="assignment" />
+      <working_directory value="$PROJECT_DIR$/assign5/assign5_1" />
+      <root_directory value="$PROJECT_DIR$" />
+      <kind value="PACKAGE" />
+      <package value="assignment/assign5/assign5_1" />
+      <directory value="$PROJECT_DIR$" />
+      <filePath value="$PROJECT_DIR$" />
+      <framework value="gotest" />
+      <pattern value="^\QTestCosts\E$" />
+      <method v="2" />
+    </configuration>
     <configuration default="true" type="GoTestRunConfiguration" factoryName="Go Test">
       <module name="assignment" />
       <working_directory value="$PROJECT_DIR$" />
@@ -103,6 +120,7 @@
     </configuration>
     <recent_temporary>
       <list>
+        <item itemvalue="Go Test.TestCosts in assignment/assign5/assign5_1" />
         <item itemvalue="Go Build.go build assignment/assign6" />
         <item itemvalue="Go Build.go build assignment" />
       </list>
diff --git a/assign3/assignment3.go b/assign3/assignment3.go
index 93cfd49..5c98130 100644
--- a/assign3/assignment3.go
+++ b/assign3/assignment3.go
@@ -1,78 +1,151 @@
 package main
 
-import "fmt"
+import (
+	"bufio"
+	"fmt"
+	"os"
+	"strconv"
+	"strings"
+)
 
-type BankAccount struct {
-	Owner   string
-	Balance float64
+// Task represents a single task in our tracker.
+type Task struct {
+	ID          int
+	Description string
+	Completed   bool
 }
 
-// a value receiver is appropriate here. Changes to 'b' inside this method
-// would only affect a copy, not the 'myAccount' variable in main.
-func (b BankAccount) DisplayBalance() {
-	fmt.Printf("%s's current balance: $%.2f\n", b.Owner, b.Balance)
+// TaskTracker manages the collection of tasks and generates unique IDs.
+type TaskTracker struct {
+	tasks     []Task
+	nextIDGen func() int
 }
 
-// Deposit is a method with a POINTER RECEIVER (b *BankAccount).
-// It increases the balance of the account by the given amount.
-func (b *BankAccount) Deposit(amount float64) {
-	if amount <= 0 {
-		fmt.Printf("Deposit amount must be positive. Skipping deposit of $%.2f.\n", amount)
-		return
+// idGenerator is a closure that generates unique sequential integer IDs.
+// It encapsulates the 'id' counter, so it's not a global variable.
+func idGenerator() func() int {
+	id := 0
+	return func() int {
+		id++
+		return id
 	}
-	b.Balance += amount // This modifies the original BankAccount's balance
-	fmt.Printf("Deposited $%.2f. New balance: $%.2f\n", amount, b.Balance)
 }
 
-// Withdraw is a method with a POINTER RECEIVER (b *BankAccount).
-// It decreases the balance, but only if sufficient funds exist.
-func (b *BankAccount) Withdraw(amount float64) {
-	if amount <= 0 {
-		fmt.Printf("Withdrawal amount must be positive. Skipping withdrawal of $%.2f.\n", amount)
-		return
-	}
-	if b.Balance >= amount {
-		b.Balance -= amount // This modifies the original BankAccount's balance
-		fmt.Printf("Withdrew $%.2f. New balance: $%.2f\n", amount, b.Balance)
-	} else {
-		fmt.Printf("Insufficient funds for withdrawal of $%.2f. Current balance: $%.2f.\n", amount, b.Balance)
+// NewTaskTracker creates and initializes a new TaskTracker instance.
+// It also sets up the unique ID generator.
+func NewTaskTracker() *TaskTracker {
+	return &TaskTracker{
+		tasks:     []Task{},
+		nextIDGen: idGenerator(),
 	}
 }
 
-// TryToModifyBalance is a standalone function that demonstrates passing a BankAccount by value.
-// Any changes to 'b' within this function will only affect a copy, not the original variable passed.
-func TryToModifyBalance(b BankAccount, amount float64) {
-	b.Balance += amount // This only modifies the *copy* of BankAccount passed in
-	fmt.Printf("  (Inside TryToModifyBalance func): Balance changed to $%.2f\n", b.Balance)
+// AddTask adds a new task to the tracker.
+// It uses a pointer receiver (*TaskTracker) because it modifies the TaskTracker's state (its 'tasks' slice).
+func (tt *TaskTracker) AddTask(description string) {
+	newID := tt.nextIDGen()
+	newTask := Task{
+		ID:          newID,
+		Description: description,
+		Completed:   false,
+	}
+	tt.tasks = append(tt.tasks, newTask)
+	fmt.Printf("Task Added: %d - %s\n", newTask.ID, newTask.Description)
 }
 
-func main() {
-	fmt.Println("--- Bank Account System Demonstration ---")
+// ListTasks displays all pending tasks.
+// It uses a pointer receiver (*TaskTracker) because it operates on the TaskTracker's 'tasks' slice,
+// even though it doesn't modify it directly in this function (good practice for methods operating on collections).
+func (tt *TaskTracker) ListTasks() {
+	fmt.Println("\nPending Tasks:")
+	foundPending := false
+	for _, task := range tt.tasks {
+		if !task.Completed {
+			fmt.Printf("%d: %s\n", task.ID, task.Description)
+			foundPending = true
+		}
+	}
+	if !foundPending {
+		fmt.Println("No pending tasks.")
+	}
+}
 
-	myAccount := BankAccount{
-		Owner:   "Alice",
-		Balance: 100.00, // Initial balance
+// CompleteTask marks a task as completed given its ID.
+// It uses a pointer receiver (*TaskTracker) because it modifies the state of a Task within the tracker's slice.
+func (tt *TaskTracker) CompleteTask(id int) {
+	taskFound := false
+	for i := range tt.tasks {
+		if tt.tasks[i].ID == id {
+			if tt.tasks[i].Completed {
+				fmt.Printf("Task %d is already completed.\n", id)
+			} else {
+				tt.tasks[i].Completed = true
+				fmt.Printf("Marking task %d as completed: %s\n", id, tt.tasks[i].Description)
+			}
+			taskFound = true
+			break
+		}
 	}
-	fmt.Println("\n--- Initial State ---")
-	myAccount.DisplayBalance()
-	fmt.Println("\n--- Deposit Funds ---")
-	myAccount.Deposit(50.50)
-	myAccount.DisplayBalance() // Observe the change
-	fmt.Println("\n--- Attempting Withdrawals ---")
-	myAccount.Withdraw(25.00)
-	myAccount.DisplayBalance()
-	myAccount.Withdraw(200.00)
-	myAccount.DisplayBalance()
+	if !taskFound {
+		fmt.Printf("Task with ID %d not found.\n", id)
+	}
+}
 
-	fmt.Println("\n--- Final State ---")
-	myAccount.DisplayBalance()
-	fmt.Println("\n--- Demonstrating Value Receiver vs. Pointer Receiver Effect ---")
-	tempAccount := BankAccount{Owner: "Bob", Balance: 50.00}
-	fmt.Printf("Bob's balance BEFORE TryToModifyBalance: $%.2f\n", tempAccount.Balance)
-	TryToModifyBalance(tempAccount, 20.00) // Pass by value
-	fmt.Printf("Bob's balance AFTER TryToModifyBalance: $%.2f (Did not change externally)\n", tempAccount.Balance)
+// displayMenu prints the interactive menu options to the console.
+func displayMenu() {
+	fmt.Println("\n--- Personal Task Tracker ---")
+	fmt.Println("1. Add a new task")
+	fmt.Println("2. List all pending tasks")
+	fmt.Println("3. Mark a task as completed")
+	fmt.Println("4. Exit")
+	fmt.Print("Choose an option: ")
+}
+
+// getUserInput reads a line of text from the standard input.
+func getUserInput() string {
+	reader := bufio.NewReader(os.Stdin)
+	input, _ := reader.ReadString('\n')
+	return strings.TrimSpace(input)
+}
 
-	fmt.Printf("Alice's balance BEFORE Deposit (again): $%.2f\n", myAccount.Balance)
-	myAccount.Deposit(10.00) // This actually modifies myAccount
-	fmt.Printf("Alice's balance AFTER Deposit (again): $%.2f (Changed externally)\n", myAccount.Balance)
+// main function orchestrates the CLI interaction.
+func main() {
+	tracker := NewTaskTracker()
+
+	for {
+		displayMenu()
+		choiceStr := getUserInput()
+		choice, err := strconv.Atoi(choiceStr)
+		if err != nil {
+			fmt.Println("Invalid choice. Please enter a number between 1 and 4.")
+			continue
+		}
+
+		switch choice {
+		case 1:
+			fmt.Print("Enter task description: ")
+			description := getUserInput()
+			if description == "" {
+				fmt.Println("Task description cannot be empty.")
+				continue
+			}
+			tracker.AddTask(description)
+		case 2:
+			tracker.ListTasks()
+		case 3:
+			fmt.Print("Enter ID of task to mark as completed: ")
+			idStr := getUserInput()
+			id, err := strconv.Atoi(idStr)
+			if err != nil {
+				fmt.Println("Invalid ID. Please enter a valid number.")
+				continue
+			}
+			tracker.CompleteTask(id)
+		case 4:
+			fmt.Println("Exiting Task Tracker. Goodbye!")
+			return
+		default:
+			fmt.Println("Invalid option. Please choose a number between 1 and 4.")
+		}
+	}
 }
diff --git a/assign5/assign5_1/assign_5_1.go b/assign5/assign5_1/assign_5_1.go
index 84fc85b..9ed768e 100644
--- a/assign5/assign5_1/assign_5_1.go
+++ b/assign5/assign5_1/assign_5_1.go
@@ -32,6 +32,17 @@ func calArea(s Shape) float64 {
 	return s.Area()
 }
 
+func costs(s Shape) float64 {
+	switch s.(type) {
+	case square:
+		return s.Area() * 100
+	case rectangle:
+		return s.Area() * 20
+	default:
+		return s.Area()
+	}
+}
+
 func main() {
 
 	rect := rectangle{b: 10, l: 20}
@@ -43,4 +54,10 @@ func main() {
 	sq := square{l: 10}
 	squareArea := calArea(sq)
 	fmt.Printf("Area of square: %.2f\n", squareArea)
+
+	cost := costs(sq)
+	fmt.Printf("Cost of square: %.2f\n", cost)
+
+	cost = costs(rect)
+	fmt.Printf("Cost of recct: %.2f\n", cost)
 }
diff --git a/assign5/assign5_1/shape_test.go b/assign5/assign5_1/shape_test.go
new file mode 100644
index 0000000..14fcda5
--- /dev/null
+++ b/assign5/assign5_1/shape_test.go
@@ -0,0 +1,45 @@
+package main
+
+import "testing"
+
+func TestCosts(t *testing.T) {
+	tests := []struct {
+		name         string
+		shape        Shape
+		expectedCost float64
+	}{
+		{
+			name:         "Square_Cost",
+			shape:        square{l: 5},
+			expectedCost: 25 * 100,
+		},
+		{
+			name:         "Rectangle_Cost",
+			shape:        rectangle{l: 4.222222, b: 1},
+			expectedCost: 4.222222 * 20,
+		},
+		{
+			name:         "Square_ZeroArea",
+			shape:        square{l: 0},
+			expectedCost: 0,
+		}, {
+			name:         "react_ZeroArea",
+			shape:        rectangle{l: 0, b: 0},
+			expectedCost: 0,
+		}}
+
+	for _, tt := range tests {
+		t.Logf("Running test case: %s \n", tt.name)
+
+		gotCost := costs(tt.shape)
+		tol := 1e-4
+		currentToll := gotCost - tt.expectedCost
+
+		if currentToll > tol {
+			t.Errorf("FAIL: Test %s  For shape %T (Area: %f), expected cost %f, but got %f",
+				tt.name, tt.shape, tt.shape.Area(), tt.expectedCost, gotCost)
+		} else {
+			t.Logf("PASS: Test %s", tt.name)
+		}
+	}
+}

From e417fe0ece99693d2830a00ce90ebed8617b8549 Mon Sep 17 00:00:00 2001
From: malnadhudga <gourish.hegde@zop.dev>
Date: Thu, 19 Jun 2025 11:17:24 +0530
Subject: [PATCH 2/2] comleted the testing for assignment3

---
 assign3/assign3_test.go | 81 +++++++++++++++++++++++++++++++++++++++++
 assign3/assignment3.go  | 41 ++++++++++-----------
 2 files changed, 101 insertions(+), 21 deletions(-)
 create mode 100644 assign3/assign3_test.go

diff --git a/assign3/assign3_test.go b/assign3/assign3_test.go
new file mode 100644
index 0000000..56b0ce3
--- /dev/null
+++ b/assign3/assign3_test.go
@@ -0,0 +1,81 @@
+package main
+
+import (
+	"fmt"
+	"testing"
+)
+
+func TestCompleteTask(t *testing.T) {
+	tracker := NewTaskTracker()
+	tracker.AddTask("Read a book")
+	tracker.AddTask("Play")
+	tracker.AddTask("Clean")
+
+	success, msg := tracker.CompleteTask(2)
+	if !success {
+		t.Errorf("Expected task 2 to be completed successfully, but it failed: %s", msg)
+	}
+	fmt.Println(msg)
+	if msg != "Marking task 2 as completed: Play" {
+		t.Errorf("Incorrect success message. Got: '%s'", msg)
+	}
+	if !tracker.tasks[1].Completed {
+		t.Errorf("Task 2 should be marked as completed")
+	}
+
+	success, msg = tracker.CompleteTask(2)
+	if success {
+		t.Errorf("Expected task 2 to not be completed, but it succeeded.")
+	}
+	if msg != "Task 2 is already completed." {
+		t.Errorf("Incorrect already completed message '%s'", msg)
+	}
+
+	success, msg = tracker.CompleteTask(99)
+	if success {
+		t.Errorf("Expected task 99 to not be completed, but it succeeded.")
+	}
+	if msg != "Task with ID 99 not found." {
+		t.Errorf("Incorrect not found message. Got: '%s'", msg)
+	}
+
+	if tracker.tasks[0].Completed {
+		t.Errorf("Task 1 should not be completed")
+	}
+	if tracker.tasks[2].Completed {
+		t.Errorf("Task 3 should not be completed")
+	}
+}
+
+func TestListTasks(t *testing.T) {
+	tracker := NewTaskTracker()
+
+	expected := "Pending Tasks:\nNo pending tasks."
+	if tracker.ListTasks() != expected {
+		t.Errorf("Expected '%s', got '%s'", expected, tracker.ListTasks())
+	}
+
+	tracker.AddTask("Task A")
+	expected = "Pending Tasks:\n1: Task A\n"
+	if tracker.ListTasks() != expected {
+		t.Errorf("Expected '%s', got '%s'", expected, tracker.ListTasks())
+	}
+
+	tracker.AddTask("Task B")
+	expected = "Pending Tasks:\n1: Task A\n2: Task B\n"
+	if tracker.ListTasks() != expected {
+		t.Errorf("Expected '%s', got '%s'", expected, tracker.ListTasks())
+	}
+
+	tracker.CompleteTask(1)
+	expected = "Pending Tasks:\n2: Task B\n"
+	if tracker.ListTasks() != expected {
+		t.Errorf("Expected '%s', got '%s'", expected, tracker.ListTasks())
+	}
+
+	tracker.CompleteTask(2)
+	expected = "Pending Tasks:\nNo pending tasks."
+	if tracker.ListTasks() != expected {
+		t.Errorf("Expected '%s', got '%s'", expected, tracker.ListTasks())
+	}
+}
diff --git a/assign3/assignment3.go b/assign3/assignment3.go
index 5c98130..922ca22 100644
--- a/assign3/assignment3.go
+++ b/assign3/assignment3.go
@@ -40,9 +40,9 @@ func NewTaskTracker() *TaskTracker {
 	}
 }
 
-// AddTask adds a new task to the tracker.
+// AddTask adds a new task to the tracker and returns the added Task.
 // It uses a pointer receiver (*TaskTracker) because it modifies the TaskTracker's state (its 'tasks' slice).
-func (tt *TaskTracker) AddTask(description string) {
+func (tt *TaskTracker) AddTask(description string) Task {
 	newID := tt.nextIDGen()
 	newTask := Task{
 		ID:          newID,
@@ -50,45 +50,42 @@ func (tt *TaskTracker) AddTask(description string) {
 		Completed:   false,
 	}
 	tt.tasks = append(tt.tasks, newTask)
-	fmt.Printf("Task Added: %d - %s\n", newTask.ID, newTask.Description)
+	return newTask
 }
 
 // ListTasks displays all pending tasks.
 // It uses a pointer receiver (*TaskTracker) because it operates on the TaskTracker's 'tasks' slice,
 // even though it doesn't modify it directly in this function (good practice for methods operating on collections).
-func (tt *TaskTracker) ListTasks() {
-	fmt.Println("\nPending Tasks:")
+func (tt *TaskTracker) ListTasks() string {
+	s := "Pending Tasks:\n"
 	foundPending := false
 	for _, task := range tt.tasks {
 		if !task.Completed {
-			fmt.Printf("%d: %s\n", task.ID, task.Description)
+			s += fmt.Sprintf("%d: %s\n", task.ID, task.Description)
 			foundPending = true
 		}
 	}
 	if !foundPending {
-		fmt.Println("No pending tasks.")
+		s += "No pending tasks."
 	}
+	return s
 }
 
 // CompleteTask marks a task as completed given its ID.
+// It returns a boolean indicating if the task was found and its completion status was changed,
+// and a string message describing the outcome.
 // It uses a pointer receiver (*TaskTracker) because it modifies the state of a Task within the tracker's slice.
-func (tt *TaskTracker) CompleteTask(id int) {
-	taskFound := false
+func (tt *TaskTracker) CompleteTask(id int) (bool, string) {
 	for i := range tt.tasks {
 		if tt.tasks[i].ID == id {
 			if tt.tasks[i].Completed {
-				fmt.Printf("Task %d is already completed.\n", id)
-			} else {
-				tt.tasks[i].Completed = true
-				fmt.Printf("Marking task %d as completed: %s\n", id, tt.tasks[i].Description)
+				return false, fmt.Sprintf("Task %d is already completed.", id)
 			}
-			taskFound = true
-			break
+			tt.tasks[i].Completed = true
+			return true, fmt.Sprintf("Marking task %d as completed: %s", id, tt.tasks[i].Description)
 		}
 	}
-	if !taskFound {
-		fmt.Printf("Task with ID %d not found.\n", id)
-	}
+	return false, fmt.Sprintf("Task with ID %d not found.", id)
 }
 
 // displayMenu prints the interactive menu options to the console.
@@ -129,9 +126,10 @@ func main() {
 				fmt.Println("Task description cannot be empty.")
 				continue
 			}
-			tracker.AddTask(description)
+			addedTask := tracker.AddTask(description)
+			fmt.Printf("Task Added: %d - %s\n", addedTask.ID, addedTask.Description)
 		case 2:
-			tracker.ListTasks()
+			fmt.Println(tracker.ListTasks())
 		case 3:
 			fmt.Print("Enter ID of task to mark as completed: ")
 			idStr := getUserInput()
@@ -140,7 +138,8 @@ func main() {
 				fmt.Println("Invalid ID. Please enter a valid number.")
 				continue
 			}
-			tracker.CompleteTask(id)
+			_, msg := tracker.CompleteTask(id)
+			fmt.Println(msg)
 		case 4:
 			fmt.Println("Exiting Task Tracker. Goodbye!")
 			return