Hi! Thanks for the PR!

I'm not sure what the intent of these changes is... the added test case is not in the canonical-data, and the added __ne__ method is never used.

However, there are other changes in the canonical data that have not been reproduced here. Would you like to work on those?

If so, please make the appropriate changes, then use magic words to link this PR to issue #1620.

The __ne__ function is actually used in the __eq__ function on some cases. I needed to implement it in order to pass the tests.

The test case I added does test a case that is mentioned in the README, but is never tested.

I will add the other tests when I will have time.

My apologies. I did not analyze the situation thoroughly; you are correct on both counts.

The ne function is actually used in the eq function on some cases. I needed to implement it in order to pass the tests.

Interesting that none of the existing tests had an issue with this. What version of Python are you running with?

The test case I added does test a case that is mentioned in the README, but is never tested.

In that case, it should be added to the canonical data. I have created exercism/problem-specifications#1414 to address this, and have tagged you as a co-author on the commit (since you technically created the test case here).

Interesting that none of the existing tests had an issue with this. What version of Python are you running with?

I'm running python 2.7.15rc1 on Ubuntu. I'm not an expert on python, but it seems to be a difference between python 2.7 and 3 (https://docs.python.org/3/reference/datamodel.html#object.__ne__ and https://docs.python.org/2.7/reference/datamodel.html#object.__ne__)

Just a couple more things:

• As sgf-parsing: add test case for multiple properties problem-specifications#1414 adds the test case you are proposing, I would like to wait until that has been merged before merging this.
• Since we are targeting sgf-parsing: add test case for multiple properties problem-specifications#1414, please change the referenced canonical data version in sgf_parsing_test.py to v1.2.0
• Use magic words to link this PR to issue sgf-parsing: update tests to v1.1.0 #1620 as updating to v1.2.0 will also cover v1.1.0
• Change to PR title to something like: "<exercise-name>: update tests to <new version>"

Thanks for the comments!

You are right this will close #1620

Is there a way to mark the PR as blocked by a different one?

You are right this will close #1620

The magic words must be in the PR description to properly link. I spoke too soon.

Is there a way to mark the PR as blocked by a different one?

Not at this time, no. I'll just have to watch the other PR until it is merged (no problem since it is my PR).

Merged; thanks for working on this!