Increase test coverage about mongo-obj-framework

[cunhazera]

@JPDSousa can you describe how to build the project and run it?
I tried to build it with mvn clean install, but did not work

[JPDSousa]

@cunhazera The mvn clean install command will run the tests, which require an active mongodb connection in port 27020. If you have mongodb installed, simply type mongod --port 27020 in one console (this will create a mongodb server), and run mvn clean install in a separate terminal.

[archie-swif]

Hi folks!
I'm planning to add some tests for Or/AndQuery.

[JPDSousa]

@archie-swif That's really cool! Actually, the whole And/Or system might be a bit fragile ... i implemented it long time ago an never got the time to properly test it. But i totally encourage you to do it! I think it is a great feature (and totally useful for updates).

If you have any doubt or suggestion just share it! (might need to open a new issue for that)

[tminussi]

@JPDSousa thanks for merging ec31910. Step by step and we'll get a high coverage!

Count on me.

[boaglio]

@JPDSousa is anything left to test ?

[JPDSousa]

@boaglio there is still some basic code in the collection's package, and in the gridfs package. I'm currently working in a new cool feature which will bring some new code to the project as well...

[JPDSousa]

@tminussi I've made some upgrades to the parser package (#32), and decided to warn you in case your testing any parser code. The new logic i'm using is still incomplete (but functional) and it's still not fully tested, but i'll add some base tests as soon as i can!

[tminussi]

@JPDSousa awesome. Thanks for letting me know!

[Nipel-Crumple]

@JPDSousa hi, I'd like to contribute too. Can I just take some untested piece of code, write tests and submit pr?

[JPDSousa]

@Nipel-Crumple sure! If you have any doubt just let me know!

[Nipel-Crumple]

@JPDSousa I'd want to test SmofUpdate class, and think about how to create test method for org.smof.collection.SmofUpdate#rename? Maybe you have some ideas? Cause if I rename field, the mapping is broken and field is null after find() from MongoDb

[JPDSousa]

You're right @Nipel-Crumple . Well, to be fair, i didn't think much about it when i added the org.smof.collection.SmofUpdate#rename method, as it is a bit useless to the project.

Maybe leave it untested for now. I'm going to tag it as @Deprecated (or even remove) in a future patch, while i think of a cool application for the MongoDB's renaming feature (useful for the project). In the mean time, if you have some ideas feel free to share 😄

[JPDSousa]

For anyone who is looking for code to test: #33 has brought a whole new set of querying features, some of them which are not tested yet. Feel free to extend ReadOpTest with some new cool query tests.

[Nipel-Crumple]

Okay, I got it, thanks a lot 🙂 So I'll try write tests for other methods of this class

[lavanyajain]

JPDSousa Any class that I can pickup here? if so please add jira and assign it to me

[JPDSousa]

Hey @lavanyajain this project is not active. Hence, we're not promoting future developments.