joelthomastrenser/Training-Team2-Zenvy-Jan26
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Refactor authentication: replace tuple with DTO, namespace config

Browse Source
This commit is contained in:
Joel Thomas
2026-04-07 00:55:44 +05:30
parent fb35ee6098
commit e7332af1e7
7 changed files with 14 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Trenser.Zenvy/Trenser.Zenvy/controllers/ZenvyController.cpp
+1 -1
View File
@@ -1,7 +1,7 @@
#include "ZenvyController.h"
//Authentication
AuthenticationContext ZenvyController::login(const std::string& email, const std::string& password)
AuthenticationDTO ZenvyController::login(const std::string& email, const std::string& password)
{
return m_authenticationManagementService->login(email, password);
}
Trenser.Zenvy/Trenser.Zenvy/controllers/ZenvyController.h
+1 -3
View File
@@ -13,8 +13,6 @@
#include "TicketManagementService.h"
#include "Enums.h"
using AuthenticationContext = std::tuple<Enums::LoginStatus, Enums::EmployeeType, Enums::EmployeeDesignation>;
class ZenvyController
{
private:
@@ -42,7 +40,7 @@ public:
m_ticketManagementService(std::make_shared<TicketManagementService>()) {};
//Authentication
AuthenticationContext login(const std::string& email, const std::string& password);
AuthenticationDTO login(const std::string& email, const std::string& password);
void logout();
void changePassword(const std::string&);
};
Trenser.Zenvy/Trenser.Zenvy/services/ApplicationConfig.h
+3
View File
@@ -1,6 +1,9 @@
#pragma once
namespace Config
{
namespace Authentication
{
constexpr const char* DEFAULT_PASSWORD = "password";
}
}
Trenser.Zenvy/Trenser.Zenvy/services/AuthenticationManagementService.cpp
+2 -2
View File
@@ -2,7 +2,7 @@
#include "AuthenticationManagementService.h"
#include "ApplicationConfig.h"
std::tuple<Enums::LoginStatus, Enums::EmployeeType, Enums::EmployeeDesignation> AuthenticationManagementService::login(const std::string& email, const std::string& password)
AuthenticationDTO AuthenticationManagementService::login(const std::string& email, const std::string& password)
{
employeeMap& employees = m_dataStore.getEmployees();
Enums::LoginStatus loginStatus = Enums::LoginStatus::USER_NOT_FOUND;
@@ -14,7 +14,7 @@ std::tuple<Enums::LoginStatus, Enums::EmployeeType, Enums::EmployeeDesignation>
{
if (employee.second->getEmployeePassword() == password)
{
if (password == Config::DEFAULT_PASSWORD)
if (password == Config::Authentication::DEFAULT_PASSWORD)
{
loginStatus = Enums::LoginStatus::FIRST_LOGIN;
}
Trenser.Zenvy/Trenser.Zenvy/services/AuthenticationManagementService.h
+4 -1
View File
@@ -1,18 +1,21 @@
#pragma once
#include <string>
#include <map>
#include <tuple>
#include <utility>
#include <stdexcept>
#include "DataStore.h"
#include "Enums.h"
using AuthenticationDTO = std::tuple<Enums::LoginStatus, Enums::EmployeeType, Enums::EmployeeDesignation>;
class AuthenticationManagementService
{
private:
DataStore& m_dataStore;
public:
AuthenticationManagementService() : m_dataStore(DataStore::getInstance()) {};
std::tuple<Enums::LoginStatus, Enums::EmployeeType, Enums::EmployeeDesignation> login(const std::string& username, const std::string& password);
AuthenticationDTO login(const std::string& username, const std::string& password);
void logout();
void changePassword(const std::string&);
};
Trenser.Zenvy/Trenser.Zenvy/utilities/Validator.cpp
+1 -1
View File
@@ -33,7 +33,7 @@ bool util::isEmailValid(const std::string& email) {
bool util::isPasswordValid(const std::string& password)
{
if (password == Config::DEFAULT_PASSWORD)
if (password == Config::Authentication::DEFAULT_PASSWORD)
{
return false;
}
Trenser.Zenvy/Trenser.Zenvy/views/UserInterface.cpp
+1 -1
View File
@@ -65,7 +65,7 @@ void UserInterface::login()
util::read(email);
std::cout << "Enter password: ";
util::read(password);
AuthenticationContext authenticationContext = m_controller->login(email, password);
AuthenticationDTO authenticationContext = m_controller->login(email, password);
Enums::LoginStatus loginStatus = std::get<0>(authenticationContext);
Enums::EmployeeType employeeType = std::get<1>(authenticationContext);
Enums::EmployeeDesignation employeeDesignation = std::get<2>(authenticationContext);