From 7a6916c3ff03df1d05474081466451481e716227 Mon Sep 17 00:00:00 2001
From: amazing-username <kundeng94@gmail.com>
Date: Wed, 5 Jun 2019 20:51:56 -0400
Subject: [PATCH] Added CLI functionality for uploading a song #7. Next thing
 that is up adding CLI functionality for downloading songs via id

---
 CMakeLists.txt                 |  4 +++
 src/Managers/CommitManager.cpp | 33 ++++++++++++++-----------
 src/Managers/CommitManager.h   |  1 +
 src/Managers/TokenManager.cpp  | 12 ++++++++-
 src/Managers/TokenManager.h    |  3 +++
 src/Managers/UserManager.cpp   |  1 -
 src/Models/API.h               | 16 ++++++++++++
 src/Parsers/APIParser.cpp      | 45 ++++++++++++++++++++++++++++++++++
 src/Parsers/APIParser.h        | 23 +++++++++++++++++
 src/Syncers/Download.cpp       |  6 +++++
 src/Syncers/Download.h         |  8 +++++-
 src/Syncers/SyncerBase.h       | 15 ++++++++++++
 src/Syncers/Upload.cpp         | 26 +++++++++++++++++---
 src/Syncers/Upload.h           | 19 ++++++++------
 14 files changed, 185 insertions(+), 27 deletions(-)
 create mode 100644 src/Models/API.h
 create mode 100644 src/Parsers/APIParser.cpp
 create mode 100644 src/Parsers/APIParser.h
 create mode 100644 src/Syncers/SyncerBase.h

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 03175c7..9035c98 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -47,6 +47,7 @@ set(SOURCES
 	src/Managers/FileManager.cpp
 	src/Managers/TokenManager.cpp
 	src/Managers/UserManager.cpp
+	src/Parsers/APIParser.cpp
 	src/Syncers/Download.cpp
 	src/Syncers/Upload.cpp
 	src/Utilities/Conversions.cpp
@@ -57,13 +58,16 @@ set(HEADERS
 	src/Managers/FileManager.h
 	src/Managers/TokenManager.h
 	src/Managers/UserManager.h
+	src/Models/API.h
 	src/Models/Flags.h
 	src/Models/IcarusAction.h
 	src/Models/Song.h
 	src/Models/Token.h
 	src/Models/UploadForm.h
 	src/Models/User.h
+	src/Parsers/APIParser.h
 	src/Syncers/Download.h
+	src/Syncers/SyncerBase.h
 	src/Syncers/Upload.h
 	src/Utilities/Conversions.h
 )
diff --git a/src/Managers/CommitManager.cpp b/src/Managers/CommitManager.cpp
index 73cf4f8..6044ae2 100644
--- a/src/Managers/CommitManager.cpp
+++ b/src/Managers/CommitManager.cpp
@@ -2,6 +2,9 @@
 
 #include<iostream>
 
+#include"Models/API.h"
+#include"Models/Song.h"
+#include"Parsers/APIParser.h"
 #include"Syncers/Upload.h"
 
 #include"TokenManager.h"
@@ -14,6 +17,9 @@ using std::string;
 
 using Managers::TokenManager;
 using Managers::UserManager;
+using Models::API;
+using Models::Song;
+using Parsers::APIParser;
 using Models::IcarusAction;
 using Syncers::Upload;
 
@@ -38,6 +44,7 @@ namespace Managers
 			case deleteAct:
 				break;
 			case downloadAct:
+				downloadSong();
 				break;
 			case retrieveAct:
 				break;
@@ -55,16 +62,23 @@ namespace Managers
 			{"upload", uploadAct}
 		};
 	}
+	void CommitManager::downloadSong()
+	{
+		// TODO: Implement this functionality
+	}
 	void CommitManager::uploadSong()
 	{
 		cout<<"Uploading song..."<<endl;
 		UserManager usrMgr{icaAction};
 		auto user = usrMgr.retrieveUser();
 
-		TokenManager tk{user};
+		APIParser apiPrs{icaAction};
+		auto api = apiPrs.retrieveAPI();
+
+		TokenManager tk{user, api};
 		auto token = tk.requestToken();
 
-		string songPath{};
+		Song song{};
 
 		for (auto arg : icaAction.flags)
 		{
@@ -73,21 +87,12 @@ namespace Managers
 
 			if (flag.compare("-s") == 0)
 			{
-				songPath.assign(arg.value);
+				song.songPath.assign(arg.value);
 			}
 		}
 
-		Upload upld{};
-		upld.uploadSong(token, songPath);
-
-		// TODO: Implement functionality for uploading
-		//
-		// Need to the following:
-		// 1. Parse the login credentials from the Flag model x
-		// 2. Retrieve a token -- implement token management x
-		// 3. Parse song path from the Flag model x Make a parser class
-		// 4. Parse the HTTP API endpoint Make parser class
-		// 5. Upload the song x
+		Upload upld{api};
+		upld.uploadSong(token, song);
 	}
 	#pragma Functions
 }
diff --git a/src/Managers/CommitManager.h b/src/Managers/CommitManager.h
index 35e4dd8..2fede64 100644
--- a/src/Managers/CommitManager.h
+++ b/src/Managers/CommitManager.h
@@ -16,6 +16,7 @@ namespace Managers
 		void commitAction();
 	private:
 		void initializeMapActions();
+		void downloadSong();
 		void uploadSong();
 
 		enum ActionValues
diff --git a/src/Managers/TokenManager.cpp b/src/Managers/TokenManager.cpp
index 568ff56..1136b86 100644
--- a/src/Managers/TokenManager.cpp
+++ b/src/Managers/TokenManager.cpp
@@ -11,6 +11,7 @@ using std::endl;
 using json = nlohmann::json;
 
 using Managers::TokenManager;
+using Models::API;
 using Models::Token;
 using Models::User;
 
@@ -21,6 +22,13 @@ namespace Managers
 	{
 		this->user = user;
 	}
+	TokenManager::TokenManager(const User user, API api)
+	{
+		this->user = user;
+		this->api = api;
+		this->api.endpoint = "api/" + api.version 
+			+ "/login";
+	}
 	#pragma Constructors
 
 
@@ -33,7 +41,9 @@ namespace Managers
 		usrObj["username"] = user.username;
 		usrObj["password"] = user.password;
 
-		auto r = cpr::Post(cpr::Url{""},
+		cout<<"Sending request for token"<<endl;
+		auto url = api.url + api.endpoint;
+		auto r = cpr::Post(cpr::Url{url},
 				cpr::Body{usrObj.dump()},
 		   cpr::Header{{"Content-Type", "application/json"}});
 
diff --git a/src/Managers/TokenManager.h b/src/Managers/TokenManager.h
index b064881..bdd8c0f 100644
--- a/src/Managers/TokenManager.h
+++ b/src/Managers/TokenManager.h
@@ -1,6 +1,7 @@
 #ifndef TOKENMANAGER_H_
 #define TOKENMANAGER_H_
 
+#include"Models/API.h"
 #include"Models/Token.h"
 #include"Models/User.h"
 
@@ -10,9 +11,11 @@ namespace Managers
 	{
 	public:
 		TokenManager(const Models::User);
+		TokenManager(const Models::User, Models::API);
 
 		Models::Token requestToken();
 	private:
+		Models::API api;
 		Models::User user;
 	};
 }
diff --git a/src/Managers/UserManager.cpp b/src/Managers/UserManager.cpp
index df87fbe..198d084 100644
--- a/src/Managers/UserManager.cpp
+++ b/src/Managers/UserManager.cpp
@@ -16,7 +16,6 @@ namespace Managers
 	UserManager::UserManager(User user)
 	{
 		this->user = user;
-		this->user = User{};
 	}
 	UserManager::UserManager(const IcarusAction icaAct)
 	{
diff --git a/src/Models/API.h b/src/Models/API.h
new file mode 100644
index 0000000..74c53f3
--- /dev/null
+++ b/src/Models/API.h
@@ -0,0 +1,16 @@
+#ifndef API_H_
+#define API_H_
+
+#include<string>
+
+namespace Models
+{
+	struct API
+	{
+		std::string url;
+		std::string endpoint;
+		std::string version;
+	};
+}
+
+#endif
diff --git a/src/Parsers/APIParser.cpp b/src/Parsers/APIParser.cpp
new file mode 100644
index 0000000..1e4e67a
--- /dev/null
+++ b/src/Parsers/APIParser.cpp
@@ -0,0 +1,45 @@
+#include"APIParser.h"
+
+using Models::API;
+using Models::IcarusAction;
+
+namespace Parsers
+{
+	#pragma
+	APIParser::APIParser(IcarusAction icaAct)
+	{
+		this->icaAct = icaAct;
+		api = API{};
+		parseAPI();
+	}
+	#pragma endregion
+
+
+	#pragma
+	API APIParser::retrieveAPI() const
+	{
+		return api;
+	}
+
+	void APIParser::parseAPI()
+	{
+		auto flags = icaAct.flags;
+
+		for (auto flag :  flags)
+		{
+			auto arg = flag.flag;
+			auto value = flag.value;
+
+			if (arg.compare("-h") == 0)
+			{
+				api.url = value;
+			}
+		}
+
+		// TODO: For now I will hard code
+		// the api version since I am only
+		// on version 1
+		api.version = "v1";
+	}
+	#pragma functions
+}
diff --git a/src/Parsers/APIParser.h b/src/Parsers/APIParser.h
new file mode 100644
index 0000000..2cb1c34
--- /dev/null
+++ b/src/Parsers/APIParser.h
@@ -0,0 +1,23 @@
+#ifndef APIPARSER_H_
+#define APIPARSER_H_
+
+#include"Models/API.h"
+#include"Models/IcarusAction.h"
+
+namespace Parsers
+{
+	class APIParser
+	{
+	public:
+		APIParser(Models::IcarusAction);
+
+		Models::API retrieveAPI() const;
+	private:
+		void parseAPI();
+
+		Models::API api;
+		Models::IcarusAction icaAct;
+	};
+}
+
+#endif
diff --git a/src/Syncers/Download.cpp b/src/Syncers/Download.cpp
index bbbd3d8..2b367f4 100644
--- a/src/Syncers/Download.cpp
+++ b/src/Syncers/Download.cpp
@@ -10,9 +10,15 @@ using std::endl;
 using std::ofstream;
 using std::string;
 
+using Models::API;
+
 namespace Syncers
 {
 	Download::Download() { }
+	Download::Download(API api)
+	{
+		this->api = api;
+	}
 	Download::Download(string filePath)
 	{
 		downloadFilePath = filePath;
diff --git a/src/Syncers/Download.h b/src/Syncers/Download.h
index 1506136..1a097ca 100644
--- a/src/Syncers/Download.h
+++ b/src/Syncers/Download.h
@@ -4,12 +4,18 @@
 #include<iostream>
 #include<string>
 
+#include"Models/API.h"
+#include"Models/Song.h"
+
+#include"SyncerBase.h"
+
 namespace Syncers
 {
-	class Download
+	class Download : SyncerBase
 	{
 	public:
 		Download();
+		Download(Models::API);
 		Download(std::string);
 
 		void downloadSong(int);
diff --git a/src/Syncers/SyncerBase.h b/src/Syncers/SyncerBase.h
new file mode 100644
index 0000000..8f03fcf
--- /dev/null
+++ b/src/Syncers/SyncerBase.h
@@ -0,0 +1,15 @@
+#ifndef SYNCERBASE_H_
+#define SYNCERBASE_H_
+
+#include"Models/API.h"
+
+namespace Syncers
+{
+	class SyncerBase
+	{
+	protected:
+		Models::API api;
+	};
+}
+
+#endif
diff --git a/src/Syncers/Upload.cpp b/src/Syncers/Upload.cpp
index f7e41d3..476a4a4 100644
--- a/src/Syncers/Upload.cpp
+++ b/src/Syncers/Upload.cpp
@@ -15,6 +15,8 @@ using std::string;
 using json = nlohmann::json;
 
 using Managers::FileManager;
+using Models::API;
+using Models::Song;
 using Models::UploadForm;
 
 using namespace cpr;
@@ -27,6 +29,11 @@ namespace Syncers
 		this->songPath = filePath;
 		this->fMgr = FileManager(songPath);
 	}
+	Upload::Upload(API api)
+	{
+		this->api = api;
+		this->api.endpoint = "song/data";
+	}
 	Upload::Upload(UploadForm formData)
 	{
 		this->url = formData.url;
@@ -52,15 +59,18 @@ namespace Syncers
 		cout<<"Finished"<<endl;
 
 	}
-	void Upload::uploadSong(const Models::Token token, const std::string songPath)
+	void Upload::uploadSong(const Models::Token token, Song song)
 	{
 		try
 		{
+			auto url = retrieveUrl();
+
+			cout<<"url "<<url<<endl;
 			string auth{token.tokenType};
 			auth.append(" " + token.accessToken);
-			auto r = cpr::Post(cpr::Url{""},
+			auto r = cpr::Post(cpr::Url{url},
                    		cpr::Multipart{{"key", "small value"},
-                                {"file", cpr::File{songPath}}},
+                                {"file", cpr::File{song.songPath}}},
 		   		cpr::Header{{"authorization", auth}}
 				);
 
@@ -74,6 +84,14 @@ namespace Syncers
 		cout<<"Finished"<<endl;
 	}
 
+	string Upload::retrieveUrl()
+	{
+		string url = api.url + "api/" + api.version + "/" +
+			api.endpoint;
+			
+		return url;
+	}
+	#pragma
 	void Upload::configureSongDemo()
 	{
 		int id = 0;
@@ -135,4 +153,6 @@ namespace Syncers
 
 		return jObj;
 	}
+	#pragma Testing
+	#pragma Functions
 }
diff --git a/src/Syncers/Upload.h b/src/Syncers/Upload.h
index cc9db7e..4a31b8e 100644
--- a/src/Syncers/Upload.h
+++ b/src/Syncers/Upload.h
@@ -6,6 +6,7 @@
 #include<nlohmann/json.hpp>
 
 #include"Managers/FileManager.h"
+#include"Models/API.h"
 #include"Models/Song.h"
 #include"Models/Token.h"
 #include"Models/UploadForm.h"
@@ -18,24 +19,28 @@ namespace Syncers
 		public:
 			Upload();
 			Upload(std::string);
+			Upload(Models::API);
 			Upload(Models::UploadForm);
 
 			void uploadSong();
-			void uploadSong(const Models::Token, const std::string);
+			void uploadSong(const Models::Token, Models::Song);
 		private:
 			Managers::FileManager fMgr;
+			Models::API api;
 			Models::Song song;
-			std::string apiUrl{"http://192.168.1.3"};
-			std::string apiEndPoint{"/api/song/data"};
-			std::string songPath;
-			std::string url;
-			int port{9349};
+			std::string apiUrl{""}; // Not being used
+			std::string apiEndPoint{""}; // Not being used
+			std::string songPath; // Not being used
+			std::string url; // Not being used
+			int port{9349}; // Not being used
+
+			std::string retrieveUrl();
 
 			void configureSongDemo();
 			void printSongDetails();
 			void printJsonData(nlohmann::json);
 
-			nlohmann::json serializeObject();
+			nlohmann::json serializeObject(); // Not being used
 	};
 }