Skip to content

Database version in features #5627

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Jun 3, 2019
Merged

Database version in features #5627

merged 4 commits into from
Jun 3, 2019

Conversation

oallouch
Copy link
Contributor

I added databaseVersion and engine in serverInfo (and in StorageAdapter).
I also corrected the old test (it failed apparently).

Note that I installed a local MongoDB. The features.spec.js tests pass with no error, but I still get a "Top Level suite" error:
Could not find a MongoDB version matching {"version":"${MONGODB_VERSION:=4.0.4}","arch":"x86_64","platform":"win32","branch":"master","bits":"64","de bug":false,"ext":".zip","distro":"2008plus-ssl","filenamePlatform":"win32"}

oallouch added 2 commits May 30, 2019 20:24
. adding engine and database in the StorageAdapter interface and implementations
@codecov
Copy link

codecov bot commented May 31, 2019

Codecov Report

Merging #5627 into master will increase coverage by 0.03%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5627      +/-   ##
==========================================
+ Coverage   94.13%   94.16%   +0.03%     
==========================================
  Files         129      129              
  Lines        9235     9244       +9     
==========================================
+ Hits         8693     8705      +12     
+ Misses        542      539       -3
Impacted Files Coverage Δ
src/Adapters/Storage/Mongo/MongoStorageAdapter.js 92.53% <100%> (+0.8%) ⬆️
src/Routers/FeaturesRouter.js 100% <100%> (ø) ⬆️
...dapters/Storage/Postgres/PostgresStorageAdapter.js 96.82% <100%> (-0.08%) ⬇️
src/RestWrite.js 93.64% <0%> (+0.17%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e370b93...6bfeb5a. Read the comment docs.

Copy link
Member

@dplewis dplewis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, just a few changes from me.

{"version":"${MONGODB_VERSION:=4.0.4}","arch":"x86_64","platform":"win32","branch":"master","bits":"64","de bug":false,"ext":".zip","distro":"2008plus-ssl","filenamePlatform":"win32"}

Is a windows issue to resolve install mongodb 4.0.4.
If not you can change package.json
from

"test": "cross-env MONGODB_VERSION=${MONGODB_VERSION:=4.0.4} MONGODB_STORAGE_ENGINE=mmapv1 TESTING=1 jasmine",

to the following temporarily and run your own mongodb locally (if you do, don't commit it)

cross-env TESTING=1 jasmine

@oallouch
Copy link
Contributor Author

oallouch commented Jun 3, 2019

@dplewis I checked "Resolve Conversation" . I don't know if I can do it or if you prefer to do it.

@dplewis dplewis merged commit 7fc0d45 into parse-community:master Jun 3, 2019
@dplewis
Copy link
Member

dplewis commented Jun 3, 2019

Thanks for the PR. Maybe we can get this in the Dashboard.

@oallouch
Copy link
Contributor Author

oallouch commented Jun 4, 2019

or in the REST doc :)

@oallouch
Copy link
Contributor Author

oallouch commented Jun 5, 2019

I've started the dashboard thing.

dplewis added a commit that referenced this pull request Jun 13, 2019
dplewis added a commit that referenced this pull request Jun 13, 2019
* Fix #5678

* Revert "Fix #5678"

This reverts commit 106b6dd.

* revert #5627
UnderratedDev pushed a commit to UnderratedDev/parse-server that referenced this pull request Mar 21, 2020
* adding database.version in the serverInfo (only MongoDB, it gives undefined when using Postgres)

* . correction of old 'features' tests
. adding engine and database in the StorageAdapter interface and implementations

* . version retrieval done in performInitialization
. PostgreSQL version

* performInitialization now returns a Promise
UnderratedDev pushed a commit to UnderratedDev/parse-server that referenced this pull request Mar 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants