diff options
author | Mike Frysinger <vapier@google.com> | 2019-06-12 17:42:43 -0400 |
---|---|---|
committer | David Pursehouse <dpursehouse@collab.net> | 2019-07-11 04:34:08 +0000 |
commit | 4f42a9706715c9c5add46632343f7108aabcd530 (patch) | |
tree | 24bfce2b7a4ec06e31aae95f36d03d0b7c2a9e96 /SUBMITTING_PATCHES.md | |
parent | 2b7daff8cb0bb6a0836ca0cbc18f7f86b5eb0c86 (diff) | |
download | git-repo-4f42a9706715c9c5add46632343f7108aabcd530.tar.gz |
run_tests: add a helper for invoking unittests
This makes it very easy for people to run all our unittests with just
`./run_tests`. There doesn't seem to be any other way currently to
quickly invoke any of the tests.
Change-Id: I1f9a3745fa397a1e797bd64065c2ba7f338de4a1
Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/227613
Tested-by: David Pursehouse <dpursehouse@collab.net>
Reviewed-by: David Pursehouse <dpursehouse@collab.net>
Diffstat (limited to 'SUBMITTING_PATCHES.md')
-rw-r--r-- | SUBMITTING_PATCHES.md | 26 |
1 files changed, 20 insertions, 6 deletions
diff --git a/SUBMITTING_PATCHES.md b/SUBMITTING_PATCHES.md index 07f76616..e8a72e87 100644 --- a/SUBMITTING_PATCHES.md +++ b/SUBMITTING_PATCHES.md | |||
@@ -1,3 +1,5 @@ | |||
1 | [TOC] | ||
2 | |||
1 | # Short Version | 3 | # Short Version |
2 | 4 | ||
3 | - Make small logical changes. | 5 | - Make small logical changes. |
@@ -52,17 +54,29 @@ Run `flake8` on changes modules: | |||
52 | 54 | ||
53 | flake8 file.py | 55 | flake8 file.py |
54 | 56 | ||
55 | Note that repo generally follows [Google's python style guide] | 57 | Note that repo generally follows [Google's python style guide] rather than |
56 | (https://google.github.io/styleguide/pyguide.html) rather than [PEP 8] | 58 | [PEP 8], so it's possible that the output of `flake8` will be quite noisy. |
57 | (https://www.python.org/dev/peps/pep-0008/), so it's possible that | 59 | It's not mandatory to avoid all warnings, but at least the maximum line |
58 | the output of `flake8` will be quite noisy. It's not mandatory to | 60 | length should be followed. |
59 | avoid all warnings, but at least the maximum line length should be | ||
60 | followed. | ||
61 | 61 | ||
62 | If there are many occurrences of the same warning that cannot be | 62 | If there are many occurrences of the same warning that cannot be |
63 | avoided without going against the Google style guide, these may be | 63 | avoided without going against the Google style guide, these may be |
64 | suppressed in the included `.flake8` file. | 64 | suppressed in the included `.flake8` file. |
65 | 65 | ||
66 | [Google's python style guide]: https://google.github.io/styleguide/pyguide.html | ||
67 | [PEP 8]: https://www.python.org/dev/peps/pep-0008/ | ||
68 | |||
69 | |||
70 | ## Running tests | ||
71 | |||
72 | There is a [`./run_tests`](./run_tests) helper script for quickly invoking all | ||
73 | of our unittests. The coverage isn't great currently, but it should still be | ||
74 | run for all commits. | ||
75 | |||
76 | Adding more unittests for changes you make would be greatly appreciated :). | ||
77 | Check out the [tests/](./tests/) subdirectory for more details. | ||
78 | |||
79 | |||
66 | ## Check the license | 80 | ## Check the license |
67 | 81 | ||
68 | repo is licensed under the Apache License, 2.0. | 82 | repo is licensed under the Apache License, 2.0. |