test(feel): verify non-existing member access

[nikku]

Ensures the following is true:

{ a: 1 }.b = null

null.b = null

Related to #537

[baldimir]

@dmn-tck/contributors please review

[opatrascoiu]

Thank you for the tests.

They look correct to me.

However, there are only 6 test cases that test the selection for lists and contexts. I recommend moving the tests for lists (4) in 0069-feel-list and the ones for context (2) in 0057-feel-context. It will make our life easier when designing new tests to improve the coverage - have tests for sections 10.3.2.5 Lists and filters and 10.3.2.6 Context in only two separate files.

Also, move the expected values where they belong, in the .xml file not in the dmn file. The expected value can be mentioned in a comment in the dmn file. I believe @StrayAlien used this pattern in lots of our test cases.

[baldimir]

Hi @nikku, could you please check the last comment from @opatrascoiu?

[nikku]

@baldimir Thanks for the reminder. This is on my TODO-list. I just was not able to get into it.

[nikku]

@opatrascoiu to verify I understand you correctly:

I recommend moving the tests for lists (4) in 0069-feel-list and the ones for context (2) in 0057-feel-context.

• Shall I move the new tests to 0069-feel-list and 0057-feel-context, or shall I move tests in 0069-feel-list and 0057-feel-context somewhere? If so, where to?

Also, move the expected values where they belong, in the .xml file not in the dmn file.

• Could you share a source code reference to the "recommended test pattern"? I'm happy to follow what is most reasonable for you but I don't see a clear pattern across the code base at the moment yet.

[opatrascoiu]

@nikku Regarding the test patterns: For example, it is recommended to have test cases related to a certain behaviour (e.g. a certain function, class) in the same place (e.g. file).

Please move the new tests related to lists (e.g. filters) to 0069-feel-list and the ones related to contexts (e.g. path navigation) in 0057-feel-context.

Thank you.

[nikku]

@opatrascoiu I updated this PR, moving the tests around as you suggested. Please have another look.

[opatrascoiu]

@nikku Apologies for the delay. Thank you very much for the contribution. The tests look good to me.