Steve Howell
4f1108cc27
refactor: Pass in a sub to remove_user_from_stream().
2017-03-05 11:55:09 -08:00
Steve Howell
190b2ab1d3
refactor: Pass in a sub to invite_users_to_stream().
2017-03-05 11:55:09 -08:00
Steve Howell
8a4dee4a80
bug fix: Handle bad streams for compose-invite feature.
...
We now handle missing subs more gracefully when you click on the
link to invite somebody you at-mentioned in a stream message.
2017-03-05 11:55:09 -08:00
Steve Howell
6183a0d8b2
bug fix: Handle invalid streams in would_receive_message().
2017-03-05 11:55:09 -08:00
Steve Howell
4b98ec0a12
bug fix: Handle errors in at-mention typeaheads.
...
If a stream doesn't exist, prevent executing code that leads
to needless warnings.
2017-03-05 11:55:09 -08:00
Steve Howell
434f228135
refactor: Avoid nesting in the compose invite handler.
...
We exit early now if email is undefined, rather than
having a big if statement.
2017-03-05 11:55:09 -08:00
Steve Howell
24631a7405
refactor: Pass in a sub to subs.toggle_pin_to_top_stream().
2017-03-05 11:55:09 -08:00
Steve Howell
32c2dc63f1
refactor: Pass in a sub to subs.toggle_home().
2017-03-05 11:55:09 -08:00
Steve Howell
3a93a7310e
refactor: Pass in a sub to set_stream_property().
...
This helps us localize the legacy need to pass stream names
(vs. ids) to the back end to set stream properties.
2017-03-05 11:55:09 -08:00
Steve Howell
1904871d77
refactor: Start to deprecate get_stream_name().
...
When possible, we should use get_sub_from_target() instead of
get_stream_name(), not only because it often saves a lookup later,
but it also makes it easier to audit the code for name vs. id
bugs. (When you rename a stream, there can be races where you
use the old stream name instead of the more durable stream_id).
This commit handles the easy cases where the caller directly
wanted the sub, not the name.
2017-03-05 11:55:09 -08:00
Steve Howell
df6d644788
refactor: Extract get_sub_for_target().
2017-03-05 11:55:09 -08:00
Steve Howell
a4376efd0f
refactor: De-dup code using subs.sub_or_unsub().
2017-03-05 11:55:09 -08:00
Steve Howell
67e558f905
refactor: Pass in a full sub to sub_or_unsub().
2017-03-05 11:55:09 -08:00
Steve Howell
c38bbea6b9
Add tests for subs.sub_or_unsub().
2017-03-05 11:55:09 -08:00
Steve Howell
8c61bb69cc
casper: Fix flakiness with uploading realm icon.
2017-03-05 11:42:43 -08:00
Tim Abbott
67219cf660
puppet: Use restart-server for weekly server restarts.
...
Using `supervisorctl restart all` carried longer downtime (since it
just restarts everything at the same moment) and was less under our
control; I'm not sure it had any advantages.
2017-03-05 11:36:10 -08:00
Tim Abbott
b6f53d6c14
coverage: bring zerver/lib/validator.py to 100% coverage.
2017-03-05 00:53:27 -08:00
Tim Abbott
2f8bb1b1cd
coverage: Add test for REST requests to /json API unauthed.
2017-03-05 00:53:27 -08:00
Tim Abbott
c8e38aaa55
coverage: Add coverage for OPTIONS HTTP method.
2017-03-05 00:53:26 -08:00
Tim Abbott
e95f139308
coverage: bring zerver/lib/response.py to 100% coverage.
2017-03-05 00:53:26 -08:00
Tim Abbott
89decf6ded
coverage: bring zerver/lib/request.py to 100% coverage.
2017-03-05 00:53:26 -08:00
Tim Abbott
e7ff3415ab
coverage: bring test_helpers.py to 100% coverage.
2017-03-05 00:53:26 -08:00
Tim Abbott
546fb9199d
coverage: bring test_classes.py to 100% coverage.
2017-03-05 00:53:26 -08:00
Tim Abbott
e3cfb256dd
coverage: Bring zerver.lib.narrow.py to 100% coverage.
2017-03-05 00:53:26 -08:00
Tim Abbott
56b1f79fe3
integrations: Fix email integration being listed unconditionally.
2017-03-05 00:53:26 -08:00
Tim Abbott
464928fdcc
bugdown: Fix mypy error with python 3 coverage.
2017-03-05 00:53:26 -08:00
Tim Abbott
a53240a47c
test_subs: Fix buggy assertion statement.
2017-03-04 23:57:56 -08:00
Tim Abbott
5050e42212
test-backend: Document # nocoverage comments.
2017-03-04 23:47:16 -08:00
Tim Abbott
e7389b4162
test-backend: Require 100% coverage on tests by default.
2017-03-04 23:39:34 -08:00
Tim Abbott
6c12a49d04
test_auth_backends: Remove now-unnecessary compatibility code.
...
This code was added as part of the Django 1.10 migration to make our
tests work with both Django 1.8 and 1.10. Now that we're on 1.10,
it's no longer required.
2017-03-04 23:38:46 -08:00
Tim Abbott
39704bac6c
test_bugdown: Remove unused common_bugdown_test.
2017-03-04 23:35:46 -08:00
Tim Abbott
1a8bc5f383
coverage: Bring test_decorators to 100% coverage.
2017-03-04 23:34:28 -08:00
Tim Abbott
c73c3c52f8
test_email_mirror: Fix TestCommandMTA.
...
The test didn't actually use the mock, due to an incorrect import
path.
2017-03-04 23:28:48 -08:00
Tim Abbott
6ea3a9cb1d
coverage: Bring test_templates.py to 100% coverage.
2017-03-04 23:28:48 -08:00
Tim Abbott
8d9cab2339
test_narrow: Use AssertionError properly.
2017-03-04 23:19:16 -08:00
Tim Abbott
d8171b2efc
coverage: Bring tests.py to 100% coverage.
2017-03-04 23:17:36 -08:00
Tim Abbott
768307d921
coverage: Bring test_events to 100% coverage.
2017-03-04 23:12:32 -08:00
Tim Abbott
d4f0e394e9
test_unread: Bring to 100% test coverage.
...
This actually fixes a test bug where we weren't actually verifying
whether other users' messages were being marked as read.
2017-03-04 23:09:45 -08:00
Tim Abbott
538570b7c1
test_subs: Bring to 100% test coverage.
2017-03-04 23:07:56 -08:00
Tim Abbott
065651b4fe
coverage: Exclude uncovered lines in test_signup.
2017-03-04 23:01:50 -08:00
Tim Abbott
ff9f827c9a
coverage: Don't require coverage in six.PY2/PY3 branches.
2017-03-04 22:51:35 -08:00
Tim Abbott
5ee33e59b8
coverage: Don't require coverage for NotImplementedErrors.
2017-03-04 22:48:21 -08:00
Tim Abbott
401ae3db97
coverage: Don't complain about if False coverage.
...
This fixes a test failure on master.
2017-03-04 22:48:14 -08:00
Tim Abbott
bc0cd7eb90
coverage: Move coverage config to tools/coveragerc.
2017-03-04 22:48:05 -08:00
Tim Abbott
63664264b8
test-backend: Disable verbose coverage display.
2017-03-04 22:48:05 -08:00
Tim Abbott
60b6a56da8
deps: Upgrade handlebars to latest version.
...
Fixes #3939 .
2017-03-04 22:05:23 -08:00
Rafid Aslam
33129059f4
deps: Upgrade and move `handlebars` from `static/third` to `npm`.
...
- Remove `handlebars.runtime.js` from static/third and fetch it from npm
- Upgrade `handlebars` to 3.0.3.
I change the test since there is a patch about line, written in
handlebars'
v2.0.0-beta.1 release note:
"Lines containing only block statements and whitespace are now removed."
Fixes part of #1709 .
2017-03-04 21:49:02 -08:00
Tim Abbott
bd0d6bb9d2
docs: Add discussion of VFL to front-end-build-process.md.
...
Also, link to this from various Python files where relevant.
2017-03-04 21:14:17 -08:00
Tim Abbott
7a048133bf
docs: Fix typo in link to translating docs.
2017-03-04 20:58:50 -08:00
Tim Abbott
b62cd866e1
docs: Significantly update static asset pipeline docs.
2017-03-04 20:58:30 -08:00