770 Commits (8a9d0b36a344c5e1a26bdd01da7f088a08050d82)

Author	SHA1	Message	Date
Jan Sandbrink	ca03e6543a	fix specs	9 years ago
Oliver Günther	0610280e6f	Cleanup	9 years ago
Oliver Günther	c16c1e75d4	Extend SysController to allow WS to check / update storage	9 years ago
Jan Sandbrink	6c1e4dbf4f	send aggregated notifications ... - only sending the last mail of an aggregation chain - displaying the aggregated state of the change - replaces old classes for mail notifications - old special error handling for missing users is obsoleted, since recipients are determined at jobs runtime - special cases for missing author and journal are considered	9 years ago
Oliver Günther	9336cff8bb	Extended specs ... This commit adds / extends the following specs: RepositoriesController: Ensure empty repository page is shown for empty repositories adapters/subversion: Test correct use of --login, --password arguments	9 years ago
Oliver Günther	caf5e4a706	Formatting and clean up	9 years ago
Oliver Günther	c5b200d7a0	Review corrections ... - Remove repositories paths using project.id instead of `project_id` - Allow managed repositories to be unlinked in the instance (But doesn't remove repository directory). - Remove leftover settings partial	9 years ago
Oliver Günther	e1bdb41bcd	More complete specs wrt. coverage, cleanup	9 years ago
Oliver Günther	1e8c873544	Remove filesystem adapter ... This commit removes the entire FileSystem repository and adapter. It rids all specs (core and legacy) of remaining uses of this adpater and replaces it with a temporary subversion repository, where necessary. We also need to remove remaining filesystem instances from the database, otherwise users will receive errors when accessing said repositories on the Frontend. This does not destroy any data on filesystem, but database information on the repositories (remarks) are lost.	9 years ago
Oliver Günther	05b27c1b98	Asynchronous creation, deletion of repositories from OP. ... This PR begins the push towards management of repository lifetime from within OpenProject. What this currently provides is the following: - A mixin `ManageableRepository`, included by `Repository::{Git, Subversion}` to provide an interface to repo management - Two services and delayed_job workers for creating (`CreateManagedRepositoryService`) and deleting (`DeleteRepositoryService`) - Customized rendering of subforms in the repository settings for arbitrary SCM vendors. TODOS: - [x] Provide sane ground to work on ( cf. PR #3149 ) - [x] Rework SCM settings form (`{scm}_tags` is currently HTML generated from within a Helper.) - [x] Ease integration of arbitrary SCM vendors - [x] Display meaningful error / warning when user looks at bare repository (currently outputs 500, as Git HEAD / svn revision is not available) - [x] I18n - [ ] Specs (in progress) - [x] Fix legacy specs - [ ] Decide what to do with `Filesystem` adapter. Open Questions: - **Configuration**: Currently replaces the existing options in `configuration.yml`. - **Implementation specifics**: - Sub-projects repositories reside in a hierarchy below their parents - How should the delayed_job react upon failure? (esp. creation of repositories) Relevant work package: https://community.openproject.org/work_packages/20416 Merge dependent on PR #3149	9 years ago
Jan Sandbrink	138d3dba29	fix some specs ... - error messages now contain punctuation - when :save returns false, there should be errors present - errors in form representers are now API errors	9 years ago
peter	6a7ab0edb8	fix invalid response for /api/v2/authentication.json	9 years ago
Alex Coles	8338b7a6e1	Update OP::Authentication to use uppercase headers ... Signed-off-by: Alex Coles <alex@alexbcoles.com>	9 years ago
Jan Sandbrink	2854584cb2	use more clear scopes for autocompletion ... lets see if that really makes stuff more understandable... I (personally) found the old definition of "all" to be too narrow. Now there is: scope == "all": I want to search over all WPs scope == "relatable": I want to search over all WPs that are relatable to WPs in the given project (via project_id) no scope: If I specified a project search over the WPs of that one, otherwise search everywhere (like "all")	9 years ago
Markus Kahl	c24c58a273	fixed specs; use default realm; test custom realm	9 years ago
Markus Kahl	bcd981df5c	consider X-Authentication-Scheme; opt scoped realm	9 years ago
Jan Sandbrink	9eb7cfbe9b	actually allow the status to be set while moving ... apparently the status_id param was set to expect an array of values while being moved in this commit: d7f399ab8d Probably because the spec started to expect that when it was moved to be an RSpec test. I am not sure if this was ever a valid thing outside the tests...	9 years ago
Jan Sandbrink	d3ceffddfc	actually allow the status to be set while moving ... apparently the status_id param was set to expect an array of values while being moved in this commit: d7f399ab8d Probably because the spec started to expect that when it was moved to be an RSpec test. I am not sure if this was ever a valid thing outside the tests...	10 years ago
Jens Ulferts	9b93316283	explicitly expecting a setting	10 years ago
Jan Sandbrink	516c9a5f3e	move mock method out of production code	10 years ago
Jan Sandbrink	bc3437c3b0	replace create_uploaded_file in specs ... mock_uploaded_file creates a Rack::Test::UploadedFile, which should behave equally to a real uploaded file, but is also treated correctly by Rack::Test	10 years ago
Alexander Bach	9d641a30f6	Address styling issues	10 years ago
Alexander Bach	a670032b83	Covered #includes_for_columns in tests	10 years ago
Markus Kahl	ee4d977c86	another logout back URL spec for good measure	10 years ago
Alex Coles	2efd6f4e45	Move legacy specs under spec/legacy ... Signed-off-by: Alex Coles <alex@alexbcoles.com>	10 years ago
Alex Coles	0d9e465108	Replace deprecated shoulda #set_the_flash in specs ... Signed-off-by: Alex Coles <alex@alexbcoles.com>	10 years ago
Markus Kahl	c3fa89e5db	disable password choice config	10 years ago
Jens Ulferts	b37ea687e2	showing error messages on scm updates	10 years ago
Jens Ulferts	fce72938e3	prevent path traversal via filesystem scm ... Setting up a filesystem scm now requires to specify a whitelist via the configuration	10 years ago
Alex Coles	fda43ac304	Replace VersionsController#create RJS with jQuery ... Matches `CategoriesController#create` implementation: see 4cc9aaf8. Signed-off-by: Alex Coles <alex@alexbcoles.com>	10 years ago
Alex Coles	07d4ea4edd	Fix version select style on wp form inline create ... Signed-off-by: Alex Coles <alex@alexbcoles.com>	10 years ago
Alex Coles	ebbc9e3d7d	Prefer #not_to over #to_not in specs ... Signed-off-by: Alex Coles <alex@alexbcoles.com>	10 years ago
Alex Coles	fed972eefa	Mark broken QueriesController policy spec pending ... Signed-off-by: Alex Coles <alex@alexbcoles.com>	10 years ago
Alex Coles	3785fdc9ed	Use #describe, #context with String arg not symbol ... The semantics of `describe` have changed in RSpec 3: passing a symbol rather than a string will change the value of the described object. See discussion here: https://github.com/rspec/rspec-core/issues/1114 Signed-off-by: Alex Coles <alex@alexbcoles.com>	10 years ago
Alex Coles	944b89d83e	Convert specs to RSpec 3.2.0 syntax with Transpec ... This conversion is done by Transpec 3.0.7 with the following command: transpec -f * 34 conversions from: it { should ... } to: it { is_expected.to ... } * 14 conversions from: obj.stub_chain(:message1, :message2) to: allow(obj).to receive_message_chain(:message1, :message2) * 7 conversions from: obj.stub(:message) to: allow(obj).to receive(:message) * 4 conversions from: be_true to: be_truthy * 3 conversions from: obj.should_receive(:message) to: expect(obj).to receive(:message) * 2 conversions from: failure_message_for_should { } to: failure_message { } * 2 conversions from: failure_message_for_should_not { } to: failure_message_when_negated { } * 1 conversion from: Klass.any_instance.stub(:message => value) to: allow_any_instance_of(Klass).to receive_messages(:message => value) * 1 conversion from: be_false to: be_falsey * 1 conversion from: obj.should_not_receive(:message) to: expect(obj).not_to receive(:message) * 1 conversion from: stub('something') to: double('something') * 1 addition of: RSpec.configure { |c| c.infer_spec_type_from_file_location! } For more details: https://github.com/yujinakayama/transpec#supported-conversions	10 years ago
Markus Kahl	85e91372a1	more comments	10 years ago
Markus Kahl	de309f0eee	polish	10 years ago
Markus Kahl	104b70cc62	work package notification mail specs	10 years ago
Jan Sandbrink	5d63732d9f	fix broken specs for CF controller ... - did expect the wrong things - did send the wrong things -> tests pass tough they shouldn't	10 years ago
Florian Kraft	d585bafbf6	fix timelog controller spec ... Signed-off-by: Florian Kraft <f.kraft@finn.de>	10 years ago
Jens Ulferts	5e2e44ee70	fix previewing work package journal notes	10 years ago
Jens Ulferts	f0933aac12	adapt spec to changed dom	10 years ago
Florian Kraft	00c257541c	move integration test to scenario	10 years ago
Florian Kraft	af2230e20d	fix project associations test	10 years ago
Florian Kraft	1a2ea133f2	fix tests for wiki and permitted params	10 years ago
Jens Ulferts	04554b5fda	fix work package controller spec	10 years ago
Florian Kraft	22853c8b46	Fix the coding style via rubocop	10 years ago
Florian Kraft	cb2f3f30d6	Fix error where anyone could read the diff of a WP ... With this fix it is no longer possible to view a description diff without the necessary permissions. The error was reported by Jan Sandbrink in https://community.openproject.org/work_packages/16974.	10 years ago
Florian Kraft	9f82ba85eb	fix styling by using rubocop -a	10 years ago
Florian Kraft	59fc511c7d	fix positioning of arguments for hound	10 years ago