Skip to content

Commit

Permalink
Remove premature test expansion to provide coverage
Browse files Browse the repository at this point in the history
  • Loading branch information
davidcmoulton committed Dec 18, 2024
1 parent cf8ad83 commit 150614c
Showing 1 changed file with 18 additions and 20 deletions.
38 changes: 18 additions & 20 deletions test/Serializer/ArticleVoRNormalizerTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -150,7 +150,7 @@ public function normalizeProvider() : array
->withDecisionLetter(promise_for(new ArticleSection(new ArraySequence([new Paragraph('Article 09560 decision letter text')]), '10.7554/eLife.09560decisionLetter', 'decision-letter-id')))
->withAuthorResponse(promise_for(new ArticleSection(new ArraySequence([new Paragraph('Article 09560 author response text')]), '10.7554/eLife.09560authorResponse', 'author-response-id')))
->withElifeAssessmentArticleSection(promise_for(new ArticleSection(new ArraySequence([new Paragraph('Article 09560 elife assessment text')]), '10.7554/eLife.09560elifeAssessment', 'elife-assessment-id')))
->withElifeAssessment(new ElifeAssessment(['important'], ['solid']))
// ->withElifeAssessment(new ElifeAssessment(['important'], ['solid']))
->withRecommendationsForAuthors(promise_for(new ArticleSection(new ArraySequence([new Paragraph('Article 09560 recommendations for authors text')]), '10.7554/eLife.09560recommendationsForAuthors', 'recommendations-for-authors-id')))
->withPublicReviews(new ArraySequence([new PublicReview('Public review 1', new ArraySequence([new Paragraph('Public review 1 content')]))]))
->__invoke(),
Expand Down Expand Up @@ -449,8 +449,8 @@ public function normalizeProvider() : array
'scietyUri' => 'https://elife-assessment.com',
'doi' => '10.7554/eLife.09560elifeAssessment',
'id' => 'elife-assessment-id',
'significance' => ['important'],
'strength' => ['solid'],
// 'significance' => ['important'],
// 'strength' => ['solid'],
],
'recommendationsForAuthors' => [
'title' => 'Recommendations for authors',
Expand Down Expand Up @@ -584,7 +584,7 @@ function ($test) {
->withDecisionLetter(promise_for(new ArticleSection(new ArraySequence([new Paragraph('Article 09560 decision letter text')]), '10.7554/eLife.09560decisionLetter', 'decision-letter-id')))
->withDecisionLetterDescription(new ArraySequence([new Paragraph('Article 09560 decision letter description')]))
->withAuthorResponse(promise_for(new ArticleSection(new ArraySequence([new Paragraph('Article 09560 author response text')]), '10.7554/eLife.09560authorResponse', 'author-response-id')))
->withElifeAssessment(new ElifeAssessment(['landmark'], ['solid']))
// ->withElifeAssessment(new ElifeAssessment(['landmark'], ['solid']))
->withElifeAssessmentArticleSection(promise_for(new ArticleSection(new ArraySequence([new Paragraph('Article 09560 elife assessment text')]), '10.7554/eLife.09560elifeAssessment', 'elife-assessment-id')))
->withElifeAssessmentTitle(promise_for('eLife assessment'))
->withElifeAssessmentScietyUri(promise_for('https://elife-assessment-09560.com'))
Expand Down Expand Up @@ -655,20 +655,20 @@ function ($test) {
'curationLabels' => ['Landmark', 'Exceptional'],
'figuresPdf' => 'http://www.example.com/figures',
'impactStatement' => 'A new hominin species has been unearthed in the Dinaledi Chamber of the Rising Star cave system in the largest assemblage of a single species of hominins yet discovered in Africa.',
'elifeAssessment' => [
'title' => 'eLife assessment',
'content' => [
[
'type' => 'paragraph',
'text' => 'Article 09560 elife assessment text',
],
],
'scietyUri' => 'https://elife-assessment-09560.com',
'doi' => '10.7554/eLife.09560elifeAssessment',
'id' => 'elife-assessment-id',
'significance' => ['landmark'],
'strength' => ['solid'],
]
// 'elifeAssessment' => [
// 'title' => 'eLife assessment',
// 'content' => [
// [
// 'type' => 'paragraph',
// 'text' => 'Article 09560 elife assessment text',
// ],
// ],
// 'scietyUri' => 'https://elife-assessment-09560.com',
// 'doi' => '10.7554/eLife.09560elifeAssessment',
// 'id' => 'elife-assessment-id',
// 'significance' => ['landmark'],
// 'strength' => ['solid'],
// ]
],
function (ApiTestCase $test) {
$test->mockArticleCall('09560', true, true, 1);
Expand Down Expand Up @@ -731,8 +731,6 @@ function (ApiTestCase $test) {
},
],
];
unset($datasets['complete']);
unset($datasets['complete snippet']);
return $datasets;
}

Expand Down

0 comments on commit 150614c

Please sign in to comment.