Skip to content

Add module to warn about excessive amount of autoloaded options #124

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

Merged
merged 14 commits into from
Mar 18, 2022
Merged

Add module to warn about excessive amount of autoloaded options #124

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
3528dd8
adding autoloaded options check to site health
manuelRod Jan 19, 2022
b5049c0
adding tests
manuelRod Jan 20, 2022
eac9521
moving file
manuelRod Jan 20, 2022
d3f7d52
Merge branch 'trunk' into add/sitehealth-autoloaded-options
manuelRod Jan 20, 2022
74932f8
Adding resource link to actions
manuelRod Jan 24, 2022
11dd3f3
simplify code
manuelRod Feb 8, 2022
be7b17d
some small refactor, + adding better documentation to filters
manuelRod Feb 15, 2022
814bb4a
nitpick
manuelRod Feb 28, 2022
a1ebb71
nitpicks
manuelRod Mar 11, 2022
389a0ff
moving mock response class to dataproviders.
manuelRod Mar 14, 2022
7c6a9f7
updating some comments
manuelRod Mar 15, 2022
b6a1949
simplifying test using wp_generate_password instead of custom random …
manuelRod Mar 16, 2022
f45fc53
Merge branch 'release/1.0.0-beta.2' into add/sitehealth-autoloaded-op…
manuelRod Mar 18, 2022
725bbd4
marking as experimental
manuelRod Mar 18, 2022
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
Prev Previous commit
Next Next commit
simplifying test using wp_generate_password instead of custom random … 
…string generator
  • Loading branch information
@manuelRod
manuelRod committed Mar 16, 2022
commit b6a1949fa6a0c61422a03a309b14931d94e03b73
18 changes: 1 addition & 17 deletions tests/modules/site-health/audit-autoloaded-options/audit-autoloaded-options-test.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ public function test_perflab_aao_autoloaded_options_size() {
* @param int $bytes bytes to load in options.
*/
public static function set_autoloaded_option( $bytes = 800000 ) {
$heavy_option_string = self::random_string_generator( $bytes );
$heavy_option_string = wp_generate_password( $bytes );
add_option( self::AUTOLOADED_OPTION_KEY, $heavy_option_string );
}

Expand All @@ -75,22 +75,6 @@ public static function delete_autoloaded_option() {
delete_option( self::AUTOLOADED_OPTION_KEY );
}

/**
* Generate random string with certain $length.
*
* @param int $length Length ( in bytes ) of string to create.
* @return string
*/
protected static function random_string_generator( $length ) {
$seed = 'abcd123';
$length_seed = strlen( $seed );
$string = '';
for ( $x = 0; $x < $length; $x++ ) {
$string .= $seed[ rand( 0, $length_seed - 1 ) ];
}
return $string;
}

/**
* This is the information we are adding into site_status_tests hook.
*
Expand Down