mirror of
https://github.com/opensourcepos/opensourcepos.git
synced 2026-06-05 22:24:24 -04:00
Tests cover: - Valid email acceptance - Invalid email rejection with row-specific error - XSS payload sanitization in email field - Mixed valid/invalid email handling - Email with special characters sanitization Verifies fixes for customer email import vulnerability.
226 lines
8.3 KiB
PHP
226 lines
8.3 KiB
PHP
<?php
|
|
|
|
namespace Tests\Controllers;
|
|
|
|
use CodeIgniter\Test\CIUnitTestCase;
|
|
use CodeIgniter\Test\DatabaseTestTrait;
|
|
use CodeIgniter\Test\FeatureTestTrait;
|
|
use CodeIgniter\Config\Services;
|
|
use App\Models\Customer;
|
|
use App\Models\Employee;
|
|
|
|
class CustomersCsvImportTest extends CIUnitTestCase
|
|
{
|
|
use DatabaseTestTrait;
|
|
use FeatureTestTrait;
|
|
|
|
protected $migrate = true;
|
|
protected $migrateOnce = true;
|
|
protected $refresh = false;
|
|
protected $namespace = null;
|
|
|
|
protected Customer $customer;
|
|
protected Employee $employee;
|
|
|
|
protected function setUp(): void
|
|
{
|
|
parent::setUp();
|
|
|
|
$this->customer = model(Customer::class);
|
|
$this->employee = model(Employee::class);
|
|
|
|
helper('test');
|
|
}
|
|
|
|
protected function tearDown(): void
|
|
{
|
|
parent::tearDown();
|
|
}
|
|
|
|
protected function loginAsEmployee(): void
|
|
{
|
|
$session = Services::session();
|
|
$session->set('person_id', 1);
|
|
$session->set('menu_group', 'office');
|
|
}
|
|
|
|
protected function createCsvFile(array $rows): string
|
|
{
|
|
$tempFile = tempnam(sys_get_temp_dir(), 'csv_test_');
|
|
|
|
$handle = fopen($tempFile, 'w');
|
|
foreach ($rows as $row) {
|
|
fputcsv($handle, $row);
|
|
}
|
|
fclose($handle);
|
|
|
|
return $tempFile;
|
|
}
|
|
|
|
public function testValidEmailIsAccepted(): void
|
|
{
|
|
$this->loginAsEmployee();
|
|
|
|
$csvContent = [
|
|
['First Name', 'Last Name', 'Gender', 'Consent', 'Email', 'Phone', 'Address 1', 'Address 2', 'City', 'State', 'Zip', 'Country', 'Comments', 'Company', 'Account Number', 'Discount', 'Discount Type', 'Taxable'],
|
|
['John', 'Doe', '1', '1', 'john.doe@example.com', '555-1234', '123 Main St', '', 'Springfield', 'IL', '62701', 'US', '', '', '', '', '', '']
|
|
];
|
|
|
|
$tempFile = $this->createCsvFile($csvContent);
|
|
|
|
$_FILES['file_path'] = [
|
|
'name' => 'test.csv',
|
|
'type' => 'text/csv',
|
|
'tmp_name' => $tempFile,
|
|
'error' => UPLOAD_ERR_OK,
|
|
'size' => filesize($tempFile)
|
|
];
|
|
|
|
$result = $this->post('/customers/importCsvFile');
|
|
|
|
$result->assertOK();
|
|
$result->assertJSONExact(['success' => true, 'message' => 'Customers imported successfully']);
|
|
|
|
$importedCustomer = $this->customer->where('email', 'john.doe@example.com')->first();
|
|
$this->assertNotNull($importedCustomer);
|
|
|
|
unlink($tempFile);
|
|
}
|
|
|
|
public function testInvalidEmailIsRejected(): void
|
|
{
|
|
$this->loginAsEmployee();
|
|
|
|
$csvContent = [
|
|
['First Name', 'Last Name', 'Gender', 'Consent', 'Email', 'Phone', 'Address 1', 'Address 2', 'City', 'State', 'Zip', 'Country', 'Comments', 'Company', 'Account Number', 'Discount', 'Discount Type', 'Taxable'],
|
|
['John', 'Doe', '1', '1', 'not-an-email', '555-1234', '123 Main St', '', 'Springfield', 'IL', '62701', 'US', '', '', '', '', '', '']
|
|
];
|
|
|
|
$tempFile = $this->createCsvFile($csvContent);
|
|
|
|
$_FILES['file_path'] = [
|
|
'name' => 'test.csv',
|
|
'type' => 'text/csv',
|
|
'tmp_name' => $tempFile,
|
|
'error' => UPLOAD_ERR_OK,
|
|
'size' => filesize($tempFile)
|
|
];
|
|
|
|
$result = $this->post('/customers/importCsvFile');
|
|
|
|
$result->assertOK();
|
|
|
|
$resultBody = json_decode($result->getJSON(), true);
|
|
$this->assertFalse($resultBody['success'], 'Import should fail for invalid email');
|
|
$this->assertStringContainsString('Row 1', $resultBody['message'], 'Error message should reference failing row');
|
|
$this->assertStringContainsString('Invalid email format', $resultBody['message'], 'Error message should mention email validation');
|
|
|
|
$importedCustomer = $this->customer->where('email', 'not-an-email')->first();
|
|
$this->assertNull($importedCustomer, 'Customer with invalid email should not be imported');
|
|
|
|
unlink($tempFile);
|
|
}
|
|
|
|
public function testXssPayloadInEmailIsSanitized(): void
|
|
{
|
|
$this->loginAsEmployee();
|
|
|
|
$maliciousEmail = '<script>alert("xss")</script>@example.com';
|
|
|
|
$csvContent = [
|
|
['First Name', 'Last Name', 'Gender', 'Consent', 'Email', 'Phone', 'Address 1', 'Address 2', 'City', 'State', 'Zip', 'Country', 'Comments', 'Company', 'Account Number', 'Discount', 'Discount Type', 'Taxable'],
|
|
['John', 'Doe', '1', '1', $maliciousEmail, '555-1234', '123 Main St', '', 'Springfield', 'IL', '62701', 'US', '', '', '', '', '', '']
|
|
];
|
|
|
|
$tempFile = $this->createCsvFile($csvContent);
|
|
|
|
$_FILES['file_path'] = [
|
|
'name' => 'test.csv',
|
|
'type' => 'text/csv',
|
|
'tmp_name' => $tempFile,
|
|
'error' => UPLOAD_ERR_OK,
|
|
'size' => filesize($tempFile)
|
|
];
|
|
|
|
$result = $this->post('/customers/importCsvFile');
|
|
|
|
$result->assertOK();
|
|
|
|
$importedCustomer = $this->customer->where('email LIKE', '%example.com')->first();
|
|
|
|
$this->assertNotNull($importedCustomer, 'Customer should be imported after sanitization');
|
|
$this->assertStringNotContainsString('<script>', $importedCustomer->email, 'Script tags should be removed');
|
|
$this->assertStringNotContainsString('</script>', $importedCustomer->email, 'Script tags should be removed');
|
|
|
|
unlink($tempFile);
|
|
}
|
|
|
|
public function testMixedValidAndInvalidEmails(): void
|
|
{
|
|
$this->loginAsEmployee();
|
|
|
|
$csvContent = [
|
|
['First Name', 'Last Name', 'Gender', 'Consent', 'Email', 'Phone', 'Address 1', 'Address 2', 'City', 'State', 'Zip', 'Country', 'Comments', 'Company', 'Account Number', 'Discount', 'Discount Type', 'Taxable'],
|
|
['Valid', 'User', '1', '1', 'valid@example.com', '555-1111', '123 Main St', '', 'City1', 'ST', '12345', 'US', '', '', '', '', '', ''],
|
|
['Invalid', 'User', '1', '1', 'invalid-email', '555-2222', '456 Oak Ave', '', 'City2', 'ST', '23456', 'US', '', '', '', '', '', ''],
|
|
['Another', 'Valid', '1', '1', 'another@example.com', '555-3333', '789 Pine Rd', '', 'City3', 'ST', '34567', 'US', '', '', '', '', '', '']
|
|
];
|
|
|
|
$tempFile = $this->createCsvFile($csvContent);
|
|
|
|
$_FILES['file_path'] = [
|
|
'name' => 'test.csv',
|
|
'type' => 'text/csv',
|
|
'tmp_name' => $tempFile,
|
|
'error' => UPLOAD_ERR_OK,
|
|
'size' => filesize($tempFile)
|
|
];
|
|
|
|
$result = $this->post('/customers/importCsvFile');
|
|
|
|
$result->assertOK();
|
|
|
|
$validCustomer1 = $this->customer->where('email', 'valid@example.com')->first();
|
|
$this->assertNotNull($validCustomer1, 'Valid customer should be imported');
|
|
|
|
$validCustomer2 = $this->customer->where('email', 'another@example.com')->first();
|
|
$this->assertNotNull($validCustomer2, 'Another valid customer should be imported');
|
|
|
|
$invalidCustomer = $this->customer->where('email', 'invalid-email')->first();
|
|
$this->assertNull($invalidCustomer, 'Invalid email customer should not be imported');
|
|
|
|
unlink($tempFile);
|
|
}
|
|
|
|
public function testEmailWithSpecialCharactersIsSanitized(): void
|
|
{
|
|
$this->loginAsEmployee();
|
|
|
|
$emailWithSpecialChars = 'test"user@example.com';
|
|
$csvContent = [
|
|
['First Name', 'Last Name', 'Gender', 'Consent', 'Email', 'Phone', 'Address 1', 'Address 2', 'City', 'State', 'Zip', 'Country', 'Comments', 'Company', 'Account Number', 'Discount', 'Discount Type', 'Taxable'],
|
|
['Test', 'User', '1', '1', $emailWithSpecialChars, '555-1234', '123 Main St', '', 'Springfield', 'IL', '62701', 'US', '', '', '', '', '', '']
|
|
];
|
|
|
|
$tempFile = $this->createCsvFile($csvContent);
|
|
|
|
$_FILES['file_path'] = [
|
|
'name' => 'test.csv',
|
|
'type' => 'text/csv',
|
|
'tmp_name' => $tempFile,
|
|
'error' => UPLOAD_ERR_OK,
|
|
'size' => filesize($tempFile)
|
|
];
|
|
|
|
$result = $this->post('/customers/importCsvFile');
|
|
|
|
$result->assertOK();
|
|
|
|
$importedCustomer = $this->customer->where('email LIKE', '%example.com')->first();
|
|
|
|
$this->assertNotNull($importedCustomer, 'Sanitized email should be imported');
|
|
$this->assertStringNotContainsString('"', $importedCustomer->email, 'Quote characters should be sanitized');
|
|
|
|
unlink($tempFile);
|
|
}
|
|
} |