artox/incubator-pagespeed-ngx
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Use check_stat_op in apache_debug_gzip_test.

Browse Source 
This prevents a break in ModPagespeedFetchWithGzip if prior tests run.
This commit is contained in:
Steve Hill
2016-09-02 18:49:38 -04:00
parent 36c62c91fe
commit 76ed6ef9bc
1 changed files with 10 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
pagespeed/apache/gzip_test.sh
+10 -5
View File
@@ -47,6 +47,13 @@ if [ "$SECONDARY_HOSTNAME" != "" ]; then
fi
start_test Efficacy of ModPagespeedFetchWithGzip
# TODO(sligocki): The serf_fetch_bytes_count should be available on
# this vhost's pagespeed_admin/statistics page. Why isn't it?
GLOBAL_STATISTICS_URL="$PRIMARY_SERVER/pagespeed_global_admin/statistics?PageSpeed=off"
STATS=$OUTDIR/gzip_efficacy_stats
$WGET_DUMP $GLOBAL_STATISTICS_URL > $STATS.1
# Note: The client request will not served with gzip because we do not
# have an Accept-Encoding header, we are testing that the backend fetch
# uses gzip.
@@ -54,10 +61,8 @@ EXAMPLE_BIG_CSS="$EXAMPLE_ROOT/styles/big.css.pagespeed.ce.01O-NppLwe.css"
echo $WGET -O /dev/null --save-headers "$EXAMPLE_BIG_CSS"
$WGET -O /dev/null --save-headers "$EXAMPLE_BIG_CSS" 2>&1 \
| head | grep "HTTP request sent, awaiting response... 200 OK"
# TODO(sligocki): The serf_fetch_bytes_count should be available on
# this vhost's pagespeed_admin/statistics page. Why isn't it?
STATISTICS_URL="$PRIMARY_SERVER/pagespeed_global_admin/statistics?PageSpeed=off"
bytes=$(scrape_stat serf_fetch_bytes_count)
check [ $bytes -gt 200 -a $bytes -lt 500 ]
$WGET_DUMP $GLOBAL_STATISTICS_URL > $STATS.2
check_stat_op $STATS.1 $STATS.2 serf_fetch_bytes_count 200 -gt
check_stat_op $STATS.1 $STATS.2 serf_fetch_bytes_count 500 -lt
check_failures_and_exit