A bug that escaped unit testing?

[ogourment]

This one seemed to override option: false.
To investigate.

MeteorSettings.setDefaults({ public:
  {
    blog: {
      "blogPath": "/blog",
      "manager": { "option": true }
    }
  }
});

With settings.json:

{
  "public": {
   "blog": {
      "blogPath": "/blog",
      "manager": {
        "option": false
      }
    }
}

[ogourment]

I added a unit test (although this case is already covered), but the test succeeded while it didn't work (the setting was overriden) in a real app.

[adeubank]

I saw false properties being overridden as well. I submitted a pull request that seems to fix a case similar to this.

[ogourment]

The specific issue is fixed thanks to @adeubank, but there's still a problem with unit tests not working correctly. Still to investigate...