[GitHub] brooklyn-server pull request #841: BROOKLYN-538: add failing test

[aledsage <gi...@git.apache.org>]

GitHub user aledsage opened a pull request:

    https://github.com/apache/brooklyn-server/pull/841

    BROOKLYN-538: add failing test

    Adds a test that demonstrates https://issues.apache.org/jira/browse/BROOKLYN-538
    
    I'm going to mark issue as "won't fix" for the following reasons:
    * The exception is benign - the sensor gets the expected value of `null`
    * The exception happens deep within the `JsonPath` library - it might well be the right behaviour for it to fail if you ask for path `$` on a null object.
    * Working around it in Brooklyn code would be added complexity for no real benefit.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/aledsage/brooklyn-server BROOKLYN-538-json-parsing-null

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/brooklyn-server/pull/841.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #841
    
----
commit 67f1661dbf6963423aaecb8dbf1399f00e356f55
Author: Aled Sage <al...@gmail.com>
Date:   2017-09-21T12:49:47Z

    BROOKLYN-538: add failing test

----


---

[GitHub] brooklyn-server pull request #841: BROOKLYN-538: add failing test

[aledsage <gi...@git.apache.org>]

Github user aledsage closed the pull request at:

    https://github.com/apache/brooklyn-server/pull/841


---

[GitHub] brooklyn-server issue #841: BROOKLYN-538: add failing test

[aledsage <gi...@git.apache.org>]

Github user aledsage commented on the issue:

    https://github.com/apache/brooklyn-server/pull/841
  
    Closing this PR - it is linked to from https://issues.apache.org/jira/browse/BROOKLYN-538, so we can refer back to it if we ever decide to do anything with BROOkLYN-538 other than "won't fix".


---