3631 Commits (d4d5acbd4575a290f73c12367cdf6405dbca178e)

Author	SHA1	Message	Date
Jan Sandbrink	992888df2a	add configuration endpoint to APIv3	9 years ago
Jan Sandbrink	68d521ca85	add link for adding attachments to work package	9 years ago
Jan Sandbrink	1f7485d5bf	fix HTTP 500 in WP form ... - occured when an invalid type was provided by the client	9 years ago
Jan Sandbrink	7e53b9ac04	remove spec expecting german translations ... german translations are not part of OpenProject anymore (they are now provided via Crowdin)	9 years ago
Jan Sandbrink	1b5bb5ba98	use AR errors as mock	9 years ago
Jan Sandbrink	88e052a53e	style ... [ci skip]	9 years ago
Jan Sandbrink	17940bf02c	change validation of WP children ... - do not add error on a symbol called :#123 (123 is the childrens ID) - this is against the semantics of ActiveRecord::Errors - creating symbols in a loop used to be a bad idea (might be better today) - it breaks error messages in the APIv3 (which is due to APIv3 working beyond the borders of AR errors, not the best of ideas...) - use :base instead, the place for unlocatable errors - include the fact that #123 refers to a child element in the error message Basically I feel that this makes the children validation cleaner and better. But somehow I fear, that there is some code out there that depends on the fact that error keys like :#123 existed...	9 years ago
Jan Sandbrink	bc30b28558	extract attribute name conversion into extra class ... - consider special renaming rules for well known attribute names -> currently covering wp attributes and createdAt, updatedAt	9 years ago
Jan Sandbrink	b374ebdd7f	store messages alongside error symbols ... - allows to skip translation part in ErrorBase - allows to have correct translations containing placeholder values	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
Jan Sandbrink	dcd2a3b03c	rework error instantiation ... - multiple errors are no longer a special case of specific errors - there is a separate error class to represent multiple API errors - convert ActiveRecord errors to API errors in a single place, ideally work with API errors from there on	9 years ago
Jan Sandbrink	c586a9cc8c	mock the model in model_contract_spec	9 years ago
Jan Sandbrink	13b9042293	treat all 422 errors the same for forms ... That means: Do render the form for all HTTP 422 errors, problems are highlighted within the form	9 years ago
Jan Sandbrink	8f110d8b93	adapt specs to new contract behaviour ... - also refine specs for done-ratio to check for non-fail when not changed - use shared examples for parent properties also in done ratio specs	9 years ago
Jan Sandbrink	ac9cffe741	remove tautology and add apropriate specs	9 years ago
Jan Sandbrink	e9059a97ee	style ... [ci skip]	9 years ago
Jan Sandbrink	ce0b906d91	dirty trick to allow using a Hash as mock ... also removes default value from initializer, which was never sane	9 years ago
Jan Sandbrink	0b7b4bf329	extract error merging into contract	9 years ago
Jan Sandbrink	218c75e765	fix update_by! user when not providing notes ... - previously the created Journal Entry would have been made by the specified user or the User.current, dependent whether :notes were provided or not - now the journal entry is always made by the specified user	9 years ago
Jens Ulferts	f2ae9be55c	increasing spec robustness, hopefully	9 years ago
peter	6a7ab0edb8	fix invalid response for /api/v2/authentication.json	9 years ago
Markus Kahl	bd0df07090	fixed specs	9 years ago
Alex Coles	1d1d56217d	Without message chain, prefer #receive in spec ... Signed-off-by: Alex Coles <alex@alexbcoles.com>	9 years ago
Alex Coles	985a15f08a	Remove obsolete TODO in VersionsController spec ... [ci skip] Signed-off-by: Alex Coles <alex@alexbcoles.com>	9 years ago
Alex Coles	8338b7a6e1	Update OP::Authentication to use uppercase headers ... Signed-off-by: Alex Coles <alex@alexbcoles.com>	9 years ago
Alex Coles	69aaa74e5f	Update OP::Authentication to use uppercase headers ... Signed-off-by: Alex Coles <alex@alexbcoles.com>	9 years ago
Alex Coles	0436446257	Revert "DRY WorkPackageSchema#available_custom_fields spec" ... This reverts commit 8e47f600b0.	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
Alex Coles	8e47f600b0	DRY WorkPackageSchema#available_custom_fields spec ... Signed-off-by: Alex Coles <alex@alexbcoles.com>	9 years ago
Alex Coles	837265c257	Fix API v3 WorkPackageSchema for AR::Relation ... Signed-off-by: Alex Coles <alex@alexbcoles.com>	9 years ago
Alex Coles	ae59a6985a	Teach User.authorize_within ActiveRecord::Relation ... Signed-off-by: Alex Coles <alex@alexbcoles.com>	9 years ago
Alex Coles	ee12f7ea82	Add explicit join references in specs ... Signed-off-by: Alex Coles <alex@alexbcoles.com>	9 years ago
Alex Coles	81de05e5fe	Remove deprecated Relation#all in specs ... Signed-off-by: Alex Coles <alex@alexbcoles.com>	9 years ago
Alex Coles	59b05568e9	Remove deprecated Relation#all in views ... Signed-off-by: Alex Coles <alex@alexbcoles.com>	9 years ago
Alex Coles	2d9cfd97f4	Remove deprecated Relation#all in controllers ... Signed-off-by: Alex Coles <alex@alexbcoles.com>	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
Markus Kahl	d854df5175	suppress browser prompt on authenticate failure	9 years ago
Alex Coles	3355458fc8	Remove Array expectation in legacy Type spec ... Signed-off-by: Alex Coles <alex@alexbcoles.com>	9 years ago
Oliver Günther	e0af219a36	Fix invalid time sheet link ... This adds specs for top menu items	9 years ago
Jan Sandbrink	393fe4e70a	please the hound	9 years ago
Jan Sandbrink	3a30834ce0	tests for execute_as behaviour	9 years ago
Jan Sandbrink	f491d3d9ef	split specs for mail notification jobs per class	9 years ago
Alex Coles	b2790b8db1	Migrate deprecated .update_all with conditions ... Signed-off-by: Alex Coles <alex@alexbcoles.com>	9 years ago
Alex Coles	0f9240037a	Fix TimeEntry cleanup on work package bulk delete ... Fix `WorkPackage.cleanup_associated_before_destructing_if_required` semantics so that the first argument can either be an individual `WorkPackage` (as the spec originally expected) or a collection (as in actual usages). Signed-off-by: Alex Coles <alex@alexbcoles.com>	9 years ago
Alex Coles	424889897a	Migrate AR finder/query methods in specs ... Signed-off-by: Alex Coles <alex@alexbcoles.com>	9 years ago
Alex Coles	ce93701029	Fix syntax (w/Rubocop) in specs ... Signed-off-by: Alex Coles <alex@alexbcoles.com>	9 years ago
Alex Coles	57618b25ec	Replace dynamic finder usages with #find_by, etc. ... This patch replaces all dynamic finders, for the sake of consistency, although only some methods are deprecated. See: https://github.com/rails/activerecord-deprecated_finders#active-record-deprecated-finders * Revert some `User#find_by_login` usages in cuke steps accidentally removed in 74228b59. User Story # 20325 Signed-off-by: Alex Coles <alex@alexbcoles.com>	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
Jens Ulferts	1fc2661375	another safeguard for the inplace edit spec	9 years ago