Skip to content
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.

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

Update tests for new git version #223

Merged
merged 1 commit into from
May 3, 2024

Conversation

Patrick-Beuks
Copy link
Contributor

When cloning a project with git 2.43.2 some more files are added that increase the size of the repository, invalidating the size test.

on such file is a pack-*.rev file added in 2.41.0 https://github.blog/2023-06-01-highlights-from-git-2-41/#on-disk-reverse-indexes-by-default

Another one is sendemail-validate.sample, but could not quickly find in what version it was introduced.

There have also been some other updates to files to make them larger.

In total the difference between git version 2.40.1 and 2.43.2 is 4028 bytes.

I have updated the tests to increase the size by 4 kb, also for the minimum size as with new git versions it does not seems to be close to this number.

Size diff:

@@ -1,7 +1,8 @@
-git version 2.40.1
+git version 2.43.2
 
 .git/objects/pack/pack-646f69826eb454b74c466756d98d499a57e881b8.pack:40247
 .git/objects/pack/pack-646f69826eb454b74c466756d98d499a57e881b8.idx:11656
+.git/objects/pack/pack-646f69826eb454b74c466756d98d499a57e881b8.rev:1564
 .git/refs/remotes/origin/HEAD:32
 .git/refs/heads/master:41
 .git/index:1002
@@ -9,13 +10,13 @@
 .git/config:243
 .git/packed-refs:568
 .git/HEAD:23
-.git/logs/refs/remotes/origin/HEAD:161
-.git/logs/refs/heads/master:161
-.git/logs/HEAD:161
+.git/logs/refs/remotes/origin/HEAD:211
+.git/logs/refs/heads/master:211
+.git/logs/HEAD:211
 .git/info/exclude:240
 .git/hooks/pre-receive.sample:544
 .git/hooks/fsmonitor-watchman.sample:4726
-.git/hooks/pre-commit.sample:1643
+.git/hooks/pre-commit.sample:1649
 .git/hooks/commit-msg.sample:896
 .git/hooks/pre-applypatch.sample:424
 .git/hooks/pre-rebase.sample:4898
@@ -26,3 +27,4 @@
 .git/hooks/push-to-checkout.sample:2783
 .git/hooks/applypatch-msg.sample:478
 .git/hooks/update.sample:3650
+.git/hooks/sendemail-validate.sample:2308

@alexandresalome
Copy link
Member

Thank you for all the details and for contributing

@alexandresalome alexandresalome merged commit a8bfcd1 into gitonomy:main May 3, 2024
5 checks passed
@Patrick-Beuks Patrick-Beuks deleted the update-size-test branch May 3, 2024 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants