X-Git-Url: https://arthur.barton.de/cgi-bin/gitweb.cgi?a=blobdiff_plain;f=t%2Ftest-prune-older;h=c460e821c677adbe7ac5c15ac5adb828a765f715;hb=45136f27bb239a48d696e21d0754cec87e8b05ef;hp=11aa861b580a5b5f6a29b013a6a1fbac54de400e;hpb=e291bccf8ea6e14902460eb9701a3e15108e4958;p=bup.git diff --git a/t/test-prune-older b/t/test-prune-older index 11aa861..c460e82 100755 --- a/t/test-prune-older +++ b/t/test-prune-older @@ -5,16 +5,15 @@ exec "$bup_python" "$0" ${1+"$@"} """ # end of bup preamble -from __future__ import print_function +from __future__ import absolute_import, print_function from collections import defaultdict from difflib import unified_diff from itertools import chain, dropwhile, groupby, takewhile from os import environ, chdir from os.path import abspath, dirname -from pipes import quote from random import choice, randint from shutil import copytree, rmtree -from subprocess import PIPE, Popen, check_call +from subprocess import PIPE from sys import stderr from time import localtime, strftime, time import os, random, sys @@ -24,51 +23,26 @@ sys.path[:0] = [abspath(script_home + '/../lib'), abspath(script_home + '/..')] top = os.getcwd() bup_cmd = top + '/bup' -from buptest import test_tempdir +from buptest import ex, exo, test_tempdir from wvtest import wvfail, wvpass, wvpasseq, wvpassne, wvstart +from bup import compat from bup.helpers import partition, period_as_secs, readpipe -def logcmd(cmd): - if isinstance(cmd, basestring): - print(cmd, file=stderr) - else: - print(' '.join(map(quote, cmd)), file=stderr) - -def exc(cmd, shell=False): - logcmd(cmd) - check_call(cmd, shell=shell) - -def exo(cmd, stdin=None, stdout=True, stderr=False, shell=False, check=True): - logcmd(cmd) - p = Popen(cmd, - stdin=None, - stdout=(PIPE if stdout else None), - stderr=PIPE, - shell=shell) - out, err = p.communicate() - if check and p.returncode != 0: - raise Exception('subprocess %r failed with status %d, stderr: %r' - % (' '.join(argv), p.returncode, err)) - return out, err, p - -def bup(*args): - return exo((bup_cmd,) + args)[0] - -def bupc(*args): - return exc((bup_cmd,) + args) - def create_older_random_saves(n, start_utc, end_utc): with open('foo', 'w') as f: pass - exc(['git', 'add', 'foo']) - utcs = sorted(randint(start_utc, end_utc) for x in xrange(n)) + ex(['git', 'add', 'foo']) + utcs = set() + while len(utcs) != n: + utcs.add(randint(start_utc, end_utc)) + utcs = sorted(utcs) for utc in utcs: with open('foo', 'w') as f: f.write(str(utc) + '\n') - exc(['git', 'commit', '--date', str(utc), '-qam', str(utc)]) - exc(['git', 'gc', '--aggressive']) + ex(['git', 'commit', '--date', str(utc), '-qam', str(utc)]) + ex(['git', 'gc', '--aggressive']) return utcs # There is corresponding code in bup for some of this, but the @@ -90,7 +64,7 @@ def expected_retentions(utcs, utc_start, spec): return utcs utcs = sorted(utcs, reverse=True) period_start = dict(spec) - for kind, duration in period_start.iteritems(): + for kind, duration in compat.items(period_start): period_start[kind] = utc_start - period_as_secs(duration) period_start = defaultdict(lambda: float('inf'), period_start) @@ -98,17 +72,17 @@ def expected_retentions(utcs, utc_start, spec): utcs = list(dropwhile(lambda x: x >= period_start['all'], utcs)) matches = takewhile(lambda x: x >= period_start['dailies'], utcs) - dailies = [min(day_utcs) for yday, day_utcs + dailies = [max(day_utcs) for yday, day_utcs in groupby(matches, lambda x: localtime(x).tm_yday)] utcs = list(dropwhile(lambda x: x >= period_start['dailies'], utcs)) matches = takewhile(lambda x: x >= period_start['monthlies'], utcs) - monthlies = [min(month_utcs) for month, month_utcs + monthlies = [max(month_utcs) for month, month_utcs in groupby(matches, lambda x: localtime(x).tm_mon)] utcs = dropwhile(lambda x: x >= period_start['monthlies'], utcs) matches = takewhile(lambda x: x >= period_start['yearlies'], utcs) - yearlies = [min(year_utcs) for year, year_utcs + yearlies = [max(year_utcs) for year, year_utcs in groupby(matches, lambda x: localtime(x).tm_year)] return chain(all, dailies, monthlies, yearlies) @@ -149,7 +123,7 @@ def result_diffline(x): def check_prune_result(expected): actual = sorted([int(x) for x in exo(['git', 'log', - '--pretty=format:%at'])[0].splitlines()]) + '--pretty=format:%at']).out.splitlines()]) if expected != actual: for x in expected: print('ex:', x, strftime('%Y-%m-%d-%H%M%S', localtime(x)), @@ -161,6 +135,11 @@ def check_prune_result(expected): wvpass(expected == actual) +environ['GIT_AUTHOR_NAME'] = 'bup test' +environ['GIT_COMMITTER_NAME'] = 'bup test' +environ['GIT_AUTHOR_EMAIL'] = 'bup@a425bc70a02811e49bdf73ee56450e6f' +environ['GIT_COMMITTER_EMAIL'] = 'bup@a425bc70a02811e49bdf73ee56450e6f' + seed = int(environ.get('BUP_TEST_SEED', time())) random.seed(seed) print('random seed:', seed, file=stderr) @@ -175,25 +154,26 @@ with test_tempdir('prune-older-') as tmpdir: now = int(time()) three_years_ago = now - (60 * 60 * 24 * 366 * 3) chdir(tmpdir) - exc(['git', 'init', 'work']) + ex(['git', 'init', 'work']) + ex(['git', 'config', 'gc.autoDetach', 'false']) wvstart('generating ' + str(save_population) + ' random saves') chdir(tmpdir + '/work') save_utcs = create_older_random_saves(save_population, three_years_ago, now) chdir(tmpdir) - test_set_hash = exo(['git', 'show-ref', '-s', 'master'])[0].rstrip() - ls_saves = bup('ls', 'master').splitlines() + test_set_hash = exo(['git', 'show-ref', '-s', 'master']).out.rstrip() + ls_saves = exo((bup_cmd, 'ls', 'master')).out.splitlines() wvpasseq(save_population + 1, len(ls_saves)) wvstart('ensure everything kept, if no keep arguments') - exc(['git', 'reset', '--hard', test_set_hash]) - _, errmsg, proc = exo((bup_cmd, - 'prune-older', '-v', '--unsafe', '--no-gc', - '--wrt', str(now)) \ - + ('master',), - stdout=False, stderr=True, check=False) - wvpassne(proc.returncode, 0) - wvpass('at least one keep argument is required' in errmsg) + ex(['git', 'reset', '--hard', test_set_hash]) + proc = ex((bup_cmd, + 'prune-older', '-v', '--unsafe', '--no-gc', + '--wrt', str(now)) \ + + ('master',), + stdout=None, stderr=PIPE, check=False) + wvpassne(proc.rc, 0) + wvpass('at least one keep argument is required' in proc.err) check_prune_result(save_utcs) @@ -204,19 +184,19 @@ with test_tempdir('prune-older-') as tmpdir: # some outside the save range. three_years_ago - period_scale['m'], now): - exc(['git', 'reset', '--hard', test_set_hash]) + ex(['git', 'reset', '--hard', test_set_hash]) expected = sorted(expected_retentions(save_utcs, now, spec)) - exc((bup_cmd, - 'prune-older', '-v', '--unsafe', '--no-gc', '--wrt', str(now)) \ - + period_spec_to_period_args(spec) \ - + ('master',)) + ex((bup_cmd, + 'prune-older', '-v', '--unsafe', '--no-gc', '--wrt', str(now)) \ + + period_spec_to_period_args(spec) \ + + ('master',)) check_prune_result(expected) # More expensive because we have to recreate the repo each time wvstart('running %d generative gc tests on %d saves' % (prune_gc_cycles, save_population)) - exc(['git', 'reset', '--hard', test_set_hash]) + ex(['git', 'reset', '--hard', test_set_hash]) copytree('work/.git', 'clean-test-repo', symlinks=True) for spec in unique_period_specs(prune_gc_cycles, # Make it more likely we'll have @@ -226,8 +206,8 @@ with test_tempdir('prune-older-') as tmpdir: rmtree('work/.git') copytree('clean-test-repo', 'work/.git') expected = sorted(expected_retentions(save_utcs, now, spec)) - exc((bup_cmd, - 'prune-older', '-v', '--unsafe', '--wrt', str(now)) \ - + period_spec_to_period_args(spec) \ - + ('master',)) + ex((bup_cmd, + 'prune-older', '-v', '--unsafe', '--wrt', str(now)) \ + + period_spec_to_period_args(spec) \ + + ('master',)) check_prune_result(expected)