Skip to content

test: add comprehensive unit test coverage across all major components #153

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Copilot wants to merge 10 commits into
base: main
Choose a base branch
from
Open

test: add comprehensive unit test coverage across all major components #153

Show file tree
Hide file tree
Changes from 9 commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
423d7c2
plan: add comprehensive unit test coverage
Copilot May 16, 2026
26dd37c
test: add comprehensive unit test coverage across all major components
Copilot May 16, 2026
aa12459
fix: regenerate pnpm-lock.yaml to match package.json (rollup ^4.60.2)
Copilot May 17, 2026
32b3bd3
Potential fix for pull request finding
MaximillianGroup May 17, 2026
9076a35
test: gitignore phpunit result cache; assert exact db_table_missing e…
Copilot May 17, 2026
1d26e42
Potential fix for pull request finding
MaximillianGroup May 17, 2026
99c8502
fix(getUserOS): reorder UA map so Android/iOS match before Linux/Mac;…
Copilot May 17, 2026
b769709
Potential fix for pull request finding
MaximillianGroup May 18, 2026
4af745d
Potential fix for pull request finding
MaximillianGroup May 18, 2026
25b9212
test: fix Android/iOS UA assertions to match corrected getUserOS() or…
Copilot May 18, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .gitignore
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ src/node_modules/
# Ignore cache files from code quality tools
.php-cs-fixer.cache
.phpstan-cache/
.phpunit.result.cache


# -----------------------------------------------------------------
Expand Down
1 change: 0 additions & 1 deletion .phpunit.result.cache
View file
Open in desktop

This file was deleted.

122 changes: 62 additions & 60 deletions pnpm-lock.yaml
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

9 changes: 6 additions & 3 deletions src/StarUserEnv.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@
private static ?array $snapshot_cache = null;

// -------------------------------------------------------------------------
// I. IDENTITY HELPERS (MODERN MODEL)

Check failure on line 76 in src/StarUserEnv.php

View workflow job for this annotation

GitHub Actions / PHP Code Quality 

Property Starisian\SparxstarUEC\StarUserEnv::$snapshot_cache type has no value type specified in iterable type array.
// -------------------------------------------------------------------------

/**
Expand Down Expand Up @@ -746,15 +746,18 @@
public static function getUserOS(): string
{
$user_agent = strtolower(self::getUserAgent());
// More-specific patterns must appear before any pattern that is a
// substring of them (e.g. 'android' before 'linux', 'iphone' before
// 'mac os x', 'windows phone' before 'windows').
$map = [
'windows phone' => 'Windows Phone',
'ipad|ipod|iphone' => 'iOS',
'android' => 'Android',
'windows' => 'Windows',
'macintosh|mac os x|macos' => 'Mac',
'linux' => 'Linux',
'ipad|ipod|iphone' => 'iOS',
'android' => 'Android',
'blackberry' => 'BlackBerry',
'webos' => 'webOS',
'windows phone' => 'Windows Phone',
];

foreach ($map as $needle => $label) {
Expand Down Expand Up @@ -784,7 +787,7 @@
if (preg_match('/Chrome/i', $user_agent)) {
return 'Chrome';
}

Check failure on line 790 in src/StarUserEnv.php

View workflow job for this annotation

GitHub Actions / PHP Code Quality 

Method Starisian\SparxstarUEC\StarUserEnv::getEnvironmentSnapshot() return type has no value type specified in iterable type array.
if (preg_match('/Safari/i', $user_agent)) {
return 'Safari';
}
Expand Down Expand Up @@ -860,7 +863,7 @@
/**
* Retrieve the current HTTP method (GET, POST, etc.).
*/
public static function getRequestMethod(): string

Check failure on line 866 in src/StarUserEnv.php

View workflow job for this annotation

GitHub Actions / PHP Code Quality 

Constant SPX_ENV_CHECK_TEXT_DOMAIN not found.
{
return sanitize_text_field(self::get_server_value('REQUEST_METHOD'));
}
Expand All @@ -883,7 +886,7 @@
* Get the current WordPress environment type.
*/
public static function getWpEnvironmentType(): string
{

Check failure on line 889 in src/StarUserEnv.php

View workflow job for this annotation

GitHub Actions / PHP Code Quality 

Constant SPX_ENV_CHECK_TEXT_DOMAIN not found.
if (function_exists('wp_get_environment_type')) {
return wp_get_environment_type();
}
Expand Down
Loading
Loading