Skip to content

Commit 59103a7

Browse files
committed
Tests should test importing name instead of _id
1 parent 9c486d2 commit 59103a7

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

admin_tests/preprint_providers/test_views.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -239,7 +239,7 @@ def test_export_to_import_new_provider(self):
239239
res = self.view.get(self.request)
240240
content_dict = json.loads(res.content)
241241

242-
content_dict['fields']['_id'] = 'new_id'
242+
content_dict['fields']['name'] = 'Awesome New Name'
243243
data = StringIO(unicode(json.dumps(content_dict), 'utf-8'))
244244
self.import_request.FILES['file'] = InMemoryUploadedFile(data, None, 'data', 'application/json', 500, None, {})
245245

@@ -249,7 +249,7 @@ def test_export_to_import_new_provider(self):
249249
new_provider = PreprintProvider.objects.get(id=provider_id)
250250

251251
nt.assert_equal(res.status_code, 302)
252-
nt.assert_equal(new_provider._id, 'new_id')
252+
nt.assert_equal(new_provider.name, 'Awesome New Name')
253253
nt.assert_equal(new_provider.subjects.all().count(), 1)
254254
nt.assert_equal(new_provider.licenses_acceptable.all().count(), 1)
255255
nt.assert_equal(new_provider.subjects.all()[0].text, self.subject.text)
@@ -261,7 +261,7 @@ def test_export_to_import_new_provider_with_models_out_of_sync(self):
261261
res = self.view.get(self.request)
262262
content_dict = json.loads(res.content)
263263

264-
content_dict['fields']['_id'] = 'new_id'
264+
content_dict['fields']['name'] = 'Awesome New Name'
265265
content_dict['fields']['new_field'] = 'this is a new field, not in the model'
266266
del content_dict['fields']['description'] # this is a old field, removed from the model JSON
267267

@@ -274,7 +274,7 @@ def test_export_to_import_new_provider_with_models_out_of_sync(self):
274274
new_provider = PreprintProvider.objects.get(id=provider_id)
275275

276276
nt.assert_equal(res.status_code, 302)
277-
nt.assert_equal(new_provider._id, 'new_id')
277+
nt.assert_equal(new_provider.name, 'Awesome New Name')
278278

279279
def test_update_provider_existing_subjects(self):
280280
# If there are existing subjects for a provider, imported subjects are ignored

0 commit comments

Comments
 (0)