summaryrefslogtreecommitdiff
path: root/tests-clar
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | Wrap iconv stuff and write testsRussell Belfer2013-10-032-1/+58
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds a simple wrapper around the iconv APIs and uses it instead of the old code that was inlining the iconv stuff. This makes it possible for me to test the iconv logic in isolation. A "no iconv" version of the API was defined with macros so that I could have fewer ifdefs in the code itself.
| * | | | | | Simplify git_path_is_empty_dir implementationRussell Belfer2013-10-032-27/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This simplifies git_path_is_empty_dir on both Windows (getting rid of git_buf allocation inside the function) and other platforms (by just using git_path_direach), and adds tests for the function, and uses the function to simplify some existing tests.
| * | | | | | Initial iconv hookup for precomposed unicodeRussell Belfer2013-10-035-39/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This hooks up git_path_direach and git_path_dirload so that they will take a flag indicating if directory entry names should be tested and converted from decomposed unicode to precomposed form. This code will only come into play on the Apple platform and even then, only when certain types of filesystems are used. This involved adding a flag to these functions which involved changing a lot of places in the code. This was an opportunity to do a bit of code cleanup here and there, for example, getting rid of the git_futils_cleanupdir_r function in favor of a simple flag to git_futils_rmdir_r to not remove the top level entry. That ended up adding depth tracking during rmdir_r which led to a safety check for infinite directory recursion. Yay. This hasn't actually been tested on the Mac filesystems where the issue occurs. I still need to get test environment for that.
| * | | | | | Add check if we need to precompose unicode on MacRussell Belfer2013-10-031-0/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds initialization of core.precomposeunicode to repo init on Mac. This is necessary because when a Mac accesses a repo on a VFAT or SAMBA file system, it will return directory entries in decomposed unicode even if the filesystem entry is precomposed. This also removes caching of a number of repo properties from the repo init pipeline because these are properties of the specific filesystem on which the repo is created, not of the system as a whole.
* | | | | | | Add Assembla unit testBen Straub2013-10-081-0/+6
| | | | | | |
* | | | | | | tree: allow retrieval of raw attributesCarlos Martín Nieto2013-10-081-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When a tool needs to recreate the tree object (for example an interface to another VCS), it needs to use the raw attributes, forgoing any normalization.
* | | | | | | Add test for multiple thread init/shutdownBen Straub2013-10-041-0/+13
| | | | | | |
* | | | | | | index: Make _read() cope with index file creationnulltoken2013-10-041-0/+50
| | | | | | |
* | | | | | | Tabify indentationsnulltoken2013-10-041-8/+8
|/ / / / / /
* | | | | | Merge pull request #1888 from jamill/network_cancellationVicent Martí2013-10-031-6/+6
|\ \ \ \ \ \ | |_|_|_|_|/ |/| | | | | network cancellation improvements
| * | | | | Style tweaks and changes for code review feedbackJameson Miller2013-10-031-0/+2
| | | | | |
| * | | | | Fix broken online push testsJameson Miller2013-10-021-4/+0
| | | | | |
| * | | | | Support cancellation in push operationJameson Miller2013-10-021-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit adds cancellation for the push operation. This work consists of: 1) Support cancellation during push operation - During object counting phase - During network transfer phase - Propagate GIT_EUSER error code out to caller 2) Improve cancellation support during fetch - Handle cancellation request during network transfer phase - Clear error string when cancelled during indexing 3) Fix error handling in git_smart__download_pack Cancellation during push is still only handled in the pack building and network transfer stages of push (and not during packbuilding).
* | | | | | Merge pull request #1887 from libgit2/ntk/topic/git_message_rawVicent Martí2013-10-031-1/+7
|\ \ \ \ \ \ | | | | | | | | | | | | | | commit: Introduce git_commit_message_raw()
| * | | | | | commit: Introduce git_commit_message_raw()nulltoken2013-10-031-1/+7
| |/ / / / /
* | | | | | HTTP: handle "relative" redirectsBen Straub2013-10-021-0/+28
|/ / / / /
* | | | | clone: put the callbacks struct directly in the clone optionsCarlos Martín Nieto2013-10-024-23/+17
| | | | | | | | | | | | | | | | | | | | There's no need for this to be a pointer to somewhere else.
* | | | | clone: bring back NULL as defaultsCarlos Martín Nieto2013-10-021-0/+6
| | | | | | | | | | | | | | | | | | | | This wasremoved as part of the large culling a few commits ago.
* | | | | clone: re-allow using a custom remote nameCarlos Martín Nieto2013-10-021-0/+9
| | | | | | | | | | | | | | | | | | | | | | | | | This is a small thing that by itself doesn't quite justify making the user use clone_into.
* | | | | clone: remove more options from basic cloneCarlos Martín Nieto2013-10-021-49/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The basic clone function is there to make it easy to create a "normal" clone. Remove a bunch of options that are about changing the remote's configuration.
* | | | | clone: remove the autotag optionCarlos Martín Nieto2013-10-021-33/+0
| | | | | | | | | | | | | | | | | | | | | | | | | Downloading all tags is part of what makes it a clone instead of simply a fetch.
* | | | | clone: implement git_clone_intoCarlos Martín Nieto2013-10-021-0/+39
| | | | | | | | | | | | | | | | | | | | | | | | | This allows you to set up the repository and remote as you which to have them before performing the clone operation.
* | | | | remote: move the credentials callback to the structCarlos Martín Nieto2013-10-023-6/+12
| | | | | | | | | | | | | | | | | | | | | | | | | Move this one as well, letting us have a single way of setting the callbacks for the remote, and removing fields from the clone options.
* | | | | remote: put the _download() callback with the othersCarlos Martín Nieto2013-10-027-16/+52
| |/ / / |/| | | | | | | | | | | | | | | | | | | The text progress and update_tips callbacks are already part of the struct, which was meant to unify the callback setup, but the download one was left out.
* | | | Merge pull request #1882 from linquize/config-subsection-fixVicent Martí2013-10-012-0/+40
|\ \ \ \ | | | | | | | | | | Config subsection name should allow to have ']' and '\\' should allow to escape any characters
| * | | | Add negative test cases for config header with invalid charactersLinquize2013-10-011-0/+22
| | | | |
| * | | | Add test case to test ']' and '\\' characters in config subsectionLinquize2013-10-012-0/+18
| | |_|/ | |/| |
* | | | Merge pull request #1879 from libgit2/redir-refactorVicent Martí2013-09-301-0/+53
|\ \ \ \ | | | | | | | | | | Redir refactor
| * | | | Make sure utility doesn't leak memoryBen Straub2013-09-261-0/+9
| | | | |
| * | | | Deploy gitno_connection_data into transport (winhttp)Ben Straub2013-09-261-4/+6
| | | | | | | | | | | | | | | | | | | | ...and have that call manage replaced memory in the output structure.
| * | | | Migrate redirect URL handling to common utilityBen Straub2013-09-251-0/+42
| |/ / /
* | | | Merge pull request #1412 from jamill/push_progressVicent Martí2013-09-301-31/+53
|\ \ \ \ | | | | | | | | | | Initial Implementation of progress reports during push
| * | | | Initial Implementation of progress reports during pushJameson Miller2013-09-301-31/+53
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds the basics of progress reporting during push. While progress for all aspects of a push operation are not reported with this change, it lays the foundation to add these later. Push progress reporting can be improved in the future - and consumers of the API should just get more accurate information at that point. The main areas where this is lacking are: 1) packbuilding progress: does not report progress during deltafication, as this involves coordinating progress from multiple threads. 2) network progress: reports progress as objects and bytes are going to be written to the subtransport (instead of as client gets confirmation that they have been received by the server) and leaves out some of the bytes that are transfered as part of the push protocol. Basically, this reports the pack bytes that are written to the subtransport. It does not report the bytes sent on the wire that are received by the server. This should be a good estimate of progress (and an improvement over no progress).
* | | | | Merge pull request #1881 from libgit2/ignore-submodules-in-stashVicent Martí2013-09-304-43/+129
|\ \ \ \ \ | | | | | | | | | | | | Never consider submodules for stashing
| * | | | | Ensure submodule repos and indices are freedignore-submodules-in-stashBen Straub2013-09-301-4/+16
| | | | | | | | | | | | | | | | | | | | | | | | ...before the helper's cleanup method tries to delete their files.
| * | | | | Test that submodules don't affect stashingJustin Spahr-Summers2013-09-274-43/+117
| | |/ / / | |/| | |
* | | | | Merge pull request #1878 from libgit2/ntk/fix/warningsVicent Martí2013-09-302-5/+5
|\ \ \ \ \ | |_|/ / / |/| | | | Fix x86/x64 size_t related warnings
| * | | | Fix warningsnulltoken2013-09-262-5/+5
| |/ / /
* | | | commit: Trim message leading newlinesnulltoken2013-09-301-3/+18
| | | | | | | | | | | | | | | | Fix libgit2/libgit2sharp#522
* | | | Tabify indentationsnulltoken2013-09-301-82/+82
|/ / /
* | | Make tests pass if XDG config existsRussell Belfer2013-09-241-9/+14
| | |
* | | Disconnect path string to preserve after redirectRussell Belfer2013-09-241-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The subtransport path was relying on pointing to data owned by the remote which meant that after a redirect, the updated path was getting lost for future requests. This updates the http transport to strdup the path and maintain its own lifetime. This also pulls responsibility for parsing the URL back into the http transport and isolates the functions that parse and free that connection data so that they can be reused between the initial parsing and the redirect parsing.
* | | Fix up filebuf tests a bitRussell Belfer2013-09-241-5/+5
| | |
* | | Fix incorrect return code in crlf filterRussell Belfer2013-09-244-16/+45
| | | | | | | | | | | | | | | | | | The git_buf_text_gather_stats call returns a boolean indicating if the file looks like binary data. That shouldn't be an error; it should be used to skip CRLF processing though.
* | | Fix warnings on Windows 64-bit buildRussell Belfer2013-09-231-1/+1
| | |
* | | merge: reverse array and length parameter orderCarlos Martín Nieto2013-09-221-2/+2
| | | | | | | | | | | | Make it pair up with the one for commits. This fixes #1691.
* | | Suppress noise from running Valgrind on OSX 10.8Ben Straub2013-09-191-0/+28
| | |
* | | Update clar and recreate cl_assert_equal_szRussell Belfer2013-09-172-1/+5
| | | | | | | | | | | | | | | | | | | | | This updates clar to the version without cl_assert_equal_sz and then adds a new version of that macro the clar_libgit2.h. The new version works around a strange issue that seemed to be arising on release builds with VS 10 64-bit builds.
* | | Bug fixes and cleanupsRussell Belfer2013-09-171-7/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This contains a few bug fixes and some header and API cleanups. The main API change is that filters should now use GIT_PASSTHROUGH to indicate that they wish to skip processing a file instead of GIT_ENOTFOUND. The bug fixes include a possible out-of-range buffer access in the ident filter, a filter ordering problem I introduced into the custom filter tests on Windows, and a filter buf NUL termination issue that was coming up on Linux.
* | | More filtering tests including orderRussell Belfer2013-09-172-6/+94
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds more tests of filters, including the ident filter when mixed with custom filters. I was able to combine with the reverse filter and demonstrate that the order of filter application with the default priority constants matches the order of core Git. Also, this fixes two issues in the ident filter: preventing ident expansion on binary files and avoiding a NULL dereference when dollar sign characters are found without Id.
View Lorry Controller Status