Skip to content
This repository has been archived by the owner on Aug 30, 2021. It is now read-only.

fix(dependency): Mongoose 4.4.7 bug #1264

Merged
merged 1 commit into from
Mar 13, 2016

Conversation

mleanos
Copy link
Member

@mleanos mleanos commented Mar 12, 2016

Fixes an issue with Mongoose 4.4.7 that causes the database connection.db property to be undefined. This issue is causing our builds to fail.

Sets the Mongoose version to ~4.4.3 <4.4.7 for now until we investigate further. This should solve the issue of our failing builds, when the dropdb Grunt/Gulp task is ran with the E2E tests.

For more context:
this line https://github.com/meanjs/mean/blob/master/gruntfile.js#L277 was throwing an exception during our builds, since the dropdb task is run with the E2E tests. This seemed to cause the Protractor tasks to time-out. However, there was no indication of this exception being thrown from our dropdb task when running with grunt test or in the Travis builds. I had to explicitly run grunt test:e2e to see this exception that led me to this fix.

Fixes an issue with Mongoose `4.4.7` that causes the database
connection.db property to be undefined. This issue is causing our builds
to fail.

Sets the Mongoose version to `~4.4.3 <4.4.7` for now until we
investigate further. This should solve the issue of our failing builds,
when the `dropdb` Grunt/Gulp task is ran with the E2E tests.
@mleanos
Copy link
Member Author

mleanos commented Mar 13, 2016

Merging this since it's a small change, and we need to have our builds passing. It's disruptive to all the contributors that the builds aren't passing.

@meanjs/contributors

@mleanos mleanos self-assigned this Mar 13, 2016
@mleanos mleanos added this to the 0.5.0 milestone Mar 13, 2016
mleanos added a commit that referenced this pull request Mar 13, 2016
@mleanos mleanos merged commit a5e72cf into meanjs:master Mar 13, 2016
@lirantal
Copy link
Member

@mleanos
Ok but let's keep investigating what's the issue here.

mleanos added a commit to mleanos/mean that referenced this pull request Mar 18, 2016
Upgrades the Mongoose version to 4.4.8.

The need to limit Mongoose version to be less than 4.4.7 is no longer
necessary due to a bug fix released with 4.4.8.

Resolves the need for meanjs#1264
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