Skip to content
This repository was archived by the owner on Mar 24, 2026. It is now read-only.

disable kelp update test#2276

Merged
NateBrady23 merged 1 commit intoTechEmpower:masterfrom
knewmanTE:disable-kelp-update-test
Sep 16, 2016
Merged

disable kelp update test#2276
NateBrady23 merged 1 commit intoTechEmpower:masterfrom
knewmanTE:disable-kelp-update-test

Conversation

@knewmanTE
Copy link
Copy Markdown
Contributor

As mentioned in #2235, some frameworks are (I assume unintentionally) modifying the structure of the MongoDB World documents by removing the id field, leaving only the _id and randomNumber fields. For example, the collection goes from this:

{ "_id" : 1, "id" : 1, "randomNumber" : 7229 }
{ "_id" : 2, "id" : 2, "randomNumber" : 5321 }
{ "_id" : 3, "id" : 3, "randomNumber" : 9144 }
{ "_id" : 4, "id" : 4, "randomNumber" : 9358 }
{ "_id" : 5, "id" : 5, "randomNumber" : 4414 }
{ "_id" : 6, "id" : 6, "randomNumber" : 5760 }
{ "_id" : 7, "id" : 7, "randomNumber" : 6196 }
{ "_id" : 8, "id" : 8, "randomNumber" : 5389 }
{ "_id" : 9, "id" : 9, "randomNumber" : 5945 }
{ "_id" : 10, "id" : 10, "randomNumber" : 6208 }

to this:

{ "_id" : 1, "id" : 1, "randomNumber" : 7776 }
{ "_id" : 2, "id" : 2, "randomNumber" : 7637 }
{ "_id" : 3, "randomNumber" : 3636 }
{ "_id" : 4, "randomNumber" : 4423 }
{ "_id" : 5, "id" : 5, "randomNumber" : 9595 }
{ "_id" : 6, "randomNumber" : 5993 }
{ "_id" : 7, "id" : 7, "randomNumber" : 3051 }
{ "_id" : 8, "id" : 8, "randomNumber" : 5884 }
{ "_id" : 9, "randomNumber" : 3341 }
{ "_id" : 10, "randomNumber" : 8440 }

Out of fairness to the other frameworks that need to access the same MongoDB collections, this PR disables the update test for Perl/kelp until the issue has been resolved.

ping @Naturalist

@knewmanTE knewmanTE modified the milestones: Round 14, Round 15, Round 13 Sep 16, 2016
@NateBrady23 NateBrady23 merged commit 5a4cf63 into TechEmpower:master Sep 16, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants