Commit c77fe0f

n2ygksliverc

authored and

committed

Allow OPTIONS request to be used on RelationshipView (#633)

1 parent 22476fe commit c77fe0fCopy full SHA for c77fe0f

File tree

+36

-4

lines changed

+36

-4

lines changed

Lines changed: 1 addition & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -15,6 +15,7 @@ any parts of the framework not mentioned in the documentation should generally b`
`15`	`15`	`* Avoid exception when trying to include skipped relationship`
`16`	`16`	* Don't swallow `filter[]` params when there are several
`17`	`17`	`* Fix DeprecationWarning regarding collections.abc import in Python 3.7`
	`18`	`+* Allow OPTIONS request to be used on RelationshipView.`
`18`	`19`
`19`	`20`	`## [2.7.0] - 2019年01月14日`
`20`	`21`

Lines changed: 33 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -274,6 +274,39 @@ def test_new_comment_data_patch_to_many_relationship(self):`
`274`	`274`
`275`	`275`	`assert Comment.objects.filter(id=self.second_comment.id).exists()`
`276`	`276`
	`277`	`+ def test_options_entry_relationship_blog(self):`
	`278`	`+ url = reverse(`
	`279`	`+ 'entry-relationships', kwargs={'pk': self.first_entry.id, 'related_field': 'blog'}`
	`280`	`+ )`
	`281`	`+ response = self.client.options(url)`
	`282`	`+ expected_data = {`
	`283`	`+ "data": {`
	`284`	`+ "name": "Entry Relationship",`
	`285`	`+ "description": "",`
	`286`	`+ "renders": [`
	`287`	`+ "application/vnd.api+json",`
	`288`	`+ "text/html"`
	`289`	`+ ],`
	`290`	`+ "parses": [`
	`291`	`+ "application/vnd.api+json",`
	`292`	`+ "application/x-www-form-urlencoded",`
	`293`	`+ "multipart/form-data"`
	`294`	`+ ],`
	`295`	`+ "allowed_methods": [`
	`296`	`+ "GET",`
	`297`	`+ "POST",`
	`298`	`+ "PATCH",`
	`299`	`+ "DELETE",`
	`300`	`+ "HEAD",`
	`301`	`+ "OPTIONS"`
	`302`	`+ ],`
	`303`	`+ "actions": {`
	`304`	`+ "POST": {}`
	`305`	`+ }`
	`306`	`+ }`
	`307`	`+ }`
	`308`	`+ assert response.json() == expected_data`
	`309`	`+`
`277`	`310`
`278`	`311`	`class TestRelatedMixin(APITestCase):`
`279`	`312`

Lines changed: 2 additions & 4 deletions

Original file line number	Diff line number	Diff line change
`@@ -29,10 +29,8 @@ class ResourceIdentifierObjectSerializer(BaseSerializer):`
`29`	`29`
`30`	`30`	`def __init__(self, args, *kwargs):`
`31`	`31`	`self.model_class = kwargs.pop('model_class', self.model_class)`
`32`		`- if 'instance' not in kwargs and not self.model_class:`
`33`		`- raise RuntimeError(`
`34`		`- 'ResourceIdentifierObjectsSerializer must be initialized with a model class.'`
`35`		`- )`
	`32`	`+ # this has no fields but assumptions are made elsewhere that self.fields exists.`
	`33`	`+ self.fields = {}`
`36`	`34`	`super(ResourceIdentifierObjectSerializer, self).__init__(args, *kwargs)`
`37`	`35`
`38`	`36`	`def to_representation(self, instance):`

Comments

(0)