Skip to content

Fix unit tests

Fix unit tests #606

Triggered via push April 14, 2025 22:09
Status Success
Total duration 1m 34s
Artifacts 1

php.yml

on: push
Matrix: basic-tests
Security checks
29s
Security checks
Sanity checks
15s
Sanity checks
Quality control
25s
Quality control
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 1 warning
PossiblyInvalidOperand: lib/Auth/Ldap.php#L732
lib/Auth/Ldap.php:732:109: PossiblyInvalidOperand: Cannot concatenate with a array<array-key, string>|string (see https://psalm.dev/163)
PossiblyInvalidArgument: lib/Auth/Ldap.php#L730
lib/Auth/Ldap.php:730:30: PossiblyInvalidArgument: Argument 1 of SimpleSAML\Module\ldap\Auth\Ldap::bind expects string, but possibly different type array<array-key, string>|string provided (see https://psalm.dev/092)
RiskyTruthyFalsyComparison: lib/Auth/Ldap.php#L673
lib/Auth/Ldap.php:673:22: RiskyTruthyFalsyComparison: Operand of type int|null contains type int, which can be falsy and truthy. This can cause possibly unexpected behavior. Use strict comparison instead. (see https://psalm.dev/356)
TypeDoesNotContainType: lib/Auth/Ldap.php#L655
lib/Auth/Ldap.php:655:13: TypeDoesNotContainType: array<array-key, mixed> does not contain false (see https://psalm.dev/056)
PossiblyInvalidArrayOffset: lib/Auth/Ldap.php#L490
lib/Auth/Ldap.php:490:35: PossiblyInvalidArrayOffset: Cannot access value on variable $attribute using offset value of 'count', expecting int<0, max> (see https://psalm.dev/116)
PossiblyInvalidArgument: lib/Auth/Ldap.php#L473
lib/Auth/Ldap.php:473:50: PossiblyInvalidArgument: Argument 2 of ldap_get_entries expects resource, but possibly different type array<array-key, resource>|resource provided (see https://psalm.dev/092)
PossiblyInvalidArgument: lib/Auth/Ldap.php#L464
lib/Auth/Ldap.php:464:52: PossiblyInvalidArgument: Argument 2 of ldap_count_entries expects resource, but possibly different type array<array-key, resource>|resource provided (see https://psalm.dev/092)
PossiblyInvalidArgument: lib/Auth/Ldap.php#L452
lib/Auth/Ldap.php:452:71: PossiblyInvalidArgument: Argument 2 of ldap_count_entries expects resource, but possibly different type array<array-key, resource>|resource provided (see https://psalm.dev/092)
PossiblyInvalidArgument: lib/Auth/Ldap.php#L300
lib/Auth/Ldap.php:300:49: PossiblyInvalidArgument: Argument 2 of ldap_first_entry expects resource, but possibly different type array<array-key, resource>|resource provided (see https://psalm.dev/092)
PossiblyInvalidArgument: lib/Auth/Ldap.php#L279
lib/Auth/Ldap.php:279:51: PossiblyInvalidArgument: Argument 2 of ldap_count_entries expects resource, but possibly different type array<array-key, resource>|resource provided (see https://psalm.dev/092)
Security checks
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

Artifacts

Produced during runtime
Name Size Digest
build-data Expired
268 KB
sha256:b803c421dff0d8ed52a5ecf45be90e17b8f49f64aee8ef3f6ea102f2cc42e21a