diff options
| author | Gavin Mak <gavinmak@google.com> | 2023-03-11 06:46:20 +0000 |
|---|---|---|
| committer | LUCI <gerrit-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2023-03-22 17:46:28 +0000 |
| commit | ea2e330e43c182dc16b0111ebc69ee5a71ee4ce1 (patch) | |
| tree | dc33ba0e56825b3e007d0589891756724725a465 /tests/test_repo_trace.py | |
| parent | 1604cf255f8c1786a23388db6d5277ac7949a24a (diff) | |
| download | git-repo-ea2e330e43c182dc16b0111ebc69ee5a71ee4ce1.tar.gz | |
Format codebase with black and check formatting in CQ
Apply rules set by https://gerrit-review.googlesource.com/c/git-repo/+/362954/ across the codebase and fix any lingering errors caught
by flake8. Also check black formatting in run_tests (and CQ).
Bug: b/267675342
Change-Id: I972d77649dac351150dcfeb1cd1ad0ea2efc1956
Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/363474
Reviewed-by: Mike Frysinger <vapier@google.com>
Tested-by: Gavin Mak <gavinmak@google.com>
Commit-Queue: Gavin Mak <gavinmak@google.com>
Diffstat (limited to 'tests/test_repo_trace.py')
| -rw-r--r-- | tests/test_repo_trace.py | 68 |
1 files changed, 36 insertions, 32 deletions
diff --git a/tests/test_repo_trace.py b/tests/test_repo_trace.py index 5faf2938..e4aeb5de 100644 --- a/tests/test_repo_trace.py +++ b/tests/test_repo_trace.py | |||
| @@ -22,35 +22,39 @@ import repo_trace | |||
| 22 | 22 | ||
| 23 | 23 | ||
| 24 | class TraceTests(unittest.TestCase): | 24 | class TraceTests(unittest.TestCase): |
| 25 | """Check Trace behavior.""" | 25 | """Check Trace behavior.""" |
| 26 | 26 | ||
| 27 | def testTrace_MaxSizeEnforced(self): | 27 | def testTrace_MaxSizeEnforced(self): |
| 28 | content = 'git chicken' | 28 | content = "git chicken" |
| 29 | 29 | ||
| 30 | with repo_trace.Trace(content, first_trace=True): | 30 | with repo_trace.Trace(content, first_trace=True): |
| 31 | pass | 31 | pass |
| 32 | first_trace_size = os.path.getsize(repo_trace._TRACE_FILE) | 32 | first_trace_size = os.path.getsize(repo_trace._TRACE_FILE) |
| 33 | 33 | ||
| 34 | with repo_trace.Trace(content): | 34 | with repo_trace.Trace(content): |
| 35 | pass | 35 | pass |
| 36 | self.assertGreater( | 36 | self.assertGreater( |
| 37 | os.path.getsize(repo_trace._TRACE_FILE), first_trace_size) | 37 | os.path.getsize(repo_trace._TRACE_FILE), first_trace_size |
| 38 | 38 | ) | |
| 39 | # Check we clear everything is the last chunk is larger than _MAX_SIZE. | 39 | |
| 40 | with mock.patch('repo_trace._MAX_SIZE', 0): | 40 | # Check we clear everything is the last chunk is larger than _MAX_SIZE. |
| 41 | with repo_trace.Trace(content, first_trace=True): | 41 | with mock.patch("repo_trace._MAX_SIZE", 0): |
| 42 | pass | 42 | with repo_trace.Trace(content, first_trace=True): |
| 43 | self.assertEqual(first_trace_size, | 43 | pass |
| 44 | os.path.getsize(repo_trace._TRACE_FILE)) | 44 | self.assertEqual( |
| 45 | 45 | first_trace_size, os.path.getsize(repo_trace._TRACE_FILE) | |
| 46 | # Check we only clear the chunks we need to. | 46 | ) |
| 47 | repo_trace._MAX_SIZE = (first_trace_size + 1) / (1024 * 1024) | 47 | |
| 48 | with repo_trace.Trace(content, first_trace=True): | 48 | # Check we only clear the chunks we need to. |
| 49 | pass | 49 | repo_trace._MAX_SIZE = (first_trace_size + 1) / (1024 * 1024) |
| 50 | self.assertEqual(first_trace_size * 2, | 50 | with repo_trace.Trace(content, first_trace=True): |
| 51 | os.path.getsize(repo_trace._TRACE_FILE)) | 51 | pass |
| 52 | 52 | self.assertEqual( | |
| 53 | with repo_trace.Trace(content, first_trace=True): | 53 | first_trace_size * 2, os.path.getsize(repo_trace._TRACE_FILE) |
| 54 | pass | 54 | ) |
| 55 | self.assertEqual(first_trace_size * 2, | 55 | |
| 56 | os.path.getsize(repo_trace._TRACE_FILE)) | 56 | with repo_trace.Trace(content, first_trace=True): |
| 57 | pass | ||
| 58 | self.assertEqual( | ||
| 59 | first_trace_size * 2, os.path.getsize(repo_trace._TRACE_FILE) | ||
| 60 | ) | ||
