delta/python-packages/gitpython.git/git/test/test_diff.py, branch 2.1.0

src: import os.path as osp

2016-10-16T00:46:31+00:00

cygwin, #533: polish abs-paths in `git add` commands

2016-10-16T00:45:55+00:00

+ Modify TCs - no main-code changes.
+ FIXed:
  + `TestSubmodule.test_git_submodules_and_add_sm_with_new_commit()`
  + TestDiff.test_diff_with_staged_file()

- Cygwin TCs failing:
  - PY2: err: 12, fail: 2
  - PY3: err: 11, fail: 2

io, dif: #519: FIX DIFF freeze when reading from GIL

2016-09-28T01:35:38+00:00

+ CAUSE: In Windows, Diffs freeze while reading Popen streams,
probably buffers smaller; good-thin(TM) in this case because reading a
Popen-proc from the launching-thread freezes GIL.  The alternative to
use `proc.communicate()` also relies on big buffers.
+ SOLUTION: Use `cmd.handle_process_output()` to consume Diff-proc
streams.
+ Retroffited `handle_process_output()` code to support also
byte-streams, both Threading(Windows) and Select/Poll (Posix) paths
updated.

- TODO: Unfortunately, `Diff._index_from_patch_format()` still slurps
input; need to re-phrase header-regexes linewise to resolve it.

test: Start using `ddt` library for TCs

2016-09-28T01:35:38+00:00

+ DataDriven TCs for identifying which separate case failed.
+ appveyor: rework matrix, conda3.4 cannot install in develop mode

Win, #519: FIX with_rw_directory() to remove read-only dirs

2016-09-28T01:35:38+00:00

+ Stop using gitdb's respective helper.
+ Fix files chmod(555) which CANNOT DELETE on Windows (but do on Linux).

win, #519: proc.terminate() instead of kill(SIGTERM)

2016-09-26T18:54:15+00:00

+ test_diff: replace asserts with unittest-asserts.

win: GC.collect on all TC.tearDown to fix appveyor hang runs

2016-09-26T11:37:16+00:00

+ Fixed the hangs at `test_git:TestGit.test_handle_process_output()`.
[travisci skip]

fix(diff): use explicit change-type if possible

2016-07-23T14:45:40+00:00

That way, we do not have to figure the change type out by
examining the diff object.

It's implemented in a way that should yield more desireable results
as we keep the change-type that git is providing us with.

Fixes #493

Store raw path bytes in Diff instances

2016-06-14T21:09:22+00:00

Previously, the following fields on Diff instances were assumed to be
passed in as unicode strings:

  - `a_path`
  - `b_path`
  - `rename_from`
  - `rename_to`

However, since Git natively records paths as bytes, these may
potentially not have a valid unicode representation.

This patch changes the Diff instance to instead take the following
equivalent fields that should be raw bytes instead:

  - `a_rawpath`
  - `b_rawpath`
  - `raw_rename_from`
  - `raw_rename_to`

NOTE ON BACKWARD COMPATIBILITY:
The original `a_path`, `b_path`, etc. fields are still available as
properties (rather than slots).  These properties now dynamically decode
the raw bytes into a unicode string (performing the potentially
destructive operation of replacing invalid unicode chars by "�"'s).
This means that all code using Diffs should remain backward compatible.
The only exception is when people would manually construct Diff
instances by calling the constructor directly, in which case they should
now pass in bytes rather than unicode strings.

See also the discussion on
https://github.com/gitpython-developers/GitPython/pull/467

Don't choke on (legitimately) invalidly encoded Unicode paths

2016-06-06T10:16:11+00:00