Skip to content

Commit f4bace7

Browse files
authored
Merge pull request vladwulf#20 from SarenT/main
fix: 🐛 📝 suggestion for a proper field check closes vladwulf#19
2 parents 37687b6 + 5ff9af2 commit f4bace7

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

‎test/app.e2e-spec.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -206,7 +206,7 @@ describe('App e2e', () => {
206206
Authorization: 'Bearer $S{userAt}',
207207
})
208208
.expectStatus(200)
209-
.expectBodyContains('$S{bookmarkId}');
209+
.expectBodyContains('$S{bookmarkId}'); //.expectJsonMatch({id: '$S{bookmarkId}'}) would have been the correct way of testing to prevent false positive matches with other numbers, user id etc.
210210
});
211211
});
212212

0 commit comments

Comments
 (0)